AI
Иначе все это крайне сложно в отладке и дебаге
Size: a a a
2020 April 06
AI
И тест на каждый конкретный класс. Иначе будешь огребать
AI
Композицией класс собирай, и тесты композицией можно.
PK
ну для каждого парсера класс создаю
они отличаются только парой функций и конструктором в котором только одна переменная меняется
они отличаются только парой функций и конструктором в котором только одна переменная меняется
Почему нельзя сделать просто include с нужной логикой?
а
Почему нельзя сделать просто include с нужной логикой?
я так и делаю
AI
я так и делаю
Дык ты выше Class.new do делал )
AI
Это и потенциальные утечки и в целом геморой для поддержки
а
Дык ты выше Class.new do делал )
и в нем инклуд
AI
и в нем инклуд
А почему просто норм класс не сделать? И вместо make_parser будет choose_parser условно
AI
Ты ща себе зскладываешь очень большой геморой на будущее, или даже ближайшее
а
А почему просто норм класс не сделать? И вместо make_parser будет choose_parser условно
потому что код увеличится в 4 раза
а так — оно срабатывает как темплейты в плюсах
а так — оно срабатывает как темплейты в плюсах
AI
потому что код увеличится в 4 раза
а так — оно срабатывает как темплейты в плюсах
а так — оно срабатывает как темплейты в плюсах
А темплейты в плюсах тоже геморой
AI
Ты типа дженерики строишь
AI
Но тут имхо лучше в 4 раза больше кода, но нормальные тесты и стектрейсы
AI
Строить дженерики без статических гарантий имхо так себе идея, но мысль вроде уловил
AI
Огребать или нет, решать тебе. Надеюсь это код твоего пет проекта, потому что на работе я бы не стал подкладывать такую свинью коллегам.
AI
Чем проще и тупее тем часто лучше.
PK
def klass_maker(class_name, params)
klass = "
class #{class_name}
include SomeModule
def initialize(#{params})
@#{params[:var_name]} = #{params[:var_value]}
end
end
"
end
PK
раз уж такая пляска