掲示板で、編集や削除のために番号を指定する時、その番号がない場合「該当する投稿はありません」と表示したい

Question

PHP5.2.4(バージョンは指定による)
掲示板を作る課題に取り組んでいます。
in_arrayで分岐して実現しようとしました。
編集番号指定のところで試していますが、
現在は、送信した番号があっても表示されてしまいます。
昨日はうまくいっていたはずなのですが、コードを直した時にどこか変えてしまったせいかうまくいかなくなってしまいました。
具体的には、投稿内に送信した番号があるときにも「該当する投稿がありません」と表示されてしまいます。
コードで言うと、
$exの中に$H_noがあるのに$hMesaageがechoされます。
何が悪いのでしょうか。

var_dumpして気づいたのですが、なぜか送信した番号の投稿が$exの中からなくなっています。

 if ( !in_array( $H_no, $ex, true )|| empty( $ex ) ) { //投稿番号の中に送信された番号がないか、ファイルの中身が無い時
      $hMessage = "該当する投稿がありません";
}
```のように書きました。
表示させるフォームは以下。
```php
<!--編集番号指定用フォーム-->
<form method="post">
  <label for="H_no" >編集対象番号(半角)</label>
  <input type="namber" name="H_no" id="H_no" required="required">
  <input type="submit" value="編集">
  <?php echo $hMessage; ?>
</form>

全体は以下です。

<?php
$fname = "keijiban.txt"; //保存するファイル
$name = filter_input( INPUT_POST, "name" ); //投稿者の名前
$comment = filter_input( INPUT_POST, "comment" ); //投稿するコメント
$D_no = filter_input( INPUT_POST, "D_no" ); //削除番号の変数定義
$H_no = filter_input( INPUT_POST, "H_no" ); //編集番号の定義
$hidden = filter_input( INPUT_POST, "hidden" ); //hiddenの送信
$psw = filter_input( INPUT_POST, "psw" ); //passwordの送信
$chpsw = filter_input( INPUT_POST, "chpsw" ); //照合するパスワード
$date = date( "Y-m-d H:i" ); //投稿日時
$delimiter = "<>"; //区切り文字
$search = array( "\r\n", "\r", "\n", "<>" ); //置き換え時の検索文字
$replace = array( "<<改行1>>", "<<改行2>>", "<<改行3>>", "&lt;&gt;" ); //置き換える文字
//投稿番号の定義
$rname = str_replace( $search, $replace, $name ); //置き換えられた名前
$rcomment = str_replace( $search, $replace, $comment ); //置き換えられたコメント
if ( is_file( $fname ) ) { //ファイルの存在確認。
  //最後の行にプラス1
  $rows = file( $fname );
  $lastline = $rows[ count( $rows ) - 1 ];
  $num = explode( $delimiter, $lastline );
  $lastnum = $num[ 0 ] + 1;
} else { //ファイルが無かった場合変数の定義を１とする
  $lastnum = 1;
}
//書き込み内容
$hozon = implode( $delimiter, array( $lastnum, $rname, $rcomment, $date, $psw ) );

//投稿部分
if ( !empty( $name ) && !empty( $comment ) && !empty( $psw ) && empty( $hidden ) ) { //もし名前とコメントが送信されたら
  $fp = fopen( $fname, "a" ); //追記モードでファイルを開く
  if ( flock( $fp, LOCK_EX ) ) { //ロックする
    fwrite( $fp, $hozon . "\r\n" ); //指定したファイルに追記モードで書き込み
  }
  flock( $fp, LOCK_UN );
  fclose( $fp );
  header( 'Location: keijiban.php', true, 303 );
  exit;
}

//編集番号指定
elseif ( !empty( $H_no ) ) { //もしPOSTで編集用の番号が送信されたら
  $rows = file( $fname ); //ファイルを配列として読み込む
  foreach ( $rows as $row ) { //ループ
    $ex = explode( $delimiter, $row ); //<>で分割し投稿番号取り出す
if ( $H_no == $ex[ 0 ] ) { //送信と書き込みの番号が同じなら
      $h_no = $ex[ 0 ];
      $h_name = $ex[ 1 ];
      $h_comment = $ex[ 2 ];
      $hMessage = "";
    }
	  if ( !in_array( $H_no, $ex, true )|| empty( $ex ) ) { //投稿番号の中に送信された番号がないか、ファイルの中身が無い時
      $hMessage = "該当する投稿がありません";
		  var_dump($H_no);
var_dump($ex);
var_dump($h_no);
    }else{
		  $hMessage = "";
	  }	  
  }
  }


