phpにて、pdoを利用したデータの削除を行いたいです。

###データの削除をPDOを通して行いたい。

ある企業の課題でphpを使って管理画面を作成しているのですが、
phpのpdoを利用した削除機能の実装につまづいています。
どなたかご協力ください。お願い致します。

###現状のコード

php
1
2  <?php
3 $res = null;
4    try {
5      $dsn = 'mysql:dbname=test_db;host=133.167.38.105;charset=utf8';
6      $user = 'test';
7      $password = 'password';
8    
9      $PDO = new PDO($dsn, $user, $password); 
10      $PDO->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
11          
12      $PDO->query("SET NAMES utf8;");
13      $sql = "DELETE FROM news　WHERE id = $page"; 
14      $stmt = $PDO->prepare($sql); 
15      $res = $dsn->query($sql);
16    
17    } catch (PDOException $e) {
18    exit('データベースに接続できませんでした。' . $e->getMessage());
19    }
20  ?>

php
1$sql = "DELETE FROM news WHERE id = $page"; 
2

こちらの$pageでは、$_GET[id]を代入しており、クエリーで検索をかけております。
エラー文は


Fatal error: Uncaught Error: Call to a member function query() on string in /var/www/html/delete_news.php:95 Stack trace: #0 {main} thrown in /var/www/html/delete_news.php on line 95

こちらです。

###行ったこと

php
1 $res = null;

変数の初期化を行いました。

###わからない部分

php
1$res = $dsn->query($sql);

こちらで$resに対して#sqlをクエリ処理することはわかるのですが、エラー文のfunction query()とは、クエリの中に何も入っていないという意味だと思うのですが、どうにか解決方法を教えていただけますと幸いです。

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

回答1件

ベストアンサー

PHP
1$sql = "DELETE FROM news　WHERE id = $page";

直接変数をSQL文に入れるのは辞めましょう。プリペアドステートメントを推奨します。

PHP
1$sql = "DELETE FROM news　WHERE id = :page";
2$stmt = $PDO->prepare($sql); 
3$stmt->bindValue(':page', $page, PDO::PARAM_INT);// $pageがIntegerの場合
4$stmt->execute();