D
А разделение кода там, не?
Size: a a a
2020 July 10
D
Редух отдельно, Реакт отдельно
OR
Редух отдельно, Реакт отдельно
И что это разделение даст?
ei
И что это разделение даст?
Тоже интересно)
D
Просто комфортнее как то становится, компактнее, по ощущениям)
RU
я заметил только один минус. мне сложней писать тесты, и я все чаще тестирую теперь "фичу целиком", а не каждый компонент в отдельности
RU
потому что впадло мокать useSelector/useDispatch
АБ
Просто комфортнее как то становится, компактнее, по ощущениям)
компактнее это как? когда у тебя по 2 файла вместо одного?
D
Да, когда в каждом файле весь код помещается на 1 максимум 2 экрана
VO
компактнее это как? когда у тебя по 2 файла вместо одного?
Можно написать контейнер в том же файле)
RU
Да, когда в каждом файле весь код помещается на 1 максимум 2 экрана
у меня ментальный барьер ~100 строк. пока все влазит
ei
У меня компоненты в основном ~50 строк
RU
просто если у тебя в контейнерах столько логики, что не влазит, то ты все равно чет не так делаешь)
АБ
Да, когда в каждом файле весь код помещается на 1 максимум 2 экрана
это не имеет практического смысла. гораздо проще проскроллить чем прыгать по миллиарду файлов. никто конеш не топит делать компоненты в тыщи строк, но ради того чтобы вынести connect делать отдельный файл та еще хуйня)
ei
я заметил только один минус. мне сложней писать тесты, и я все чаще тестирую теперь "фичу целиком", а не каждый компонент в отдельности
Так разделение на компоненты и контейнеры вообще ничего не меняет, ведь у тебя во вьюшке снизу так же будут контейнеры которые так же придется мокать
RU
Так разделение на компоненты и контейнеры вообще ничего не меняет, ведь у тебя во вьюшке снизу так же будут контейнеры которые так же придется мокать
а так я могу экспортнуть "чистый" компонент и тестить его
D
это не имеет практического смысла. гораздо проще проскроллить чем прыгать по миллиарду файлов. никто конеш не топит делать компоненты в тыщи строк, но ради того чтобы вынести connect делать отдельный файл та еще хуйня)
Очень бесит скроллить)))
ei
а так я могу экспортнуть "чистый" компонент и тестить его
У тебя на фичу 1 контейнер сверху был?
н#
я кстати тоже не могу понять, где ПР, как такое добавили, на каком основании
смотри ишью с названием scarf
RU
смотри ишью с названием scarf
уже нашел