Юрий
da
Юрий
Юрий
Roman
Юрий
Огромное спасибо!!!!!
Oleg
такой вопрос... хуки ведь полезны? компоненты на хуках при прочих равных легче разрабатывать, поддерживать?
Александр
Igor
А зачем что-то доказывать? Если версия реакта позволяет — делай сам)
Александр
окей, а как доказать команде ценность хуков? могу я вообще что-то сделать будучи джуном?
Зависит от команды, вообще говоря можете показать им пример какой-либо из вашей кодовой базы переписанный на хуках
Mark
А зачем что-то доказывать? Если версия реакта позволяет — делай сам)
А потом получить по шапке)
Oleg
А зачем что-то доказывать? Если версия реакта позволяет — делай сам)
меня просто иногда тянет переписать компонент (особенно если он маленький) с класса на функцию. или переписать тот компонент, с которым я работаю. как-то уменьшить его и тп.
Anonymous
Oleg
Так и нужно делать: переписывай
ну так и делал. но мне сказали "не лезь, оно тебя сожрёт"
Igor
Ну переписывать вот чревато. Какой-то новый компонент — почему бы нет?
Anonymous
ну так и делал. но мне сказали "не лезь, оно тебя сожрёт"
Тогда ливай с тимы, скорее, это она тебя сожреёт
Oleg
Тогда ливай с тимы, скорее, это она тебя сожреёт
ну не согласен. тима опытная, мне есть чему поучиться.
Anonymous
Дело твоё
Igor
Я бы тоже напрягся, если бы джун полез переписывать что-то с нуля) Может, на то были причины
Александр
Я бы тоже напрягся, если бы джун полез переписывать что-то с нуля) Может, на то были причины
Переписать с целью показать что вот мол смотрите как на хуках проще и понятнее, и на ревью выставить, почему нет, тут цель то именно показать в сравнении два подхода
Александр
И при чем тут джун иди не джун, хотя если в тиме дедовщина, и право высказать мнение есть только у сеньоров, то лучше начать подыскивать другое место работы
Ruslan
Igor
Тут видимо срабатывает знание одно из заповедей айти «работает — не трогай». Но нежелание команды внедрять новые фичи конечно заставляет задуматься, да
Valentin
Зависит от команды, вообще говоря можете показать им пример какой-либо из вашей кодовой базы переписанный на хуках
Может ты сам попробуешь создать пример. Взяв за основу плохо читаемый компонент. А ещё лучше и хорошо читаемый. И покажешь преимущество и сам попробуешь и будешь более уверено понимать нужны хуки или не нужны?
Oleg
Тут видимо срабатывает знание одно из заповедей айти «работает — не трогай». Но нежелание команды внедрять новые фичи конечно заставляет задуматься, да
выглядит, как наиболее адекватное мнение. короче, ладно. оставлю переписи на потом. спасибо.
wolfe
выглядит, как наиболее адекватное мнение. короче, ладно. оставлю переписи на потом. спасибо.
я бы переписывал в случае необходимости или отсутствии тасков на данный момент. просто если оно работает, зачем это трогать? да и это человеко-часы к тому же
а новые компоненты можно писать на хуках
Mark
Причина?
Я к тому, что тима видимо про хуки только слышали, ну может поигрались на каких-нибудь тудушках. Иначе почему вопрос стоит как "доказать ценность хуков"? Получается команда и не знает об их преимуществах.
Тогда кто будет поддерживать такой код? Только сам Олег? А если он уйдет через месяц-два. Заставить всю тиму сидеть учить хуки? Как сказали выше – это человеко-часы.
Разумеется нет ничего плохого в том, чтобы написать часть компонентов на хуках и показать тиме. Но вот "плюй и делай как хочешь" – звучит так себе
Igor
А чего их учить-то?)
Igor
Это не с jquery на реакт переписывать же
wolfe
Я к тому, что тима видимо про хуки только слышали, ну может поигрались на каких-нибудь тудушках. Иначе почему вопрос стоит как "доказать ценность хуков"? Получается команда и не знает об их преимуществах.
Тогда кто будет поддерживать такой код? Только сам Олег? А если он уйдет через месяц-два. Заставить всю тиму сидеть учить хуки? Как сказали выше – это человеко-часы.
Разумеется нет ничего плохого в том, чтобы написать часть компонентов на хуках и показать тиме. Но вот "плюй и делай как хочешь" – звучит так себе
хуки релизнулись под слоганом "No breaking changes" и их использование на данный момент является необязательным.
мы не знаем всех ньюансов, поэтому не нужно делать поспешных выводов.
единственное — если команда действительно не знает, то можно предложить ознакомиться
Default
Я к тому, что тима видимо про хуки только слышали, ну может поигрались на каких-нибудь тудушках. Иначе почему вопрос стоит как "доказать ценность хуков"? Получается команда и не знает об их преимуществах.
Тогда кто будет поддерживать такой код? Только сам Олег? А если он уйдет через месяц-два. Заставить всю тиму сидеть учить хуки? Как сказали выше – это человеко-часы.
Разумеется нет ничего плохого в том, чтобы написать часть компонентов на хуках и показать тиме. Но вот "плюй и делай как хочешь" – звучит так себе
Если тиме нравится акробатика с рекомпоусом или плодить классы на каждую необходимость в локальном стейте, то флаг в руки и барабан на шею, собсна
Default
И ваще что значит «переходить» на хуки? Это ж не значит классы по всему коду выпилить, это просто в будущем не вкорячивать класс ради лайфсайкла/стейта
Valentin
Oleg
Пзд. Рили?
что? ну подозрительно же выглядит, когда джун пытается переписать что-то. к тому же я пару раз допускал ошибки при переписи. эти ошибки не связаны были с хуками, но всё же.
Artur
что делать, если ушло понимание, что делать 😂
Default
Default
Дали тебе задачку, сделай два варианта, с хуками и без
Valentin
что? ну подозрительно же выглядит, когда джун пытается переписать что-то. к тому же я пару раз допускал ошибки при переписи. эти ошибки не связаны были с хуками, но всё же.
Подозрительно выглядит, когда челик предлагает хуки, но не готов и один компонент написать на них, что бы доказать эффективность. В общем ты уже такой как и твоя тима. Она тебя сожрала. Не поберёг ты себя. Код тебе пухом
Alexey
что? ну подозрительно же выглядит, когда джун пытается переписать что-то. к тому же я пару раз допускал ошибки при переписи. эти ошибки не связаны были с хуками, но всё же.
так не надо выпиливать при переписывании.
есть superComponent, пишешь новый superComponentOnHooks рядом, показываешь тиме, проверяете, всё работает? - profit
Alexey
отдельным коммитом перенос на новый компонент, отдельным выпиливание старого, чтобы можно было легко откатить изменения
DjKaira
Задача. Распечатать кусок html блока, указав кол-во копий и размер бумаги для распечатки. Проблема в том, что размеры разные и неудобно каждый раз менять параметры печати при распечатке. Кто знает, в какую сторону копать ? Нужно чтоб просто нажал на кнопку распечатать, появилось окно браузера распечатки с уже проставленными, нужными параметрами
Default
так не надо выпиливать при переписывании.
есть superComponent, пишешь новый superComponentOnHooks рядом, показываешь тиме, проверяете, всё работает? - profit
Ещё раз, зачем переписывать существующий код?
Михаил
интересно что за консервативная жилка в некоторых присутствует. хуки уже пол года стабле. и до сих пор спорят что лучше что хуже )) @sovasergey у вас уже вышел первый аудио подкаст? Как то слушал превью.
wolfe
что? ну подозрительно же выглядит, когда джун пытается переписать что-то. к тому же я пару раз допускал ошибки при переписи. эти ошибки не связаны были с хуками, но всё же.
как правило, ошибки при переписывании случаются достаточно часто.
переписывание — это не плохо, просто нужно грамотно время распределять, как по мне
Alexey
Ещё раз, зачем переписывать существующий код?
хуки - реюз кода (логику легко вынести можно при необходимости), выглядят обычно лаконичнее и читаются проще (это уже субъективно, но у нас большинство компонентов с классов в функции стали выглядеть "опрятнее")
Oleg
хуки - реюз кода (логику легко вынести можно при необходимости), выглядят обычно лаконичнее и читаются проще (это уже субъективно, но у нас большинство компонентов с классов в функции стали выглядеть "опрятнее")
ну вот кстати реюза кода я не заметил. либо я что-то не так делаю, либо это "просто компонент на хуках".
Default
хуки - реюз кода (логику легко вынести можно при необходимости), выглядят обычно лаконичнее и читаются проще (это уже субъективно, но у нас большинство компонентов с классов в функции стали выглядеть "опрятнее")
Не, ты не понял. Зачем переписывать то, что уже написано и работает, если у тебя нет соответствующей таски?
Alexey
Ещё раз, зачем переписывать существующий код?
остальных "плюшек", что пишут в доках по реакту я не понял если честно. Особенно про "классы путают людей и кампутеры"
Bogdan
окей, а как доказать команде ценность хуков? могу я вообще что-то сделать будучи джуном?
не докажешь, пока в очко не залезут, хотя они же не вытеснили хоки и рендер пропы, но я не могу придумать чем они хуже
🦜
хуки - реюз кода (логику легко вынести можно при необходимости), выглядят обычно лаконичнее и читаются проще (это уже субъективно, но у нас большинство компонентов с классов в функции стали выглядеть "опрятнее")
не только реюз кода, у тебя еще уходят баги с this, когда стейт не тот, который ты ожидал
Михаил
ну вот кстати реюза кода я не заметил. либо я что-то не так делаю, либо это "просто компонент на хуках".
можно несколько хуков в один вынести и потом переиспользовать.
Alexey
Не, ты не понял. Зачем переписывать то, что уже написано и работает, если у тебя нет соответствующей таски?
Например потому, что программировать мне нравится, а зарплата - приятное дополнение, а не наоборот
Artur
https://pastebin.com/XPgemLJu
Не подскажите, почему данные в выпадающем списке не подтягиваются? Апишка работает
Михаил
не только реюз кода, у тебя еще уходят баги с this, когда стейт не тот, который ты ожидал
стрелочные функции все решали )
wolfe
можно несколько хуков в один вынести и потом переиспользовать.
HOC, но с другой стороны и проще
Oleg
можно несколько хуков в один вынести и потом переиспользовать.
ну... я не знаю, что это и как делается. где почитать? в доке?
Default
Например потому, что программировать мне нравится, а зарплата - приятное дополнение, а не наоборот
Код это не самоцель в коммерческом проекте. Это некая решённая проблема. Каждая написанная строчка кода это деньги на прогера, деньги на ревьюера, на тестирование и цена багов, которые там попадутся. Уже написанный код не надо переписывать, если у него нет фатальных недостатков, мешающих дальнейшему использованию. Новый код лучше писать с использованием новых фич, при рефакторинге можно перепилить, просто так переписывать нахер никому не надо.
Alexey
Код это не самоцель в коммерческом проекте. Это некая решённая проблема. Каждая написанная строчка кода это деньги на прогера, деньги на ревьюера, на тестирование и цена багов, которые там попадутся. Уже написанный код не надо переписывать, если у него нет фатальных недостатков, мешающих дальнейшему использованию. Новый код лучше писать с использованием новых фич, при рефакторинге можно перепилить, просто так переписывать нахер никому не надо.
Значит объявят классы фатальным недостатком, сообщество сожрёт и будет переписывать. JS-сообщество же.
Artem
Люди добрые! Подскажите пожалуйста. Есть список кривых дат, которые нужно обработать (лишние знаки, пробелы, месяц словом). Я их обрабатываю пользовательским хуком и возвращаю массив с исправленными значениями. И далее мне в компоненте этот массив нужно обработать и сделать возможность сортировки. В чем боль. В каком виде мне из пользовательского хука возвращать даты? Если в .toLocaleString(), то он будет сортировать как строки. Если возвращать в new Date формате ругается примерно так:
Objects are not valid as a React child (found: Fri Jun 15 2018 00:00:00 GMT+0500 (Екатеринбург, стандартное время)). If you meant to render a collection of children, use an array instead.
Если перевести в миллисекунды (аля геттайм). То незнаю как вернуть обратно в дату. Но суть в том, что после сортировки даты должны уже отображаться как dd.mm.yy
Paul
Rita
Кто работал с анимацией js spring? Есть вопрос
Rita
Анимация резко заканчивается, если прервана другой анимацией. Что с этим можно сделать? Если анимацию не прерывать и дать ей закончиться, то работает гладко
Rita
<Spring
native
from={{ opacityFrom: 0, opacityTo: 1 }}
to={{ opacityFrom: 1, opacityTo: 0 }}
config={{ duration: 2000, easing: quadInOut }}
key={`${this.state.activeSector}_${this.state.prevSector}`}
>
{({ opacityFrom, opacityTo }) => {
return (
<>
{this.renderPieContent(this.state.prevSector, amount, {
opacity: opacityTo,
})}
{this.renderPieContent(this.state.activeSector, amount, {
opacity: opacityFrom,
})}
</>
);
}}
</Spring>
Ivan
спасибо, а обратно как в дату перевести?
https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/Date
Ruslan
Я не вижу пока смысла переписывать на хуки, хотя при этом у нас практически the cutting edge, но я считаю хуки - проходной технологией