foreachの中に配列のデータを1つずつ出力したいです

Question

現在、自主制作にてカレンダーの中にデータを出力するアプリを作っています。

カレンダーをforeachにて回しているのですが、そのカレンダーの日付ごとにデータを一つずつ出力したいです。

<?php
//カレンダー
    //現在時刻の設定
    date_default_timezone_get('Asia/Tokyo');

    $year=date('Y');
    $month=date('m');

    //月末日を取得
    $end_month=date('t',strtotime($year.$month.'01'));

    //月初めの曜日を取得
    $first_week=date('w',strtotime($year.$month.'01'));

    //月末の曜日を取得
    $last_week=date('w',strtotime($year.$month.
    $end_month));

    $calender=[];
    $j=0;

    //1日が始まるまでの穴埋め
    for($i=0; $i<$first_week; $i++){
        $calender[$j][]='';
    }

    //1日から月末までループ
    for($i=1;$i<=$end_month;$i++){
        //日曜日まで進んだら改行
        if(isset($calender[$j]) && count($calender[$j]) === 7){
            $j++;
        }
        $calender[$j][]=$i;
    }

    //月末曜日までの穴埋め
    for($i = count($calender[$j]);$i<7;$i++){
        $calender[$j][]='';
    }

    $aryweek=['日','月','火','水','木','金','土'];
?>

//出力したいデータカラム
    //ログイン者のdataカラムを取り出す
    $datas=$db->prepare('SELECT data FROM post WHERE member_id=:member_id');

    //バインド
    $datas->bindParam(':member_id',$mem['member_id'],PDO::PARAM_STR);

    //実行
    $datas->execute();
    $data=$datas->fetchAll();

    //ログイン者のcreatedカラムを取り出す
    $createds=$db->prepare('SELECT created FROM post WHERE member_id=:member_id');

    //バインド
    $createds->bindParam(':member_id',$mem['member_id'],PDO::PARAM_STR);

    //実行
    $createds->execute();
    $created=$createds->fetchAll();

<!--カレンダー-->
<table border="5">
            <tr>
                <?php foreach($aryweek as $week){;?>
                    <th>
                        <?php echo $week;?>    
                    </th>
                <?php };?>
            </tr>

            <!-- カレンダーのマスを成形する-->
            <?php foreach($calender as $tr){;?>
                <tr>
                <?php foreach($tr as $td){;?>
                    
                    <?php if($td !=date('j')){;?>
                        <td>
                            <?php 
                              //日付出力
                              echo $td
                             ;?>
                        </td>
                    <?php }else{?>
                        <td class="today">
                        　　　<!--todayに色付け-->
                        </td>
                    <?php };?>
             　　<?php };?>
                </tr>
            <?php };?>    
        </table>

試したこと

下記コードを//日付出力の箇所で試してみました。
結果は＄td一つに対して$dataの全データが出力されている状況です。

for($i=0;$i<=$end_month;$i++){
   if($created[$i]['created']==date('Y-m'.'-'.$td)){
       echo $data[$i]['data'];
      }
}

###お願いします
今回の質問では、「foreachの中に配列のデータを1つずつ出力したい」ことなのですが
自分で考えて作ったものなので、気になる点やもっと効率のいいコードがあったら教えてほしいです。

Comments

[m.ts10806]

何が問題ですか？

Answer 1

データベースからの読み出しの箇所、なぜ2度に分けて読み出しているのか。
これ、一度にできますよね。
それに、もしもデータが当月分だけじゃなく複数の年月に渡った場合に全部のデータを抽出すると邪魔なので、
一ヶ月分のカレンダーを作ろうとしているのであれば、念の為にcreatedを$yearと$monthで区切ってみるのも加えてみたり。
それに、ORDER BY句がないので、抽出される順序が不定になってるので補ってみたり。

//出力したいデータカラム
    //ログイン者のdataカラムを取り出す
    $stmt=$db->prepare('SELECT data, dayofmonth(created)+1 as "day" FROM post WHERE member_id = :member_id and year(created) = :year and month(created) = :month ORDER BY created');

    //バインド
    $datas->bindValue(':member_id', $mem['member_id'], PDO::PARAM_STR);
    $datas->bindValue(':year', $year, PDO::PARAM_INT);
    $datas->bindValue(':month', $month, PDO::PARAM_INT);

    //実行
    $datas->execute();
    $data = $datas->fetchAll();

これなら、日付の「日」と投稿データをセットで得られますよ。
MySQL前提でDAYOFMONTH()を使いましたが、1日目がゼロではじまるため、＋１しています。
MySQL 5.6のマニュアルでは→12.7 日付および時間関数

なお、安直に「year(created) = :year and month(created) = :month」とした箇所は、
いちいち関数による演算を加えると遅くなる傾向にあるため、
より負担なく高速にするためには
当月の初日の文字列と最終日の文字列を用意してBETWEENするのが良いかと。
「created BETWEEN DATE('2021-02-01') AND DATE('2021-02-28')」
になるように
「created BETWEEN DATE(:gessho) AND DATE(:getsumatsu)」
みたいにしてこれもbindValue()で与えればいいわけで。

---

もう一つ。
構造の話になってくるのですが、
質問者さんは二次元配列（週単位の配列）を組んで実装していますが、
日曜日から週を始めるためには二次元配列を組まなくてもいい方法があります。

日付を管理する配列だけのシンプルさにして、
曜日を判定して土曜日であればtr要素を出力して週を改めるってだけで実現できます。
日付は常に＋１方向で増えていくだけなので、単純な一つのfor()ループだけです。
その日付のデータを出力する前や後に、曜日の判定処理を加えるのです。

tr出力
↓
当月1日の曜日ギャップを調整するfor()ループでtd出力
↓
曜日ごとデータ出力for()ループ開始
↓
その日付データを出力（曜日ごとの色替えや今日の色替えなどもここで）
↓
もしも土曜日ならtr出力して週を改める
↓
ループ終了、繰り返し。
↓
末日以降の曜日ギャップを調整するfor()ループでtd出力
↓
忘れずに閉じtr出力して終了

という流れで、組めるはず。
曜日の繰り返しなんてしなくても組めます。

Comments

[退会済みユーザー]

MySQL詳しくないのでデバッグよろしく。