сжечь
увы, даже после того как все в команде поняли что это не самый корректный подход, он сказал, процитирую: "да безполезно меня убеждать. Я уверен в правильности своего подхода. Да, хук если не нравится, это вкусовщина, можно переделать."
А как ты понял, что это вообще за код и что он делает? Ну, типа, почему это плохо, мне задача не понятна
Ну этот хук вызывается на разных страницах, когда можно просто взять внутренность хука и заинлайнить на самих страницах)
ну до конца я не понимаю, но тут явно какой-то код, который понимает, на какой странице он запущен и в зависимости от этого делает разное. мне кажется это явным говнокодом. в том смысле, что при добавлении новых страниц нужно будет постоянно расширять эту функцию
это было бы первое предупреждение, ещё бы 2 и уволил бы к чёртовой матери
React Router v7
Та ну хз, где-то придется все равно расширять, можно придумать кейс, где похожий подход был бы допустим, как если бы у всех страниц должен быть общий контекст и хук менеджит общий стейт для них, но видимо это не тот случай
Тут просто не понятно что такое "скрин"
Там по-любому уже есть какой-нибудь компонент или модуль под каждую страницу. И вот в нём надо под конкретное место писать конкретный набор элементов.
screen это страница, просто route
Так я ж говорю, я хз что делает этот код 😁
Обсуждают сегодня