sqlエラーの原因が分かりません

以下のコードにおきまして、sqlエラーが発生しています。エラーの原因が何か分からないため、教えて頂けましたら幸いです。

php
1<?php
2
3include 'shop_class.php';
4
5error_reporting(E_ALL);
6ini_set("display_errors",1);
7
8$shop = new Shop('mysql:host=localhost;dbname=ec_website','root','12345');
9if($_POST){
10  // var_dump($_POST);
11  // exit();
12  $user_id = intval($_POST['user_id']);
13  $product_name = $_POST['name'];
14  $img = $_POST['img'];
15  $price = $_POST['price'];
16  $product_id = intval($_POST['product_id']);
17  $item_check = $shop->cart_check($user_id);
18
19  if(!$item_check){
20    // try{
21      $shop->add_select_item($user_id,$product_id);
22    // }catch(PDOException $e){
23    //   echo $e->getMessage();
24    // }
25    
26    echo __LINE__;
27    exit();
28
29  }else{
30    $shop->update($product_id);
31    echo __LINE__;
32    exit();
33  }
34
35
36
37}
38
39
40
41?>
42
43<!DOCTYPE html>
44<html lang="ja">
45<head>
46  <meta charset="UTF-8">
47  <link type="text/css" rel="stylesheet" href="./css/top.css">
48  <title>カートページ</title>
49</head>
50<body>
51  <header class="header_box">
52    <a href="top.php">
53      <img class="logo" src="./images/logo.png" alt="CodeShop">
54    </a>
55
56    <p class="user_name">
57      ユーザー名:<?php echo $_COOKIE['name']; ?>
58    </p>
59    <a href="cart.php">
60      <img src="./images/cart.png">
61    </a>
62    <a href="logout.php" class="logout">
63      ログアウト
64    </a>
65    </header>
66    <div class="cart_content">
67      <h3>ショッピングカート</h3>
68
69    </div>
70  </body>
71  </html>

php
1<?php
2
3class Shop
4{
5
6  private $dbh;
7  
8  public function  __construct($dsn,$user,$password){
9    $dbh = new PDO($dsn,$user,$password);
10    $dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);
11    $dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES,false);
12    $this->dbh = $dbh;
13  }
14
15  public function h($str){
16    return htmlspecialchars($str,ENT_QUOTES,'UTF-8');
17  }
18
19  public function write($user_name,$password){
20
21    $stmt = $this->dbh->prepare("INSERT INTO user(user_name,password,created_at,updated_at)VALUES(:user_name,:password,now(),now())");
22    $stmt->bindValue(':user_name',$user_name,PDO::PARAM_STR);
23    $stmt->bindValue(':password',$password,PDO::PARAM_STR);
24    $stmt->execute();
25
26
27  }
28
29  public function user_check($user_name){
30    $stmt = $this->dbh->prepare("SELECT user_name FROM user WHERE user_name = :user_name");
31    $stmt->bindValue(':user_name',$user_name,PDO::PARAM_STR);
32    $stmt->execute();
33    $result = $stmt->fetchALL(PDO::FETCH_ASSOC);
34    if(count($result) > 0){
35     return '同じユーザー名が既に登録されています'; 
36    }
37    return 'ok';
38    
39  }
40
41  public function login_check($user_name){
42    $stmt = $this->dbh->prepare(" SELECT id,password FROM user WHERE user_name = :user_name ");
43    $stmt->bindValue(':user_name',$user_name,PDO::PARAM_STR);
44    $stmt->execute();
45    $user_info = $stmt->fetch(PDO::FETCH_ASSOC);
46    return $user_info;
47
48  }
49
50  public function add_item($name,$price,$img,$status){
51    $stmt = $this->dbh->prepare("INSERT INTO item(name,price,img,status,created_at,updated_at) VALUES(:name,:price,:img,:status,now(),now())");
52    $stmt->bindValue(':name',$name,PDO::PARAM_STR);
53    $stmt->bindValue(':price',$price,PDO::PARAM_INT);
54    $stmt->bindValue(':img',$img,PDO::PARAM_STR);
55    $stmt->bindValue(':status',$status,PDO::PARAM_INT);
56    $stmt->execute();
57    $stmt = $this->dbh->prepare("SELECT id,created_at FROM item WHERE name = :name");
58    $stmt->bindValue(':name',$name,PDO::PARAM_STR);
59    $stmt->execute();
60    $item_data = $stmt->fetch(PDO::FETCH_ASSOC);
61    return $item_data;
62
63  }
64
65  public function add_stock($item_id,$stock,$date){
66    $stmt = $this->dbh->prepare("INSERT INTO stock(item_id,stock,created_at,updated_at)VALUES(:item_id,:stock,:created_at,:updated_at)");
67    $stmt->bindValue(':item_id',$item_id,PDO::PARAM_INT);
68    $stmt->bindValue(':stock',$stock,PDO::PARAM_INT);
69    $stmt->bindValue(':created_at',$date,PDO::PARAM_STR);
70    $stmt->bindValue('updated_at',$date,PDO::PARAM_STR);
71    $stmt->execute();
72
73  }
74
75  public function display_user(){
76    $stmt = $this->dbh->prepare("SELECT user_name,created_at FROM user");
77    $stmt->execute();
78    $user_list = $stmt->fetchAll(PDO::FETCH_ASSOC);
79    return $user_list;
80
81  }
82
83  public function check_product_name($product_name){
84    $stmt = $this->dbh->prepare("SELECT name FROM item WHERE name = :name");
85    $stmt->bindValue(':name',$product_name,PDO::PARAM_STR);
86    $stmt->execute();
87    $item_name = $stmt->fetch(PDO::FETCH_ASSOC);
88    return $item_name;
89  }
90
91  public function display_product(){
92    $stmt = $this->dbh->prepare("SELECT id,name,price,img,status FROM item");
93    $stmt->execute();
94    $product_list = $stmt->fetchAll(PDO::FETCH_ASSOC);
95    return $product_list;
96
97  }
98
99  public function display_quantity(){
100    $stmt = $this->dbh->prepare("SELECT item_id,stock FROM stock");
101    $stmt->execute();
102    $stock_list = $stmt->fetchAll(PDO::FETCH_ASSOC);
103    return $stock_list;
104  }
105  
106  public function change_stock($change_stock,$id){
107    $stmt = $this->dbh->prepare("UPDATE stock SET stock = :change_stock WHERE item_id = :id");
108    $stmt->bindValue('change_stock',$change_stock,PDO::PARAM_INT);
109    $stmt->bindValue('id',$id,PDO::PARAM_INT);
110    $stmt->execute();
111
112  }
113
114  public function change_public_status($id){
115    $stmt = $this->dbh->prepare("UPDATE item SET status = 0 WHERE id = :id");
116    $stmt->bindValue('id',$id,PDO::PARAM_INT);
117    $stmt->execute();
118  }
119  public function change_private_status($id){
120    $stmt = $this->dbh->prepare("UPDATE item SET status = 1 WHERE id = :id");
121    $stmt->bindValue('id',$id,PDO::PARAM_INT);
122    $stmt->execute();
123  }
124
125  public function delete($id){
126    $stmt = $this->dbh->prepare("DELETE FROM item WHERE id = :id");
127    $stmt->bindValue('id',$id,PDO::PARAM_INT);
128    $stmt->execute();
129    $stmt = $this->dbh->prepare("DELETE FROM stock WHERE item_id = :id");
130    $stmt->bindValue('id',$id,PDO::PARAM_INT);
131    $stmt->execute();
132  }
133
134  public function line_up_product(){
135    $stmt = $this->dbh->prepare("SELECT id,name,price,img FROM item WHERE status = 1");
136    $stmt->execute();
137    $list_product = $stmt->fetchALL(PDO::FETCH_ASSOC);
138    return $list_product;
139  }
140  public function cart_check($user_id){
141    $stmt = $this->dbh->prepare("SELECT item_id FROM cart WHERE user_id = :user_id");
142    $stmt->bindValue('user_id',$user_id,PDO::PARAM_INT);
143    $stmt->execute();
144    $cart_user_id = $stmt->fetch(PDO::FETCH_ASSOC);
145    return $cart_user_id;
146
147  }
148
149  public function add_select_item($user_id,$product_id){
150    $stmt = $this->dbh->prepare("INSERT INTO cart(user_id,item_id,amount,created_at,updated_at)VALUES(:user_id,:product_id,1,now(),now()");
151    $stmt->bindValue(':user_id',$user_id,PDO::PARAM_INT);
152    $stmt->bindValue(':product_id',$product_id,PDO::PARAM_INT);
153    $stmt->execute();
154
155  }
156
157  public function update($item_id){
158    $stmt = $this->dbh->prepare("UPDATE cart SET amount = amount+1 WHERE item_id = :item_id");
159    $stmt->bindValue(':item_id',$item_id,PDO::PARAM_INT);
160    $stmt->execute();
161  }
162
163}

