【MySQL】INSERT文でのシンタックスエラーを解決したい

Question

PHP(Version7.1)で以下のコードを実施したとき

<?php
function addPictOverViewTable(PDO $db, String $picture_id, $title, $category){

    try {
        $sql = "INSERT INTO picts_overview (picture_id, title, category, pins_count, register_datetime, update_datetime) VALUES (:picture_id, :title, :category, :pins_count, :register_datetime, :update_datetime)";
        $stmt = $db -> prepare($sql);
        $stmt -> bindParam(':picture_id', $picture_id, PDO::PARAM_STR);
        $stmt -> bindParam(':title', $title, PDO::PARAM_STR);
        $stmt -> bindParam(':category', $category, PDO::PARAM_STR);
        $stmt -> bindValue(':pins_count', 0, PDO::PARAM_INT);
        $datetime = new DateTime();
        $stmt -> bindValue(':register_datetime', $datetime->format('Y-m-d H:i:s'), PDO::PARAM_STR);
        $stmt -> bindValue(':update_datetime', $datetime->format('Y-m-d H:i:s'), PDO::PARAM_STR);
    } catch(PDOException $e) {
        echo $e->getMessage();//Remove or change message in production code
    }
}

以下のエラーが発生します。

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 ')' at line 7

どうやらINSERT文が間違っているエラーのようですが、一向に原因が分からず、困り果てております。
予約語の可能性や、スキーマとの不一致なども確認してみましたが、正しそうでした。

少しでも何かヒントになる情報を頂ければ、大変有り難く思います。
宜しくお願い致します。

Comments

[MasahikoHirata]

ざっくり見ると”INSERT INTO picts_overview (”の”（”が閉じてないのでは？。生成したSQLを先ず表示して確認されたし。

[hassan_x]

ご回答ありがとうございます。今一度確認しましたところ、閉じられていました。もしくは自分の見方誤りでしょうか？

Answer 1

executeメソッドを呼んでないのは、単に引用を省略したからでしょうか?
execute前であれば手元の環境でも試せるなと思い試したところ、特にエラーなく終了しました。executeしてないので、MySQLまで行っていないので当然ですが、そうすると、示していただいているエラーは、引用されている箇所ではないところで発生していることになります。
もう少し広い範囲を引用して頂いたほうがよいかと思います。

Answer 2

よくソースを見ずに回答していました。間違いです。すみません。
下記の定義のテーブルでは実行できました。

create table picts_overview (picture_id text, title text, category text, pins_count int, register_datetime timestamp, update_datetime timestamp);

addPictOverViewTable(PDO $db, String $picture_id, $title, $category){

このなかに「$pins_count」がふくまれておらず、値がNULLなのでエラーになっているのでは、無いでしょうか？

＝＝＞picture_idにはコンスタント０値がセットされていました。

Comments

[King_of_Flies]

$stmt -> bindValue(':pins_count', 0, PDO::PARAM_INT);
この記述であっても、pins_countに0が入らないということでしょうか？

詳しく教えてください。

[A.Ichi]

すみません気が付きませんでした。その通りです。大丈夫です。

Answer 3

多数のご回答とアドバイスありがとうございました。
遅ればせながらお礼申し上げます。

お恥ずかしいですが、別の部分のファンクションでエラーになっていました。
ockeghem様の言うとおりでした。

ありがとうございました。

Answer 4

DoctrineのDBALを使うと簡単にデータベースの操作ができます。

valuesに数字を振ってあるのはsetParameterの第1引数に渡す数値を間違えないようにするためにしています。

use Doctrine\DBAL\Configuration;
use Doctrine\DBAL\DriverManager;

$config = new Configuration();

$dbParams = array(
    'driver' => 'pdo_mysql',
	'host' => 'localhost',
    'user' => 'root',
    'password' => '',
    'dbname' => 'dbname'
);

$conn = DriverManager::getConnection($dbParams, $config);
$builder = $conn->createQueryBuilder();

$bulder->insert('picts_overview')
    ->values(array(
        'picture_id' => '?',　// 0
        'title' =>  '?',　　　// 1
        'category' => '?', 　// 2
        'pins_count' => '?',　　　　　// 3
        'register_datetime' => '?',　// 4
        'update_datetime' => '?',　　// 5
    ))
    ->setParameter(0, $picture_id)
    ->setParameter(1, $title)
    ->setParameter(2, $category)
    ->setParameter(3, 0)
    ->setParameter(4, "NOW()")
    ->setParameter(5, "NOW()")
    ->execute();

Answer 5

一応同じ名称をつかって試してみましたが特に問題はなさそうです
violation: 1064 はカラム名などを間違えるとよくでるエラーなので
タイプミスなどよくよく見直してみて下さい

ちなみにいくつか気になる点があります。

• bindValue()で数値をダイレクトにいれたり、日付をPHPでフォーマットしていれたりしていますが

無駄だと思います。

  $sql = "INSERT INTO `picts_overview`"
  $sql.= "(picture_id, title, category, pins_count, register_datetime, update_datetime)";
  $sql.= " VALUES (:picture_id, :title, :category, 0, NOW(), NOW())";

• id(picture_id)に文字列系のデータをつかっていますが、

idは可能な限りint系のデータにしておいたほうが無難です。

• bindParamもユーザー関数の引数を引き継ぐならbindValueで十分かと

Answer 6

$datetime_regist = new DateTime();
$stmt -> bindValue(':register_datetime', $datetime_regist->format('Y-m-d H:i:s'), PDO::PARAM_STR);
$datetime_update = new DateTime();
$stmt -> bindValue(':update_datetime', $datetime_update->format('Y-m-d H:i:s'), PDO::PARAM_STR);

これくらいしか思いつかないんですが、試してみていただけますか。

追記ー
やっぱりこれをしたところで変数に格納する値は変わらないので、
意味がないと思いました。

見なかったことにしてください。

Comments

[King_of_Flies]

お詫びと言ってはなんですが、
BindParamかBindValueで変数の評価に違いがあるようです。
回答につながるかわかりませんが、
一読してみてください

http://qiita.com/_dozen_/items/f6239cf1bdab6f8b0026