單選按鈕驗證導致驗證失敗的其餘部分

Question

無線驗證有效，但其餘的驗證無效。我做錯了什麼？

function validateRadio(radios) { 
    for (i = 0; i < radios.length; ++i) { 
     if (radios[i].checked) return true; 
    } 
    return false; 
} 

function validateForm() { 
    if (validateRadio(document.forms["pancettaForm"]["updateShip"])) { 
     return true; 
    } else { 
     alert("Please tell us how you would like to update your order."); 
     return false; 
    } 
} 

var x = document.forms["pancettaForm"]["order-number"].value; 
if (x == null || x == "") { 
    alert("Please provide your order number."); 
    return false; 
} 
var x = document.forms["pancettaForm"]["full-name"].value; 
if (x == null || x == "") { 
    alert("Please provide your full name, or the recipients name if you are updating shipping information."); 
    return false; 
} 
var x = document.forms["pancettaForm"]["phone"].value; 
if (x == null || x == "") { 
    alert("Please provide a phone number in case of delivery questions."); 
    return false; 
} 
var display = document.getElementById('address').style.display; 
if (display == 'block') { 
    var x = document.forms["pancettaForm"]["address"].value; 
    if (x == null || x == "") { 
     alert("Please provide your address."); 
     return false; 
    } 
} 
var display = document.getElementById('city').style.display; 
if (display == 'block') { 
    var x = document.forms["pancettaForm"]["city"].value; 
    if (x == null || x == "") { 
     alert("Please provide your city."); 
     return false; 
    } 
} 
var display = document.getElementById('state').style.display; 
if (display == 'block') { 
    if (document.pancettaForm.state.value == "- Select State -") { 
     alert("Please provide your state."); 
     return false; 
    } 
} 
var display = document.getElementById('zip').style.display; 
if (display == 'block') { 
    var x = document.forms["pancettaForm"]["zip"].value; 
    if (x == null || x == "") { 
     alert("Please provide your zip code."); 
     return false; 
    } 
} 
var display = document.getElementById('newShipDate').style.display; 
if (display == 'block') { 
    if (document.pancettaForm.state.value == "- Select Date -") { 
     alert("Please choose your new shipping date."); 
     return false; 
    } 
} 

Answer 1

只是扭轉測試，因此您不必返回

if(!validateRadio (document.forms["pancettaForm"]["updateShip"])) 
      { 
       alert("Please tell us how you would like to update your order."); 
       return false; 
      } 

    // continue 

你有射頻的測試後的尾架這樣的腳本的其餘部分只是漂浮在網絡空間

最後也只返回一次，並且沒有var x多次，並且在如何訪問表單元素方面一致，並且我還修復了正在測試狀態的日期測試

function validateForm() { 
    var x,display,form = document.forms["pancettaForm"]; 
    if (!validateRadio(form["updateShip"])) { 
    alert("Please tell us how you would like to update your order."); 
    return false; 
    } 

    x = form["order-number"].value; 
    if (x == null || x == "") { 
    alert("Please provide your order number."); 
    return false; 
    } 
    x = form["full-name"].value; 
    if (x == null || x == "") { 
    alert("Please provide your full name, or the recipients name if you are updating shipping information."); 
    return false; 
    } 
    x = form["phone"].value; 
    if (x == null || x == "") { 
    alert("Please provide a phone number in case of delivery questions."); 
    return false; 
    } 
    display = form["address"].style.display; 
    if (display == 'block') { 
    x = form["address"].value; 
    if (x == null || x == "") { 
     alert("Please provide your address."); 
     return false; 
    } 
    } 
    display = form["city"].style.display; 
    if (display == 'block') { 
    x = form["city"].value; 
    if (x == null || x == "") { 
     alert("Please provide your city."); 
     return false; 
    } 
    } 
    display = form['state'].style.display; 
    if (display == 'block') { 
    x = form['state'].value; 
    if (x == "- Select State -") { 
     alert("Please provide your state."); 
     return false; 
    } 
    } 
    display = form['zip'].style.display; 
    if (display == 'block') { 
    x = form["zip"].value; 
    if (x == null || x == "") { 
     alert("Please provide your zip code."); 
     return false; 
    } 
    } 
    display = form["newShipDate"].style.display; 
    if (display == 'block') { 
    x = form["newShipDate"].value; 
    if (x.value == "- Select Date -") { 
     alert("Please choose your new shipping date."); 
     return false; 
    } 
    } 

    return true; 
}