#628 progress default page changed #1590
Conversation
There was a problem hiding this comment.
Я бы предложил переписать код. Смотрите, у нас есть сервисы https://github.com/Hexlet/hexlet-sicp/tree/main/app/Services
и там можно создать chapterService и добавить всю логику по работе с главами.
Т.е. если пользователь отмечает прочитанной главы - то проверяем, все ли они относятся к mainChapter.
Соответственно, нам бы оставалось просто брать главные главы книги и главные главы пользователя, и строить запрос, чтобы получать первую главу, которая еще не отмечена прочитанной.
| @@ -21,7 +21,19 @@ public function __invoke(): View | |||
| $user->load('readChapters', 'exerciseMembers'); | |||
|
|
|||
| $chapters = Chapter::with('children', 'exercises')->get(); | |||
| $mainChapters = $chapters->where('parent_id', null); | |||
| $mainChapters = $chapters->where('parent_id', null)->sortBy('id'); | |||
There was a problem hiding this comment.
давайте добавить scope для получения главных глав.
По сортировке- надежнее использовать сортировку по path. Тогда это будет по смыслу ближе к нашей предметной области. ведь у наших глав в книге нет айдишников, есть ее номера.
|
|
||
| public function haveReadMainChapter(Chapter $chapter): bool | ||
| { | ||
| $subChapters = ChapterHelper::getChapterDescendants($chapter); |
There was a problem hiding this comment.
На мой взгляд неправильно прокидывать в модель хелпер.
Хелперы служат для удобного доступа к некоторым данным, чтобы не дулировать код. Чаще всего они используются во вьюхах, контроллерах. Но на самом деле некоторые из них уже содержат код, который можно сказать принадлежит нашей предметной области и пора это добро закинуть в в какой-то сервис что ли.
|
@ddm14159 скажите, пожалуйста, нужна ли какая-нибудь помощь, как продвигается работа? |
#628 По умолчанию открываем первый раздел, в котором есть непрочитанные главы. Если прочитано все, то показываем начало
https://ddm-hexlet-sicp.onrender.com/ru/my