データベースからデータを出力できない

Question

現在ログイン機能をつけようと作業しています。ログイン画面でユーザー名を入力してボタンを押すとログインできるようにしたいのですが上手くいきません。下のようにコードを書いてみると、そもそもデータベースから取得されるはずのusernameが画面に表示されません。まず、echo $row['username'];でユーザー名を表示できるようにしたいです。

<?php
$dsn = 'mysql:dbname=post;host=127.0.0.1;charset=utf8';
$user = 'xxx';
$password= 'xxx';

try{
  $dbh = new PDO( $dsn, $user, $password );
}catch( PDOException $error ){
  echo "接続失敗:".$error->getMessage();
  die();
}
$a=$_POST['username'];
$sql ="select username from user where username = $a";
$sth = $dbh->prepare($sql);
foreach($sth as $row):
echo $row['username'];
endforeach;
?>

Comments

[m.ts10806]

SQL実行しているところがないですよ。
前の質問で書いたのに・・
https://teratail.com/questions/209118
「PHPでデータベースに接続するときのまとめ」の記事も提示したのに・・・

Answer 1

いくつか気になる点はありますが、 表示されない理由はクエリが実行されていないからだと思います。

$sth = $dbh->prepare($sql);

の部分ですが、 $dbh->query()はすぐに実行し、 prepare()は execute() で明示的に実行しなければなりません。なので、

$a = $_POST['username'];
$sql = "select username from user where username=$a";
$sth = $dbh->prepare($sql);
$sth->execute();

で取得はできると思います。
しかし、 セキュリティの観点上、 POSTなどユーザーから与えられる情報をSQL文に直接入れてはいけません。
プレースホルダを使用することが適切です。

$a = $_POST['username'];
$sql = "select username from user where username=:username";
$sth = $dbh->prepare($sql);
$sth->bindValue(':username', $a, PDO::PARAM_STR);
$sth->execute();

こちらを参考にしてください。
PHPでデータベースに接続するときのまとめ

Comments

[Orlofsky]

リンク先のURLのあたまにゴミが入っていませんか？

[Newbi]

余計な文字が入っていました。
ありがとうございます(^^)

Answer 2

データベース操作の基本を覚えてください。
一般的に利用できるテンプレートは以下となります。
PHP で MySQL 接続時に必要な知識(最小限版)

余裕があれば、mpyw さんの PHP でデータベースに接続するときのまとめも併せて読むと良いです。

Answer 3

こんな感じで試してみてください

$dsn = 'mysql:dbname=post;host=127.0.0.1;charset=utf8';
$user = 'xxx';
$password= 'xxx';
try{
  $pdo = new PDO( $dsn, $user, $password );
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
  $a=filter_input(INPUT_POST,'username');
  $sql ="select username from user where username = ?";
  $stmt = $pdo->prepare($sql);
  $stmt->execute([$a]);
  $rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($rows);
}catch( PDOException $error ){
  echo "接続失敗:".$error->getMessage();
  die();
}