2010-11-16 12 views
0

J'ai écrit une fonction courte pour faire quelques vérifications d'erreur pour un formulaire et je suis bloqué à une partie du code où la clause finale 'elseif' dans le code ci-dessous continue d'être exécutée, même quand il y a texte dans la zone de texte ...if-check dans HTML allé mal

pourrait vous conseiller ... s'il vous plaît .. merci

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; 



} 

Répondre

4

on devrait utiliser == pour comparer, = à affecter. Vous affectez, ajoutez donc = pour comparer.

+0

merci. quand j'ajoute le '=' supplémentaire, la vérification ne se fait pas du tout .. – Roy

+0

Vous pouvez vérifier une chaîne vide, '" "' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' '' –

+0

okay, merci! – Roy

1

Vous utilisez = comme affectation, pas d'équivalence. Utilisez == ou ===.

1

Deux problèmes:

  1. Utilisez == à titre de comparaison, non =.
  2. La chaîne vide est "", et non " ". Notez l'espace supplémentaire.

Résultat:

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