エラーの内容は以下の通りです。
「Fatal error: Uncaught PDOException: 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 1 in C:\xampp\xampp\htdocs\ec_website\shop_class.php:150 Stack trace: #0 C:\xampp\xampp\htdocs\ec_website\shop_class.php(150): PDO->prepare('INSERT INTO car...') #1 C:\xampp\xampp\htdocs\ec_website\cart.php(21): Shop->add_select_item(1, 13) #2 {main} thrown in C:\xampp\xampp\htdocs\ec_website\shop_class.php on line 150」

行動規範の内容に同意します

回答2件

ベストアンサー

エラーメッセージを読むと"がシンタックスエラー。now()"です。

SQL
1    $stmt = $this->dbh->prepare("INSERT INTO cart(user_id,item_id,amount,created_at,updated_at)VALUES(:user_id,:product_id,1,now(),now()");

VALUES(の閉じ)がないのかもしれません。

投稿2019/05/02 11:23

編集2019/05/02 11:26

xebme

総合スコア1081

toll_tree

2019/05/02 11:57

ご回答ありがとうございます。これと似たエラーに結構悩まされているので、どうにかしていきたいんですけどねー 150行目にエラーが出ていると分かっていても、気づかないものなんですよね...

xebme

2019/05/02 12:25

エラーメッセージを丹念に読むだけです。でも人に話すと自己解決することも多いです。バグ解決するために「ゴムのアヒルに話しかける」方法があります。https://en.wikipedia.org/wiki/Rubber_duck_debugging

退会済みユーザー

2019/05/02 12:27

横から。EclipseとかNetbeansとかIDEと呼ばれる開発環境を導入すれば、phpやhtmlの文法くらいは指摘してくれるけど、SQLの文法までは無理なので、別途SQL単体での実行をテストする環境をphpMyAdmin他で確保すると良いかも。

xebme

2019/05/02 12:31

はい、SQLを実行してバグを早期発見するのが最も良い解決法です。

toll_tree

2019/05/02 12:35

そうですね。。まずはphpmyadminなどで、sqlの文法を直に入れて、データが入るか確認するようにしたいと思います

行動規範の内容に同意します

valueの前に空白いるのでは？

投稿2019/05/02 11:34

akirafudo6

総合スコア341

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

sqlエラーの原因が分かりません

関連した質問