"You have an error in your SQL syntax" przy wysylaniu formularza, Ja osobiscie nie widze problemu lecz moze ktos zauwazy :) Opcje

[PeeSDeOeN]

Witam!
Mój problem to wyskakujacy komunikat "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 '' at line 53". W momencie gdy probuje dodac nowa oferte. (Serwis ma na celu zamieszczanie ogloszen typu wynajem mieszkan - oczywiscie darmowy). Niestety kod jest za dlugi aby go umiescic w poscie wiec podaje linka do niego:
link do pliku php

Z gory dziekuje za kazda udzielona odpowiedz.

Powód edycji: [wookieb]:

[mlawnik]

Angielski znasz?
Jak nie, to google translate ci pomoże.


STFW

[wookieb]

Czekam na PW z poprawną nazwą tematu. Zamykam.

Powód edycji: [wookieb]: Otwieram

[PeeSDeOeN]

Doszedlem juz do tego co bylo nie tak. Lecz pozostala mi kwestia tego iz formularz nie zapisuje danych w bazie..

[konole]

A mógłbyś wstawić odpowiedzialny za to kod tutaj? Nie chce mi się ściągać plików...

[PeeSDeOeN]

A mógłbyś wstawić odpowiedzialny za to kod tutaj? Nie chce mi się ściągać plików...

[PHP] pobierz, plaintext 
$kobieta = false;
if (isset($_POST["kobieta"]))
	$kobieta = true;	
$gaz = false;
if (isset($_POST["gaz"]))
	$gaz = true;
$prad = false;
if (isset($_POST["prad"]))
	$prad = true;
$woda = false;
if (isset($_POST["woda"]))
	$woda = true;
$ogrzewanie = false;
if (isset($_POST["ogrzewanie"]))
	$ogrzewanie = true;
$wywoz_smieci = false;
if (isset($_POST["wywoz_smieci"]))
	$wywoz_smieci = true;
$internet = false;
if (isset($_POST["internet"]))
	$internet = true;
$telefon = false;
if (isset($_POST["telefon"]))
	$telefon = true;
$lokal_umeblowany = false;
if (isset($_POST["lokal_umeblowany"]))
	$lokal_umeblowany = true;
$lokal_internet = false;
if (isset($_POST["lokal_internet"]))
	$lokal_internet = true;
$lokal_kablowka = false;
if (isset($_POST["lokal_kablowka"]))
	$lokal_kablowka = true;
$lokal_telefon = false;
if (isset($_POST["lokal_telefon"]))
	$lokal_telefon = true;
$lokal_lodowka = false;
if (isset($_POST["lokal_lodowka"]))
	$lokal_lodowka = true;
$lokal_pralka = false;
if (isset($_POST["lokal_pralka"]))
	$lokal_pralka = true;
$lokal_telewizor = false;
if (isset($_POST["lokal_telewizor"]))
	$lokal_telewizor = true;
$lokal_mikrofalowka = false;
if (isset($_POST["lokal_mikrofalowka"]))
	$lokal_mikrofalowka = true;
$lokal_balkon = false;
if (isset($_POST["lokal_balkon"]))
	$lokal_balkon = true;
$lokal_wanna = false;
if (isset($_POST["lokal_wanna"]))
	$lokal_wanna = true;
$lokal_prysznic = false;
if (isset($_POST["lokal_prysznic"]))
	$lokal_prysznic = true;
$mezczyzna = false;
if (isset($_POST["mezczyzna"]))
	$mezczyzna = true;
$nie_para = false;
if (isset($_POST["nie_para"]))
	$nie_para = true;
$nie_student = false;
if (isset($_POST["nie_student"]))
	$nie_student = true;
$pracujacy = false;
if (isset($_POST["pracujacy"]))
	$pracujacy = true;
$nie_palacy = false;
if (isset($_POST["nie_palacy"]))
	$nie_palacy = true;
$bez_zwierzat = false;
if (isset($_POST["bez_zwierzat"]))
	$bez_zwierzat = true;
 