//編集機能
elseif ( !empty( $name ) && !empty( $comment ) && !empty( $hidden ) ) { //名前とコメントとhiddenが空でなかったら
  $rows = file( $fname ); //ファイルを配列として読み込む
  $fp = fopen( $fname, "w" ); //書き込みモードでファイルを開く
  if ( flock( $fp, LOCK_EX ) ) { //排他ロック出来たら
    foreach ( $rows as $row ) { //ループ
      $ex = explode( $delimiter, trim( $row ) ); //<>で分割し投稿番号取り出す
      if ( $hidden == $ex[ 0 ] && $chpsw !== $ex[ 4 ] ) {
        $hpMessage = "パスワードが違います";
        $h_no = "";
        $h_name = "";
        $h_comment = "";
        $hMessage = "";
      }

      if ( $hidden == $ex[ 0 ] && $chpsw == $ex[ 4 ] ) { //送信と書き込みの番号が同じで、且つパスワードが一致すけば
        $hensyu = implode( $delimiter, array( $hidden, $rname, $rcomment, $date, $chpsw ) );
        fwrite( $fp, $hensyu . "\r\n" ); //指定したファイルに追記モードで書き込み
      } else {
        fwrite( $fp, $row );
      }

    }
  }
  flock( $fp, LOCK_UN ); //ロック開放
  fclose( $fp );
}

//削除機能
elseif ( !empty( $D_no ) && !empty( $chpsw ) ) { //削除番号とパスワードがからでなかったら
  $rows = file( $fname ); //ファイルを配列として読み込む
  $fp = fopen( $fname, "w" ); //wモードで開く
  if ( flock( $fp, LOCK_EX ) ) { //もし排他ロックかけたら
    foreach ( $rows as $row ) { //ループ
      $ex = explode( $delimiter, trim( $row ) ); //<>で分割し投稿番号取り出す
      if ( $D_no == $ex[ 0 ] && $chpsw == $ex[ 4 ] ) { //削除番号とパスワードが一致するものはなにもしない
        $sMessage = "削除されました";
      } else { //それ以外は
        fwrite( $fp, $row ); //ファイルに書きこむ  
      }
      if ( !in_array( $D_no, $ex, true ) || empty( $ex ) ) { //投稿番号の中に送信された番号がなかった時
        $sMessage = "該当する投稿がありません";
      }
      if ( $D_no == $ex[ 0 ] && $chpsw !== $ex[ 4 ] ) {
        $sMessage = "パスワードが違います";
      }
    }
  }
  flock( $fp, LOCK_UN ); //ロック開放
  fclose( $fp );
}

?>
<!doctype html>
<html lang="ja">
<head>
<meta charset="utf-8">
<title>簡易掲示板</title>
<script>
window.addEventListener('DOMContentLoaded', function(){
document.querySelector('#delbtn').addEventListener('click',function(e){
if(!confirm('本当に削除しますか？')){
alert('キャンセルされました'); 
e.preventDefault();
}
});
});
</script>
<style>
span {
    color : red;
}
</style>
</head>
<body>
<?php
$fname = "keijiban.txt";
$search = array( "<<改行1>>", "<<改行2>>", "<<改行3>>", "&lt;&gt;", ); //置き換え時の検索文字
$replace = array( "\r\n", "\r", "\n", "<>", ); //置き換える文字
//XSS対策
function h( $str ) {
  return  nl2br(htmlspecialchars( $str, ENT_QUOTES, 'UTF-8' )) ;
}
$h_n = str_replace( $search, $replace, $h_name );
$h_c = str_replace( $search, $replace, $h_comment) ;

