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>"; } ?> 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 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 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. 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? :'( 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. 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... 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. 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. 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 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 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 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. 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. Link to comment https://forums.phpfreaks.com/topic/224110-mysql-styanx-error/#findComment-1158115 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.