Undefined index problem

andrewlondon · #1 Aug 30th, 2007, 09:52

Hi, i'm really stuck and would really apreciate some help...

I've made a simple page that will redirect a user to their desired page if they agree to a terms and conditions.... the thing is, if a user doesn't click the agree box, the page comes up but certain elements are removed from the page... the background and text formatting being the main bits....

I've viewed source on the page that loads once a user clicks submit without checking the box and it gives out the following error at the top of the page....

Notice: Undefined index: terms_ok in C:\Program Files\EasyPHP 2.0b1\www\PopCap\game_redirect.php on line 16 

If you do a refresh on this page that hasn't loaded correctly it does load the page correctly.....

Please please please help me with this guys, i'll buy whoever helps a beer!

Thanks,
Andrew

PHP: Select all


<?php 
 
//Pref Setup 
$terms_error = ''; 
$selected_game = $_GET['game']; 
$file_extension = '.php'; 
$go_where = $selected_game.$file_extension; 
 
$do_we_agree = 'no'; 
 
//What do we do once the user hits submit? 
if (isset($_POST['submitted'])) { 
    $valid = 1; 
 
if ($_POST['terms_ok']=$do_we_agree) { 
      $valid = 0; 
      $terms_error = 'Please try again.'; 
    } 
 
// If user agrees send them to their game 
            if ( $valid == 1 ) { 
                 header('Location:' .$go_where); 
                exit; 
            } 
} 
?>

HTML: Select all

<!-- Elements Begin --> 
<?php include "common_elements.html"; ?> 
 
<!-- Terms & Conditions Begin -->        
<div class="game_holder">Terms & Conditions<br><br> 
 
<!-- Terms Checkbox Begin --> 
<form enctype="multipart/form-data" method="post" action=""> 
 
<!-- Terms Text Begin --> 
<iframe width="450" height="332" src="terms.html"></iframe> 
<!-- Terms Text End --> 
 
<br> 
 
<span style="color:#666666";>I Agree to the Terms & Conditions:</span> 
 
<input type="checkbox" name="terms_ok" value="agree"></input> 
 
<span style="font-size:8pt; color:red;"><?php echo $terms_error; ?></span> 
<br> 
 
<input type="submit" name="submitted" value="Lets Go!" ></input> 
 
 
</form> 
<!-- Terms Checkbox End --> 
 
</div> 
<!-- Terms & Conditions End --> 
 
 
<!-- Ads Begin -->    
<!-- Sky --> 
   <div class="skyscraper">Sky Ad</div> 
 
<!-- Leaderboard --> 
<div class="banner">Banner Ad<div> 
<!-- Ads End -->

nate2099 · #2 Aug 30th, 2007, 10:16

As far as I understand it, if the user dosen't check that box, no value will be sent to the next page.

therefore you should say:

Code: Select all

 
if (checkboxname != "") // if it has been checked
{
    do this
}
else
{
     do this // it will only do this if the box has not  been checked
}

hope this helps.

nate2099 · #3 Aug 30th, 2007, 10:17

also, shouldn't this:

Code: Select all

 
if ($_POST['terms_ok']=$do_we_agree)

be:

Code: Select all

 
if ($_POST['terms_ok']==$do_we_agree)

could be wrong of course..

andrewlondon · #4 Aug 30th, 2007, 10:32

I tried using an else and it still isn't working, i don't think that's the problem.

My original code.... The script isn't run unless somebody presses the submit button, if they do and haven't ticked the box then valid is set to 0 and the terms_error variable is set and displayed when the page is reloaded. If the box is ticked then it is redirecting with the header function.

Using an else is pretty much the same thing as i've already been doing i think..... It's really odd because when i refresh the page the background elements return and it is displayed correctly.

PHP: Select all


<?php 
 
//Pref Setup 
$terms_error = ''; 
$selected_game = $_GET['game']; 
$file_extension = '.php'; 
$go_where = $selected_game.$file_extension; 
 
$do_we_agree = 'no'; 
 
//What do we do once the user hits submit? 
if (isset($_POST['submitted'])) { 
    $valid = 1; 
 
if ($_POST['terms_ok']=$do_we_agree) { 
      $valid = 0; 
      $terms_error = 'Please try again.'; 
    } 
 
// If user agrees send them to their game 
            if ( $valid == 1 ) { 
                 header('Location:' .$go_where); 
                exit; 
            } 
}

