2010-05-29 9 views
1

J'ai ce tableau (JQuery) où j'ajoute tout ce qu'il ressemble à des contrôles de ma forme,:Vérification des valeurs dans ce tableau, JQuery

var  name  = $("#name"), 
     surname = $("#surname"), 
     address = $("#address"), 
     phone  = $("#phone"), 
     photo  = $("#photo"), 
     grade  = $("#grade"), 
     profession = $("#profession"), 
     email  = $('#email'), 
     title  = $('#title'), 

        allFields = $([]).add(name) 
             .add(surname) 
             .add(address) 
             .add(phone) 
             .add(photo) 
             .add(grade) 
             .add(profession) 
             .add(email) 
             .add(title) 
             .add(grade); 

Je veux vérifier les valeurs de chaque élément dans la « Tous champs » tableau avec

function checkingFieldsArentEmpty(){ 

       for (var f in allFields){ 
        if(f.val() === null) 
         //if any val is null just return false 
         return false; 
       } 
       return true; 
      } 

J'ai besoin d'idées afin d'améliorer la dernière fonction. Merci.

+0

Quel aspect voulez-vous améliorer? Votre var valide n'est pas nécessaire, renvoyez juste true à la fin. – patrickmcgraw

+1

Est-ce que 'f' n'est pas déjà un objet jQuery? 'allFields [f] .val()' semble faux, ne devrait-il pas être simplement 'f.val()'? –

+0

Oui, vous avez raison, c'est faux. – Felix

Répondre

1

Vous pouvez remplacer cette fonction avec:

function checkingFieldsArentEmpty(){ 
    return !(allFields.filter(":input[value]").length != allFields.filter(":input").length); 
} 
+0

J'ai aussi des valeurs