[PDO] Błąd wykonania skryptu, SQLSTATE[HY093]: Invalid parameter number:... Opcje

[ze4lot]

Błąd:

Kod

Warning: PDOStatement::execute() [pdostatement.execute]: SQLSTATE[HY093]: Invalid parameter number: parameter was not defined in /system/autoresponder.php on line 84

Skrypt, który go generuje. Nie mam zielonego pojęcia jak to rozwiązać... proszę o pomoc.

[PHP] pobierz, plaintext 
<?
@include('engine.php');
class arEngine extends standart
{
	public function getEmailList($table, $conditions)
	{
		$this->dbs();
 
		foreach($conditions as $name => $value)
		{
			if(!is_array($value))
			{
				$cond[][$name] = $value;
			} else {
 				foreach($value as $element=>$val)
				{
					$cond[][$name] = $val;
				}
			}
		}
		$i=0;
		$j=0;
		$query = "SELECT * FROM ".$table." WHERE ";
		foreach($cond as $id => $name)
		{
			foreach($name as $name2 => $val);
			{
				$temp = $name2."=:".$name2;
				$queryElement[] = $temp;
 
				if($i>0)
				{
					$end = end($queryElement);
					if(prev($queryElement) == $temp)
					{
						++$j;
					} else {
						$j=0;
					}
					$bind[] = array(":".$name2.$j, $val);
				} else {
					$bind[] = array(":".$name2.$j, $val);
				}
 
				$i++;
			}
		}
		$queryElementSize = count($queryElement);
		$j=0;
		for($i=0;$i<$queryElementSize;$i++)
		{
			if($queryElement[$i] != $queryElement[$i-1] && $queryElement[$i] != $queryElement[$i+1])
			{		
				$query .= $queryElement[$i];
				if($i<($queryElementSize-1))
				{
					$query .= " AND ";
				}				
			} else {
				if($queryElement[$i] !=  $queryElement[$i-1])
				{
					$query .= "(";
				}	
				$query .=  $queryElement[$i];
				if( $queryElement[$i] ==  $queryElement[$i+1])
				{
					$query .= $j." OR ";
				} else {
					$query .= $j.") ";
					if($queryElement[$i+1])
					{
				  		$query .= "AND ";
					}
				}
				$j++;
			} 
		}
		unset($j);
		$stmt=$this->pdo->prepare($query);
		foreach($bind as $bindValue)
		{
			$stmt->bindValue($bindValue[0], $bindValue[1]);
		}
		$stmt -> execute();
		while($row = $stmt -> fetch())
		{
			foreach($row as $name->$value)
			{
				$list[$name] = $value;
			}
		}
		$stmt -> closeCursor();
		if(count($list)>0)
		{
			return $list;
		} else {
			return false;
		}
	}
}
 
$conds = array('cos1'	=> array(1, 2, 'abc'),
					'cos2'	=> 'foo',
					'cos3'	=> 'bar'
					);
 
$ar = new arEngine;
$email_list = $ar -> getEmailList('konta', $conds);
print_r($email_list);
?>
[PHP] pobierz, plaintext 

engine.php

[PHP] pobierz, plaintext 
class standart
	{
	...		
		public function dbs() {
			$db_user = "usuniete by mod";
			$db_pass = "usuniete by mod";
			$db_dsn = "usuniete by mod";
			$this->pdo = new PDO($db_dsn, $db_user, $db_pass);
		}
	...
	}
[PHP] pobierz, plaintext 

Powód edycji: [nospor]: usunalem dane dostępowe do bazy

[Zyx]

Na mój gust Twój super-rozbudowany algorytm generowania zapytania ma jakiś błąd i albo podpina za mało argumentów pod zapytanie, albo generuje za dużo argumentów. Sprawdź w pierwszej kolejności, jaka jest postać uzyskanego zapytania oraz jakie dane są do niego pakowane.

[nospor]

a ja dodam jeszcze, że Twoj problem z obiektówką nie ma żadnego związku, więc przenoszę.

[phpion]

A ja dodam jeszcze... (IMG:style_emoticons/default/winksmiley.jpg) , że świetnym posunięciem było publiczne podawanie danych dostępowych do bazy danych (IMG:style_emoticons/default/smile.gif)

[ze4lot]

najwyższa pora pracować w dzień a nie w nocy...
zaraz się biorę za przerabianie. Dzięki

*** EDIT BŁĄD ZNALEZIONY ***

zapytanie $query:

[SQL] pobierz, plaintext 
SELECT * FROM konta WHERE (cos=:cos0 OR cos=:cos1 OR cos=:cos2) AND foo=:foo AND bar=:bar
[SQL] pobierz, plaintext 

pętla tworząca bindy generuje:

[PHP] pobierz, plaintext 
bindValue(:cos0, wartosc1);
bindValue(:cos1, wartosc2);
bindValue(:cos2, wartosc3);
bindValue(:foo0, wartosc4);
bindValue(:bar0, wartosc5);
[PHP] pobierz, plaintext 

czyli

bindy, które się nie powtarzają (foo i bar) mają dopisaną wartość '0' natomiast w zapytaniu są bez tego zera.

*** EDIT ***

może jest tu jakaś osoba co dysponuje większą ilością czasu i potrafi moje wypociny zamienić na prawdziwe OOP? na prawdę przydała by mi się porządna lekcja

Ten post edytował ze4lot 16.11.2009, 15:29:51