2010-11-16 3 views
0

Я написал короткую функцию для выполнения некоторых проверок ошибок для формы и застрял на части кода, где окончательное предложение elseif в коде ниже продолжает выполняться, даже если есть текст в текстовом поле ...if-check in HTML gone awry

могли бы вы посоветовать ... спасибо ..

function errorCheck(){ 

    if(!isInteger(document.getElementById("appleQty").value)){ 
    alert('Please key in an integer in the Apple Quantity text box.'); 
    document.getElementById("appleQty").value=""; 
    document.getElementById("appleQty").focus(); 
    return false; 

    } 
    else if(!isInteger(document.getElementById("orangeQty").value)){ 
    alert('Please key in an integer in the Orange Quantity text box.'); 
    document.getElementById("orangeQty").value=""; 
    document.getElementById("orangeQty").focus(); 
    return false; 
    } 
    else if(!isInteger(document.getElementById("bananaQty").value)){ 
    alert('Please key in an integer in the Banana Quantity text box.'); 
    document.getElementById("bananaQty").value=""; 
    document.getElementById("bananaQty").focus(); 
    return false; 

    } 
    else if(document.getElementById("user").value = " "){ /!-Problem, keeps getting repeated--> 
    document.getElementById("user").focus(); 
    alert('Please key in your name.'); 

    return false; 


    } 
    return true; 



} 

ответ

4

следует использовать == для сравнения, = назначить. Вы назначаете, поэтому добавьте дополнительный = для сравнения.

+0

спасибо. когда я добавляю дополнительный '=', проверка вообще не выполняется. – Roy

+0

Возможно, вы захотите проверить пустую строку '' '' вместо '' '', которая представляет собой одно пространство. –

+0

хорошо получилось, спасибо! – Roy

1

Вы используете = как назначение, а не эквивалентность. Используйте == или ===.

1

Две проблемы:

  1. Использование == для сравнения, не =.
  2. Пустая строка: "", а не " ". Обратите внимание на дополнительное пространство.

Результат:

else if (document.getElementById("user").value == "") {