Неопределенная переменная в "представлении" Laravel. Почему так и как исправить?

Question

[Dmitrii]

Контроллер:

public function GetStudents($i) {
        try {
            $dbconnect = DB::connection()->getPDO();
            $dbname = DB::connection()->getDatabaseName();
        } catch(Exception $e) {
            return redirect('/exception');
        }

        if (isset($i)) {
            $students = Students::all()->where('class','=',$i);
        } else {
            $students = Students::all();
        }

        return view('students', compact('students'));

    }

Route (маршут):

Route::get('/students/{i}', 'App\Http\Controllers\StudentsController@GetStudents')->name('GetStudents');

Преставление (View):

@foreach($students as $student)
                    <tr>
                        <td>
                            @php
                                echo $i++;
                            @endphp
                        </td>
                        <td>{{$student->student_name}}</td>
                        <td>{{$student->class}}</td>
                    </tr>
                @endforeach

Что я делаю не так?

Comments

[Dmitrii]

TheAndrey7, "Шапка шапкой"

if (isset($i)) {
            $students = Students::all()->where('class','=',$i);
        } else {
            $students = Students::all();
        }

Однако дело это не меняет ¯\_(ツ)_/¯ 

[Dmitrii]

TheAndrey7, Если хотите могу и в вопросе отредактировать

[Дмитрий]

А откуда у вас $i во view возьмется?

Answer 1

[Дмитрий]

1. Не совсем понятно откуда у вас появляется во view переменная $i. Ее либо надо туда передавать, либо там инициализировать.
2. При этом делать этого не надо, у blade есть The Loop Variable и стоит использовать {{$loop->index}}
3. Этот код странен. Вы выбираете все из базы и начинаете фильтровать циклом. Зачем вам БД?

$students = Students::all()->where('class','=',$i);

Как бы

$students = Students::where('class','=',$i)->get();

4. Но и это не совсем правильно. Совсем правильно будет вместо

if (isset($i)) {
            $students = Students::all()->where('class','=',$i);
        } else {
            $students = Students::all();
        }

сделать:

$students = Students::when($i, function($builder, $i){ return $builder->where('class', '=', $i); } )->get();

Вот только параметр $i у вас обязательный - что в роуте что в методе. Каким образом у вас может возникнуть ситуация когда $i в методе необьявлена? По идее никогда - у вас вылетит эксепшн

5. Смысл этого кода какой?

try {
            $dbconnect = DB::connection()->getPDO();
            $dbname = DB::connection()->getDatabaseName();
        } catch(Exception $e) {
            return redirect('/exception');
        }

Comments

[ThunderCat]

и стоит использовать {{$loop->index}}

скорее {{$loop->iteration}}

[Дмитрий]

ThunderCat, возможно. Но я не увидел попыток начать с 1 а не с 0

[ThunderCat]

Дмитрий, По разметке видно что в колонку пытались вставить нумерацию. А по уровню кода в целом понятно что делалось методом святого тыка...