А что сказать человеку, который фанатично пишет логику в хуках и сеттерах в модели?)

что он балбес

а ты определи, что такое контроллер, модель и вью

Ну модель это твой маппинг на таблицу в дб

А что сказать человеку, который фанатично пишет логику в хуках и сеттерах в модели?)

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

действительно балбес

и выстрелит себе в ногу в будущем

можно же на вьюхах логику ебашить

всё тот же?

да)

Ну модель это твой маппинг на таблицу в дб

o'rly?

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

кстати да

можно же на вьюхах логику ебашить

может что-то путное наконец-то скажешь или так и будешь по строчке комментировать?

он не останется без работы

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

но он уходит, а поддерживать его мне. При этом он не дает мне писать нормально)

может что-то путное наконец-то скажешь или так и будешь по строчке комментировать?

хочу и комментирую

но он уходит, а поддерживать его мне. При этом он не дает мне писать нормально)

бейсбольная бита

но он уходит, а поддерживать его мне. При этом он не дает мне писать нормально)

в каком плане не дает?

это очень забавно)

но он уходит, а поддерживать его мне. При этом он не дает мне писать нормально)

сожги его трудовую

в каком плане не дает?

Говорит, что вся прилага уже написана так, как она написана и нужно консистентно поддерживать это говно)

Говорит, что вся прилага уже написана так, как она написана и нужно консистентно поддерживать это говно)

ты все не успокаиваешься?)

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

ты все не успокаиваешься?)

ну проблема все еще есть)

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

не стоит

тут есть дети

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

def included_publisher_ids=(ids) (ids || []).each do |id| TrafficTarget.where( target_type: 'Account', mode: 'include', target_id: id, targetable: self.id, targetable_type: 'Supply', ).first_or_create end traffic_targets .where(target_type: 'Account', mode: 'include') .where.not(target_id: ids).each(&:destroy) end def excluded_publisher_ids=(ids) (ids || []).each do |id| TrafficTarget.where( target_type: 'Account', mode: 'exclude', target_id: id, targetable: self.id, targetable_type: 'Supply', ).first_or_create end traffic_targets .where(target_type: 'Account', mode: 'exclude') .where.not(target_id: ids).each(&:destroy) end

парочка сеттеров в модели

их там больше

там же удаляется, лол)

ids || []

чет

ids || []

это единственное что тебя смущает?))))

ну это сразу

def included_publisher_ids=(ids) (ids || []).each do |id| TrafficTarget.where( target_type: 'Account', mode: 'include', target_id: id, targetable: self.id, targetable_type: 'Supply', ).first_or_create end traffic_targets .where(target_type: 'Account', mode: 'include') .where.not(target_id: ids).each(&:destroy) end def excluded_publisher_ids=(ids) (ids || []).each do |id| TrafficTarget.where( target_type: 'Account', mode: 'exclude', target_id: id, targetable: self.id, targetable_type: 'Supply', ).first_or_create end traffic_targets .where(target_type: 'Account', mode: 'exclude') .where.not(target_id: ids).each(&:destroy) end

про DRY, видимо, не слышали

ids || []

про это я вообще молчу. Основная проблема в архитектуре

лан

Он считает, что модель как бы разделитель между вью и базой, а не котроллер и свято в это верит

печальненько

печальненько

он еще отрицает серивс обжекты как таковые. считает что это не рельс вей

?

ну, правильно делает

по канонам должно быть

ага, узнал от них от меня. Сам пишет на рельсах 7 лет, а это мой первый проект на рельсах

дхх завещал

кстати да, DHH за что топит?

офк

за татарское нашествие в моделях?

ну у него свое понимание модели

особенный тоже

может ты и работаешь с dhh?

Алло. дхх не пишет уже очень давно.

может ты и работаешь с dhh?

ор

кстати да, DHH за что топит?

за консерны

за консерны

ууу сюка

а вообще

где-то у меня ссыль оежит, на тред в реддите

Алло. дхх не пишет уже очень давно.

Это объясняет, почему он редакторы в рельсы добавляет, а не что-то нормальное

Это объясняет, почему он редакторы в рельсы добавляет, а не что-то нормальное

Ну тип редакторы тоже нужны.

Недавно мы хотели взять его.

где он в дискуссии про рефакторинг участвовал, и там примеры кода кидал

Из-за reach text/

Алло. дхх не пишет уже очень давно.

ну коммиты коммитит в рельсу

где он в дискуссии про рефакторинг участвовал, и там примеры кода кидал

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

где он в дискуссии про рефакторинг участвовал, и там примеры кода кидал

код ослеплял своим совершенством?

но когда происходит адище в сеттере то это другое дело

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

интересно, а что там было?

код ослеплял своим совершенством?

норм код был

интересно, а что там было?

3 строчки вынес в сервис обжект, по его словам его за это уволили, чувак запилил гист на эту тему, туда подключился dhh

ERROR: S client not available

3 строчки вынес в сервис обжект, по его словам его за это уволили, чувак запилил гист на эту тему, туда подключился dhh

это где его уволили за это?

ага

да

это где его уволили за это?

да

помню

есть у тебя тот гист?

кого уволили?

увольнять конечно за это хуй знает

не, просто помню случай

кого уволили?

dhh?))

кого уволили?

парня, который вынес в сервис

dhh?))

он со мной работает)

типа бля

увольнять за такое

это чё вообще

бля

дайте гист

увольнять за такое

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

прямо топово было бы

дайте гист

в институте интриги открыли кое что интересное (ц)

https://gist.github.com/justinko/2838490

оно?

https://gist.github.com/justinko/2838490#gistcomment-356060

ответ dhh)

а не позже?

по моему недавно же прям было

хотя вроде оно

не, это оно

его обсуждали

кто нибудь плиз скажите что увольнять за такое нельзя