La validation 'Check for Letter' ne fonctionne pas

Question

J'exécute certaines fonctions pour vérifier que le texte saisi par l'utilisateur dans un formulaire contient au moins un chiffre et une lettre. Cela ne semble pas fonctionner. Étant donné que les deux fonctions sont presque identiques, je vais poster un d'entre eux:

function hasALetter(textField){ 
    //returns true or false based on whether or not the text field has at least one letter inside it  
    console.log("Checking for letters..."); 

    var hasLetter = false; 

    for(var i=0, checkLength=textField.length; i<checkLength; i++){ 
     var letter = textField.substr(i,1); 
     console.log("letter = " + letter); 
     if(isNan(letter) == false){ 
      hasLetter = false; 
     } 
    } 
    if(hasLetter == true){ 
     return true; 
    } 
} 

Le journal (« lettre = » + lettre) montre jamais dans ma console. Il me manque probablement quelque chose de stupide, mais cela ne semble pas aller jusqu'au bout de la fonction.

Pour référence, voici comment j'appelle les fonctions:

if(pwd.value.length > 9){ 

      var pwdLetter = hasALetter(pwd); 
      var pwdNumber = hasADigit(pwd); 

      if(pwdLetter==true){ 
       if(pwdNumber==true){ 

Oui, je suis conscient qu'il est très salissant, mais je suis encore à apprendre. Je suis sûr qu'il y a des façons plus avancées/plus propres de faire cette validation, mais pour les besoins de ma scolarité, je le fais comme ça pour l'instant.

Answer 1

La meilleure façon de vérifier si une chaîne a une lettre est l'expression régulière:

function hasLetter(str){ 
    // check for characters between a and z 
    // i flag makes it case insensitive 
    return /[a-z]/i.test(str); 
} 

Si vous voulez boucle à travers la chaîne et que vous souhaitez également utiliser la fonction isNaN, ce serait faire:

function hasLetter(str){ 

    // loop through every character 
    for(var i=0; i<str.length; i++){ 

    // check if the i-th character is not a number 
    if(isNaN(str[i])){ 

     // if so, return true 
     return true; 
    } 
    } 

    // if the loop has finished and no letters have been found, return false 
    return false; 
} 

Mais je ne recommanderais pas cette méthode, car isNaN contrôles wh Le premier argument est un nombre ou non. Tout d'abord, tous les caractères qui ne sont pas des chiffres ne sont pas des lettres. Deuxièmement, l'argument que vous passez est une chaîne (str[i] renvoie un caractère de type chaîne, même s'il s'agit d'un chiffre)

Answer 2

Il est « isNaN », pas « isNan » ... ... aussi, si elle est pas un certain nombre, cela ne signifie pas nécessairement que ce n'est pas une lettre; en fait la logique semble en arrière, ou à l'envers.

Si "isNaN" renvoie true, ce n'est pas un nombre. Est-ce une lettre? Eh bien, vous devez vérifier. Si "isNaN" renvoie false, alors tout ce que vous savez c'est que c'est l'un des caractères "0" à "9".

L'ensemble est beaucoup mieux fait avec une expression régulière, mais si cela est devoirs il se peut que cette approche ne peut pas être utilisée. Vous pouvez vérifier si un caractère est une lettre avec quelque chose comme:

function isLetter(c) { 
    c = c.toUpperCase(); 
    return c >= "A" && c <= "Z"; 
} 

Answer 3

Je voudrais utiliser des expressions régulières, voici quelques fonctions rapides.

function hasALetter(text) 
    { 
     var regex = new RegExp("[a-zA-Z]"); 
     if(text.match(regex)) 
      return true; 
     else 
      return false; 
    } 

    function hasANumber(text) 
    { 
     var regex = new RegExp("[0-9]"); 
     if(text.match(regex)) 
      return true; 
     else 
      return false; 
    }