[php]重複チェックのためにSQLでCOUNTして戻ってきた数を、変数に入れたい

皆様、いつもお世話になっております。

emailというカラムを持つテーブルを対象に、重複したものがないかをチェックしたいのですが、
私の書いた以下のコードではエラーが出てしまいます。コードの書き方自体に間違いがあると思うのですが、
ご指摘願えますでしょうか。アドバイスをよろしくお願いします。

以下のエラーが発生しています。
Notice: Object of class PDOStatement could not be converted to int in /Applications/MAMP/htdocs/sample/sqltest.php on line 19

Catchable fatal error: Object of class PDOStatement could not be converted to string in /Applications/MAMP/htdocs/sample/sqltest.php on line 22

以下コードです。

php
1<?php
2
3error_reporting(E_ALL);
4ini_set('display_errors', 'On');
5
6$email = 'email@gmail.com';
7$dsn ='mysql:dbname=php_sample01;host=localhost;charset=utf8';
8$user = 'user';
9$password = 'password';
10$option = array(
11          PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
12          PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
13          PDO::MYSQL_ATTR_USE_BUFFERED_QUERY => true,
14        );
15
16$pdo = new PDO($dsn,$user,$password,$option);
17$sql = 'SELECT COUNT(email) FROM users WHERE email ="'.$email.'";';
18$checkDouble = $pdo->query($sql);
19if($checkDouble == 0){
20        echo '重複なし';
21      }else{
22        echo $checkDouble.'個の重複がありました。';  
23      }
24
25?>

KiyoshiMotoki

2016/06/19 12:39

「以下のコードではエラーが出てしまいます」だけでは、状況が分かりません。具体的に、どのようなエラーが発生したのかを明記してください。 https://teratail.com/help/question-tips#questionTips3-4

kalon

2016/06/19 12:48

ご指摘ありがとうございました。アドバイス頂いたとおり、エラーにつても記述しました。

KiyoshiMotoki

2016/06/19 13:00

情報の追加、ありがとうございます。エラーの原因・修正方法についてはmpyw様の回答に言い尽くされておりますので、そちらをご参照ください。

KiyoshiMotoki

2016/06/19 13:01 編集

2重投稿してしまいましたw

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

回答1件

ベストアンサー

$pdo->query()の返り値はPDOStatementというオブジェクトです．そこから結果を取り出す必要があります．が，そもそも

php
1$sql = 'SELECT COUNT(email) FROM users WHERE email ="'.$email.'";';

この書き方にはSQLインジェクションに対する致命的な脆弱性があるので，プリペアドステートメントとプレースホルダを使う書き方に変えてください．(備考)

html
1<?php
2
3$email = 'email@email.com'; // 外部からの入力を想定 (filter_inputを推奨)
4
5try {
6    
7    // 接続
8    $dsn = 'mysql:dbname=php_sample01;host=localhost;charset=utf8';
9    $user = 'user';
10    $password = 'password';
11    $options = [
12        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
13        PDO::ATTR_DEFAULT_FETCH_MODE => PDO::FETCH_ASSOC,
14    ];
15    $pdo = new PDO($dsn, $user, $password, $options);
16    
17    // プリペアドステートメントを用意
18    $stmt = $pdo->prepare('SELECT COUNT(*) FROM users WHERE email = ?');
19    
20    // 「?」の部分に文字列として安全に入力をあてはめて実行
21    $stmt->execute([$email]);
22    
23    // COUNTなので結果は1行，さらにその1列目を取得 
24    // 文字列なので一応整数型にキャストしておく
25    $count = (int)$stmt->fetchColumn();
26    
27} catch (PDOException $e) {
28    
29    // エラー時にはテキストでメッセージを吐かせる
30    // ステータスコードは「500 Internal Server Error」
31    header('Content-Type: text/plain; charset=UTF-8', true, 500);
32    exit('Error: ' . $e->getMessage());
33    
34}
35
36header('Content-Type: text/html; charset=UTF-8');
37
38?>
39<!DOCTYPE html>
40<title>Example</title>
41<h1>結果</h1>
42<?php if ($count === 0): ?>
43<p>重複はありません</p>
44<?php else: ?>
45<p><?=$count?>個の重複がありました</p>
46<?php endif; ?>