Телеграмм чат группы rubylang страница 11103

Size: a a a

Ruby, Rails, Hanami | dry-rb

2024 membersпожаловаться на группу

2020 December 04

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

Рубокоп такое не пропустит -> рефакторить)

источник

16:17пожаловаться #1

Eugene Shved in Ruby, Rails, Hanami | dry-rb

Viacheslav Stepanov

Рубокоп такое не пропустит -> рефакторить)

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

источник

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

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

рубокопу бы научиться бы бить по рукам, мир бы стал лучше

источник

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

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

ну стандартная настройка не пропустит manage точно)

источник

16:18пожаловаться #4

Eugene Shved in Ruby, Rails, Hanami | dry-rb

Антон Дьячук

а что тут не DRY?

Эта штука, как 3 строки встречаются 10 раз в коде.

источник

16:19пожаловаться #5

АД

Антон Дьячук... in Ruby, Rails, Hanami | dry-rb

Eugene Shved

Эта штука, как 3 строки встречаются 10 раз в коде.

ну там же разное содержимое

источник

16:19пожаловаться #6

Kernel panic in Ruby, Rails, Hanami | dry-rb

Viacheslav Stepanov

Рубокоп такое не пропустит -> рефакторить)

источник

16:21пожаловаться #7

Eugene Shved in Ruby, Rails, Hanami | dry-rb

Ну можно же прокидывать пару аргументов и присваивать в методе?

источник

16:22пожаловаться #8

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

Да там выходные параметры не везде совпадают даже

источник

16:22пожаловаться #9

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

первый 5 item_type вроде ложатся в шаблон, а потом для article уже привет, а для special вообще зоопарк

источник

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

Eugene Shved in Ruby, Rails, Hanami | dry-rb

Viacheslav Stepanov

Да там выходные параметры не везде совпадают даже

Да, я вижу, увеличить количество аргументов, то что не нил присваивать?
Далее убрать elseif заменить на return if

источник

16:23пожаловаться #11

Eugene Shved in Ruby, Rails, Hanami | dry-rb

Вынести отдельную логику в дополнительный код?

источник

16:23пожаловаться #12

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

та помоему, там слишком много функционала внесли

источник

16:24пожаловаться #13

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

Мне вообще manage метод не нравится

источник

16:24пожаловаться #14

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

это ж какраз про солид

источник

16:24пожаловаться #15

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

да

источник

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

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

он много на себя взял по-моему

источник

16:24пожаловаться #17

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

дадада

источник

16:24пожаловаться #18

Viacheslav Stepanov in Ruby, Rails, Hanami | dry-rb

это типа как давайте вместо 10 эндпоинтов сделаем один manage и будем потом параметрами рулить все)

источник

16:25пожаловаться #19

Sergii Serogin in Ruby, Rails, Hanami | dry-rb

обычно такое исторически накапливается

источник

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