Телеграмм чат группы pro

Как вот такую ссанину можно отрефакторить красиво?

def scope
      scope = AppCreationLog.all
      scope = scope.bundle_like(query) if query
      scope = scope.greater_than_limit if greater_than_limit
      scope
    end

18:29пожаловаться #1

Это точно не худшая ссанина что я видел. Кстати не уверен но последний scope вроде можно убрать

20:20пожаловаться #2

Вообще на это дело можно абстракцит накидать, и даже с dsl, но надо ли?

20:20пожаловаться #3

Alex Ilizarov

Вообще на это дело можно абстракцит накидать, и даже с dsl, но надо ли?

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

20:30пожаловаться #4

Roman Usherenko

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

Согласен, все чаще склоняюсь к большему хардкоду для более понятной бизнеслогики

20:30пожаловаться #5

я тоже раньше был за метамагию во все поля :)

20:31пожаловаться #6

сейчас чуть меньше, в большей степени благодаря работе с js :)

20:31пожаловаться #7

2019 November 28

Dmitry Gorodnichy

Как вот такую ссанину можно отрефакторить красиво?

def scope
      scope = AppCreationLog.all
      scope = scope.bundle_like(query) if query
      scope = scope.greater_than_limit if greater_than_limit
      scope
    end

почему ты считаешь это ссаниной?

09:28пожаловаться #8

Rustam Ibragimov

почему ты считаешь это ссаниной?

Да я то не считаю, тем более это я и написал) но линтер ругается.

09:34пожаловаться #9

вполне нормальный код.

линтер на что ругается? ты ведь его настраивал :)

09:35пожаловаться #10

W: Move model logic into model (scope use_count > 4)

10:05пожаловаться #11

Вот такое ему не нравится

Alexander Andreev in pro.rb (Ruby/Rails / RU)

10:05пожаловаться #12

Dmitry Gorodnichy

W: Move model logic into model (scope use_count > 4)

Расскажи ему что fat model это тоже плохо

10:22пожаловаться #13

тут у тебя логика фильтрации (спецификации), которая спецефична для каких-то определенных кейсов твоего домена, я так понимаю

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

10:29пожаловаться #14

Та я просто заигнорил постхуки пуша. Пускай ревьюверы решат пустить такое в мастер или нет

10:33пожаловаться #15

так тогда в коде останется варнинг

10:43пожаловаться #16

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

10:55пожаловаться #17

так себе у вас проверки)

11:27пожаловаться #18

Dmitry Gorodnichy

А вот это уже пиздец. Почему нельзя убрать правило для определенного пути?

11:32пожаловаться #19

Ой кажется у меня тут ссанина, поэтому отложу гавнеца чтобы санья не заметили.