А ну если так, Но тогда очень много методов будет же

Или это ничего страшного?

ну

Где-то я видел правило что метод должен быть не более 7 строк

можно не только по методам разделять

но и по классам

было время, когда действовало правило "метод должен помещаться в один экран"

было время, когда действовало правило "метод должен помещаться в один экран"

все равно хреновый метод

Ладно, буду придерживаться этому правилу

А ну если так, Но тогда очень много методов будет же

много легкочитаемых методов лучше чем мало плохочитаемых - для этого правило и придумали

много легкочитаемых методов лучше чем мало плохочитаемых - для этого правило и придумали

везде важна золотая середина

а не класс на 100 однострочных методов

Есть книга "Чистый код", вот там во главу угла поставлено это всё. И там ещё про компромисы когда излишние ограничения могут навредить и тп.

И как когда и как поступать

С примерами

Правда на Java

Но там не про язык, так что понятно

а не класс на 100 однострочных методов

а что в этом плохого?

а что в этом плохого?

бегать по всему классу чтобы понять его работу нахер надо

+

мне особенно "нравятся" классы типа def call set_post process_form update_data end

мне особенно "нравятся" классы типа def call set_post process_form update_data end

вполне норм, что не нравится?

вполне норм, что не нравится?

потому что бегаешь по методам как дурак

а особо умные там сайдэффекты создают

мне особенно "нравятся" классы типа def call set_post process_form update_data end

этож красота!

основная бизнеслогика должна проходить в главном методе, с методами хелперами.

основная бизнеслогика должна проходить в главном методе, с методами хелперами.

звучит как фигня

звучит как фигня

то что я выше скинул фигня

Тут всё зависит от того а что в этих методах

def main

у меня основная логика разделена на 5 классов и методов 10 наверно

def main

именно!

а особо умные там сайдэффекты создают

ну это уже др тема

то что я выше скинул фигня

там у тебя должен быть последовательный и инкапсулированный флоу

именно!

что именно? def main юзает классы, все просто

тоесть def call #300 строк одного метода #200 строк другого метода #еще 150 строк кучки мелких вспомогательных методов end выглядит круче?

тоесть def call #300 строк одного метода #200 строк другого метода #еще 150 строк кучки мелких вспомогательных методов end выглядит круче?

нету в call и 150 строк

То есть смысл то в том что есть метод, который делает то как назван, есть 3 метода-шага, которые делают по частям это единое целое, при этом внутри уже низкоуровневая бизнес-логика с вычислениями.

даже 50 не наберется. Если наберется то стоит подумать о других классах.

Идеально когда шаги могут быть переиспользованы

https://robots.thoughtbot.com/sandi-metz-rules-for-developers

И переиспользуются

идеальные правила

> Methods can be no longer than five lines of code. Фигня

Есть более жесткие с 3мя аргументами

нет

идеальные правила

докажи мне, что у тебя нет методов длинее 5 строк ?

да

это делает все понятным

это делает все запутанным наоборот

и заставляет тебя думать над названиями

нет это заставляет тебя без причины плодить методы

А вот 5 строк мало, 7 норм. Но вы уже ощищаете это?! Дух холивара за лимиты правил!

а потом бегать как дурак.

строк 15 норм, у меня там банально хэшик может собираться

Хеш, кстати, можно за одну строку считать, если там нет вычислений сложных внутри прям

Хеш, кстати, можно за одну строку считать, если там нет вычислений сложных внутри прям

ну да, бегай по 100 методам разбирайся какая конечная форма у хэша будет

Ну вот, собственно компромисы

Это очень тонкое искусство конечно - писать хороший код

@davydovanton только если совсем никак не разделить)

Где не 100500 строк и не 100500 вложенностей, но и не 100500 методов

Это очень тонкое искусство конечно - писать хороший код

тонкое искуство это найти золотую середину.

самое тупое, что я могу примеров 20 с головы назвать, когда не разделить

И я об этом

самое тупое, что я могу примеров 20 с головы назвать, когда не разделить

У меня такой алгоритм был, где куча матана и нельзя было просто так взять и адекватно порезать без боли и 100500 аргументов

У меня такой алгоритм был, где куча матана и нельзя было просто так взять и адекватно порезать без боли и 100500 аргументов

да банальнее пример if interactor.success? flash[:status] = 'cool' redirect_to ... else render ... end

это уже 6 строк

ERROR: S client not available

Вот, поэтому по мне так 5 мало

А так да, по факту 15 вполне себе ок

Но главное чтобы они делали одно действие, заданное именем

5 строк лимит на метод?

шта?

Ну, за исключением извращенских методов с названием doProcess и тп, особенно когда вообще не ясно что за процесс то

я очень критичен к длине своих методов, но даже для меня это слишком

шта?

Есть такие, да

10-15 самое то

15 оптимально

иногда больше если метод хорошо читается.

Есть компании где программистам платили за количество методов.... ну вы поняли....

KPI вообще ерунда.

считать сколько дохода принес погромист - ооооч сложно

а какой профит от 5 строк? как аргументируется?

ибо в куче случае ты даже не сможешь придумать нормальное название метода, чтобы вынести

если у тебя допустим 7 строк вместо 5

Аргументируется мол абстракция простая выходит и быстро понимаешь

Ну, вот так вот считают авторы правил

а называть методы в стиле do_one_and_do_two_if_conditions_true

это уже маразм, имхо

Аргументируется мол абстракция простая выходит и быстро понимаешь

вангую, что это от дяди Боба выстрелило?

он любит давать дельные советы, но не говорить о том, что это рекомендации, а не железное правило

и после программисты зарабатывают невроз от того, что не могут писать как дядя Боб

а дядя боб смееца.

я серьёзно, убил себе когда-то кучу нервов, разнося неразносимое

по советам дяди Боба, когда методы были слишком длиными, чтобы пройти его правила

да банальнее пример if interactor.success? flash[:status] = 'cool' redirect_to ... else render ... end

Плохой пример if interactor.success? redirect_to ..., flash: { status: 'cool' } else render ... end

Плохой пример if interactor.success? redirect_to ..., flash: { status: 'cool' } else render ... end

реальный метод редиректа вылетит на две строки

Плохой пример if interactor.success? redirect_to ..., flash: { status: 'cool' } else render ... end

этот пример похуже, всё-таки установка flash на отдельной строке очевиднее будет

сразу перед глазами, как отдельное действие

а не как часть редиректа