iterating and looping in javascript

Question

i have this code using javascript. where i use for loop in evaluating the text box if it has content or none.

this is the code:

 function validateTable()
 {
 var rowCount = $('#mhTable >tbody >tr').length;
 //alert(rowCount);
 for(var ctr = 1; ctr <= rowCount; ctr++)
 {
    if( $('input[name^="process['+ctr+']"]').val() != '' )
    {
        continue;

    }
    else if( $('input[name^="process['+ctr+']"]').val() == '' )
    {
        messagealertManhourCheck("Error","Please fill out the form");
    }
}
SaveSavingsData();
}

my problem is when there are many textboxes and if one of the textbox do not have any content. it will still go in the function saveSavingsData();

but it must be error.

Luke · Accepted Answer · 2015-05-13 03:35:32Z

2

function validateTable() {
    var rowCount = $('#mhTable >tbody >tr').length,
        hasErrors = false;
    for(var ctr = 1; ctr <= rowCount; ctr++) {
        if( $('input[name^="process['+ctr+']"]').val() == '' ) {
            messagealertManhourCheck("Error","Please fill out the form");
            hasErrors = true;
        }
    }
    if (!hasErrors) {
        SaveSavingsData();
    }
}

answered May 13, 2015 at 3:35

Luke

1,7361 gold badge12 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Yangguang · Accepted Answer · 2015-05-13 03:41:36Z

in your code, it will always execute the SaveSavingsData(), if you want to stop execute if a input value is '',you can:

function validateTable()
{
    var rowCount = $('#mhTable >tbody >tr').length;
    //alert(rowCount);
    for(var ctr = 1; ctr <= rowCount; ctr++)
    {
      if( $('input[name^="process['+ctr+']"]').val() != '' )
      {
        continue;

      }
      else
      {
        messagealertManhourCheck("Error","Please fill out the form");
        return;
      }
    }
    SaveSavingsData();
}

renakre · Accepted Answer · 2015-05-13 03:36:53Z

You do not need the else statement with continue, and you can use an additional variable for checking if there are any empty textboxes:

function validateTable()
{
    var rowCount = $('#mhTable >tbody >tr').length;
    var isValid = true;
    //alert(rowCount);
    for(var ctr = 0; ctr < rowCount; ctr++)//any reason your loop was starting from 1 ?? I changed it to 0
    {
       if( $('input[name^="process['+ctr+']"]').val() == '' )
       {
           isValid = false;
           messagealertManhourCheck("Error","Please fill out the form");
       }
    }
    if(isValid)
         SaveSavingsData();
}

Collectives™ on Stack Overflow

iterating and looping in javascript

3 Answers 3

Comments

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Related