[PHP][PDO] Błąd podczas bindowania Opcje

[mattix19]

Witam,
mam taki kod:

[PHP] pobierz, plaintext 
     $tab = array(':tytul' => $nazwa, ':autor' => $autor,':cena' => $cena, ':typ' => $typ,':opis' => $opis, ':rodzaj' => $rodzaj,
     ':stanowiskwgarazu' => $garaz, ':katnachyleniadachu' => $kat, ':podpiwniczenie' => $piwnica, ':poddasze' => $poddasze, ':kominek' => $kominek,
     ':szerokoscdzialki' => $szerokosc, ':dlugoscdzialki' => $dlugosc, ':powuzytkowa' => $pow_uzytk, 
     ':powzabudowy' => $pow_zabud, ':powdachu' => $pow_dach, ':typkotla' => $typ_ogrzewania, ':idpracowni' => $id_pracowni , ':idkat' => $id_kat);
 
 $stm = $this->db->prepare("INSERT INTO projekty_projekt (tytul, autor, cena, typ, opis, rodzaj, stanowisk_w_garazu,  kat_nachylenia_dachu, podpiwniczenie, poddasze, kominek, szerokosc_dzialki, dlugosc_dzialki, pow_uzytkowa, pow_zabudowy, pow_dachu, typ_kotla, projekty_pracownie_id_pracowni, projekty_kategorie_id_kategorii )
 					VALUES (:tytul, :autor, :cena, :typ, :opis, :rodzaj, :stanowiskwgarazu, :katnachyleniadachu, :podpiwniczenie, :poddasze, :kominek, :szerokoscdzialki, :dlugoscdzialki, :powuzytkowa, :powzabudowy, :powdachu, :typkotla, :idpracowni, :idkat )");	
foreach($tab as $k => $v){
 
		 	          if(is_int($v))
                    $param = PDO::PARAM_INT;
                elseif(is_bool($v))
                    $param = PDO::PARAM_BOOL;
                elseif(is_null($v))
                    $param = PDO::PARAM_NULL;
                elseif(is_string($v))
                    $param = PDO::PARAM_STR;
                    else
                    $param = '';   
                if($param)
                   $stm->bindValue($k, $v, $param);	
		 }				
$il = $stm->execute();
[PHP] pobierz, plaintext 

Problem polega na tym ze jak robie dodawanie do bazy to pojawia sie taki blad:

[PHP] pobierz, plaintext 
Warning: PDOStatement::execute() [pdostatement.execute]: SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in /home/.../admin.class.php on line 297
[PHP] pobierz, plaintext 

jednak gdy robie bindowanie z reki bez petli foreach:

[PHP] pobierz, plaintext 
		           $stm->bindValue(':tytul', $nazwa);	
             $stm->bindValue(':autor', $autor);
              $stm->bindValue(':cena', $cena);
               $stm->bindValue(':typ', $typ); 
               $stm->bindValue(':opis', $opis);
                $stm->bindValue(':rodzaj', $rodzaj);
                 $stm->bindValue(':stanowiskwgarazu', $garaz);
                 $stm->bindValue(':katnachyleniadachu', $kat);
                 $stm->bindValue(':podpiwniczenie', $piwinica);
                 $stm->bindValue(':poddasze', $poddasze);
                 $stm->bindValue(':kominek', $kominek);
                 $stm->bindValue(':szerokoscdzialki', $szerokosc);
                 $stm->bindValue(':dlugoscdzialki', $dlugosc);
                 $stm->bindValue(':powuzytkowa', $pow_uzytk);
                 $stm->bindValue(':powzabudowy', $pow_zabud);
                 $stm->bindValue(':powdachu', $pow_dach);
                 $stm->bindValue(':typkotla', $typ_ogrzewania);
                 $stm->bindValue(':idpracowni', $id_pracowni);
                 $stm->bindValue(':idkat', $id_kat);
[PHP] pobierz, plaintext 

to zamiast 400 roznych rekordow pojawia mi sie tylko w bazie 1, pierwszy. Caly kod powinien sie wykonac okolo 400 razy bo tyle rekordow pobralem z xml. Tablice stworzona mam dobrze bo var_dump ja ladnie wyswietlil. Problem twki w samym dodawaniu. Dlaczego dodaje mi jeden rekord i jak z powyzszego zapytania pozbyc sie tego bledu?

Ten post edytował mattix19 31.03.2013, 13:03:39

[Crozin]

Jeżeli zmienna nie jest typu int/string/bool/null zmiennej $param przypisujesz pusty string. Następnie masz IFa, który zbinduje wartość tylko w przypadku, gdy zmienna $param ma wartość, a ta nie będzie miała wartości (pusty string jest tożsamy z FALSE przy zwykłym porównaniu), jeżeli zmienna będzie obiektem, tablicą bądź typem float/double. else $param = ''; zamień na else $param = PDO::PARAM_STR;.

[mattix19]

dobra to dziala ale w petli for nadal dodaje mi tylko jeden 1 rekord. Czy musze zamknac jakos polaczenie z baza zeby dodal wszystkie? closeCursor nie dziala.

[Crozin]

Ale nie widać tutaj żadnej pętli, która miałaby dodać więcej niż jeden rekord. Jeżeli taka istnieje, to domyślam się, że obejmuje ona pow. kod, w takim wypadku:
1. PrepareStatement powinieneś przenieść przed pętle - nie ma potrzeby każdorazowo tego wykonywać.
2. PDOStatement::closeCursor() powinieneś umieścić zaraz za PDOStatement::execute()

[mattix19]

wyglada to dokladnie tak:

[PHP] pobierz, plaintext 
//parser to tablica z danymi z xml.
$stm = $this->db->prepare("INSERT INTO projekty_projekt (tytul, autor, cena, typ, opis, rodzaj, stanowisk_w_garazu,  kat_nachylenia_dachu, podpiwniczenie, poddasze, kominek, szerokosc_dzialki, dlugosc_dzialki, pow_uzytkowa, pow_zabudowy, pow_dachu, typ_kotla, projekty_pracownie_id_pracowni, projekty_kategorie_id_kategorii ) VALUES (:tytul, :autor, :cena, :typ, :opis, :rodzaj, :stanowiskwgarazu, :katnachyleniadachu, :podpiwniczenie, :poddasze, :kominek, :szerokoscdzialki, :dlugoscdzialki, :powuzytkowa, :powzabudowy, :powdachu, :typkotla, :idpracowni, :idkat )");
 
for ($i=0; $i < count($parser); $i++) { 
 $tab = array(':tytul' => $nazwa, ':autor' => $autor,':cena' => $cena, ':typ' => $typ,':opis' => $opis, ':rodzaj' => $rodzaj,
     ':stanowiskwgarazu' => $garaz, ':katnachyleniadachu' => $kat, ':podpiwniczenie' => $piwnica, ':poddasze' => $poddasze, ':kominek' => $kominek,
     ':szerokoscdzialki' => $szerokosc, ':dlugoscdzialki' => $dlugosc, ':powuzytkowa' => $pow_uzytk, 
     ':powzabudowy' => $pow_zabud, ':powdachu' => $pow_dach, ':typkotla' => $typ_ogrzewania, ':idpracowni' => $id_pracowni , ':idkat' => $id_kat);
 
 foreach($tab as $k => $v){
 
	 if(is_int($v))
                    $param = PDO::PARAM_INT;
                elseif(is_bool($v))
                    $param = PDO::PARAM_BOOL;
                elseif(is_null($v))
                    $param = PDO::PARAM_NULL;
                elseif(is_string($v))
                    $param = PDO::PARAM_STR;
                else
                    $param =  PDO::PARAM_STR;   
                if($param)
                   $stm->bindValue($k, $v, $param);	
		 }
			$il = $stm->execute();
			$stm->closeCursor();
}
[PHP] pobierz, plaintext 

i w bazie pojawia sie tylko jeden rekord. Powinno okolo 400.

[Crozin]

1. Nie wiem czy skonfigurowałeś PDO by rzucało wyjątkami czy nie, upewnij się jednak, że PDOStatement::execute() zawsze zwraca TRUE.
2. Nie widzę, byś gdziekolwiek zmieniał zawartość zmiennych przekazywanych jako parametry zapytania. Cały czas operujsz na tych samych danych.

[mattix19]

nie no zmienne sa tworzone juz w petli porostu nie dodalem tu. dane sa dobrze zrobione zapytanie tez lecz nie mam pojecia czemu to nie rusza.... execute() zwraca true tylko za 1 obiegiem petli potem juz false...

[Crozin]

No to jeżeli PDOStatement::execute() zwraca FALSE masz jakiś błąd. Sprawdź jaki to dokładnie błąd: http://www.php.net/manual/en/pdo.error-handling.php