第一次迭代后，for循环的JavaScript似乎退出或卡住了

Question

I have a contact us form that i want to check with javascript before submitting it to php. so when i call the function verify(), by placing alert("hello")'s throughout the code, i determined that it was exiting after the first iteration of the second loop. does anyone know why?

Heres my html form code:

<div id="errortext">
</div>
<form id="contact" method="post" action="contact.php">
    <h2>Contact Us</h2>
    <h1>Fields marked with a * are required</h1>
    <p>Full Name*</p>
    <input type="text" class="single" placeholder="Full Name" name="name" id="name" /><br />
    <p>E-Mail Address*</p>
    <input type="text" class="single" placeholder="E-Mail Address" name="email" id="email" /><br />
    <p>Phone Number</p>
    <input type="text" class="single" placeholder="Phone Number" name="number" id="number" /><br />
    <p>Subject</p>
    <input type="text" class="single" placeholder="Subject" name="subject" id="subject" /><br />
    <p>Message*</p>
    <textarea placeholder="Message" name"message" id="message"></textarea><br />
    <input type="button" id="submitb" onclick="verify()" value="SEND" />
</form>

and heres my javascript code:

<script type="text/javascript">
    function verify()
    {
        var rgood = true, nrgood = true;
        var name = document.getElementById("name");
        var email = document.getElementById("email");
        var phone = document.getElementById("number");
        var subject = document.getElementById("subject");
        var message = document.getElementById("message");
        var error = document.getElementById("errortext");

        var relements = new Array(name, email,  message);
        var nrelements = new Array(phone, subject);

        var relementsng = new Array;
        var nrelementsng = new Array;
        var relementsngp = new Array;
        var nrelementsng = new Array;

        for (var x = 0; x < relements.length; x++)
        {
            if (relements[x].value == "")
            {
                rgood = false;
                relementsng.push(relements[x]);
                relementsngp.push(relements[x].placeholder);
            }
        }

        for (var y = 0; y < nrelements.length; y++)
        {
            if (nrelements[y].value == "")
            {
                nrgood = false;
                nrelementsng.push(nrelements[y]);
                nrelementsngp.push(nrelements[y].placeholder);
            }
        }
        alert("hello");
        if (!rgood)
        {
            error.textContent = "All required fields are not filled out! These fields must be filled out: " + nrelementsngp.toString() + "!";
        }
    }
</script>

Answer 1

Your code is all right except this typo :

line23: nrelementsngp.push(nrelements[x].placeholder);
                    ^
line31: nrelementsngp.push(nrelements[y].placeholder);
                    ^

---

On my browser, the error message below was shown.

 "ReferenceError: Can't find variable: nrelementsngp" 

Hope this helps.

Answer 2

    var relementsng = new Array;
    var nrelementsng = new Array;
    var relementsngp = new Array;
    var nrelementsng = new Array; // <<---- Change this line to

To

    var nrelementsngp = new Array;

Because

for (var y = 0; y < nrelements.length; y++){
    if (nrelements[y].value == ""){
        nrgood = false;
        nrelementsng.push(nrelements[y]);
        nrelementsngp.push(nrelements[y].placeholder); // This place is causing your script to crash
    }
}

The identifier nrelementsngp isn't declared.