javascript validation du formulaire

Question

Je suis en train d'écrire une validation de formulaire javascript de PreSaveAction() comme ci-dessous:

function PreSaveAction() { 
    var number1 = getTagFromIdentifierAndTitle("INPUT","TextField","Quantity"); 
    //var ddl1 = getTagFromIdentifierAndTitle("select","DropDownChoice","Is Product Completed"); 
    var myvar = getTagFromIdentifierAndTitle("TextArea","TextField","Description"); 
    //if (dropdown1.value == "Yes" && date1.value == "") 
    if (number1.value == "" || myvar.value === "") 
    { 
     alert("Please fill all the Required Fields"); 
     return false; // Cancel the item save process 
    } 

    if(confirm("Do you want to continue with this information?")==true) 
    { 

    alert("Successfully Submitted"); 
    return true; // OK to proceed with the save item 

    } 

    return false; 

} 

et le script pour la fonction getTagFromIdentifierAndTitle

function getTagFromIdentifierAndTitle(tagName, identifier, title) { 
    var len = identifier.length; 
    var tags = document.getElementsByTagName(tagName); 

    for (var i=0; i < tags.length; i++) { 
     var tempString = tags[i].id; 
     if (tags[i].title == title && (identifier == "" || tempString.indexOf(identifier) == tempString.length - len)) { 
      return tags[i]; 
     } 
    } 
    return null; 
} 

ce que j'ai observé ci-dessus est il ne valide pas les deux champs ensemble où j'utilise et opérateur.

que je fais quelque chose de mal dans le script

S'il vous plaît aidez-moi à ce sujet.

Merci à l'avance

Answer 1

si vous voulez que le test échoue si soit des valeurs sont vides, vous voulez la propriété OU: ||

Ceci s'assurera qu'ils sont tous deux non vides.

Answer 2

var number1 = getTagFromIdentifierAndTitle("INPUT","TextField","Quantity"); (TextBox) 

var Desc1 = getTagFromIdentifierAndTitle("TextArea","TextField","Description"); (MultiLine) 


if (number1.value == "" || Desc1.value == "") //Changed here 
    { 
     alert("Please fill all the Required Fields"); 
     return false; // Cancel the item save process 
    } 
return true; 

Answer 3

if (number1.value == "" || Desc1.value == "") 
    { 
     alert("Please fill all the Required Fields"); 
     return false; // Cancel the item save process 
    } 
return true; 

Vous voulez utiliser un ou ici. Parce que vous voulez être ensemble les deux champs, non?

if (number1.value != "" && Desc1.value != "") 

Est également possible.

Pour valider le bon numéro:

number1.length && !isNaN(Number(number1.value)); //Number("") returns 0, not NaN 

Answer 4

Il est souvent préférable d'utiliser « === » au lieu de « == » puisque cela inclut une vérification du type aussi bien.

Answer 5

Vous avez plus d'un élément avec le même ID, ce qui provoque uniquement le retour du premier élément par votre fonction.

Demandez ID unique pour chaque élément puis changer la deuxième ligne à quelque chose comme:

var myvar = getTagFromIdentifierAndTitle("TextArea", "txtDescription", "Description"); 

En supposant que vous avez maintenant id="txtDescription" à l'élément textarea.

Alternativement, vous pouvez affecter la même classe à plusieurs éléments, mais vous ne voyez pas pourquoi vous ne pouvez pas donner d'ID différent.