Nikolai
Nikolai
Егор
Егор
Я уже скинул рекомендацию реакта
Егор
Daniil
Егор
Понимаю, но с тем что ничего не дает не согласен
Егор
То что в моменте ничего не дает - не означает что ничего не дает в принципе. Ты понимаешь разницу между стратегией и тактикой?
Егор
Daniil
Нуну
Дай конкретный пример где у тебя требовалось добавить реф в зависимость и это спасло тебя
Dmitriy
Nikolai
@EgorKluch Вы с Андреем звёздочкой не в одной команде случайно? А то он тут недавно рассказывал, что все хендлеры в useCallback оборачивает "на всякий случай"
Ivan
Егор
Егор
А не, ну да. Но я таких кейсов не встречал)
Егор
Daniil
Так этому в Яндексе или Альфе тебя научили?
Егор
Daniil
Но ведь должна была причина, почему ты начал такое использовать. Где ты получил выстрел в ногу
Daniil
Ведь если ты за это топишь, то был кейс реальный.
Егор
Daniil
Расскажешь зачем рефы в дипсы?
🦜
Вы целый час о нихуя ведёт дискас
Daniil
Или ты просто решил "на всякий случай"?
i
эмм... как так получается что фолс но в хранилище тру? переменная status также прописана но меняетсмя значение
Егор
Ведь если ты за это топишь, то был кейс реальный.
ну я просто когда изучал хуки очень много багов ловил. Изучал инфу читал статьи, вот и все. Не вспомню уже. Я уже рассказал, лень десятый раз писать. Просто задумываюсь - можно ли потенциально сломать случайно? Вот и все.
Есть риск - можем случайно создать баг.
Есть меры по его предотвращению - добавить лишний deps. Цена мер - практически нулевая по времени. Так почему не использовать?
Daniil
То есть, просто так решил? На всякий случай
Егор
А могут ли эти доп меры сломать код? Ну крайне меловероятно - но об этом конечно задумываюсь когда пишу код. Таких кейсов пока не встречал. Выше был один
Daniil
Бойлерплейт не ломает код, но усложняет его понимание
Dmitriy
Dmitriy
Daniil
Устранять то, что еще не проявляло себя в целом судя по ишью и комьюнити реакта.
Егор
Nikolai
Dmitriy
Nikolai
Лишний шум в коде
Daniil
Daniil
и всю команду соответственно на проекте
Егор
Егор
Спасибо
Егор
Может правила этого нет тупо. Проверю.
Егор
Но не понимаю что там может сломаться. То что ref.current юзаю в депсах?
Daniil
Писать проверки нужно к тому, что действительно может выдать сбой. Ты встречал ишью или подобное в комьюнити реакта, где рефы в дипсах решали проблему, кроме кеширования?
Егор
Не понял тебя, наверное нет. Они контролируют кэширование и колличество вызовов. Ну как бы я это и контролирую же, что бы точно вызвалось. Минус - лишний раз вызывается хук - но пока не требуется такая оптимизация, и не могу представить кейс когда потребуется.
Егор
Ну то есть это еще не видел что бы было самым узким звеном в быстродействии. Обычно нужно просто дополнительно кэшировать сложные вычисления и т.д. Ну я не работаю там где прям микрооптимизация нужна и каждая миллисекунда важна - там может быть другие правила.
Daniil
В общем, я вел к тому, что подобными действиями, ты можешь ухудшить читаемость и понимание, банально человек может не понять, зачем ты реф в зависимость засунул, ведь я уверен, такое не часто можно увидеть. Линтер хуков тоже начнет ругаться, его отключать, либо настраивать. И так поэтапно. По итогу, если у комьюнити реакта не было проблем с кейсом, который тут упорно обсуждают в треде, то я пока не вижу нужды в этих действиях. Как только - поправим. А пока это больше похоже на псевдооптимизации лично мне
Егор
В общем, я вел к тому, что подобными действиями, ты можешь ухудшить читаемость и понимание, банально человек может не понять, зачем ты реф в зависимость засунул, ведь я уверен, такое не часто можно увидеть. Линтер хуков тоже начнет ругаться, его отключать, либо настраивать. И так поэтапно. По итогу, если у комьюнити реакта не было проблем с кейсом, который тут упорно обсуждают в треде, то я пока не вижу нужды в этих действиях. Как только - поправим. А пока это больше похоже на псевдооптимизации лично мне
Понял тебя. Спасибо.
Егор
https://codepen.io/EgorKluch/pen/ZEEqZRm - а вот банальный пример сломанного кода (изначально на пустой массив и был пример с useEffect) Но я тебя, @Tchernyavsky услышал, подумаю.
Daniil
Егор
Ааа. Ну кстати да. Логично, реально
Егор
https://codepen.io/EgorKluch/pen/ZEEqZRm - вот как пример, но если добавить зависимость - ререндера все равно не будет)
Егор
Потому что это же не стейт и не пропс
Dmitriy
Что вместо момента взять?
Daniil
date-fns
Max
Max
https://github.com/you-dont-need/You-Dont-Need-Momentjs
Dmitriy
Daniil
Год назад, да ( сейчас может игра и изменилась, но суть та же
Dmitriy
Ruslan
Что там, 16.12 уже выкатывают?
invasion
Michail
Всем привет
Не кто не знает почему если я пишу у компонента style={{marginBottom: 24}}, а у этого компонента есть div className={props.default ? classes.first : classes.second}, мои инлайн стили к нему не присваиваются?
Nikolai
Nikolai
Michail
Да
Понял, и это нормально?
Nikolai
Michail
Вполне
не знаешь как по regex находить в webstorm'e?
Андрей
Привет, подскажите, пожалуйста, кто работает с mobx, почему при первом изменении тектового поля слетает фокус. Как будто реакт не запоминает это текстовое поле в доме и добавляет его заного. Пробовал указывать атрибут key, но мне не помогло
Valentin
Андрей
Перерисовывает компонент. Ты знаешь что делает реакт и где?
Понятно что он его рендерит. У меня экшен вызывается при событии onChange у текстового поля. Я подозреваю что он его заного создаёт после первого изменения, так как вызывается конструктор, но ведь при последующих изменениях этого не происходит. Значит в виртуальном доме у него есть этот инпут и реф ссылка?