bob2006 Posted January 11, 2011 Share Posted January 11, 2011 Can some one look at this code for me i dont see the problem but there is some thing wrong thank you 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 '','34448','US','3525887894')' at line 1 <?php include'db.php'; if (isset($_POST[submit])) { $frist=$_POST['firstname']; $lastname=$_POST['lastname']; $mail=$_POST['email']; $add1=$_POST['address1']; $add2=$_POST['address2']; $city=$_POST['city']; $state=$_POST['state']; $post=$_POST['postcode']; $country=$_POST['country']; $number=$_POST['phonenumber']; echo"$frist<br>$last<br>$mail<br>$add1<br>$add2<br>$city<br>$state<br>$post<br>$country<br>$number"; mysql_query("INSERT INTO user (id,username,email,last,address1,address2,City,State,Postcode,Country,Phone) VALUES (NULL,'$frist','$mail','$lastname','$add1','$add2','$city',$state','$post','$country','$number')")or die(mysql_error()); } else { echo"<form action='' method='post'> <table width='100%' border='0' cellpadding='0' cellspacing='0'> <tbody> <tr> <td width='50%' height='312' valign='top'><table width='100%' border='0' cellpadding='10' cellspacing='0'> <tbody> <tr> <td width='100'>First Name</td> <td><input type='text' name='firstname' value='' /></td> </tr> <tr> <td>Last Name</td> <td><input type='text' name='lastname' value='' /></td> </tr> <tr> <td>Email Address</td> <td><input type='text' name='email' value='' /></td> </tr> <tr> <td></td> <td></td> </tr> <tr> <td></td> <td></td> </tr> <tr> <td> </td> <td> </td> </tr> <tr> <td> </td> <td> </td> </tr> </tbody> </table></td> <td width='50%' valign='top'><table width='100%' border='0' cellpadding='10' cellspacing='0'> <tbody> <tr> <td width='100'>Address 1</td> <td><input type='text' name='address1' value='' /></td> </tr> <tr> <td>Address 2</td> <td><input type='text' name='address2' value='' /></td> </tr> <tr> <td>City</td> <td><input type='text' name='city' value='' /></td> </tr> <tr> <td>State/Region</td> <td><input type='text' name='state' value='' /></td> </tr> <tr> <td>Zip Code</td> <td><input type='text' name='postcode' size='15' value='' /></td> </tr> <tr> <td>Country</td> <td><select name='country'> <option value='AF'>Afghanistan</option> <option value='AX'>Aland Islands</option> <option value='AL'>Albania</option> <option value='DZ'>Algeria</option> <option value='AS'>American Samoa</option> <option value='AD'>Andorra</option> <option value='AO'>Angola</option> <option value='AI'>Anguilla</option> <option value='AQ'>Antarctica</option> <option value='AG'>Antigua And Barbuda</option> <option value='AR'>Argentina</option> <option value='AM'>Armenia</option> <option value='AW'>Aruba</option> <option value='AU'>Australia</option> <option value='AT'>Austria</option> <option value='AZ'>Azerbaijan</option> <option value='BS'>Bahamas</option> <option value='BH'>Bahrain</option> <option value='BD'>Bangladesh</option> <option value='BB'>Barbados</option> <option value='BY'>Belarus</option> <option value='BE'>Belgium</option> <option value='BZ'>Belize</option> <option value='BJ'>Benin</option> <option value='BM'>Bermuda</option> <option value='BT'>Bhutan</option> <option value='BO'>Bolivia</option> <option value='BA'>Bosnia And Herzegovina</option> <option value='BW'>Botswana</option> <option value='BV'>Bouvet Island</option> <option value='BR'>Brazil</option> <option value='IO'>British Indian Ocean Territory</option> <option value='BN'>Brunei Darussalam</option> <option value='BG'>Bulgaria</option> <option value='BF'>Burkina Faso</option> <option value='BI'>Burundi</option> <option value='KH'>Cambodia</option> <option value='CM'>Cameroon</option> <option value='CA'>Canada</option> <option value='CV'>Cape Verde</option> <option value='KY'>Cayman Islands</option> <option value='CF'>Central African Republic</option> <option value='TD'>Chad</option> <option value='CL'>Chile</option> <option value='CN'>China</option> <option value='CX'>Christmas Island</option> <option value='CC'>Cocos (Keeling) Islands</option> <option value='CO'>Colombia</option> <option value='KM'>Comoros</option> <option value='CG'>Congo</option> <option value='CD'>Congo, Democratic Republic</option> <option value='CK'>Cook Islands</option> <option value='CR'>Costa Rica</option> <option value='CI'>Cote D'Ivoire</option> <option value='HR'>Croatia</option> <option value='CU'>Cuba</option> <option value='CY'>Cyprus</option> <option value='CZ'>Czech Republic</option> <option value='DK'>Denmark</option> <option value='DJ'>Djibouti</option> <option value='DM'>Dominica</option> <option value='DO'>Dominican Republic</option> <option value='EC'>Ecuador</option> <option value='EG'>Egypt</option> <option value='SV'>El Salvador</option> <option value='GQ'>Equatorial Guinea</option> <option value='ER'>Eritrea</option> <option value='EE'>Estonia</option> <option value='ET'>Ethiopia</option> <option value='FK'>Falkland Islands (Malvinas)</option> <option value='FO'>Faroe Islands</option> <option value='FJ'>Fiji</option> <option value='FI'>Finland</option> <option value='FR'>France</option> <option value='GF'>French Guiana</option> <option value='PF'>French Polynesia</option> <option value='TF'>French Southern Territories</option> <option value='GA'>Gabon</option> <option value='GM'>Gambia</option> <option value='GE'>Georgia</option> <option value='DE'>Germany</option> <option value='GH'>Ghana</option> <option value='GI'>Gibraltar</option> <option value='GR'>Greece</option> <option value='GL'>Greenland</option> <option value='GD'>Grenada</option> <option value='GP'>Guadeloupe</option> <option value='GU'>Guam</option> <option value='GT'>Guatemala</option> <option value='GG'>Guernsey</option> <option value='GN'>Guinea</option> <option value='GW'>Guinea-Bissau</option> <option value='GY'>Guyana</option> <option value='HT'>Haiti</option> <option value='HM'>Heard Island & Mcdonald Islands</option> <option value='VA'>Holy See (Vatican City State)</option> <option value='HN'>Honduras</option> <option value='HK'>Hong Kong</option> <option value='HU'>Hungary</option> <option value='IS'>Iceland</option> <option value='IN'>India</option> <option value='ID'>Indonesia</option> <option value='IR'>Iran, Islamic Republic Of</option> <option value='IQ'>Iraq</option> <option value='IE'>Ireland</option> <option value='IM'>Isle Of Man</option> <option value='IL'>Israel</option> <option value='IT'>Italy</option> <option value='JM'>Jamaica</option> <option value='JP'>Japan</option> <option value='JE'>Jersey</option> <option value='JO'>Jordan</option> <option value='KZ'>Kazakhstan</option> <option value='KE'>Kenya</option> <option value='KI'>Kiribati</option> <option value='KR'>Korea</option> <option value='KW'>Kuwait</option> <option value='KG'>Kyrgyzstan</option> <option value='LA'>Lao People's Democratic Republic</option> <option value='LV'>Latvia</option> <option value='LB'>Lebanon</option> <option value='LS'>Lesotho</option> <option value='LR'>Liberia</option> <option value='LY'>Libyan Arab Jamahiriya</option> <option value='LI'>Liechtenstein</option> <option value='LT'>Lithuania</option> <option value='LU'>Luxembourg</option> <option value='MO'>Macao</option> <option value='MK'>Macedonia</option> <option value='MG'>Madagascar</option> <option value='MW'>Malawi</option> <option value='MY'>Malaysia</option> <option value='MV'>Maldives</option> <option value='ML'>Mali</option> <option value='MT'>Malta</option> <option value='MH'>Marshall Islands</option> <option value='MQ'>Martinique</option> <option value='MR'>Mauritania</option> <option value='MU'>Mauritius</option> <option value='YT'>Mayotte</option> <option value='MX'>Mexico</option> <option value='FM'>Micronesia, Federated States Of</option> <option value='MD'>Moldova</option> <option value='MC'>Monaco</option> <option value='MN'>Mongolia</option> <option value='ME'>Montenegro</option> <option value='MS'>Montserrat</option> <option value='MA'>Morocco</option> <option value='MZ'>Mozambique</option> <option value='MM'>Myanmar</option> <option value='NA'>Namibia</option> <option value='NR'>Nauru</option> <option value='NP'>Nepal</option> <option value='NL'>Netherlands</option> <option value='AN'>Netherlands Antilles</option> <option value='NC'>New Caledonia</option> <option value='NZ'>New Zealand</option> <option value='NI'>Nicaragua</option> <option value='NE'>Niger</option> <option value='NG'>Nigeria</option> <option value='NU'>Niue</option> <option value='NF'>Norfolk Island</option> <option value='MP'>Northern Mariana Islands</option> <option value='NO'>Norway</option> <option value='OM'>Oman</option> <option value='PK'>Pakistan</option> <option value='PW'>Palau</option> <option value='PS'>Palestinian Territory, Occupied</option> <option value='PA'>Panama</option> <option value='PG'>Papua New Guinea</option> <option value='PY'>Paraguay</option> <option value='PE'>Peru</option> <option value='PH'>Philippines</option> <option value='PN'>Pitcairn</option> <option value='PL'>Poland</option> <option value='PT'>Portugal</option> <option value='PR'>Puerto Rico</option> <option value='QA'>Qatar</option> <option value='RE'>Reunion</option> <option value='RO'>Romania</option> <option value='RU'>Russian Federation</option> <option value='RW'>Rwanda</option> <option value='BL'>Saint Barthelemy</option> <option value='SH'>Saint Helena</option> <option value='KN'>Saint Kitts And Nevis</option> <option value='LC'>Saint Lucia</option> <option value='MF'>Saint Martin</option> <option value='PM'>Saint Pierre And Miquelon</option> <option value='VC'>Saint Vincent And Grenadines</option> <option value='WS'>Samoa</option> <option value='SM'>San Marino</option> <option value='ST'>Sao Tome And Principe</option> <option value='SA'>Saudi Arabia</option> <option value='SN'>Senegal</option> <option value='RS'>Serbia</option> <option value='SC'>Seychelles</option> <option value='SL'>Sierra Leone</option> <option value='SG'>Singapore</option> <option value='SK'>Slovakia</option> <option value='SI'>Slovenia</option> <option value='SB'>Solomon Islands</option> <option value='SO'>Somalia</option> <option value='ZA'>South Africa</option> <option value='GS'>South Georgia And Sandwich Isl.</option> <option value='ES'>Spain</option> <option value='LK'>Sri Lanka</option> <option value='SD'>Sudan</option> <option value='SR'>Suriname</option> <option value='SJ'>Svalbard And Jan Mayen</option> <option value='SZ'>Swaziland</option> <option value='SE'>Sweden</option> <option value='CH'>Switzerland</option> <option value='SY'>Syrian Arab Republic</option> <option value='TW'>Taiwan</option> <option value='TJ'>Tajikistan</option> <option value='TZ'>Tanzania</option> <option value='TH'>Thailand</option> <option value='TL'>Timor-Leste</option> <option value='TG'>Togo</option> <option value='TK'>Tokelau</option> <option value='TO'>Tonga</option> <option value='TT'>Trinidad And Tobago</option> <option value='TN'>Tunisia</option> <option value='TR'>Turkey</option> <option value='TM'>Turkmenistan</option> <option value='TC'>Turks And Caicos Islands</option> <option value='TV'>Tuvalu</option> <option value='UG'>Uganda</option> <option value='UA'>Ukraine</option> <option value='AE'>United Arab Emirates</option> <option value='GB'>United Kingdom</option> <option value='US' selected='selected'>United States</option> <option value='UM'>United States Outlying Islands</option> <option value='UY'>Uruguay</option> <option value='UZ'>Uzbekistan</option> <option value='VU'>Vanuatu</option> <option value='VE'>Venezuela</option> <option value='VN'>Viet Nam</option> <option value='VG'>Virgin Islands, British</option> <option value='VI'>Virgin Islands, U.S.</option> <option value='WF'>Wallis And Futuna</option> <option value='EH'>Western Sahara</option> <option value='YE'>Yemen</option> <option value='ZM'>Zambia</option> <option value='ZW'>Zimbabwe</option> </select></td> </tr> <tr> <td>Phone Number</td> <td><input type='text' name='phonenumber' size='20' value='' /></td> </tr> </tbody> </table></td> </tr> </tbody> </table> <center><input type='submit' name='submit' id='submit' value='Submit' /></center> </form>"; } ?> Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/ Share on other sites More sharing options...
Psycho Posted January 11, 2011 Share Posted January 11, 2011 How is this a PHP question? Moving to MySQL forumn. You should always create your queries as a string variable so you can echo the entire query to the page for debugging purposes. But, from the little information you have provided I can see that there is no opening quotation mark for the state value. Change your code to $query = "INSERT INTO user (username, email, last, address1, address2, City, State, Postcode, Country, Phone) VALUES ('$frist', '$mail', '$lastname', '$add1', '$add2', '$city', '$state', '$post', '$country',' $number')"; mysql_query()or die("Query:<br />{$query}<br />Error:<br />".mysql_error()); EDIT: Also, if id is an auto-increment value, there is no need to include it in the query at all. EDIT #2: That code should work assuming there are no other errors, but I don't think you meant to name the variable for the first name as $frist. You used the same name when you set it and when you use it, so it would work, but I'm guessing it was a typo that you copied/pasted Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158007 Share on other sites More sharing options...
jwilson122 Posted January 11, 2011 Share Posted January 11, 2011 How is this a PHP question? Moving to MySQL forumn. He has a question about PHP and MySQL. No need to move the thread. There are tons of these kinds of questions. He is trying to insert data into a MySQL database.. So its basically both PHP and MySQL. As for your question bob2006, Try replacing: mysql_query("INSERT INTO user (id,username,email,last,address1,address2,City,State,Postcode,Country,Phone) VALUES (NULL,'$frist','$mail','$lastname','$add1','$add2','$city',$state','$post','$country','$number')")or die(mysql_error()); with mysql_query("INSERT INTO user (`id`,username,email,last,address1,address2,City,State,Postcode,Country,Phone) VALUES ('','$frist','$mail','$lastname','$add1','$add2','$city','$state','$post','$country','$number')")or die(mysql_error()); I'm not sure if that will work or not, but let me know Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158011 Share on other sites More sharing options...
Maq Posted January 11, 2011 Share Posted January 11, 2011 jwilson, I understand what you're saying but this particular issue is strictly MySQL. He was just missing a single quote before $state. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158014 Share on other sites More sharing options...
mikosiko Posted January 11, 2011 Share Posted January 11, 2011 change this mysql_query("INSERT INTO user (id, ...... for this mysql_query("INSERT INTO user (`id`,.... why? :'( Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158016 Share on other sites More sharing options...
jwilson122 Posted January 11, 2011 Share Posted January 11, 2011 change this mysql_query("INSERT INTO user (id, ...... for this mysql_query("INSERT INTO user (`id`,.... why? :'( here. Just add a " ' " right below the $state variable. That's all that's wrong. Give credit to Maq for saying that tho LOL. I didn't see it myself. Also, sometimes if you wrap your fields in the "`" such as.. (`id`,`username`) etc it will prevent it from messing with MySQL. Such as, if you have a field called "desc", it will not work, because MySQL has something called that already. BUT if you wrap it like this: "`desc`" it would work Also @ Maq, I know its a MySQL question too. Thing is, someone needs PHP knowledge to fix the problem lol. Not just SQL. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158019 Share on other sites More sharing options...
mikosiko Posted January 11, 2011 Share Posted January 11, 2011 change this mysql_query("INSERT INTO user (id, ...... for this mysql_query("INSERT INTO user (`id`,.... why? :'( Also, sometimes if you wrap your fields in the "`" such as.. (`id`,`username`) etc it will prevent it from messing with MySQL. Such as, if you have a field called "desc", it will not work, because MySQL has something called that already. BUT if you wrap it like this: "`desc`" it would work funny.... I know exactly when/why to use the backticks... the question was aimed for you to explain why you were recommending to do that when was totally unnecessary... Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158022 Share on other sites More sharing options...
Pikachu2000 Posted January 11, 2011 Share Posted January 11, 2011 change this mysql_query("INSERT INTO user (id, ...... for this mysql_query("INSERT INTO user (`id`,.... why? :'( here. Just add a " ' " right below the $state variable. That's all that's wrong. Give credit to Maq for saying that tho LOL. I didn't see it myself. Also, sometimes if you wrap your fields in the "`" such as.. (`id`,`username`) etc it will prevent it from messing with MySQL. Such as, if you have a field called "desc", it will not work, because MySQL has something called that already. BUT if you wrap it like this: "`desc`" it would work Also @ Maq, I know its a MySQL question too. Thing is, someone needs PHP knowledge to fix the problem lol. Not just SQL. 'id' isn't a MySQL reserved word, so the backticks will have no effect. Besides, that isn't where the error message showed the problem to be. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158024 Share on other sites More sharing options...
jwilson122 Posted January 11, 2011 Share Posted January 11, 2011 mikosiko, I said it before the fact I didn't know that there was a " ' " missing before $state. I've had times where when I used the "`" around things, it fixes a lot of my problems. Pikachu2000, you owned me there lolz. I gotta admit Dunno what I was rly thinking lol. I just know its helped fix a lot of issues I've had in the past. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158027 Share on other sites More sharing options...
mikosiko Posted January 11, 2011 Share Posted January 11, 2011 .... Dunno what I was rly thinking .... ditto... that explain and close the topic Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158036 Share on other sites More sharing options...
jwilson122 Posted January 11, 2011 Share Posted January 11, 2011 .... Dunno what I was rly thinking .... ditto... that explain and close the topic Mhm lol Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158038 Share on other sites More sharing options...
bob2006 Posted January 11, 2011 Author Share Posted January 11, 2011 thanks for the help and you are right i had a typo thanks for all the help Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158071 Share on other sites More sharing options...
Psycho Posted January 11, 2011 Share Posted January 11, 2011 Just add a " ' " right below the $state variable. That's all that's wrong. Give credit to Maq for saying that tho LOL. I didn't see it myself. Why are you giving credit to Maq (not that I have anything against him)? I identified the problem in the first response. ... I can see that there is no opening quotation mark for the state value. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158107 Share on other sites More sharing options...
Maq Posted January 11, 2011 Share Posted January 11, 2011 Why are you giving credit to Maq (not that I have anything against him)? I identified the problem in the first response. Quote Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158115 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.