0

我对我在 Laravel 中的代码有点困惑。我有一个会员可以订阅一门或多门课程。我有三个表:成员、课程和课程成员。

我创建了一个成员表单,从这个表单中,我可以在课程中注册成员(我需要获取课程 ID,并将记录放在数据透视表中)。

然后我有课表;我可以创建一个新课程并放入一个或多个成员(在这种情况下我需要成员代码)。这两个功能几乎相同,但参数不同。在我的解决方案中,我创建了两个具有两种不同功能的不同控制器。

课程控制器

<?php

public function addMember(Request $request)
{
    $lessonMember = new LessonMember();
    $lessonId = $request->session()->get('lessonId', 1);

    if ((!($lessonMember::where('lesson_id', '=', $lessonId)
        ->where('license_member_id', '=', $request->memberId)
        ->exists()))) {
        $lessonMember->lesson_id = $lessonId;
        $lessonMember->license_member_id = $request->memberId;
        $lessonMember->save();
        $member = LicenseMember::find($request->memberId)->member;

        return response()->json(['user_saved' => $member, 'llm' => $lessonMember, 'actualMembers' => $actualMembers]);
    }
}

成员控制器

<?php

public function addLesson(Request $request)
{
    $lessonMember = new LessonMember();
    $memberId = $request->session()->get('memberId', 1);

    if ((!($lessonMember::where('lesson_id', $request->lessonId)
        ->where('license_member_id', $memberId)
        ->exists()))) {
        $lessonMember->lesson_id = $request->lessonId;
        $lessonMember->license_member_id = $memberId;
        $lessonMember->save();
        $member = LicenseMember::find($memberId)->member;

        return response()->json(['user_saved' => $member, 'llm' => $lessonMember]);
    }
}

我对 removeFromLesson() 方法和 updateLessonMember 方法有同样的问题,但解决方案应该是相似的。它肯定不是 DRY,我想我必须在模型中(或其他地方)放一些代码,但我不知道如何继续。我想重构一个干净的解决方案。我读过关于特质的文章,但我不知道这是否是正确的遵循方式。

4

1 回答 1

0

LessonControllerMemberController扩展一个BaseController.

在该基本控制器中,创建一个名为的函数add_lesson_member(您可以随意调用它)

然后你只需要$this->add_lesson_member()在你的 LessonController 或 MemberController 中调用这个函数。

于 2018-11-28T19:45:36.760 回答