-1

我有一个小问题。我的代码正在运行，但我认为我没有以正确的方式进行操作。

在我的 GradeController 我有这个代码：

/**
 * Display a listing of the resource.
 *
 * @return \Illuminate\Http\Response
 */
public function index()
{
    if(auth()->user()->hasRole('Student')) {
        $subjects = Subject::all();

        return view('grades.student.index', compact('subjects'));
    }
}

在我看来，我以这种方式获得属于指定用户的成绩：

@foreach($subject->grades->where('student_id', '=', auth()->user()->id) as $grade)
<span class="badge badge-primary">
    {{ $grade->value }}
</span>@endforeach

在这里，我的意思是 Laravel，有更好的方法吗？因为我认为获取属于某个主题的所有成绩然后查找ID并不是很“有效”。

祝你有美好的一天。

4

2 回答 2

1

您可以使用with()急切加载帮助程序，并带有一个闭包，它将subject根据属于登录用户的成绩过滤“成绩”：

$subjects = Subject::with(['grades' => function($query) {
    $query->where('student_id', auth()->user()->id);
}])->get();

注意子句, '=',中的删除。->where()如果检查是否等于，则不需要此参数。

于 2019-06-26T08:19:35.427 回答
0

在您的控制器index()中，您可以制作一个中间件而不是auth()->user()->hasRole('Student')多次执行此检查，如果您只在这里检查一次，那么您所做的就可以了。

public function index()
{
    // refactoring 
    return view('grades.student.index', ['subject' => Subject::all()]);
}

在刀片文件中进行查询也不好，因此您可以grades直接从控制器传递

public function index()
{
    return view('grades.student.index', [
        'grades' => Grade::where('student_id', auth()->id())->get()
    ]);
}

在index刀片文件中，您现在可以使用：


@foreach($grades as $grade)
<span class="badge badge-primary">
    {{ $grade->value }}
</span>
// you can get the subject from $grade->subject
@endforeach
于 2019-06-26T08:28:29.097 回答