Jragon Posted July 20, 2010 Share Posted July 20, 2010 In this little script all i am getting is the headings <!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"> <head> <title>What would you like to be said over and over agien?</title> </head> <body> <h1>What would you like to be said over and over agien?</h1> <h6>By Rory Anderson!</h6> <?php if(isset($_POST['submit'])){ for($i=0; $i >= $times; $i++){ echo $text; echo '<br />'; } }else{ echo <<<ENDHTML <form method="post" action=""> <h2>What would you like to be said:</h2> <br /> <input type="text" name="text"/> <br /> <h2>How many times would you like it to be said:</h2> <br /> <input type="text" name="times"/> <br /> <input type="submit" value="say!" name="submit"/> </form> ENDHTML; } ?> </body> </html> when i sumit it nothing happens exsept the headings Quote Link to comment Share on other sites More sharing options...
Alex Posted July 20, 2010 Share Posted July 20, 2010 Your script is relying on register_globals. You should instead be accessing the data from the $_POST array. So $times becomes $_POST['times']. Additionally you have a flaw with your logic, the comparison operator you should be using is < not >=. if(isset($_POST['submit'])){ for($i=0; $i < $_POST['times']; $i++){ echo $_POST['text']; echo '<br />'; } Quote Link to comment Share on other sites More sharing options...
Jragon Posted July 20, 2010 Author Share Posted July 20, 2010 I have changed it to: if(isset($_POST['submit'])){ for($i=0; $i >= $_POST['times']; $i++){ echo $_POST['text']; echo '<br />'; } But I'm still getting nothing Quote Link to comment Share on other sites More sharing options...
Alex Posted July 20, 2010 Share Posted July 20, 2010 Additionally you have a flaw with your logic, the comparison operator you should be using is < not >=. That's referring to this in the for loop: $i >= $_POST['times'] Quote Link to comment Share on other sites More sharing options...
Jragon Posted July 20, 2010 Author Share Posted July 20, 2010 I saw that but it did not make much sense. Its working now. Thakns Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.