質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.49%

foreachにて2回回っているかのような現象が起きてしまいます

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 1,797

score 35

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

foreachが2回回っているかのような現象が起きてしまっています。何故<p>タグ部分が2回出力されているのかが分かりません。
以下は該当のソースコードを実行した結果になります
イメージ説明

該当のソースコード

<table class="calender_table">
    <thead>
      <tr class="day">
      @foreach (['日', '月', '火', '水', '木', '金', '土'] as $dayOfWeek)
        @if($loop->first)
          <th class="sunday">{{ $dayOfWeek }}</th>
        @elseif($loop->last)
          <th class="saturday">{{ $dayOfWeek }}</th>
        @else
        <th>{{$dayOfWeek}}</th>
        @endif
      @endforeach
      </tr>
    </thead>
    <tbody>
    @foreach($dates as $date)

    @if($date->dayOfWeek == 0)
    <tr>
    @endif
        <td>
          @if($date->month == $current_date->month)
            {{$date->day}}
          @endif

           @foreach($current_reserved_date as $reserved_date)

            @if($date->day == $reserved_date->day)

              <p>予約されています</p>

            @else

              <p>空室</p>

            @endif

            @endforeach
        </td>
     @if($date->dayOfWeek == 6)  
    </tr>
    @endif
    @endforeach
    </tbody>

  </table>
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use Carbon\Carbon;
use App\Reservation;
class Reservecontroller extends Controller
{
        //


        public function getCalendarDates($year, $month)
            {
                $date = Carbon::parse("$year-$month-1")->locale('ja_JP');
                // dd($date);
                $last = $date->copy()->startOfWeek();
                $count = $last->diffInDays($date->copy()->endOfMonth()->endOfWeek()) + 1;
                // dd($count,$date);
                for ($i = 0; $i < $count; $i++, $last->addDay()) {
                        $dates[] = $last->copy();
                }

                return $dates;

            }

        public function reserve_date()
        {
            $current_date = new Carbon();
            $current_year = $current_date->year;
            $current_month = $current_date->month;
            $dates = $this->getCalendarDates($current_year,$current_month);
            $reservations = Reservation::get(['reservation_date']);
            $reservations_array = $reservations->toArray();
            // var_dump($reservations_array);
            // dd();
            foreach($reservations_array as $key=> $value ){
                $reserved_date[] = Carbon::parse($value['reservation_date']);
                if($reserved_date[$key]->month == $current_month){
                    $current_reserved_date[] = $reserved_date[$key];
                }
            }


            // foreach($current_reserved_date)

            // dd($current_reserved_date,$dates);    


            return view('reserve.calender',compact('dates','current_date','current_reserved_date'));

        }

}

試したこと

以下はコントローラーファイルで「dd($current_reserved_date,$dates);」とした際の実行結果です。
上から$current_reserved_date,$datesの実行結果になります。
イメージ説明

補足情報(FW/ツールのバージョンなど)

laravel5.8
windows10

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2019/06/22 11:42

    もう少し丁寧にデバッグしてください。問題がループ内なのは明らかなのですから配列を見ても意味がありません

    キャンセル

  • m.ts10806

    2019/06/22 11:52

    いつまでcalenderという恥ずかしいスペルミスを引っ張り続けますか?
    直せない理由などないでしょう。むしろ直さないことによるデメリットの方が大きい。別の人にも「早く対応した方がいい」と指摘されていたにも関わらず、なぜ直しませんか?

    キャンセル

回答 2

+3

2回回っているわけではありません。
$current_reserved_dateにデータが2本あってどの日でもその分だけ回しているだけです。
そして日でしか比較していないため10日以外は全てelseに入りますね。今の作りだと年月関係なく。

つまり今の作りだとデータが増えれば増えるほどデータの件数だけ「空室」と表示されることになります。
$current_reserved_dateのkeyを0から連番ではなく年月日にしておき、そのキーとカレンダーループの年月日全てで比較したほうが良いでしょう。

なので、$reservations_arrayから$current_reserved_dateを作るロジックから見なおす必要があります。

もっと言えばReservation::get(['reservation_date']);も。
カレント年月のカレンダーしか表示させないのでしたら年月で絞ったデータのみを取得するべきです。
でないとデータが増えれば増えるほど無用なループが回り、パフォーマンスがどんどん悪くなります。

月のカレンダーに時間を表示させる必要があるかないかも検討が必要ですし、通常は表示させません。「その日のデータがあるかないか(または件数)」のみ分かればいいので、時間まで込みのデータは不要と思います。
group byで年月日で絞って日付毎のデータ件数だけ取っては。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

checkベストアンサー

0

どう直したらいいかは、この学習段階では難しそうなので。

$current_reserved_dateを作るときに、年月日を連結した文字列をキーにした連想配列にするといいと思います

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/06/23 20:15 編集

    質問者自身が「この学習段階では難しそうなので」を認識してもらえればいいのですが
    (年月日キーの件は私もそれとなく触れてますがその前のデータをなんとかすべきと気づいてもらえたらいいんですけども…私を意地でも無視したいらしいので。)

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.49%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る