来自javascript验证simplemodal表单的错误

Question

我在html网站上编辑Simplemodal联系表单以创建表单。所有的编辑都很顺利，直到我遇到javascript导致获得未定义值的问题。我正在尝试添加一些字段并删除消息和复选框字段。第一个块是原始代码，我添加了第二个块：

            if (!$('#contact-container #contact-name').val()) {
            contact.message += 'First Name is required. ';
            fail = true;
        }

        if (!$('#contact-container #contact-lname').val()) {
            contact.message += 'Last Name is required. ';
            fail = true;
        }

第一个if语句工作正常，但第二个语句在处理js时有一个未定义的值。实际表单显示正常，但此处理不起作用。以下代码是我在文件中的代码：

            <label for='contact-name'>First Name:</label>
        <input type='text' id='contact-name' class='contact-input' name='name' tabindex='1001' />
        <label for='contact-lname'>Last Name:</label>
        <input type='text' id='contact-lname' class='contact-input' name='lname' tabindex='1002' />

原件完好无损，我刚刚添加了最后两行。任何帮助，将不胜感激。谢谢！

编辑：http://www.ericmmartin.com/projects/simplemodal-demos/我遇到问题的代码是contact.js，我在上面的修改过的行中遇到了问题。

编辑：这是通过网页激活Simplemodal的代码

<div id='container'>
<div id='content'>
    <div id='contact-form'>
        <h3><a href='#' class='contact'>Support the Cause!</a></h3>
    </div>
</div>
</div>

修改后的contact.js位于下方并使用jquery和jquerySimplemodal（这两种方式都不影响代码处理的方式）。其他两个js文件是通过上面的链接。

