Jump to content

[SOLVED] Seemingly simple code not working?


zerbon

Recommended Posts

I made this simple example below that I thought should work; I'm not sure what I am missing?

 

Unless I am mistaken, when you click on one of the list items the paragraph should update to "You clicked [whatever you clicked]"?  I'm sure it is a simple mistake somewhere, but I cannot seem to see it?!  Please help.

 

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">

<head>

<title>PHP</title>

<meta http-equiv="content-type" content="text/html;charset=utf-8" />

</head>

<body>

<ul>

<li><a href="?fruit=apples">Apples</a></li>

<li><a href="?fruit=cabbages">Cabbages</a></li>

<li><a href="?fruit=rhubarb">Rhubarb</a></li>

</ul>

 

<p>You clicked <?php echo $fruit; ?></p>

</body>

</html>

 

Link to comment
Share on other sites

Your code would only work if register_globals is on, which is never a good idea. Instead you should be using the $_GET superglobal. So it would be $_GET['fruit']. Additionally you should be using isset() to confirm that the fruit index is defined within the $_GET superglobal otherwise (depending on your error reporting level) a warning will be thrown. Something like this would work:

 

if(isset($_GET['fruit']))
{
     echo '<p>You clicked ' . $_GET['fruit'] . '</p>';
}

Link to comment
Share on other sites

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.