JSHint“未定义”错误

时间:2013-09-04 11:02:06

标签: javascript jquery debugging jshint

我有这个功能,我用来验证联系表单,但是当我通过JSHint运行它时,我收到了大量的“未定义”错误。我是一个相当jquery的菜鸟,我不确定这些应该如何定义。

(function ($, document, undefined) {
$(document).ready(function(){

    // Place ID's of all required fields here.
    required = ["name", "email", "message"];
    // If using an ID other than #email or #error then replace it here
    email = $("#email");
    errornotice = $("#error");
    // The text to show up within a field when it is incorrect
    emptyerror = "Please fill out this field.";
    emailerror = "Please enter a valid e-mail.";

    $("#theform").submit(function(){
        //Validate required fields
        for (i=0;i<required.length;i++) {
            var input = $('#'+required[i]);
            if ((input.val() == "") || (input.val() == emptyerror)) {
                input.addClass("needsfilled");
                input.val(emptyerror);
                errornotice.fadeIn(750);
            } else {
                input.removeClass("needsfilled");
            }
        }
        // Validate the e-mail.
        if (!/^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/.test(email.val())) {
            email.addClass("needsfilled");
            email.val(emailerror);
        }

        //if any inputs on the page have the class 'needsfilled' the form will not submit
        if ($(":input").hasClass("needsfilled")) {
            return false;
        } else {
            errornotice.hide();
            return true;
        }
    });

    // Clears any fields in the form when the user clicks on them
    $(":input").focus(function(){       
       if ($(this).hasClass("needsfilled") ) {
            $(this).val("");
            $(this).removeClass("needsfilled");
       }
    });
});
})(jQuery, document);

以下是几个例子:

来自表单数据 - 'required'未定义。

required = ["name", "email", "message"];

'我'未定义。

for (i=0;i<required.length;i++) {

和'email'未定义。

if (!/^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-])+\.)+([a-zA-Z0-9]{2,4})+$/.test(email.val())) {

2 个答案:

答案 0 :(得分:0)

您应该使用&#34; var&#34;:

定义变量
var required = ["name", "email", "message"];
for(var i...

答案 1 :(得分:0)

嗯,这可能最终是一个风格问题。使用没有var声明的变量显然意味着您正在使用未声明的变量。在某些情况下,这是为了生成具有全局范围的变量而有目的地完成的。这当然可以(以更清洁的方式)通过使用var email;在所有闭包 的某处声明变量来实现。