jQuery(function ($) {
var contact = {
    message: null,
    init: function () {
        $('#contact-form input.contact, #contact-form a.contact').click(function (e) {
            e.preventDefault();

            // load the contact form using ajax
            $.get("data/contact.php", function(data){
                // create a modal dialog with the data
                $(data).modal({
                    closeHTML: "<a href='#' title='Close' class='modal-close'>x</a>",
                    position: ["15%",],
                    overlayId: 'contact-overlay',
                    containerId: 'contact-container',
                    onOpen: contact.open,
                    onShow: contact.show,
                    onClose: contact.close
                });
            });
        });
    },
    open: function (dialog) {
        // add padding to the buttons in firefox/mozilla
        if ($.browser.mozilla) {
            $('#contact-container .contact-button').css({
                'padding-bottom': '2px'
            });
        }
        // input field font size
        if ($.browser.safari) {
            $('#contact-container .contact-input').css({
                'font-size': '.9em'
            });
        }

        // dynamically determine height
        var h = 280;
        if ($('#contact-subject').length) {
            h += 26;
        }
        if ($('#contact-cc').length) {
            h += 22;
        }

        var title = $('#contact-container .contact-title').html();
        $('#contact-container .contact-title').html('Loading...');
        dialog.overlay.fadeIn(200, function () {
            dialog.container.fadeIn(200, function () {
                dialog.data.fadeIn(200, function () {
                    $('#contact-container .contact-content').animate({
                        height: h
                    }, function () {
                        $('#contact-container .contact-title').html(title);
                        $('#contact-container form').fadeIn(200, function () {
                            $('#contact-container #contact-name').focus();

                //          $('#contact-container .contact-cc').click(function () {
                //              var cc = $('#contact-container #contact-cc');
                    //          cc.is(':checked') ? cc.attr('checked', '') : cc.attr('checked', 'checked');
                    //      })
                    ;

                            // fix png's for IE 6
                            if ($.browser.msie && $.browser.version < 7) {
                                $('#contact-container .contact-button').each(function () {
                                    if ($(this).css('backgroundImage').match(/^url[("']+(.*\.png)[)"']+$/i)) {
                                        var src = RegExp.$1;
                                        $(this).css({
                                            backgroundImage: 'none',
                                            filter: 'progid:DXImageTransform.Microsoft.AlphaImageLoader(src="' +  src + '", sizingMethod="crop")'
                                        });
                                    }
                                });
                            }
                        });
                    });
                });
            });
        });
    },
    show: function (dialog) {
        $('#contact-container .contact-send').click(function (e) {
            e.preventDefault();
            // validate form
            if (contact.validate()) {
                var msg = $('#contact-container .contact-message');
                msg.fadeOut(function () {
                    msg.removeClass('contact-error').empty();
                });
                $('#contact-container .contact-title').html('Sending...');
                $('#contact-container form').fadeOut(200);
                $('#contact-container .contact-content').animate({
                    height: '80px'}, 
                function () {
                    $('#contact-container .contact-loading').fadeIn(200, function () {
                        $.ajax({
                            url: 'data/contact.php',
                            data: $('#contact-container form').serialize() + '&action=send',
                            type: 'post',
                            cache: false,
                            dataType: 'html',
                            success: function (data) {
                                $('#contact-container .contact-loading').fadeOut(200, function () {
                                    $('#contact-container .contact-title').html('Thank you!');
                                    msg.html(data).fadeIn(200);
                                });
                            },
                            error: contact.error
                        });
                    });
                });
            }
            else {
                if ($('#contact-container .contact-message:visible').length > 0) {
                    var msg = $('#contact-container .contact-message div');
                    msg.fadeOut(200, function () {
                        msg.empty();
                        contact.showError();
                        msg.fadeIn(200);
                    });
                }
                else {
                    $('#contact-container .contact-message').animate({
                        height: '30px'
                    }, contact.showError);
                }

            }
        });
    },
    close: function (dialog) {
        $('#contact-container .contact-message').fadeOut();
        $('#contact-container .contact-title').html('');    <!--here-->
        $('#contact-container form').fadeOut(200);
        $('#contact-container .contact-content').animate({
            height: 40
        }, function () {
            dialog.data.fadeOut(200, function () {
                dialog.container.fadeOut(200, function () {
                    dialog.overlay.fadeOut(200, function () {
                        $.modal.close();
                    });
                });
            });
        });
    },
    error: function (xhr) {
        alert(xhr.statusText);
    },
    validate: function () {
        contact.message = '';
        var fail = false;
        if (!$('#contact-name').val()) {
            contact.message += 'First Name is required. ';
            fail = true;
        }
        //#contact-container 
        if (!$('#contact-lname').val()) {
            contact.message += 'Last Name is required. ';
            fail = true;
        }
        var email = $('#contact-email').val();
        if (!email) {
            contact.message += 'Email is required. ';
            fail = true;
        }
        else {
            if (!contact.validateEmail(email)) {
                contact.message += 'Email is invalid. ';
                fail = true;
            }
        }

        var zip = $('#contact-zip').val();
        if (!zip) {
            contact.message += '2' + zip + '2';
            contact.message += 'ZIP is required. ';
            fail = true;
        }
        else{
            if (!contact.isValidZipCode(zip)) {
                contact.message += 'ZIP is invalid. ';
                fail = true;
            }
        }

        var age = $('#contact-age').val();
        if (!age){
            contact.message += 'Age is required. ';
            fail = true;
        }
        else{
            if (!contact.validateAge(age)) {
                contact.message += 'Age is invalid. ';
                fail = true;
            }
        }

        if (!$('#contact-message').val()) {
            contact.message += 'Message is required.';
            fail = true;
        }

        //if (contact.message.length > 0) {
    //      return true;
        //}
        else {
            if (fail) return false;
            else return true;
        }
    },

    isValidZipCode: function (value) {
    var re = /^\d{5}([\-]\d{4})?$/;
    return (re.test(value));
    },

    validateAge: function (age) {
        if (isNan(age)) return false;
        var a = parseInt(age);
        if (a > 0 && a < 100) return true;
        else return false;
    },

    validateEmail: function (email) {
        var at = email.lastIndexOf("@");

        // Make sure the at (@) sybmol exists and  
        // it is not the first or last character
        if (at < 1 || (at + 1) === email.length)
            return false;

        // Make sure there aren't multiple periods together
        if (/(\.{2,})/.test(email))
            return false;

        // Break up the local and domain portions
        var local = email.substring(0, at);
        var domain = email.substring(at + 1);

        // Check lengths
        if (local.length < 1 || local.length > 64 || domain.length < 4 || domain.length > 255)
            return false;

        // Make sure local and domain don't start with or end with a period
        if (/(^\.|\.$)/.test(local) || /(^\.|\.$)/.test(domain))
            return false;

        // Check for quoted-string addresses
        // Since almost anything is allowed in a quoted-string address,
        // we're just going to let them go through
        if (!/^"(.+)"$/.test(local)) {
            // It's a dot-string address...check for valid characters
            if (!/^[-a-zA-Z0-9!#$%*\/?|^{}`~&'+=_\.]*$/.test(local))
                return false;
        }

        // Make sure domain contains only valid characters and at least one period
        if (!/^[-a-zA-Z0-9\.]*$/.test(domain) || domain.indexOf(".") === -1)
            return false;   

        return true;
    },
    showError: function () {
        $('#contact-container .contact-message')
            .html($('<div class="contact-error"></div>').append(contact.message))
            .fadeIn(200);
    }
};

contact.init();});

这是由javascript调用以生成表单的PHP文件

    $output = "<div style='display:none'>
<div class='contact-top'></div>
<div class='contact-content'>

    <div class='contact-loading' style='display:none'></div>
    <div class='contact-message' style='display:none'></div>
    <form action='#' style='display:none'>
        <label for='contact-name'>First Name:</label>
        <input type='text' id='contact-name' class='contact-input' name='name' tabindex='1001' />
        <label for='contact-lname'>Last Name:</label>
        <input type='text' id='contact-lname' class='contact-input' name='lname' tabindex='1002' />
        <label for='contact-email'>Email:</label>
        <input type='text' id='contact-email' class='contact-input' name='email' tabindex='1003' />
        <label for='contact-zip'>ZIP:</label>
        <input type='text' id='contact-zip' class='contact-input' name='zip' tabindex='1004' />
        <label for='contact-age'>Age:</label>
        <input type='text' id='contact-age' class='contact-input' name='age' tabindex='1005' />

Answer 1

试试这个选择器$('#contact-lname')而不是你的选择器。在我看来，您的#contact-lname不在#contact-container中，或者您的页面上有id="contact-container"或id="contact-lname"的容器。

Answer 2

试试这个：

if ( $.trim($('input[name="name"]').val()) == "") {
        contact.message += 'First Name is required. ';
        fail = true;
    }

if ( $.trim($('input[name="lname"]').val()) == "") {
        contact.message += 'Last Name is required. ';
        fail = true;
    }

<击>

更新问题后，我假设您正在从contact.php文件加载表单元素。

$.get("data/contact.php", function(data){

然后您必须将您的联系表单验证到contact.php。

因为你的jQuery代码在刷新页面时加载了一次，并且当你的页面的所有DOM元素都被加载时，你的jquery代码无法检测你的表单元素，因为你正在加载来自$.get()的元素通过contact.php

的外部文件