-
Notifications
You must be signed in to change notification settings - Fork 3
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature Sliced review #7
Comments
Начну с того, что "хорошо"))
loripsum-generator/src/app/index.css Lines 3 to 4 in 2d9e07e
|
loripsum-generator/src/shared/ui/index.jsx Lines 1 to 7 in 2d9e07e
|
|
Чтоб компонент не разбухал loripsum-generator/src/features/copy-to-clipboard/ui.jsx Lines 3 to 21 in 2d9e07e
|
loripsum-generator/src/entities/text/model.js Lines 1 to 7 in 2d9e07e
|
Теперь к
Можно прям на уроне loripsum-generator/src/app/index.jsx Lines 4 to 7 in 2d9e07e
|
loripsum-generator/src/pages/home/index.css Lines 1 to 12 in 2d9e07e
|
loripsum-generator/src/pages/home/index.jsx Lines 30 to 55 in 2d9e07e
loripsum-generator/src/pages/home/index.css Lines 14 to 37 in 2d9e07e
|
loripsum-generator/src/features/copy-to-clipboard/ui.jsx Lines 7 to 8 in 2d9e07e
Пока норм, но потом по мере сложности фичи - понадобится директории выделять почти наверняка |
loripsum-generator/src/features/copy-to-clipboard/ui.jsx Lines 2 to 20 in 2d9e07e
loripsum-generator/src/features/copy-to-clipboard/lib.js Lines 1 to 10 in 2d9e07e
|
Потом это может неплохо так выстрелить в колено, поэтому для импорта внешних ресурсов лучше сразу использовать абсолютные импорты
loripsum-generator/src/pages/home/index.jsx Lines 6 to 11 in 2d9e07e
|
Но тем не менее рано или поздно компоненты эти начнут друг друга дергать, т.к. даже UIKit бывает разной сложности
Можно глянуть атомик и по нему потом раскладывать компоненты, чтобы они друг друга не кросс импортили жестко |
За остальное глаз не зацепился вродь, но и у проекта сложность не шибко большая на старте) На самом деле еще стоит понимать - что концепции эти часто мешает применять уже существующее понимание принципов (GRASP, DRY) + нехватка контекста, чтобы правильно применять концепции FS (критерии слоев) Но уже оч круто, что есть понимание - что можно дробить на слои логику, а не держать "на подольше" рядом с местом использования. Это рано или поздно выстреливает, особенно когда проект больше 1 человека мейнтейнит) |
Если все эти правки внести, то вполне можно добавить репе лейбл |
Если останутся вопросы - пингуй) |
This issue was moved to a discussion.
You can continue the conversation there. Go to discussion →
Опишу здесь основные моменты, надеюсь поможет 🍰
UPD: Буду офк накидывать чисто со стороны проектирования по FS
The text was updated successfully, but these errors were encountered: