PHP・MySQL　セッションを利用した掲示板の削除機能

###前提・実現したいこと

PHP・MySQLの練習として掲示板を作っています。
削除機能を付けようと参考書を見ながらやっておりますがうまくいきません。
以下の仕様を考えています。
・投稿時にパスワードを入力
・削除時に投稿時に入力したパスワードを入力し
レス番号とパスワードが一致すれば削除し、一致しなければ削除しない

###発生している問題・エラーメッセージ

レス番号とパスワードの照合が上手くいかず
削除時にどのようなパスワードを入力しても「合致しました」と表示されます。

###該当のソースコード

【delete-out.php】
<?php
session_start();
$pdo=new PDO('mysql:host=localhost;dbname=board;charset=utf8','ユーザー','パスワード');
$sql=$pdo->prepare('select * from board1 where nomber=? and pass=?');
     $sql->execute([$_SESSION['deletenomber'],$_REQUEST['pass']]);

foreach ($sql->fetchAll() as $row) {
  $_SESSION['res']=[
    'nomber'=>$row['nomber'], 'pass'=>$row['pass']];
}

if (isset($_SESSION['res'])) {
  echo "合致しました。";
} else {
  echo '合致しませんでした。';
}
?>

【delete-in.php】
<?php
 $i=$_REQUEST['nomber'];

session_start();
 unset($_SESSION['deletenomber']);
 $_SESSION['deletenomber']=$i;

 echo $_SESSION['deletenomber'];
 ?>

<!doctype html>
<html>
<head>
 <meta charset="utf-8"/>
 <body>

  <p>投稿時に設定した削除パスワードを入力して下さい</p>
   <form action="delete-out.php" method="post">
     <input type="password" name="pass">
     <input type="submit" value="削除">
   </form>

 </body>
</head>
</html>

$_SESEION['deletenomber']にはレス番号が入っています。

追記：$_SESSION['res']を確認のためにechoで書き出しをしようと思い
echo $_SESSION['res'];をいれましたが
Parse error: syntax error, unexpected 'is' (T_STRING)が出てしまいました。
echoで書き出しはできなのでしょうか。

追記2：delete-in.phpのソースも追加いたしました。

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

回答1件

ベストアンサー

お示しのコード中の

php
1foreach ($sql->fetchAll() as $row) {
2  $_SESSION['res']=[
3    'nomber'=>$row['nomber'], 'pass'=>$row['pass']];
4}

の部分でいきなりセッション登録するのではなく、

php
1if ($sql is not false) { 
2  foreach ($sql->fetchAll() as $row) {
3    if (!is_null($row)) {
4      $_SESSION['res']=[
5        'nomber'=>$row['nomber'], 'pass'=>$row['pass']];
6      }
7  }
8}

といった戻り値のチェックに合格すればセッション処理を実行するように直すとよいのではないでしょうか。

投稿2016/12/30 15:04

seastar3

総合スコア2285

shabo

2016/12/30 16:20

seastar3様回答ありがとうございます。お教えいただいたソースを試しに丸っとコピーさせていただいたところ Parse error: syntax error, unexpected 'is' (T_STRING) inのエラーが出てしまいました。私自信がis not falseや!is_nullという文を使用するのが初めてなもので、仕様を理解しないままに使ってしまい申し訳ありません。

seastar3

2016/12/30 18:24

　確かめずに回答して失礼しました。is not false はできませんね。 is not false を !== に置き換えて実行してみて下さい。　phpのpdo処理のexecute()メソッドはエラーをfalseで戻すのみなので、try catch が効かないようです。自分で戻り値を確かめなければいけません。　また、関数の動作確認は例えば、 ``` php <?php $a = false; if (!is_null($a)) { echo "nullじゃない。<br>"; } else { echo "nullだ。<br>"; } ``` などのように自分で短い検証コードを試しつつ、用法を理解していきましょう。

shabo

2017/01/09 21:10

返信が遅くなり大変申し訳ございません。ご教示いただいたようにコードを少しずつ学び、小さなスクリプトを試し学習することにしました。質問させていただいた件は下記のソースを -------------------------------------------- if (isset($_SESSION['res'])) { echo "合致しました。"; } else { echo '合致しませんでした。'; } こちらにすることで正常に真偽が表示されるようになりました。ありがとうございました。 -------------------------------------------- if (isset($_SESSION['res']['nomber']['pass'])) { echo "合致しました。"; } else { echo '合致しませんでした。'; } --------------------------------------------

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