Вложенный foreach в Laravel двумя разными моделями

Попробуйте заменить

$choices = Choice::where('user_id',Auth::id())->pluck('question_number')->toArray();

с $choices = Choice::where('user_id',Auth::id())->get(); как когда выщипываешь ключ question_number не получишь и получите ошибку trying t o get property of undefined

1 решение

Вы используете идентификатор выбора в файле лезвия, но в запросе вы не загружаете идентификатор Поэтому необходимо загрузить как id, так и question_number

$choices = Choice::where('user_id',Auth::id())->get(['id','question_number'])->toArray();

Также, когда вы используете toArray (), вам нужно изменить

Из{{ $choice->question_number }}

К{{ $choice['question_number'] }}

то же самое для выбора редактировать ссылку href = "choices/{{ $choice['id'] }}/edit"

Или просто удалите toArray() из запроса.

2 решение

Теперь, если вы хотите загрузить выбор вместе с темой, и у вас есть идентификатор темы в модели выбора

$duplicates = Question::selectRaw("count('id') as total, topic_id")->with('topic', 'topic.choices')->groupBy('topic_id')->get(); // Id for this table is your question number or any realtion between the Question Model and Choice Model
$choices = Choice::where('user_id',Auth::id())->pluck('question_number','topic_id')->toArray();
return view('choices.index',compact('duplicates','choices'));


@foreach ($duplicates as $duplicate)
    <tr>
        <td style = "text-align: center;">{{ $duplicate->topic->id }}</td>
        <td style = "text-align: center;">{{ $duplicate->topic->title }}</td>
        <td style = "text-align: center;">{{ $duplicate->total }}</td>
        <td style = "text-align: center;">
             {{ isset($choices[$duplicate->topic->id]) ? $duplicate->topic->id : '' }}
        </td>
        <td>
        <a class = "btn btn-default" href = "choices/{{ isset($choices[$duplicate->topic->id]) ? $choices[$duplicate->topic->id] : '' }}/edit">Шинэчлэх</a></td>
    </tr>
@endforeach

пытается получить доступ к $choice->id из цикла foreach в вашем представлении

вы использовали щипать () в своем контроллере

поэтому, на ваш взгляд, вы не можете использовать $choice->question_number

так что вам нужно сменить контроллер запрос или вы можете использовать ниже

@foreach ($duplicates as $duplicate)
  @foreach ($choices as $choice)
  // here only can use like this
      {{ $choice }}
      //{{ $choice->question_number }}
    @endforeach           

// this $choice->id out of foreach loop
<a class = "btn btn-default" href = "choices/{{ $choice->id }}/edit">Шинэчлэх</a></td> 
@endforeach

или, как сказал @Mayuri Pansuriya, вам нужно изменить запрос контроллера, тогда вы можете получить доступ

$choices = Choice::where('user_id',Auth::id())->get();

тогда вы можете получить доступ в своем представлении

<td style = "text-align: center;">
       @foreach ($choices as $choice)
           {{ $choice->question_number }}
       @endforeach
</td>

В вашем контроллере не используйте toArray (), если вы хотите, чтобы оба элемента были объектами. Лучший способ сделать это в представлении - использовать array_combine (), если объекты имеют одинаковую длину.

Контроллер

$duplicates = Question::selectRaw("count('id') as total, topic_id")->with('topic', 'topic.choices')->groupBy('topic_id')->get()->toArray();
$choices = Choice::where('user_id',Auth::id())->get()->toArray();
return view('choices.index',compact('duplicates','choices'));

Вид

@for($i=0; $i< count($duplicates); $i++){
   <tr>
        <td style = "text-align: center;">{{ $duplicates[$i]['topic']['id'] }}</td>
        <td style = "text-align: center;">{{ $duplicates[$i]['topic']['title'] }}</td>
        <td style = "text-align: center;">{{ $duplicates[$i]['total'] }}</td>
        <td style = "text-align: center;">{{ $choices[$i]['question_number'] }}</td>
        <td><a class = "btn btn-default" href = "choices/{{ $choices[$i]['id'] }}/edit">Шинэчлэх</a></td>
    </tr>
@endfor

Показанная вами ошибка связана с попыткой доступа к массиву с использованием синтаксиса объекта:

{{ $choice->question_number }}

$choices = Choice::where('user_id',Auth::id())->pluck('question_number');

Если это единственная проблема, просто не используйте toArray () в коллекции и оставьте ее как объект или используйте синтаксис массива.

Лучше объедините все эти таблицы и сохраните их в одной переменной. Затем передайте соединения в представлении с помощью compact () метода контроллера. Затем выполните foreach, сохранив одну переменную. Это сделает вас чистым и лучше.