if ( !empty( $H_no ) && $H_no == $h_no ) {
  echo "<span>" . $h_no . "番</span>の投稿を編集をします<BR>";
}
?>
<!--投稿フォーム-->
<form method="post" >
  <label for="name-field">お名前<span>【必須】</span> </label>
  <input type="hidden" name="hidden" value="<? echo $h_no; ?>">
  <input type="text" name="name" id="name-field" required="required" value="<?php echo $h_n ; ?>">
  <br>
  <label for="comment">コメント<span>【必須】</span></label>
  <textarea name="comment" cols="30" rows="3" id="comment" required="required"><?php echo $h_c ;?></textarea>
  <br>
  <?php
  if ( !empty( $H_no ) ) {
    echo "投稿時設定したパスワードを入力して下さい<span>【必須】</span><input type='password' name='chpsw' required='required'>";
  } else {
    echo "<label for='psw' >パスワード設定</label><span>【必須】</span>
	<input type='password' name='psw' required='required'>";
  }
  ?>
  <input type="submit" value="<?php if (!empty( $H_no ) ){ echo '編集'; }else{ echo '投稿'; } ?>">
  <?php echo $hpMessage;?>
</form>
<!--削除番号指定用フォーム-->
<form method="post" name="sakujo">
  <label for="D_no" >削除対象番号(半角)</label>
  <input type="namber" name="D_no" id="D_no" required="required">
  <br>
  <label for="spsw" >削除パスワード</label>
  <input type="password" name="chpsw" required="required">
  <input name="btn" type="submit" value="削除" id="delbtn">
  <?php echo $sMessage; ?>
</form>

<!--編集番号指定用フォーム-->
<form method="post">
  <label for="H_no" >編集対象番号(半角)</label>
  <input type="namber" name="H_no" id="H_no" required="required">
  <input type="submit" value="編集">
  <?php echo $hMessage; ?>
</form>
<?php
//投稿一覧
if ( !empty( $fname ) ) {
  $rows = file( $fname );
  foreach ( $rows as $row ) {
    $c = explode( $delimiter, $row );
    $c = str_replace( $search, $replace, $c );
    echo h( $c[ 0 ] ) . "<br>";
    echo h( $c[ 1 ] ) . "<br>";
    echo h( $c[ 2 ] ) . "<br>";
    echo h( $c[ 3 ] ) . "<br>";

  }
} else {
  echo "<p>投稿はまだありません</p>";
}
var_dump( $h_name );
var_dump( h( $c[ 2 ] ) );


?>
</body>
</html>

Comments

[退会済みユーザー]

「うまくいかなくなってしまいました。」なにがどうなってる？ 具体的に示してくれないと的確な回答に繋がりません。

[MakotoIshizawa]

すみません。
具体的には、投稿内に送信した番号があるときにも「該当する投稿がありません」と表示されてしまいます。
コードで言うと、
$exの中に$H_noがあるのに$hMesaageがechoされます。

Answer 1

      if ( !in_array( $D_no, $ex, true ) || empty( $ex ) ) { //投稿番号の中に送信された番号がなかった時
        $sMessage = "該当する投稿がありません";
      }

この処理、何をしようとしているか説明できますか？

$exは投稿データ1行分がデリミタで分解されたものは格納されている配列ですが、
その配列の中をin_array()で検索することって妥当でしょうか。
わたしはNOだと思います。
掲示板投稿の名前もしくはコメントに投稿番号とみなせる数字が書かれていたらヒットしてしまいかねません。
（仮に、ファイルに保存されている投稿データから、投稿番号だけ洗い出したものを配列の保持しているのであれば、in_array()を使う目もあるかもしれませんが、同時にパスワードも検証するとなるとやはり破綻しかねません。）

そこにさらにempty( $ex )って、この処理の流れの中で成り立たないことではないでしょうか。

あくまで一例ですが、
foreach()ループの外側でフラグ変数を宣言しておいてfalseで初期化しておいて、
foreach()ループ内で投稿番号とパスワードがもしも一致したならば、そのフラグ変数をtrueにするようにしておけば、
foreach()ループを抜けた時点でもfalseであれば一致するものがなかったと判断できます。

Comments

[MakotoIshizawa]

ありがとうございます。
$exの中身の件、考えが浅はかでした。
フラグ変数というのが初耳で理解が追いついていないですが、調べてなんとか試してみます。

[MakotoIshizawa]

ありがとうございました。
うまくいきました。
フラグというやり方を知ってちょっとだけ幅が広がりました。
勉強になりました。