Почему не работает PDO и плейсхолдеры?

Question

[teodor7teodor7]

Почему не получается построить запрос используя плейсхолдеры. Использую openserver для тестирования

if (isset($sql)) {
             $sql = "INSERT INTO  comments ( :columnts ) VALUES ( 1233 )";
  $params = 'id';
            $stm = self::$dbh->prepare($sql );
            $stm->bindParam(':columnts', $params);
            $stm->execute();
            return true;
        }
        return false;

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '? ) VALUES ( 1233 )' at line 1' in C:\OpenServer\domains\commentWall.loc\app\core\db\DbAdapter.php:89 Stack trace: #0 C:\OpenServer\domains\commentWall.loc\app\core\db\DbAdapter.php(89): PDO->prepare('INSERT INTO co...') #1 C:\OpenServer\domains\commentWall.loc\app\core\Model.php(41): app\core\db\DbAdapter->query('INSERT INTO co...', Array) #2 C:\OpenServer\domains\commentWall.loc\app\controllers\ControllerMain.php(36): app\core\Model->create(Array, Array) #3 C:\OpenServer\domains\commentWall.loc\app\Router.php(38): app\controllers\ControllerMain->actionIndex(NULL) #4 C:\OpenServer\domains\commentWall.loc\App.php(14): app\Router::execute(Array) #5 C:\OpenServer\domains\commentWall.loc\index.php(14): App::execute(Array) #6 {main} thrown

Answer 1

[Nik Gubin]

Если я не ошибаюсь, то плейсхолдером нельзя забиндить имя колонки. Только значения.

Comments

[D3lphi]

Вы не ошибаетесь.

[teodor7teodor7]

$sql = "INSERT INTO comments ( :columnts ) VALUES ( 1233 )";
суть в том что мне нужно сделать запрос универсальным то есть что бы таблица имя колонки и значения можно было подставлять.

$params[':tableName'] = $this->tableName;
$params[':columns'] = 'id'; 
$params[':value'] = 122;
$sql = "INSERT INTO  :tableName ( :columnts ) VALUES ( :value )";
$stm = self::$dbh->prepare($sql );
$stm->execute( $params);

то же не работает. Получатся что нужно по старинке $sql = "INSERT INTO $params[':tableName'] ( $params[':columns'] ) VALUES ( $params[':value'] )"; так?

[Евгений Вольф]

Мне тоже так кажется, поддерживаю предыдущего комментатора :D

[Nik Gubin]

teodor7teodor7: я могу вас натолкнуть к поиску построителей запросов. Query Builder. К примеру в Laravel это выглядит так (с вашими параметрами пишу пример):

DB::table($this->tableName)->insert([
    'id' => 122 // вместо 'id' можно указать так же и переменную
]);

[Boris Köln]

teodor7teodor7: К чему такая упоротость? Три комментатора с высоким рейтингом говорят, что забиндить можно только значения, а вы не верите им. Ну, я буду четвертым.

Можно только так (если имя таблицы и имя колонки задаете вы в коде, а не получаете от клиента):

$sql = "INSERT INTO {$this->tableName} ( id ) VALUES ( :value )";

[Nik Gubin]

teodor7teodor7: если знакомы с composer и используете в проекте, то можете покопаться в этом пакете.

Но в вашем случае проще будет проверить переменную на допустимые значения, а после просто "вшить" в строку с запросом, а значение сбиндить через плейсхолдер. Вы же успокоите меня и не скажете, что имя колонки прилетает от пользователя?

[teodor7teodor7]

Boris Korobkov: Та нет никаких упоростей) Просто документацию читаю видимо не дапонял.

Answer 2

[Сергей Рыбаков]

Попробуй вместо placeholder использовать value