I have a div which contain lots of input[type=text] fields. I want to put a check that user must have fill all the fields otherwise show error message. For this i tried :

if ($('div').children("input[type=text]").attr("value") == '') 
 flag = false; 
else 
 flag = true;

But this will not check all the text field, it only checks the first-child . How can i check all the text field ?

I have a div which contain lots of input[type=text] fields. I want to put a check that user must have fill all the fields otherwise show error message. For this i tried :

if ($('div').children("input[type=text]").attr("value") == '') 
 flag = false; 
else 
 flag = true;

But this will not check all the text field, it only checks the first-child . How can i check all the text field ?

• javascript
• jquery

Share Follow asked Jul 5, 2012 at 9:49 Tom RiderTom Rider 2,81577 gold badges4444 silver badges6666 bronze badges

Add a ment  | 

3 Answers 3

Sorted by: Reset to default 3

Try to use find() instead

 if ($('div').find("input[type=text]").val() == '')

To check over all input items you need to iterate over them using each().

Correct code would be:

$(function() {
    $('input:submit').click(function(){
        var flag = true;
        $('div').find("input[type=text]").each(function(){
            if($(this).val() === '') flag = false
        });

        alert(flag);
    });
});​

See demo on jsFiddle.

This code return true if all input[type=text] fields inside div are filled and false if at least one is empty.

The standard way

if(!$('div').find('#idOfTextField').val()){
 //empty
}

if($("div").find('input:text[value=""]').length > 0)
{
 alert("error");
 return false;
}