[SOLVED] Need some simple help

[Lime]

Well I am making a number guessing game. It works just fine. But if the person doesn't pick a number and just clicks submit, it says that you got the number wrong. How can I make it say something like "Please go back and select a number." Thanks to all that can help. Here is my current script with the radiobuttons to select a number.

 

<form action="if.php" method="post">
Please select a number to guess.<br>
<input type="radio"
name="num" value="1">1   
<input type="radio"
name="num" value="2">2   
<input type="radio"
name="num" value="3">3   
<input type="radio"
name="num" value="4">4   
<input type="radio"
name="num" value="5">5<br>
<input type="submit" value="Guess that number!">
</form>

<?php
$cpu_number = rand(1, 5);

If (isset ($_POST['num'])) {
$number = $_POST['num'];
}

    If ($number == $cpu_number) { 
        echo "You guessed $number... Your number the same! You win!";

    } elseif ($number != $cpu_number) { 
        echo "You guessed $number... Your number does not match. Sorry, you lose. The computer's number was $cpu_number.";

   } elseif (noNumberSelected) {
echo "You haven't pressed a number. Please go back and select a number.";
}
?>

[bobsch]

If looks like you need a loop to do that. Do you know how to make a while loop? Basically, you can set up a while(x = false) loop, where x is a boolean variable. When they get the question right, set x to true, and it will exit the loop. Otherwise, it will continue to loop through the selections.

 

Err...I guess that would only work if the options were inside the PHP code, and you had a while loop around all that. I'm not sure, but someone else can probably help you out.

[br0ken]

<?php
$cpu_number 	= rand(1, 5);
$number 	= (int)$_POST['num'];

switch($number)
{
  case $cpu_number:
echo "You guessed $number... Your number is the same! You win!";
  break;
  case 0:
echo "You haven't selected a number. Please go back and select a number.";
  break;
  default:
echo "You guessed $number... Your number does not match. Sorry, you lose. The computer's number was $cpu_number.";
  break;
}

?>

[Lime]

Okay, thats one way of doing it. I'll try it.

 

P.S. Please forget the:

   } elseif (noNumberSelected) {
echo "You haven't pressed a number. Please go back and select a number.";

That was a sad attempt that I forgot to take out...

[jelly]

<?php

    $number = $_POST['num'];
    $back_link = $_SERVER['HTTP_REFERER'];

         if (empty($number) || (int)$number == 0)
             {
                  echo ('Please go back and select a number. ');
                  echo ('<a href="'.$back_link.'">Go back</a>');
             }
             else
             {
                    // your code
             }
?>

[xtopolis]

Or simply set one of them to checked to begin with.

 

<input type="radio" name="num" value="1" checked="yes" />

[Lime]

br0ken, with ur code it automatically says no number has been selected. =\

 

jelly, testing yours now.

[br0ken]

It should only say that if the $number is zero...

 

Anyways by the looks of it Jelly's code should work so just go for that

[kenrbnsn]

The following code has been tested and works fine:

<form action="" method="post">
Please select a number to guess.<br>
<?php
for ($i = 1; $i < 6; $i++)
echo '<input type="radio" name="num" value="' . $i . '">' . $i;
?>
<br><input type="submit" name="submit" value="Guess that number!">
</form>

<?php

If (isset($_POST['submit'])) {
$cpu_number = rand(1, 5);
$number = $_POST['num'];
switch ($number) {
           case $cpu_number:
        echo "You guessed $number... Your number the same! You win!";
                break;
   case 0:
                echo "You haven't pressed a number. Please go back and select a number.";
                break;
           default:
                echo "You guessed $number... Your number does not match. Sorry, you lose. The computer's number was $cpu_number.";
}
}
?>

 

Ken

[Lime]

Thanks Ken! Another challenge for you. Can you get it to hide the radiobuttons when it says the "You haven't pressed a number." thing?

[kenrbnsn]

Here's one way -- I modified your script a little to be more interesting...

 

<?php
session_start();
If (isset($_POST['submit'])) {
       $cpu_number = $_SESSION['cpu_guess'];
       unset($_SESSION['cpu_guess']);
       $number = $_POST['num'];
switch ($number) {
	case $cpu_number:
        echo "You guessed $number... Your number the same! You win!";
		  break;
   case 0:
			echo "You haven't pressed a number. Please go back and select a number.";
			break;
	default: 
	     echo "You guessed $number... Your number does not match. Sorry, you lose. The computer's number was $cpu_number.";
}
}
else {
    $guesses = rand(5,15);
    $_SESSION['cpu_guess'] = rand(1,$guesses);
?>
<form action="" method="post">
I'm thinking of a number between 1 and <?php echo $guesses ?> Can you guess what it is?.<br>
<?php
    for ($i = 1; $i <= $guesses; $i++)
echo '<input type="radio" name="num" value="' . $i . '">' . $i;
?>
<br><input type="submit" name="submit" value="Guess that number!">
</form>
<?php } ?>

 

Ken

[Lime]

Thanks Ken, its perfect! Thats really all I need. This thread is ready to be locked.

[kenrbnsn]

We don't lock threads like this. But you can mark it "Solved" by clicking the "Topic Solved" button at the bottom of the replies.

 

Ken

[Lime]

Kk, and that is a very cool feature.