[PHP]PDO UPDATE problem, jw Opcje

[miras]

[PHP] pobierz, plaintext 
	public function update_weather($id, $data) {
        if (!empty($_POST['check'])) {
            $i = 0;
            $len = count($data);
            foreach($data as $name => $val):
                if ($i!=$len-1) {
                    if ($i==$len-2) {
                        $n .= $name.'=?';
                        $v .= $val;
                    } else {
                        $n .= $name.'=?,';
                        $v .= $val.",";
                    }
                }
                $i++;
            endforeach;
 
                $this->activ = $this->pdo->prepare("UPDATE weather SET $n WHERE id=?");
               $this->activ->execute(array($v, $id));
 
    	}	
    }
[PHP] pobierz, plaintext 

Błąd:

Invalid parameter number: number of bound variables does not match number of tokens' in ....

Ma ktoś pomysł?

[nospor]

masz jedno ? a przypisujesz dwie wartosci... komunikat bledy wyrazne ci na to wskazuje. Masz miec tylko wartosci ile masz znakow zapytania

[kapslokk]

Masz jeden pytajnik, a przesyłasz w array'u 2 wartości?

[miras]

No ok, ale

[PHP] pobierz, plaintext 
var_dump(count(explode(",", $v)));  == 66
 
i 
 
var_dump(count(explode(",", $n)));  == 66 
[PHP] pobierz, plaintext 

[nospor]

zrozum, ze znaki zapytania maja isc do PREPARE. Do EXECUTE przekazywac masz tylko wartosci.
A teraz w prepare masz tylko jedno ? a w execute dajesz dwie wartosci...

[kapslokk]

Ale masz $n w zapytaniu, a nie pytajnik, czyli do zbindowania zostaje CI jeden pytajnik, a chcesz bindować 2 wartości.

@nospor
Czy Ty zawsze musisz być szybszy?

Ten post edytował kapslokk 16.11.2015, 18:14:28

[miras]

Znaki zapytania przechowuje właśnie zmienna $n, dump tej zmiennej:

[PHP] pobierz, plaintext 
 
data=?,gdansk_d=?,gdansk_n=?,suwalki_d=?,suwalki_n=?,olsztyn_d=?,olsztyn_n=?,szczecin_d=?, itd do 66...
 
[PHP] pobierz, plaintext 

dump $v

[PHP] pobierz, plaintext 
0000-00-00,6,6,6,6,6,66,6,6,6,6,6,6,666,6,666,6,6,6,66,6,6,6,6,66,6,6,  itd do 66...
[PHP] pobierz, plaintext 

Ten post edytował miras 16.11.2015, 18:17:52

[nospor]

Ok, ale do EXECUTE masz przekazac tablice wartosci. A ty teraz przekazujesz tekst oddzielony przecinkami pod $v (IMG:style_emoticons/default/smile.gif)

[miras]

czyli to co jest pod $v zmienić na $v[] = $val, nie łapie już...

[nospor]

No bo potem w EXECUTE wkladasz tablice w tablice. No skup sie troche. Ma byc jedno wymiarowa tablica (IMG:style_emoticons/default/smile.gif)

[miras]

[PHP] pobierz, plaintext 
$this->activ->execute($v, array($id));
[PHP] pobierz, plaintext 

Również bez rezultatu....

[kapslokk]

[PHP] pobierz, plaintext 
$values = explode(',',$v);
$values[] = $id;
$this->activ->execute($values);
[PHP] pobierz, plaintext 

Tylko zamiast explodować, zrób sobie wcześniej tak, żeby Ci zamiast tego stringu generowało tablicę, na koniec doklej do niej $id.

Ten post edytował kapslokk 16.11.2015, 18:37:40