データベースに情報が1つしか格納されない。

Question

仕様

①ユーザーにカレンダーのtextarea部分に情報を書き込み送信してもらう。
②同スクリプトにて受け取った情報をデータベースに入れる。
③書き込まれた情報をデータベースから取り出し各textareaに表示させる。

というものを考えています。

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

自分は今上記の②の部分でつまずいています。

データベースに1つしか情報を送れていない。のが発生している問題です。

おそらくforや配列の部分がこの問題が起きている理由だと考えているのですが、どのようにコードを組めばうまくいくのかわからず質問させていただきました。
お忙しいところ恐縮ですが、ご回答いただけましたら幸いです。

該当のソースコード

<?php
$ym = null;
$timestamp = null;
$today = null;
$html_title = null;
$prev = null;
$next = null;
$day_count = null;
$youbi = null;
$weeks = null;
$week = null;
$day = null;
$day_count = null;
$option = null;
$pdo = null;
$message_array = array();
$text_msg = array();
$content = array();
//データベースの接続情報
define( 'DB_HOST', 'localhost');
define( 'DB_USER', 'root');
define( 'DB_PASS', 'growry1293');
define( 'DB_NAME' , 'calender');


session_start();

try{
  $option = array(
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
    PDO::MYSQL_ATTR_MULTI_STATEMENTS => false);
    $pdo = new PDO('mysql:charset=UTF8;dbname='.DB_NAME.';host='.DB_HOST,DB_USER,DB_PASS, $option);
}catch(PDOException $e){
  $error_msg[]= $e -> getMessage();
}

if (isset(($_POST["update"]))) {
    for($i = 0; $i<=30; $i++) {
        if(!empty($_POST["text_msg"][$i])) {
            $text_msg[$i]=preg_replace('/\A[\p{C}\p{Z}]++|[\p{C}\p{Z}]++\z/u', '', $_POST["text_msg"][$i]);
        }else{
            $text_msg[$i] = null;
        }
    }

  if(empty($error_msg)){
//ここで1つしか情報が送られていない。

      try {
          // 値をセット
          for($i = 0; $i<=30; $i++){
            if(!empty($text_msg[$i])){
              // SQL作成
              $stmt = $pdo->prepare("INSERT INTO msg_box (text_msg) VALUES ( :text_msg)");

              $stmt->bindParam(':text_msg', $text_msg[$i], PDO::PARAM_STR);
              //sqlクエリの実行
              $res = $stmt->execute();
              //コミット
              $res = $pdo->commit();
            }
          }
      } catch(Exception $e) {
          echo "$e";
      }
      if($res) {
          $_SESSION['success_msg'] = "書き込みに成功しました";
      } else {
          $error_msg[] = "書き込みに失敗しました。";
      }
        header("Location:./calender.php");
        exit;
    }
}

if(!empty($pdo)) {
  //メッセージのデータを取得する
  $sql = "SELECT text_msg FROM msg_box";
  $message_array = $pdo->query($sql);
}
?>
<!-----------カレンダープログラム--------------->

<?php
//タイムゾーンを設定
date_default_timezone_set('Asia/Tokyo');


//前月・次月リンクが選択された場合は、GETパラメーターから年月を取得
if(isset($_GET['ym'])){
    $ym = $_GET['ym'];
}else{
    //今月の年月を表示
    $ym = date('Y-m');
}

//タイムスタンプ（どの時刻を基準にするか）を作成し、フォーマットをチェックする
//strtotime('Y-m-01')
$timestamp = strtotime($ym . '-01');
if($timestamp === false){//エラー対策として形式チェックを追加
    //falseが返ってきた時は、現在の年月・タイムスタンプを取得
    $ym = date('Y-m');
    $timestamp = strtotime($ym . '-01');
}

//今月の日付　フォーマット　
$today = date('Y-m-j');

//カレンダーのタイトルを作成　例）2020年10月
$html_title = date('Y年n月', $timestamp);//date(表示する内容,基準)

//前月・次月の年月を取得
//strtotime(,基準)
$prev = date('Y-m', strtotime('-1 month', $timestamp));
$next = date('Y-m', strtotime('+1 month', $timestamp));


//該当月の日数を取得
$day_count = date('t', $timestamp);

//１日が何曜日か
$youbi = date('w', $timestamp);

//カレンダー作成の準備
$weeks = [];
$week = '';

//第１週目：空のセルを追加
//str_repeat(文字列, 反復回数)
$week .= str_repeat('<td></td>', $youbi);

