Please review code and identify error

Anon

PLEASE INDICATE THE FOLLOWING:
Profession:
<input type="checkbox" name="visitor_profession[]" value="lighting showroom">Lighting Showroom
<input type="checkbox" name="visitor_profession[]" value="designer">Designer
<input type="checkbox" name="visitor_profession[]" value="architect">Architect
<input type="checkbox" name="visitor_profession[]" value="electrical supply">Electrical Supply Wholesaler
<input type="checkbox" name="visitor_profession[]" value="purchasing agent">Purchasing Agent
<input type="checkbox" name="visitor_profession[]" value="sales representative">Sales Representative
<input type="checkbox" name="visitor_profession[]" value="other">Other
<?php
reset($visitor_profession);
while( list($visitor_id, $value) = each($visitor_profession) )
{
if($value == "on")
{
db_query("INSERT visitor_profession ".
"(visitor_id, value) ".
"VALUES ($visitor_id, $value)");

							}
				}

				?>						
				<b>Please Specify if Your Profession is not Listed</b>
				<p><font face="arial" size="2">PROFESSION:<input type="text" name="visitor_profession_specified" size="48">
                <p><font face="arial" color="red" size="2"><b>REQUESTS CAN
                NOT BE PROCESSED WITHOUT TOTALLY COMPLETED FORM</b></font>

                <p><input type="submit" name="submit" value="Submit" value="Reset">
          </table>



          </FORM>
		  <?php
		  }
		  ?>

Any help would be greatly appreciated,
Debra

Anon

What the heck is if ($value = "on")... while this makes perfect logic sense, how does the computer know what "on" is? All the computer knows is in

$visitor_profession[0] = "lighting showroom";
etc... etc...

That is all it knows. So when you check and see if
$visitor_profession[0] == "on";

It says "Nope the string 'lighting showroom' is not equal to the string 'on'."

I think that is right... I never really use list and each (more of an iterative i,j,k guy. 🙂 ) Try this instead

for($i = 0; $i > sizeof($visitor_profession); $i++) {

db_query("INSERT visitor_profession ".
"(visitor_id, $visitor_profession[$i]) ". "VALUES ($visitor_id, $visitor_profession[$i])");

}

Unless I am missing something. I think that is what you are tring to do?!?