nate2099 · #5 Aug 30th, 2007, 10:38

hmm ok, but wont this statement:

Code: Select all

 
($_POST['terms_ok']=$do_we_agree)

always be true because of the = instead of ==

also if the user doesn't check the box, $_POST['terms_ok'] will equal ""

wont it??

andrewlondon · #6 Aug 30th, 2007, 10:46

I've changed the code a little. You say that if a user doesn't tick the box then the $_POST['terms_ok'] variable will containt no data, all that will happen then is the $terms_error variable gets set and displayed on the following page. I don't see how the contents of the $_POST_['terms_ok'] variable stops certain elements of the page from loading but will appear if you do a refresh....

<?php
//Pref Setup
$terms_error = '';
$selected_game = $_GET['game'];
$file_extension = '.php';
$go_where = $selected_game.$file_extension;
$do_we_agree = 'yes';
//What do we do once the user hits submit?
if (isset($_POST['submitted'])) {
$valid = 1;
$terms = $_POST['terms_ok'];
if ( $terms!==$do_we_agree) {
$valid = 0;
$terms_error = 'You must select \'Yes\' to continue.';
}

else {
// If user agrees send them to their game

header('Location:' .$go_where);
exit;
}
}
?>

andrewlondon · #7 Aug 30th, 2007, 10:50

Very strange, when i put the files on my server rather than on my desktop machine it loads and works no problem....

:-(

nate2099 · #8 Aug 30th, 2007, 10:56

lol, so long as it works I guess..

When the user ticks the box, the $terms_ok will equal whatever you put in the value field for the checkbox in the form. It won't equal yes or no..

nate2099 · #9 Aug 30th, 2007, 11:01

sry if u know this already, but doesn't equal is !=
equals is ==

!== is valid but it means something else, i forget what.

you could also say if (!$terms) means if $terms is not set, ie if they didn't check the box.

andrewlondon · #10 Aug 30th, 2007, 11:12

Quote:

Originally Posted by nate2099

lol, so long as it works I guess..

When the user ticks the box, the $terms_ok will equal whatever you put in the value field for the checkbox in the form. It won't equal yes or no..

if the box is ticked the $_POST['terms_ok'] will equal 'yes' because that is what i set its value to.

I've set a variable at the top of the code that also equals yes.... the final bit of code i put up is saying that if $_POST['terms_ok'] doesn't equal $do_we_agree then set the $valid to 0, if it does match then $valid equals 1 so redirect.....

I didn't know that !== is different to != so i'll look into it....

Thanks.

nate2099 · #11 Aug 30th, 2007, 12:20

Quote:

if the box is ticked the $_POST['terms_ok'] will equal 'yes' because that is what i set its value to.

not according to the html in the file above...

anyways, glad it's working!!

andrewlondon · #12 Aug 31st, 2007, 10:12

Ahhh, well in the code i posted above it will be set to "agree", i changed this and set the value to yes which is why i was saying it will be set to yes.... Oops! ;D

Rakuli · #13 Sep 2nd, 2007, 12:19

I think you have it sorted out here but must point out that !== & != although essentially doing a match comparison are quite different.

When PHP compares two values using != or ==, it will do variable type conversion. So, the integer 2 and the string '2' will match becuase PHP will convert the string to an integer based on its value. Similarly when checking say if($var) it is comparing againts a boolean value and whatever the variable contains will be converted to a boolean so an integer of 0 will translate to fale.

When you use === or !== PHP doesn't do this type conversion for you so in the example above 2 will not equal '2' becuase they are different variable types and 0 wil not equal false for the same reason.

This is an important difference when working with forms as all values are strings so you have to be careful you aren't using === to compare againts integers or booleans from a $_REQUEST value.

Hope that makes sense..

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Undefined Index error with DB	Sphinx111	PHP Forum	3	Apr 21st, 2008 20:37
design & code problem - movie index strategy and loading movies	betso	Web Page Design	2	Sep 11th, 2006 20:35
Undefined index using $GET_	LostProphet	PHP Forum	7	Aug 22nd, 2006 11:15
Index Server search results problem	starrbuck	Classic ASP	0	Feb 22nd, 2006 17:20

		Webforumz.com > Main Forums > Program Your Website > PHP Forum
Undefined index problem