Problem z PDO, bindowanie parametrów, PDOStatement::execute(): SQLSTATE[HY093] Opcje

[MMySlime]

Witam,

mam problem. Wywala mi errora gdy (tak sądzę) próbuję zbindować parametry w takim oto sobie kodzie:

[PHP] pobierz, plaintext 
 
 public function select($table,$where = NULL,$add = NULL,$how = '*'){
 
        if($where != NULL) $where = 'WHERE '.$where;
 
        $select = $this->db->prepare('SELECT :how FROM :table :where :add');
 
        $select->bindParam(':how', $how);
        $select->bindParam(':table', $table);
        $select->bindParam(':where', $where);
        $select->bindParam(':add', $add);
 
        $select->execute();
 
    }
 
	public function insert( $table, $value){
 
        $key = array_keys($value);
 
        $value = array_values($value);
 
        $i = 1;
 
        $ij = 0;
 
        $j = null;
 
        foreach($key as $keys){
            $j++;   
        }
 
        $values = '';
 
        foreach($key as $keyss){
            $values .= $values . $keyss . ' = ?, ';
        }
 
        $values = substr( $values, 0, -2 );
 
        $insert = $this->db->prepare('INSERT INTO '.$table.' SET '.$values);
 
        while($i < $j){
            $insert->bindParam($i,$value[$ij]); 
            $i++; 
            $ij++;
        } 
 
        $insert->execute();
 
}
[PHP] pobierz, plaintext 

Dokładny kod błędu:

[PHP] pobierz, plaintext 
Warning:  PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in (Jakaś ścieżka) on line 85
[PHP] pobierz, plaintext 

Nie ma co owijać w bawełnę, powiem że nie wiem zupełnie co ten błąd oznacza.
Myślę że dobrze zbindowałem wartości ale skoro PHP wywala błąd to coś jest nie tak...
Próbowałem różnych metod jednak żadna z "moich" nie działa...
Prosiłbym o pomoc.

Dziękuję za wszystkie odpowiedzi (IMG:style_emoticons/default/smile.gif)

[abort]

Ilość parametrów (parametry to ciąg znaków zaczynający się od dwukropka) podanych w $this->db->prepare(); musi być równa ilości bindowań.
W listingu na początku w liniach 6-11 masz ok, ale już w liniach 41-44 tak nie jest.

[zegarek84]

funkcja "select" raczej nie powinna działać - nie chce mi się sprawdzać ale tam nie rozumiesz idei bindowania i to zapytanie mogłeś po prostu skleić jako tekst...

w funkcji "insert" masz jeden podstawowy błąd:

Kod

$values .= $values . $keyss . ' = ?, ';

zdecyduj się na
$values .= $keyss . ' = ?, ';
lub
$values = $values . $keyss . ' = ?, ';

poza tym zastanawia mnie, czy nie powinno Ci zwrócić niepoprawne zapytanie - nie wiem czy na końcu INSERT może być przecinek i czy to poprawne, zrobił bym to raczej inaczej... dlaczego też w pętli zliczasz wielkość tablicy jak są do tego funkcje??...

[MMySlime]

@abort: Specjalnie dlatego zrobiłem while aby bindowała wszystkie parametry które są liczbami a nie ciągami znaków, jeżeli dobrze w php.net piszą to można wykorzystywać liczby zamiast stringów (IMG:style_emoticons/default/wink.gif)

@zegarek84: Rzeczywiście zrobiłem błąd, skoro dodaję wartość do stringa to nie dodaje tej samej wartości :/ I jakby co spójrz na funkcję substr, obcina ona końcówkę którą jest ", "

Dzięki za odpowiedzi (IMG:style_emoticons/default/wink.gif)

==EDIT==

@abort: A więc tak... strzeliłem faila. Miałem w kodzie napisać bindValue a napisałem bindParam ._.

==EDIT2==

Tak czy siak nie działa... zastosowałem się do waszych poleceń i nadal "wywala błęda" :/

==EDIT3==

O matko już 3 edit... w każdym bądź razie popełniłem błąd, tu chodziło głównie o while, zmienna $i była zła, na początku powinna być 0 i przed wykonaniem bindowania powiększyć ją o jeden, w tym cała magia oraz problem.
Ważne że sobie poradziłem i problem z głowy. Dzięki wszystkim za pomoc (IMG:style_emoticons/default/wink.gif)

Ten post edytował MMySlime 16.11.2012, 20:51:12

[abort]

To może inaczej, wrzuć ten kod:

[PHP] pobierz, plaintext 
$prepare = 'INSERT INTO '.$table.' SET '.$values;
$insert = $this->db->prepare($prepare);
 
$binds = '';
while($i < $j){
$insert->bindParam($i,$value[$ij]);
$binds .= ':' . $i . '=' . $value[$ij] . ' ';
$i++;
$ij++;
}
 
// Dobrze byłoby wiedzieć, co tu wypluje...
echo 'Prep:' . $prep . ',   bindings:' . $binds';
$insert->execute();
[PHP] pobierz, plaintext 

I pokaż co wypluje. Druga sprawa: czy w linii 21 zaprezentowanego kodu nie masz czasem literówki?

Ten post edytował abort 16.11.2012, 22:00:19

[MMySlime]

@abort: Dzięki ale już nie trzeba, poradziłem sobie gdyż był błąd w pętli while (IMG:style_emoticons/default/wink.gif) Zamiast $i = 1 powinno być 0 oraz przez bindem $i++, automatycznie to powiększało zmienną i usuwało problem.

==EDIT znowu==

U mnie już nie ma błędu ale u ciebie jest (IMG:style_emoticons/default/wink.gif) Po $binds nie powinno być '

Ten post edytował MMySlime 16.11.2012, 22:33:41

[abort]

U mnie już nie ma błędu ale u ciebie jest (IMG:style_emoticons/default/wink.gif) Po $binds nie powinno być '

A i owszem - ale takie rzeczy zdarzają się (mimo wszystko) przy pisaniu "na gorąco". Nie miało to na celu świadome wprowadzenie Cię w błąd.