Почему не работает SQL запрос?

Question

[Николай Громов]

Здравствуйте, подскажите почему не работает sql запрос? Вот код с запросом:

function getProductsFromArray($itemsIds){
    global $pdo;
    try {
        $strIds = implode($itemsIds, ', ');
        $sql = "SELECT *
             FROM `products` 
             WHERE 
             id in ({$strIds})";
        $stmt = $pdo -> query($sql);
    d($stmt);
    return createSmartyRsArray($stmt);
    } catch (Exception $e) {
        exit($e->getMessage());
    }
}

Переменная $strIds в запросе написан как видеоуроке, на и без скобок фигурных и обычных он выдает ошибку одну и туже
SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near '' at line 4
функция d($stmt) просто отладочная, но до нее дело не доходит.
В переменную $strIds ID приходят нормально.

Comments

[Евгений Куманин]

implode(', ', $itemsIds);

[Дмитрий С]

SQL запрос сам покажите, а код его генерирующий.

Answer 1

[Yevgeniy Kisselyov]

implode неправильный
должно быть
$strIds = implode(', ', $itemsIds);
а лучше даже (если id в таблице не int)
$strIds = "'" . implode("','", $itemsIds) . "'";
и в запросе скобки фигурные убрать можно

непонятно как при таком implode может быть

В переменную $strIds ID приходят нормально.

Answer 2

[Дмитрий Энтелис]

1. Если начали использовать PDO - используйте его правильно: phpfaq.ru/pdo
2. Если собираете запрос руками - первое с чего нужно начинать - посмотреть на сам sql запрос.
Вставляете после $sql = die( $sql ) - и сразу многое станет понятней

Comments

[Николай Громов]

function getProductsFromArray($itemsIds){
    global $pdo;
    try {
        $strIds = implode(', ', $itemsIds);
        $stmt = $pdo -> prepare("SELECT *
             FROM products 
             WHERE 
             id = :id");           
        $stmt -> bindParam(':id', $strIds);
        $stmt -> execute(); 
    return createSmartyRsArray($stmt);
    } catch (Exception $e) {
        exit($e->getMessage());
    }
}

Вот так правильнее?
Прошлый код я писал с урока что бы разобраться почему не работает!

[Дмитрий Энтелис]

Николай Громов:
про sql - правильнее.

в целом:
1) globals не используйте в принципе. Используете или паттерн singleton, или хотя бы передавайте в конструктор

class ProductsModel 
{
   private $pdo;

   function __construct ($pdo)
   {
      $this->pdo = $pdo;
   }

   function getProductsFromArray($itemsIds)
   {
      ...
   }
}

2) посмотрите на www.php-fig.org/psr/psr-2 , мне лично лишние пробелы в "$stmt -> execute(); " очень режут глаз