[RESOLVED] Issue nesting to If loops

[dmacman1962]

Hi all,

I have two radio buttons that the user can select (yes and no). The issue I am having is if they choose the 'Yes' button, I then need to ensure the fill out the field labelled "HowMany'.

Here is what I tried, but it does not work:

if ($ColleaguesUse >0) {
	and(strlen($HowMany) >0 and strlen($HowMany) <1)
		{
		Back();
		die("<p align='center'><font face='Arial' size='3' color='#FF0000'>You did not specify how many of your colleagues use ITCM videos.<br><br>Please use the 'Back' button and check Yes.</font></p>");
		}
}

and I tried this:

if ($ColleaguesUse >0) {
	and if (strlen($HowMany) >0 and strlen($HowMany) <1)
		{
		Back();
		die("<p align='center'><font face='Arial' size='3' color='#FF0000'>You did not specify how many of your colleagues use ITCM videos.<br><br>Please use the 'Back' button and check Yes.</font></p>");
		}
}

Anyone have any direction with this issue?

Thanks,

Don

[SargeZT]

A more elegant way to do it would be:

if (!empty($ColleaguesUse) AND empty($HowMany)) 
        { 
        Back(); 
        die("<p align='center'><font face='Arial' size='3' color='#FF0000'>You did not specify how many of your colleagues use ITCM videos.<br><br>Please use the 'Back' button and check Yes.</font></p>"); 
        } 

But, if you could tell us what the Radio Button's name is in the form, and just more information in general, we could probably do a bit more for you.

[Installer]

You should look at your string length comparisons. The code will evaluate true only if the length of $howmany is greater than zero and is less than one (i.e. more than none and fewer than any).

You also have too many curly braces. Remove the one after the $colleaguesuse line and one of the ending ones.

[dmacman1962]

SargeZT, your solution worked perfectly!

Thanks!!!

To answer your questions;

1) The Radio buttons are called 'ColleaguesUse' and 'Yes = 1' and 'No = 0' .

2) The field to be filled out for the 'Yes' radio button is called 'HowMany' .

But I do not think your code needs modified, unless you see something I am not.

Thanks for the quick responses!

Don

[SargeZT]

Good deal. I don't see any problems, I was just making sure that 0 = no.

Good luck!

[Installer]

Just one caveat: if the user somehow puts a non-numeric (even a space) into "$howMany" then the if clause will fail. Maybe you're already checking for that elsewhere, but if not, you should. Perhaps by using--gulp--"ctype_digit()".

[SargeZT]

Yeah, I suck at validating forms.

[dmacman1962]

Thanks all,

I am still 'refining' the form. Yes, I hate 'validating' forms.

I wish everyone "filtered it thru a brain cell' before filling out forms.

🙂

Don