Mattio2187 Posted June 21, 2009 Share Posted June 21, 2009 i am fairly new with php/mysql. I thought I was doing something pretty straight forward. just taking a chosen radio button value and entering into a single field of a table. Here is the error im getting: Error: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') VALUE (three)' at line 1 And here is my php code, I can't find the syntax error: <?php $con = mysql_connect("localhost","mattsol1_user","user"); if (!$con) { die('Could not connect: ' . mysql_error()); } mysql_select_db("mattsol1_pollthree", $con); $sql= "INSERT INTO vote (select) VALUES ('$_POST[select]')" ; if (!mysql_query($sql,$con)) { die('Error: ' . mysql_error()); } echo "Thank You"; mysql_close($con); ?> code is live at mattsolano.com/zink/voting.html any feedback would be appreciated Quote Link to comment Share on other sites More sharing options...
Andy-H Posted June 21, 2009 Share Posted June 21, 2009 "select" is a mysql reserved word and needs to be surrounded by backticks, or better yet, you could change the name of the field. <?php $con = mysql_connect("localhost","mattsol1_user","user"); if (!$con) { die('Could not connect: ' . mysql_error()); } mysql_select_db("mattsol1_pollthree", $con); $sql= "INSERT INTO vote (`select`) VALUES ('" . mysql_real_escape_string($_POST['select']) . "')" ; if (!mysql_query($sql,$con)) { die('Error: ' . mysql_error()); } echo "Thank You"; mysql_close($con); ?> Quote Link to comment Share on other sites More sharing options...
Ken2k7 Posted June 21, 2009 Share Posted June 21, 2009 $sql = sprintf("INSERT INTO vote (`select`) VALUES ('%s')", mysql_real_escape_string($_POST['select'])); You get PHP notices if you don't put quotes around select in $_POST['select']. Quote Link to comment Share on other sites More sharing options...
Mattio2187 Posted June 21, 2009 Author Share Posted June 21, 2009 Thanks, a ton. Like I said, I'm just starting out so i wouldn't have caught that. I'm not sure exactly what some of the other stuff you added is, but i'm going to trust that its better code than what I had. Thanks again 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.