u
но это прям такое себе
Size: a a a
2021 May 31
AD
Что бы его нормально назвать и упростить основную логику
NB
читабельность, например, мы ради неё делаем очень многое
AD
И не разбираться в коде каждый раз как в первый раз
CR
я про тот случай когда метод вызывается один раз и в одном месте.
Ну если у тебя там не так, я просто не рассматривал, то норм имхо.
Ну если у тебя там не так, я просто не рассматривал, то норм имхо.
NB
боюсь даже представить насколько критическим должна быть та часть системы где это даст прирост производительности :D
AD
Так что плохого, в том, что он в одном месте один раз вызывается?
u
Ну я помню когда в универе был, была отдельная каста ребят у нас, которые на сях байтоёбством занимались
u
Мало ли, кому-то нравится
NB
с лямбдой ещё вопрос будет по куче/памяти я думаю, в если она в переменной какой - то и GC
AD
Если что, я правда не могу понять в чем проблема и зачем читаемость усложнять
NB
Не ну для олимпиадок там да, дело понятное. Там каждая наносекунда важна и всё такое
А в энтерпрайзе вопросы другие всё-таки
А в энтерпрайзе вопросы другие всё-таки
CR
ну когда ты как новый разраб ты начинаешь смотреть - где такой метод еще вызвается и почему он например не приватный, но у тебя он приватный это ладно
DV
Ну я тоже подумал, что проще вынести в метод - но я подумал о разделении абстракции. В ифе мы решаем что выполняем, а детали выполнения - смотрим по запросу, если нужно.
AD
И в чем проблема, если он в рамках мелкого класса?
CR
в том что таких методов может быть дофига, хоть и будут они лежать в привате все.
NB
Так а что в этом плохого? 🤔
AD
Так может быть что угодно, тут надо смотреть каждый пример конкретный тогда
NB
Это вы ещё Sequel::Model не видели на 2000+ строк кода :D
CR
плохо тем что неудобно - ориентироваться и бегать по коду вниз к методу изучать его и обратно к тому месту где он вызывается - не удобно.