HOME       >>       Programming

Shout Box Help


KansukeKojima

Alright... so this is really my first time using databases at all. Anyways, I need some help.

http://forums.xisto.com/no_longer_exists/

I used a tutorial from spoono.com (http://forums.xisto.com/no_longer_exists/) to create this... everything seems to work, except that it won't show what is posted in the shoutbox.....

my code is below.... and since I hardly know anything about databases, I'm sure you'll find an error quickly.

Thanks!

<?php//host name passwordmysql_connect("localhost","name","password");//select databasemysql_select_db("shoutboxdatabase");if($submit){//date function$time=date ("h:ia d/j/y");//insert into shoutbox table$result=MYSQL_QUERY("INSERT INTO shoutbox (id,name,message,time)"."VALUES ('NULL','$name', '$message','$time')");}//last five messages$result = mysql_query("select * from shoutbox order by id desc limit 5");//while loopwhile($r=mysql_fetch_array($result)){//variables from table$time=$r["time"];$id=$r["id"];$message=$r["message"];$name=$r["name"];?><?php $time ?><br /><?php echo $name ?><br /><?php echo $message ?><br /><?php } ?><form action="<?php echo $php_self ?>" method="post"><INPUT TYPE='TEXT' value='Name' NAME='name' SIZE=30 maxlength='100'><br /><INPUT TYPE='TEXT' value='Message' NAME='message' size=30 maxlength='100'><input type="submit" name="submit" value="Submit"></form>


shadowx

The first thing i notice is this line:

mysql_select_db("shoutboxdatabase");

Is this how it is in the code or have you changed the DB name to hide it?

The reason i ask is that with T17 you need to prefix the DB name with your hosting username EG: If my cpanel login name was "shadowx" (which it isnt by the way...) My DB name would be "shadowx_shoutboxdatabase"

I dont have time to look through the code so if this isnt the problem someone else will have to help!

KansukeKojima

yeah... just never mind all the db connect stuff... I changed it to hide the info....


sonesay

Either check through cpanel if entries are being inserted into the database using myphpadmin or what ever its called. Then if entries are being inserted there check your actually receiving results from the query by using

$found = mysql_num_rows($result);echo $found;

if your getting the correct return entries then we can assume its correct and the error is on the loop where you try and output the results. How you have done it I never do

while($r=mysql_fetch_array($result)){//variables from table$time=$r["time"];$id=$r["id"];$message=$r["message"];$name=$r["name"];?><?php $time ?><br /><?php echo $name ?><br /><?php echo $message ?><br /><?php } ?>

while($r=mysql_fetch_array($result)){//variables from table$time=$r["time"];$id=$r["id"];$message=$r["message"];$name=$r["name"]; echo $time."<br />"; echo $name."<br />"; echo $message."<br />";} ?>

I would just keep the while loop in side the same php tags. I dont know what the effect of it doing your way does but obviously it did not pick up the error when you tried to just do "$time" with no echo . just try it this way and see if it works.


jlhaslip

Can you get results from the Database using this Query in the phpMyAdmin using the SQL function?

select * from shoutbox order by id desc limit 5"

Also, add this after the query in your script to see what the script is getting for results:
print_r( $result );

Of course, set error reporting at the top of the script to see if the script is tossing an error:
ini_set("display_errors", 1);error_reporting(E_ALL);


KansukeKojima

I think that this may be it.

Notice: Undefined variable: submit in /home/kansuke/public_html/shoutbox.php on line 11Resource id #3

... what is the undefined variable in the code?

and it rejects the query you said to use...

sonesay

you didnt say if that was your entire code you posted. Looking at it now $submit is not defined as when its submitted your checking if($submitrefer to it as $_POST['submit'];


truefusion

A couple of suggestions:

Replace:

while($r=mysql_fetch_array($result)){//variables from table$time=$r["time"];$id=$r["id"];$message=$r["message"];$name=$r["name"];?><?php $time ?><br /><?php echo $name ?><br /><?php echo $message ?><br /><?php } ?>
With:
while($r=mysql_fetch_assoc($result)){//variables from tableecho $r["time"]."<br/>\n".$r["id"]."<br/>\n".$r["name"].":<br/>\n".$r["message"]."<br/>\n";}?>
to save some space, and to receive an associative array from the result.

And since the form is going to be submitted to the same page it's in, you can have simply this:
<form action="" method="post">



electriic ink

Also, your output code reads this:

 

<br /><b>Notice</b>: Undefined variable: submit in <b>/home/kansuke/public_html/shoutbox.php</b> on line <b>11</b><br />Resource id #3<form action="<br /><strong class='bbc'><b>Notice</b>: Undefined variable: php_self in <b>/home/kansuke/public_html/shoutbox.php</b> on line <b>37</b></strong><br />" method="post"><INPUT TYPE='TEXT' value='name' NAME='name' SIZE=30 maxlength='100' /><br /><INPUT TYPE='TEXT' value='message' NAME='message' SIZE=30 maxlength='100' /><br /><input type="submit" name="submit" value="submit"></form>

The error is on the following line in the source code:

 

<form action="<?php echo $php_self ?>" method="post">

Which should read:

 

<form action="<?php echo $SERVER['PHP_SELF']; ?>" method="post">


KansukeKojima

you didnt say if that was your entire code you posted. Looking at it now $submit is not defined as when its submitted your checking if($submit
refer to it as $_POST['submit'];


I think that would maybe solve it... but, where am I supposed to place it? I'm not quite clear on where you want me to place $_POST['submit'].

sonesay

This whole block in your original code would never return true since its $submit is not defined in PHP yet.

if($submit){...}

So you can either change it so you define $submit before its used or just change $submit to $_POST['submit']; directly. Submit is coming from your button named 'submit'.

// this $submit = $_POST['submit'];if($submit){..}// or thisif($_POST['submit']){..}


KansukeKojima

w00t! It works.

sonesay, what you said worked perfectly! But, an error said that the name and message variables were not defined... I almost posted another message saying that it would not work, but I added $name = $_POST['name']; and $message = $_POST['message']; and now it works perfectly! Thank you everyone that helped!

Now I just have to work on some styling and stuff...

thanks!

**EDIT**
ahh... crap... ok one problem left... whenever someone visits the page... it re-posts the previous message...... hmm... how would I fix that....

here is my code again...

<?ini_set("display_errors", 1);error_reporting(E_ALL);//the host, name, and password for your mysqlmysql_connect("localhost","not tellin you!","not tellin");//select the databasemysql_select_db("not tellin");$submit = $_POST['submit'];$name = $_POST['name'];$message = $_POST['message'];if($submit){ //use the PHP date function for the time $time=date("h:ia d/j/y"); // inserting it into the shoutbox table which we made in the mysql statements before $result=MYSQL_QUERY("INSERT INTO shoutbox (id,name,message,time)". "VALUES ('NULL','$name', '$message','$time')");}?><?//returning the last 5 messages$result = mysql_query("select * from shoutbox order by id desc limit 5");//the while loopwhile($r=mysql_fetch_assoc($result)){//variables from tableecho $r["time"]."<br/>\n".$r["id"]."<br/>\n".$r["name"].":<br/>\n".$r["message"]."<br/>\n";}?><form action="" method="post"><INPUT TYPE='TEXT' value='name' NAME='name' SIZE=30 maxlength='100' /><br /><INPUT TYPE='TEXT' value='message' NAME='message' SIZE=30 maxlength='100' /><br /><input type="submit" name="submit" value="submit"></form>


sonesay

ini_set("display_errors", 1);error_reporting(E_ALL);//the host, name, and password for your mysqlmysql_connect("localhost","not tellin you!","not tellin");//select the databasemysql_select_db("not tellin");$submit = $_POST['submit'];$name = $_POST['name'];$message = $_POST['message'];if($submit){

I think whats causing the error reporting for the first time you come to the page is the upper block of code where it says error reporting etc. Its catching and displaying the error when post vars submit, name, and message are assigned but are null I'm guessing. I would try and remove the error reporting statements in the first two lines and test it out. I think that way you can still try and assign the post variables and if they are null which would be in the case where someone first loads the page it shouldn't report the error. Not sure if this is good practice but if someone has a better way please post.


KansukeKojima

well... this is retarded... the error messages are gone now... but it keeps reposting the last message you typed whenever you refresh teh page.... what the hell...


coolcat50

Um Kansuke try this.

<?//the host, name, and password for your mysqlmysql_connect("localhost","not tellin you!","not tellin");//select the databasemysql_select_db("not tellin");$submit = $_POST['submit'];$name = $_POST['name'];$message = $_POST['message'];if($submit){ //use the PHP date function for the time $time=date("h:ia d/j/y"); // inserting it into the shoutbox table which we made in the mysql statements before $result=MYSQL_QUERY("INSERT INTO shoutbox (id,name,message,time)". "VALUES ('NULL','$name', '$message','$time')"); $_POST['submit'] = 0; $_POST['message'] = ""; $_POST['name'] = "";}?><?//returning the last 5 messages$result = mysql_query("select * from shoutbox order by id desc limit 5");//the while loopwhile($r=mysql_fetch_assoc($result)){//variables from tableecho $r["time"]."<br/>\n".$r["id"]."<br/>\n".$r["name"].":<br/>\n".$r["message"]."<br/>\n";}?><form action="" method="post"><INPUT TYPE='TEXT' value='name' NAME='name' SIZE=30 maxlength='100' /><br /><INPUT TYPE='TEXT' value='message' NAME='message' SIZE=30 maxlength='100' /><br /><input type="submit" name="submit" value="submit"></form>

EDIT: Fixed BBCode

sonesay

The reason why its keeps reposting is because the post variable are saved since its posted to the same page. When you reload you reload the post lol. The only other way to do it so you can reload that page without double posting is to have it submit to a page thats designed specifically for inserting in to the database. Then redirect it back to the original page where you display form and last previous five message. I'm pretty sure that will work so you need an extra php page.I dont think its possible to destroy post variables after a post. I could be wrong but I dont think we have acess to anything like that.. doubt it lol.


jlhaslip

Does this work any better?
I have adjusted it a little so that it checks to see if the Form is 'set' before assigning the variables used.

<?ini_set("display_errors", 1);error_reporting(E_ALL);//the host, name, and password for your mysqlmysql_connect("localhost","not tellin you!","not tellin");//select the databasemysql_select_db("not tellin");if(isset($_POST['submit'])){ $submit = $_POST['submit']; $name = $_POST['name']; $message = $_POST['message']; //use the PHP date function for the time $time=date("h:ia d/j/y"); // inserting it into the shoutbox table which we made in the mysql statements before $result=MYSQL_QUERY("INSERT INTO shoutbox (id,name,message,time)". "VALUES ('NULL','$name', '$message','$time')");}?><?//returning the last 5 messages$result = mysql_query("select * from shoutbox order by id desc limit 5");//the while loopwhile($r=mysql_fetch_assoc($result)){//variables from tableecho $r["time"]."<br/>\n".$r["id"]."<br/>\n".$r["name"].":<br/>\n".$r["message"]."<br/>\n";}?><form action="" method="post"><INPUT TYPE='TEXT' value='name' NAME='name' SIZE=30 maxlength='100' /><br /><INPUT TYPE='TEXT' value='message' NAME='message' SIZE=30 maxlength='100' /><br /><input type="submit" name="submit" value="submit"></form>


KansukeKojima

Thanks jlhaslip, I tried what you posted unfortuneately, it does not work. But, I've found something that does, and unless someone can come up with a better method... <iframes>!Fortuneately, it keeps the shoutbox from re-posting... so I'll probably be using them... unfortuneately xD... If someone has a better idea...I'll use it.


sonesay

if you really need to do it on the same page and not like how I said previous post with submitting to another page then redirecting back to the original. that way save post variables are lost then..You can probably do it with some javascript. Set a flag for when the submit button is acutally pressed. That way you can distinguish between page reloads and real submissions. I wont give you exact code but that is the idea. Have a onclick function on your button to set flag that it is a real submit then check on your if statement code. If you really need the exact logic and code reply lol.


KansukeKojima

Err... yeah wait... I'll try out what you said in your previous post...

*EDIT*

Ok, I did what you suggested in the previous post. Works like a charm! Thank you so much!!

feel free to test it out

http://forums.xisto.com/no_longer_exists/ (click the 'Open Shoutbox Window' link)

EDIT 2

bbcode is now included.



VIEW DESKTOP VERSION REGISTERGET FREE HOSTING

Xisto.com offers Free Web Hosting to its Members for their participation in this Community. We moderate all content posted here but we cannot warrant full correctness of all content. While using this site, you agree to have read and accepted our terms of use, cookie and privacy policy. Copyright 2001-2019 by Xisto Corporation. All Rights Reserved.