Почему возникает ошибка в запросе?

Question

[ortsuev33]

var_dump($arrayId);
		$count=count($arrayId);
		$dbconn=Db::getConnection();
		$sql='SELECT * FROM city WHERE id=?';

			$query=$dbconn->prepare($sql);
		
				for ($i=0;$count>$i;  $i++) {
					$query->execute(
		      		 $arrayId );

		   		


		}
		$res=$query->fetchAll(PDO::FETCH_ASSOC);
		var_dump($res);

	}

Comments

[ortsuev33]

В переменной $arrayId массив ,Когда отправляю один запрос все нормально

Answer 1

[Rsa97]

Без текста ошибки непонятно, но подозреваю, что из-за неправильной передачи массива $arrayId.
К тому же, то, что вы написали, в принципе работать не будет. Вам вернутся данные только последнего выполненного запроса.

Comments

[ortsuev33]

Rsa97 $res=$query->fetchAll(PDO::FETCH_ASSOC); если переместить это в цикл будет же работать!?

array(2) { [0]=> int(10) [1]=> int(7) }
Fatal error: Uncaught PDOException: SQLSTATE[HY093]: Invalid parameter number: number of bound variables does not match number of tokens in D:\OSPanel\domains\flip.ru\models\Advert.php:438 Stack trace: #0 D:\OSPanel\domains\flip.ru\models\Advert.php(438): PDOStatement->execute(Array) #1 D:\OSPanel\domains\flip.ru\models\Advert.php(460): Advert::checkCityDb(Array) #2 D:\OSPanel\domains\flip.ru\models\Advert.php(198): Advert::checkCityId(Array) #3 D:\OSPanel\domains\flip.ru\controllers\FormController.php(4): Advert::AdvertAdd() #4 D:\OSPanel\domains\flip.ru\components\Router.php(51): FormController->actionAdd() #5 D:\OSPanel\domains\flip.ru\index.php(19): Router->run() #6 {main} thrown in D:\OSPanel\domains\flip.ru\models\Advert.php on line 438

[Rsa97]

ortsuev33, Тогда будете каждый раз перезаписывать $res.
Циклы при обращении к базе данных - зло. Перепишите запрос с использованием IN и получайте все нужные данные одним запросом.

$arrayId = array_filter($arrayId, function($a) { return is_numeric($a); });
$sql='SELECT * FROM city WHERE id IN (' . implode(',', $arrayId) . ')';

[Rsa97]

ortsuev33, У вас один параметр подставляется в запросе? Значит в execute должен быть массив строго с одним элементом.
Вам же английским по белому пишет:

Invalid parameter number: number of bound variables does not match number of tokens

[ortsuev33]

Rsa97, я английский не знаю(( а insert тоже можно делать через IN просто у меня еще такой код

$sql='INSERT INTO adCoor 	(idAd,
									idCity,
									lon,
									lat,
									nameHouse,
									numberHouse
									)
					VALUES (?,?,?,?,?,?)';

		
		$query=$dbconn->prepare($sql);
		
		for ($i=0;$count>$i;  $i++) {
				
    $query->execute(array(
        $dataId,
        $data['idCity'][$i],
        $data['lon'][$i],
        $data['lat'][$i],
        $data['nameHouse'][$i], 
        $data['numberHouse'][$i]

    ));

   		


}

и почему циклы зло в случае бд? потому что execute делает каждый раз запрос к бд(в цикле)?

[Rsa97]

ortsuev33, IN - это условие, проверяющее вхождение значения в массив.
А английский учите. Без него в компьютерных отраслях работать практически невозможно. Хотя бы со словарём, но понимать надо.

[ortsuev33]

Rsa97, понятно ,спасибо большое