11

JSLint 验证错误“将此与前面的 var 语句结合起来”

我如何结合这个,所以我不会得到 JSLint 验证错误?我在 getClassName 函数的代码行中收到验证错误。

$(document).ready(function () {
'use strict';
// This function is used to calculate the date
function dateString(dateToDisplay) {

    var monthNames = ['January', 'February', 'March', 'April', 'May', 'June',
        'July', 'August', 'September', 'October', 'November', 'December'],
        dayNames = ['Sunday', 'Monday', 'Tuesday', 'Wednesday', 'Thursday',
            'Friday', 'Saturday'];

// Use current date if no parameter or bad date provided
    if (dateToDisplay === undefined || !(dateToDisplay instanceof Date)) {
        dateToDisplay = new Date();
    }

    return dayNames[dateToDisplay.getDay()] + ', ' // Day of week: Sunday
        + monthNames[dateToDisplay.getMonth()] + ' ' // Name of month: July
        + dateToDisplay.getDate() + ', ' // Day of month: 20
        + dateToDisplay.getFullYear(); // Year: 1969
}


$('#safetyRecord').hide(); // hides the header with the id of safetyRecord until we are ready to display it later in the code
$('#today').text(dateString()); // changes the text in the span with the id of today to the current date using the dateString function
function getClassName(days) { // this function determines  which css style to apply to the id daysSinceLastAccident and the id message based on the number of days since last accident
    if (days >= 730) { return "great"; }
    if (days >= 180) { return "good"; }
    if (days >= 60) { return "marginal"; }
    if (days >= 14) { return "poor"; }
    if (days >= 0) { return "disaster"; }
}

$('#checkRecord').click(function () { // when the checkRecord button is clicked the number of days since last accident is calculated, storing the number in a variable.
    var userEnteredDate = new Date($('#dateOfLastAccident').val()); // variable that stores user entered date
    var today = new Date(); 
    var minutes = 1000 * 60; // calculation used to convert miliseconds to minutes
    var hours = minutes * 60; // calculation used to conver minutes into hours
    var days = hours * 24; // calculation used to convert hours to days
    var years = days * 365; // calculation used to convert days into years
    var daysSinceAccident = Math.floor((today.getTime() - userEnteredDate.getTime()) / days); // calculation used to find the difference between current date and user entered date as a whole number
    var className = getClassName(daysSinceAccident); // variable clasName finds the correct css style to apply based on daysSinceAccident
    $('#daysSinceLastAccident').text(daysSinceAccident); // replaces the content of the element with id daysSinceLastAccident with the number of days accident-free
    $('#daysSinceLastAccident').removeClass();
    $('#daysSinceLastAccident').addClass(className); // applies css class style to element with id daysSinceLastAccident

    $('#safetyRecord').show(); // Using the same timeframes, a custom message styled with the appropriate css class in the paragraph with id message is displayed.
    $('#message').removeClass(); 
    $('#message').addClass(className);
    $('#message').html(className + " is the current safety record.");
});

});

4

2 回答 2

28

此错误意味着您var的某些函数中有多个语句,例如:

var x = 1;
var y = 2;

JSLint 希望您将变量声明组合在一个var语句中,例如:

var x = 1,
    y = 2;
于 2013-10-11T05:17:56.727 回答
4

我认为 JSLint 指的是您的代码的这几行(在您提供的代码的底部):

var userEnteredDate = new Date($('#dateOfLastAccident').val()); // variable that stores user entered date
var today = new Date(); 
var minutes = 1000 * 60; // calculation used to convert miliseconds to minutes
var hours = minutes * 60; // calculation used to conver minutes into hours
var days = hours * 24; // calculation used to convert hours to days
var years = days * 365; // calculation used to convert days into years
var daysSinceAccident = Math.floor((today.getTime() - userEnteredDate.getTime()) / days); // calculation used to find the difference between current date and user entered date as a whole number
var className = getClassName(daysSinceAccident); // variable clasName finds the correct css style to apply based on daysSinceAccident

您可以通过用逗号分隔每个变量来避免多次声明它们:

var userEnteredDate = new Date($('#dateOfLastAccident').val()),
    today = new Date(),
    minutes = 1000 * 60,
    hours = minutes * 60,
    days = hours * 24,
    years = days * 36,
    daysSinceAccident = Math.floor((today.getTime() - userEnteredDate.getTime()) / days),
    className = getClassName(daysSinceAccident);
于 2013-10-11T05:30:49.300 回答