if(isset($_POST['send']) && $_POST['send']==1){
	if(empty($_POST["wojewodztwo"]) && empty($_POST["miasto"]) && empty($_POST["dzielnica"]) && empty($_POST["ulica"]) && empty($_POST["nr_domu"]) && empty($_POST["typ_lokalu"]) && empty($_POST["typ_zabudowy"]) && empty($_POST["powierzchnia"]) && empty($_POST["liczba_miejsc"]) && empty($_POST["liczba_pokoi"]) && empty($_POST["lokal_wolny_od"]) && empty($_POST["koniec_wynajmu"]) && empty($_POST["cena_za_miejsce"]) && empty($_POST["cena_za_calosc"]) && empty($_POST["kaucja"]) && empty($_POST["imie"]) && empty($_POST["nazwisko"]) && empty($_POST["kontakt_telefon"]) && empty($_POST["email"]) && empty($_POST["temat_ogloszenia"]) && empty($_POST["opis_ogloszenia"])){
		ShowForm("Wypelnij pola obowiazkowe!!!");
		echo "</tr> 
						</td> 
						</table> 
					</div> 
				</div>			
				<div class=\"pasek\"> 
				</div> 
				<div id=\"copyright\"> 
					Š
				</div> 
				<div id=\"góre\"> 
					<h1><a href=\"#top\"><img src=\"img/wgore.png\">W góre</a></h1> 
				</div> 
			</div> 
		</div> 
	 </div> 
 
<!--]>--><script type=\"text/javascript\"></script><div style=\"text-align:center;font-size:11px\" class=\"cbalink\"><a href=\"http://www.cba.pl/\" title=\"darmowy hosting\">Darmowy Hosting</a> CBA.PL<br/><br/></div> 
<script type=\"text/javascript\" src=\"http://a.cba.pl/r2.js\"></script> 
<script src=\"http://www.google-analytics.com/urchin.js\" type=\"text/javascript\"></script> 
<script type=\"text/javascript\"> 
_uacct = \"UA-2289508-3\";
urchinTracker();
</script> 
</body> 
 
</html>"; 
		exit;
		}
		{
		mysql_query("insert into ogloszenia values(``,
		`1`,
		`".$_POST['wojewodztwo']."`,
		`".htmlspecialchars($_POST["miasto"])."`,
		`".htmlspecialchars($_POST["dzielnica"])."`,
		`".htmlspecialchars($_POST["ulica"])."`,
		`".$_POST["nr_domu"]."`,
		`koordynaty_mapy`,
		`".$_POST["typ_lokalu"]."`,
		`".$_POST["typ_zabudowy"]."`,
		`".$_POST["powierzchnia"]."`,
		`".$_POST['liczba_miejsc']."`,
		`".$_POST['liczba_pokoi']."`,
		`".$_POST["lokal_wolny_od"]."`,
		`".$_POST["koniec_wynajmu"]."`,
		`".$_POST["cena_za_miejsce"]."`,
		`".$_POST["cena_za_calosc"]."`,
		`".$_POST["kaucja"]."`,
		`".$gaz."`,
		`".$prad."`,
		`".$woda."`,
		`".$ogrzewanie."`,
		`".$wywoz_smieci."`,
		`".$internet."`,
		`".$telefon."`,
		`".htmlspecialchars($_POST["inne"])."`,
		`".$lokal_umeblowany."`,
		`".$lokal_internet."`,
		`".$lokal_kablowka."`,
		`'".$lokal_telefon."`,
		`".$lokal_lodowka."`,
		`".$lokal_pralka."`,
		`".$lokal_telewizor."`,
		`".$lokal_mikrofalowka."`,
		`".$lokal_balkon."`,
		`".$lokal_wanna."`,
		`".$lokal_prysznic."`,
		`".$kobieta."`,
		`".$mezczyzna."`,
		`".$nie_para."`,
		`".$nie_student."`,
		`".$pracujacy."`,
		`".$nie_palacy."`,
		`".$bez_zwierzat."`,
		`".htmlspecialchars($_POST["imie"])."`,
		`".htmlspecialchars($_POST["nazwisko"])."`,
		`".$_POST["kontakt_telefon"]."`,
		`".htmlspecialchars($_POST["email"])."`,
		`".$_POST["gadu"]."`,
		`".htmlspecialchars($_POST["temat_ogloszenia"])."`,
		`".htmlspecialchars($_POST["opis_ogloszenia"])."`,
		`brak`,
		`NOW()`");
		echo "<br>Twoje ogloszenie zostalo dodane!";
		}
	}
else ShowForm();
 
?>	
[PHP] pobierz, plaintext 

Jest ktos w stanie mi w tym pomoc??
Bo ja juz wysiadam.. Potrzebuje tylko aby formularz sie zapisywal w bazie danych.
moge nawet wkleic caly kod php w kilku postach jak ktos tak woli

[wookieb]

Temat: Jak poprawnie zadac pytanie
Przenosze.

[worek]

mnóstwo błędów podstawowych. chociaż by przy poleceniu insert

składnia polecenia wygląda mniej więcej tak

INSERT INTO [tabela] (`[kolumna z wartościami]` ,`[kolumna z wartościami]`) VALUES ('wartość' , 'wartość');

to jest polecenie dla wstawienia dwóch wartości, jak dla więcej to chyba się domyślisz co trzeba zrobić, a tak poza tym to powtórkę podstaw polecam.

Stosujesz coś koło 20 warunkó if czy nie lepiej dać foreach lub while i wtedy do tego warunek?? mniej linijek kodu i bardziej przejrzyście. Wyobraź sobie że formularz będzie miał np 100 pól do uzupełnienia. I będzie trzeba zmienić warunek np if(prad = true) wtedy zmieniaj w każdej instrukcji warunek Co do tych if-ów brakuje klamr otwierających i zamykających. Ogółem możesz wyrzucić laptopa przez okno albo spłukać w toalecie bo kod do niczego się nie nadaje. Pozdrawiam

Ten post edytował worek 30.05.2011, 09:08:23

[mortus]

1. Linia 107: Przemyśl, czy w tym miejscu koniecznie musisz użyć funkcji exit, bowiem należy pamiętać, że exit powoduje zakończenie wykonywania skryptu.
2. Linia 78: Warunek będzie spełniony, jeśli wszystkie obowiązkowe pola będą puste, a chyba powinien być spełniony, jeśli któreś ze wszystkich obowiązkowych pól będzie puste. Zamień znaki && na ||.
3. Linie 108-109: Dopóki wyżej używasz funkcji exit dopóty wszystko jest w porządku. Ale zamiast tej funkcji exit powinieneś dodać słówko else między nawiasami z tych linii.
4. Linie 110-162: W zapytaniu INSERT używasz ` (gravisów), a powinny być ' (apostrofy) lub " (cudzysłowy).

@worek A co robią ' (apostrofy) przy nazwach kolumn? Tutaj właśnie mają być ` (gravisy). Poza tym if to nie pętla, a instrukcja warunkowa i while-em czy foreach-em tego nie zastąpisz.

Ten post edytował mortus 30.05.2011, 09:09:28

[webdice]

INSERT INTO [tabela] ('[kolumna z wartościami]' ,'[kolumna z wartościami]') VALUES ('wartość' , 'wartość');

Co Ty za głupoty opowiadasz? Zaglądnij do manuala.

[worek]

Heh chwytacie za słówka oczywiście o warunek chodziło a wy od razu jazdę robicie, a czy panowie są pewni czy to są wszystkie kolumny z tabeli wypełniane?? Np Samochód jest 13 kolumną a nie jest uzupełnianą i co wtedy? Zonk. Nie znam struktury tabeli to nie wiem.

[mortus]

Stosujesz coś koło 20 warunkó if czy nie lepiej dać foreach lub while i wtedy do tego warunek?? ... Co do tych if-ów brakuje klamr otwierających i zamykających.

Zrobienie takiej pętli foreach wcale nie jest takie proste, jak sobie to wyobrażasz i żadnych klamer nie brakuje. Podstawy podstaw!

[worek]

"A róbta jak chceta" - człowiek chce pomóc to na tym kiepsko wychodzi. Powodzenia z PHP życzę.