Undefined index

[Sycohazza]

---

Hi ive been having an issue with this for a long time and i never found a fix from pure google so i have come to ask other human beings

 

heres my code:

 

<a href="index.php?page=news">News</a>

 

<?php
	define ('page', 'page');

	switch($_GET

---

)
	{


  			case "news":
    			 include "news.php"; break;

  			case "is-it-for-me":
     			include "for_me.php"; break;

	}
?>

 

This is what shows up when first entering index.php

 

Notice: Undefined index: page in C:\xampp\htdocs\template1\index.php on line 35

 

Thanks in advance,

Sycohazza

 

[noXstyle]

Hiya,

 

Are you certain that the 'page' index is passed to the index page? Only way I could reproduce the notification is not to pass the ?page= part to the url.

[Pikachu2000]

Why are you defining a constant 'page' as 'page'? If that was to clear up an "undefined constant" notice, all you would have needed to do was to quote the array index: $_GET['page'] . . .

[Sycohazza]

if i dont i get another warning

 

Notice: Use of undefined constant page - assumed 'page' in C:\xampp\htdocs\template1\index.php on line 35

 

 

with quotes i get

 

Notice: Undefined index: page in C:\xampp\htdocs\template1\index.php on line 35

[Pikachu2000]

. . . If that was to clear up an "undefined constant" notice, all you would have needed to do was to quote the array index: $_GET['page'] . . .

[Sycohazza]

sorry, i modded the other post, after i re read what you wrote b4 you reposted

[Sycohazza]

---

ok i fixed it my new code is

 

<?php
	if ($_GET){

	switch("$_GET

---

")
	{
  			case "news":
    			 include "news.php"; break;

  			case "is-it-for-me":
     			include "for_me.php"; break;

	}
	}
?>

 

 

Thanks for your help

[DavidAM]

In every configuration I have ever used, $_GET is always defined, so your IF statement will always be true. You need to test for a specific index in the $_GET array. I usually use isset:

 

if (isset($_GET['page'])){

/* NOTE: The double quotes around the entire $_GET[] expression are not needed;
	however, single (or double) quotes are needed around the index	*/
switch($_GET['page']) {  
	case "news":
		include "news.php"; break;

	case "is-it-for-me":
		include "for_me.php"; break;
	}
}

[Sycohazza]

ah isee what you mean, yea i did originally have the quote around page, i am new to learning php and i was quick to reply, the problem with if statement is that it doesnt load a default page, making the home page empty to begin with