Forum PHP.pl

Cześć. Zaczynam przygodę z PDO i podczepianiem więc proszę o wyrozumiałość. Przy próbie UPLOADowania danych napotykam na problem.

Komunikat: blad db : SQLSTATE[42000]: Syntax error or access violation: 1064 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 '( `id_pan`, `nazwa_en`, `nazwa_de`, `nazwa_fr`, `nazwa_esp`, `nazwa_pl`, `opinia' at line 1

ewentualnie komunikat typu: blad db : SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens

Sprawdziłem ilość parametrów (wykonałem opcję z dodaniem id_mia i bez id_mia , - to parametr który wskazuje mi miejsce edycji danych w bazie.
Dokonałem sprawdzenia typów zmiennych wejściowych (zgadzają się) oraz przeprowadziłem próbę na bindValue oraz bindParam

Gdzie popełniam błąd? Proszę o pomoc.

kod funkcji:

[PHP] pobierz, plaintext 
public function AktualizujMiasto($polaczenie,$id_pan,$id_mia,$nazwa_en,$nazwa_de,$nazwa_fr,$nazwa_esp,$nazwa_pl,$opinia_pl,$opinia_en,$status,$y,$x,$data1,$data2,$data3) {
 
	try
   {
 
     $this->polaczenie=$polaczenie;
	 $this->id_pan=$id_pan;
	 $this->id_mia=$id_mia;
	 $this->nazwa_pl=$nazwa_pl;
	 $this->nazwa_en=$nazwa_en;
	 $this->nazwa_de=$nazwa_de;
	 $this->nazwa_fr=$nazwa_fr;
	 $this->nazwa_esp=$nazwa_esp;
	 $this->opinia_pl=$opinia_pl;
	 $this->opinia_en=$opinia_en;
	 $this->status=$status;
	 $this->y=$y;
	 $this->x=$x;
	 $this->data1=$data1;
	 $this->data2=$data2;
	 $this->data3=$data3;
 
$stmt = $this->polaczenie->prepare('UPDATE `miasto` (  `id_pan`, `nazwa_en`, `nazwa_de`, `nazwa_fr`, `nazwa_esp`, `nazwa_pl`, `opinia_pl`, `opinia_en`, `status`, `y`, `x`, `data1`, `data2`, `data3`) 
VALUES (
		:id_pan,	
		:nazwa_en,
		:nazwa_de,
		:nazwa_fr,
		:nazwa_esp,
		:nazwa_pl,
		:opinia_pl,
		:opinia_en,
		:status,
		:y,
		:x,
		:data1,
		:data2,
		:data3)
WHERE `id_mia` =: id_miasta ');
	/*
$stmt->bindValue(':id_miasta', $this->id_mia, PDO::PARAM_INT);
$stmt->bindValue(':id_pan', $this->id_pan, PDO::PARAM_INT); // STR - ԡ񣵣h
$stmt->bindValue(':nazwa_en', $this->nazwa_en, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindValue(':nazwa_de', $this->nazwa_de, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindValue(':nazwa_fr', $this->nazwa_fr, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindValue(':nazwa_esp', $this->nazwa_esp, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindValue(':nazwa_pl', $this->nazwa_pl, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindValue(':opinia_pl', $this->opinia_pl, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindValue(':opinia_en', $this->opinia_en, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindValue(':status', $this->status, PDO::PARAM_INT, 3); 
$stmt->bindValue(':y', $this->y, PDO::PARAM_STR,10); 
$stmt->bindValue(':x', $this->x, PDO::PARAM_STR,10); 
$stmt->bindValue(':data1', $this->data1, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindValue(':data2', $this->data2, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindValue(':data3', $this->data3, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
*/
 
$stmt->bindValue(':id_miasta', $this->id_mia, PDO::PARAM_INT);
$stmt->bindParam(':id_pan', $this->id_pan, PDO::PARAM_INT); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_en', $this->nazwa_en, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_de', $this->nazwa_de, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_fr', $this->nazwa_fr, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_esp', $this->nazwa_esp, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_pl', $this->nazwa_pl, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':opinia_pl', $this->opinia_pl, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindParam(':opinia_en', $this->opinia_en, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindParam(':status', $this->status, PDO::PARAM_INT, 3); 
$stmt->bindParam(':y', $this->y, PDO::PARAM_STR,10); 
$stmt->bindParam(':x', $this->x, PDO::PARAM_STR,10); 
$stmt->bindParam(':data1', $this->data1, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindParam(':data2', $this->data2, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindParam(':data3', $this->data3, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
 
 
$ilosc = $stmt -> execute(); 
 
 if($ilosc > 0)
                        {
                                $_SESSION['info']='edycja miasta powiodła się ';
                        }
                        else
                        {
                                 $_SESSION['info']='problem z edycją miasta';
                        }
 
   }
   catch(PDOException $e)
   {
      echo 'blad db : ' . $e->getMessage();
   }  
 
 
}
[PHP] pobierz, plaintext 

Ten post edytował jeremiash 15.12.2012, 19:37:48

(IMG:style_emoticons/default/ohmy.gif) o kurcze...

_____________________________________
Na początku chcę podziękować MOTRUS-owi , za szybką i fachową pomoc. Przez niedopatrzenie traciłem mnóstwo czasu... Dzięki Ci bardzo!

Skoro temat już wisi to wrzucam poprawną składnię. Jakiemuś 'głąbowi' podobnemu do mnie , ewentualnie osobie uczącej się może się przydać (IMG:style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
 abstract class funkcyjka{
 
                private $polaczenie; 
		private $nazwa_pl;
		private $nazwa_en;
		private $nazwa_de;
		private $nazwa_fr;
		private $nazwa_esp;
		private $opinia_pl;
		private $opinia_en;
		private $status;
		private $data1;
		private $data2;
		private $data3;
		private $id_pan;
		private $id_mia;
		private $x;
		private $y;
 
public function AktualizujMiasto($polaczenie,$id_pan,$id_mia,$nazwa_en,$nazwa_de,$nazwa_fr,$nazwa_esp,$nazwa_pl,$opinia_pl,$opinia_en,$status,$y,$x,$data1,$data2,$data3) {
 
	try
   {
 
     $this->polaczenie=$polaczenie;
	 $this->id_pan=$id_pan;
	 $this->id_mia=$id_mia;
	 $this->nazwa_pl=$nazwa_pl;
	 $this->nazwa_en=$nazwa_en;
	 $this->nazwa_de=$nazwa_de;
	 $this->nazwa_fr=$nazwa_fr;
	 $this->nazwa_esp=$nazwa_esp;
	 $this->opinia_pl=$opinia_pl;
	 $this->opinia_en=$opinia_en;
	 $this->status=$status;
	 $this->y=$y;
	 $this->x=$x;
	 $this->data1=$data1;
	 $this->data2=$data2;
	 $this->data3=$data3;
 
$stmt = $this->polaczenie->prepare("UPDATE miasto SET id_pan= :id_pan , nazwa_en= :nazwa_en , nazwa_de= :nazwa_de , nazwa_fr= :nazwa_fr , nazwa_esp= :nazwa_esp , nazwa_pl= :nazwa_pl , opinia_pl= :opinia_pl , opinia_en= :opinia_en , 
status= :status , y= :y , x= :x , data1= :data1 , data2= :data2 , data3= :data3  
WHERE id_mia = :id_miasta ");
 
$stmt->bindParam(':id_miasta', $this->id_mia, PDO::PARAM_INT);
$stmt->bindParam(':id_pan', $this->id_pan, PDO::PARAM_INT); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_en', $this->nazwa_en, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_de', $this->nazwa_de, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_fr', $this->nazwa_fr, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_esp', $this->nazwa_esp, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':nazwa_pl', $this->nazwa_pl, PDO::PARAM_STR, 100); // STR - ԡ񣵣h
$stmt->bindParam(':opinia_pl', $this->opinia_pl, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindParam(':opinia_en', $this->opinia_en, PDO::PARAM_STR, 400); // STR - ԡ񣵣h
$stmt->bindParam(':status', $this->status, PDO::PARAM_INT, 3); 
$stmt->bindParam(':y', $this->y, PDO::PARAM_STR,10); 
$stmt->bindParam(':x', $this->x, PDO::PARAM_STR,10); 
$stmt->bindParam(':data1', $this->data1, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindParam(':data2', $this->data2, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
$stmt->bindParam(':data3', $this->data3, PDO::PARAM_STR, 40); // STR - ԡ񣵣h
 
 
$ilosc = $stmt -> execute(); 
 
 if($ilosc > 0)
                        {
                                $_SESSION['info']='edycja miasta powiodła się ';
                        }
                        else
                        {
                                 $_SESSION['info']='problem z edycją miasta';
                        }
 
   }
   catch(PDOException $e)
   {
      echo 'blad db : ' . $e->getMessage();
   }  
 
 
}
 
} // end funkcyjka
[PHP] pobierz, plaintext 

Pozdrawiam serdecznie

Ten post edytował jeremiash 15.12.2012, 22:04:12

Jak dla mnie to powinines tę klase przebudować.

1. Zrobić getery i setery, a nie przekazywać mnóstwo zmiennych w metodzie klasy. Co w chwili gdy nie nie chce aktualizować pierwszych 5 zmiennych? Muszę na siłe je wpisywać.
2. Masz pola prywatne w z tymi wpisami, następnie, przekazujesz je w metodzie, i zapisujesz to tych pól prywatnych, i zapisuje te pola prywatne. Po co? Skoro szybciej, prościej i łatwie było by w tym przypadku zapisać poprostu te zmienne które przekazujesz w metodzie? (i tak nie powinno być ich tyle)
3. Połączenie z baza danych ($polaczenie) nie przekazywał bym jako 1 pole w metodzie, tylko np w konstruktorze klasy, zachowując to połączenie dla inncyh metod w klasie.

Jest to moja pierwsza aplikacja w OOP więc dziękuję za rady. Owszem pkt. 3 - masz rację. Co do pkt. 2 myślę nad nim. Pkt. 1 (wczytuję dane do formularza, więc dokonuję zmian, reszta pozostaje. Nie widzę problemu.

Pozdrawiam i dzięki za zainteresowanie oraz porady