Add new example - Portfolio #651
Conversation
👷 Deploy request for pr-fsd pending review.Visit the deploys page to approve it
|
|
Привет, глянул бегло код проекта и сам сайт. Как я понимаю, сайт, в основном, статичный, в том плане, что это информационная страница, а не приложение с динамическим контентом. С этой точки зрения мне кажется, что FSD в данном проекте стоит использовать намного меньше, чем есть сейчас. Например, ограничиться только слоями shared и pages, да и то, может быть, многовато, учитывая, что тут только две страницы. Ещё немного не по назначению использованы слои сущностей и фич. На слое сущностей лежит ProjectCard, это звучит как UI-компонент в сущности project. Фичи тоже сложно оправдать, это скорее просто компоненты из shared/ui, потому что в них не зашиты бизнес-сценарии, да и не переиспользуются они. Я думаю, этот проект не стоит включать в примеры FSD. Скажи, что думаешь на этот счёт |
|
Привет @illright, спасибо за ревью проекта. В защиту скажу следующее: я старался понять и реализовать на практике каждый слой. Поэтому, как и у вас в туториале Касательно фич, как по мне, не все так однозначно. Опять же, если ссылаться на туториал, переключатель выполненной / невыполненной задачи Безусловно, можно было весь проект реализовать только в слоях shared и pages, но, как по мне, в примерах должны быть те, в которых используется как можно больше слоев. Это, как раз, и дает понять насколько может быть удобна архитектура. Чем мой проект хуче чем проект Polka который уже добавили в примеры? |
|
Понимаю, что то, что я сказал, звучит очень странно в контексте туториала на сайте ФСД. Честно признаться — туториал написан не очень хорошо, и давно уже не отражает лучшие практики использования ФСД. Просто переписать его очень непросто, этим я занимаюсь, но прогресс по этой задаче идет очень медленно.
как по мне, в примерах должны быть примеры удобных и легко понятных проектов. Цель тут — не собрать количество слоев, цель — сделать проект простым. В случае твоего проекта, дополнительные слои усложняют понимание проекта, по крайней мере, мне.
С перспективы пример проекта на ФСД, твой проект содержит излишние усложнения, учитывая, что это статичный сайт. Проект Полка содержит динамические данные и реальные "бизнес-сущности", там эта декомпозиция чуть более оправдана, хоть и все равно, имхо, излишня. |
Я еще раз просмотрел дату, когда редактировалась статья туториала и она очень свежая (28 янв. 2024 г.). По этому принцыпу у вас много висит примеров и только мне вы говорите что этот подход устарел. Устарел чуть меньше чем за месяц. Честно говоря, я не понимаю в чем проблема сказать мне что подправить чтобы этот проект подходил на пример использования методологии, а не говорить что Вам лично не нравиться этот проект? |
|
Чтоб этот проект был хорошим примером FSD, хотелось бы добавить в него больше функциональности, страниц, или динамических данных. Статья туториала, на самом деле, была написана года три назад, если не больше. У нас просто криво работают указатели последнего редактирования страниц, да и недавно там были правки грамматических ошибок. Извини, что ввели в заблуждение своей писаниной. Постараюсь поскорее выкатить свои наработки по туториалу, чтоб больше не путать людей. |
|
Сделал ПР с новым туториалом, можешь почитать, если интересно, скоро смерджу — https://deploy-preview-652--pr-fsd.netlify.app/ru/ |
|
Окей, я понял |
|
ПР застыл, закрою его |
|
Сори, у меня сильно все поменялось по этому не мог сделать проект по новой методологии. |
Пример с использованием React
https://github.com/semklim/portfolio