-
Notifications
You must be signed in to change notification settings - Fork 0
Замечания по 3 лабороторной #2
Copy link
Copy link
Open
Description
vasiliy-uskov
opened on Oct 11, 2021
Issue body actions
- Скорее всего по ts не прочитает типы в js-doc`ах для функций. Опишите, используя ts.
- Странно, что elementFunctions зависят от presentationFunctions. Зависимости должны идти от меньшего к большему но не наоборот.
- Не стоит передавать избыточные данные. Возможно работу c history стоит сосредоточить в логике работы с редактором.
- Хромает структурированность проекта.
- Работа с коллекцией - задача коллекции, а не элементов, входящих в неё. (В elementsFunctions есть функции модификации презентации, что не очень хорошо, учитывая, что есть presentationFunctions)
- Если разбивать по модулям, то до конца. Можно хранить описание состояния рядом с элементом или вынести в папку ./model. Внутри ./model есть два три пути:
- Два файла types.ts и actions.ts
- Для каждой сущности описывать отдельный файлик в котором описан тип и действия над ним.
- Совместить два первых варианта и создавать под каждую сущность отдельную папку с файлами actions и types
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels