Using radio buttons with PHP

[Brad]

Im trying to be able to select a radio button which in turn invokes the print statement to add an embed tag onto a web page. I'm not sure what is wrong with this. Here is the code
<form method="$PHP_SELF">

			<?
				if ($first == 1)
				{
					print "<embed src='im_taking_my_time.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
				}
				elseif ($second == 1)
				{
					print "<embed src='hawaiin_beach_baby.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
				}
				elseif ($third == 1)
				{
					print "<embed src='downtown.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
				}
				else
				{
					print "<br><b>Please select a tune to hear</b><br><br>";
				}

			?>

			I'm takin my time <input type="radio" name="tunes" value="first"><br>
			Hawaiin beach baby <input type="radio" name="tunes" value="second"><br>
			Downtown <input type="radio" name="tunes" value="third"><br>
			<br>

			<input type="submit" value="Go!">
			</form>

what is going wrong?!?!

[JMJimmy]

First lets start with your HTML:

I'm takin my time <input type="radio" name="tunes" value="first"><br>
Hawaiin beach baby <input type="radio" name="tunes" value="second"><br>
Downtown <input type="radio" name="tunes" value="third"><br>

this is fine - it'd be better for PHP if it was this though:

I'm takin my time <input type="radio" name="tunes" value="1"><br>
Hawaiin beach baby <input type="radio" name="tunes" value="2"><br>
Downtown <input type="radio" name="tunes" value="3"><br>

Now for your php - the problem was the variable is $_POST[tunes] that you'll want to check not $first, $second, $third. It's best, if you made the above html changes to do the following:

switch($_POST[tunes]){
  case 0:
    include(your html document here);
    break;
  case 1:
    print "<embed src='im_taking_my_time.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
    break;
  case 2:
    print "<embed src='hawaiin_beach_baby.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
    break;
  case 3:
    print "<embed src='downtown.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
    break;
  default:
    include(your html document here again);
    break;
}

This reduces duplication and allows for easy expandability.

JMJimmy

[Brad]

This doesn't work either!
switch($_POST[tunes]){
case 0:
include(your html document here);
break;
case 1:
print "<embed src='im_taking_my_time.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
break;
case 2:
print "<embed src='hawaiin_beach_baby.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
break;
case 3:
print "<embed src='downtown.mp3' console='smallconsole' autostart='true' loop='1' width='128' height='128'>";
break;
default:
include(your html document here again);
break;
}

Im getting about 6 different undefined use of constants errors. Is there a better way of getting this to work???

[aran]

perhaps you should try replaceing "your html document here" with the file name of the html document? 🙂

that's a start.

[JMJimmy]

lol - It wasn't cut & paste code - it was a suggestion & quick/dirty example on using switch and proper variables.