for($day = 1; $day <= $day_count; $day++, $youbi++) {

    $date = $ym . '-' . $day;
    //それぞれの日付をY-m-d形式で表示例：2020-01-23
    //$dayはfor関数のおかげで１日づつ増えていく

    if($today == $date) {
        //もしその日が今日なら
        $week .= '<td class="today">' . $day . "<textarea  name = 'text_msg[]'>";
    }else {
        //上２つ以外なら
        $week .= '<td>' . $day . "<textarea   name = 'text_msg[]'>";
    }
    if(!empty($text_msg[$day])){
//ここでも情報が取得できていない。
      foreach( $message_array as $value){
        $week .= $value.'</textarea>'.'</td>';
      }
    }else{
      $week .= '</textarea>'.'</td>';
    }

    if($youbi % 7 == 6 || $day == $day_count) {//週終わり、月終わりの場合
        //%は余りを求める、||はまたは
        //土曜日を取得

        if($day == $day_count) {//月の最終日、空セルを追加
            $week .= str_repeat('<td></td>', 6 - ($youbi % 7));
        }

        $weeks[] = '<tr>' . $week . '</tr>'; //weeks配列にtrと$weekを追加

        $week = '';//weekをリセット
    }
}

?>


<!-----------カレンダープログラム--------------->
<!-----------データベースに接続-------->


<!DOCTYPE html>
<html lang="ja">
<head>
    <meta charset="utf-8">
    <title>PHPカレンダー</title>
    <link rel="stylesheet" href="https://maxcdn.bootstrapcdn.com/bootstrap/3.3.7/css/bootstrap.min.css" integrity="sha384-BVYiiSIFeK1dGmJRAkycuHAHRg32OmUcww7on3RYdg4Va+PmSTsz/K68vbdEjh4u" crossorigin="anonymous">
    <link href="https://fonts.googleapis.com/css?family=Noto+Sans" rel="stylesheet">
    <style>
      .container {
        font-family: 'Noto Sans', sans-serif;
        margin-top: 80px;
        width: 80vw;
        margin: 0 auto;
      }
      h3 {
          margin-bottom: 30px;
      }
      th {
          height: 30px;
          text-align: center;
      }
      td {
          height: 100px;
      }
      textarea{
        height: 100px;
        width: auto;
      }
      .today {
          background: orange;
      }
      th:nth-of-type(1), td:nth-of-type(1) {
          color: red;
      }
      th:nth-of-type(7), td:nth-of-type(7) {
          color: blue;
      }
      .holiday{
          color: red;
      }
      .update_box{
        text-align: center;
      }
      .submit_box{
        width: 120px;
        height: 50px;
        margin: 50px auto;
      }
    </style>
</head>
<body id="">
      <div class="container">
      <h3><a href="?ym=<?php echo $prev; ?>">&lt;</a><?php echo $html_title; ?><a href="?ym=<?php echo $next; ?>">&gt;</a></h3>
      <form action="<?=$_SERVER['SCRIPT_NAME']?>" method="POST">
          <table class="table table-bordered">
          <tr>
              <th>日</th>
              <th>月</th>
              <th>火</th>
              <th>水</th>
              <th>木</th>
              <th>金</th>
              <th>土</th>
          </tr>
            <?php
              foreach ($weeks as $week) {
                  echo $week;
              }
          ?>
        </table>
          <div class = "update_box">
            <input type="submit"  class = "submit_box" value = "更新する" name = "update"></input>
          </div>
      </form>
    </div>
</body>
</html>

Comments

[dameo]

きちんと設計しましょう。
データベースには何を入れたいのでしょう？
テーブルのスキーマはどうすればいいのでしょう？
正しいスキーマであれば、コードはどう書けばいいのでしょう？
問題は見つかりましたか？
問題が見つかったら、現象を起こす最小限のコードを説明用に作りましょう。
誰もが再現できる形で、コード量を最小限に抑えたものを書きましょう。
環境も全て詳細に記載しましょう。
DDLなど事前準備の手順も全て記載が必要です。
そこまでやってなお分からなければ、質問を送信しましょう。
質問の最初の一歩を踏み出せます。

[m.ts10806]

POSTのtext_msg直ではなく、あえてforを通してるのはどういう理由でしょうか。
また、POSTのtext_msgには想定の情報は入っていたのでしょうか？

設計的なアドバイスですが「この日の」ならtext_msg[]ではなくtext_msg[20230417]のような日付情報をキーにしたほうがやりやすいと思います。
[]では0から振られるため、日付など決まった情報がある場合にリンクされないこともあります。

Answer 1

1つのループごとに新しい行を挿入しているため、ループが完了した時点でデータベースには最後に挿入された要素しか残らなくなっています。
ループ内で commit を呼び出しているため、1回のループで1つのメッセージしか保存できず、問題が発生しています。
これを修正するためには、送信された全ての情報を配列に格納してから、一度にデータベースに挿入するように変更する必要があります。

これでどうですか？