Hi there

I was wondering if someone could help with the following problem...

I have some javascript that validates a form before it submits to a process page.

It goes something like this:

function processForm(form)
{
    var message = "Please:\n";
    var badFlag = false;

    if (form.elements['content_title'].value == "")
    {
        message = message + "Enter Title.\n";
        badFlag = true;
    }

    if ((!form.elements['content_image2'].value == "") && (form.elements['content_image2_align'].value == ""))
        {
        message += "Choose whether to align image 2 to the left or right of your text.\n";
        badFlag = true;
        }

    if(badFlag == true){
        alert(message);
        return false;
        }
    return true;
}

This all works fine BUT I have up to 8 possible fields in my form which run from content_image2 to content_image8 so I thought i'd put this section of code in a for loop and wrote the following...

    for (var i=2;i<=8;i=i++)
    {
        if ((!form.elements['content_image' + i].value == "") && (form.elements['content_image' + i + '_align'].value == ""))
        {
        message += "Choose whether to align image" + i + " to the left or right of your text.\n";
        badFlag = true;
        }
    }

BUT it crashes my browser!!

I haven't done much javascript programming so I don't know what i'm doing wrong so any help would be greatly appreciated. I guess I could do it the really long way but thought this would be a better solution!

Thanks in advance

Lucy xx

Hii Lucy!!

plz check your for loop for (var i=2;i<=8;i=i++)

Error--Infinite loop because you are using i=i++ ,that assign the value of i to 2 every time.

while it should be
CaseI Correct one----> for (var i=2;i<=8;i++)

or
CaseII Correct one---->
         j=1
         for (var i=2;i<=8;i=j++)
          {
            //statements
          }

As i think you are using caseI, so use (i++) rather than (i=i++)

Hope this will help you ;)


Cheers :)

vinod

Hi Lucky!!
Alternatively you can use
  for (i=2;i<=8;i=++i)

Hope this will help you

Cheers :)

vinod

Good catch.


Woody Z http://www.learntoprogramnow.com

Ah-ha! A rooky mistake!

Thanks for all your help with this - it now works perfectly - I just changed i=i++ to i++ and hey presto, jobs a good'un!

Cheers all

Lucy xx