How to become an author
Search
Write a publication
Pull to refresh

Comments 22

Интересно, но вот только непонятно причём тут DRY. По сути он относится только к 
# lib/models/validations.rb
module Models
  module Validations
    # общий модуль c валидациями для нескольких моделей
  end
end

а всё остальное собственно к «не повторяй себя» отношения имеет мало
Total votes 9: ↑9 and ↓0+9
Ну вообще да, правильное замечание. Наверное, стоило как-то отметить, что где DRY, там, обычно, и модульность со структурированностью.
Total votes 3: ↑2 and ↓1+1
Ага, остальное — KISS и рефакторинг)
Total votes 3: ↑3 and ↓0+3
Влияет ли это как-то на скорость работы приложения, где каждая копейка дорога? Насколько я знаю все это инклудится на этапе загрузки приложения, а на этапе запроса уже не играет никакой роли, но хотелось бы подтверждения.
This comment wasn’t rated yet0
Ну, в общем случае так оно и есть.
This comment wasn’t rated yet0
Такой подход удлинняет цепочку наследования и фактически может увеличить время поиска метода при его вызове. Но не думаю, что сколько-нибудь существенно.
Total votes 2: ↑1 and ↓10
в продакшене это время добавиться только к первому обращению и то при инициализации приложения. Так что при дальнейшей работе уже все будет находиться в кеше.
This comment wasn’t rated yet0
Спасибо, буду пользовать. Про концерн не слышал даже…
Total votes 2: ↑1 and ↓10
Клевый доклад кстати, мне нравится такой подход, я на симфони пишу щас (но вобщем какая разница :)) и когда фигачишь в модели метод с запросом состоящим из кучи джойнов к другим моделям, думается иногда: «а какого черта я пишу это именно в этом классе?». Презентеры — клевый петтерн буду думать… :)
This comment wasn’t rated yet0
там где каждая копейка дорога разве используют RoR?
Total votes 4: ↑2 and ↓20
Копейки разные бывают.
Total votes 1: ↑0 and ↓1-1
ТОЛЬКО АССЕМБЛЕР!!! ТОЛЬКО ХАРДКОР!!!
Total votes 8: ↑7 and ↓1+6
Почему нет? Сервер стоит гораздо дешевле, чем найм программистов, при этом, хотя Rail-программист стоит дороже, более выгодно использовать Rails, нежели PHP, поскольку скорость разработки значительно выше. Ну а если возникнут проблемыы с производительностью, то всегда можно переписать, например на Java (сомневаюсь, что вы будете писать Twitter).
Total votes 1: ↑1 and ↓0+1
Отличный пример того как надо рефакторить код моделей. Я почему-то раньше думал, что данный подход используется только при написании больших гемов.

Буду пользоваться, исправляться. Спасибо большое!
Total votes 2: ↑2 and ↓0+2
Открыл статья в сладкой надеждле узнать что-то новенькое, а прочитал о том, как инклюдить модули… Мда… Но плюсик поставил, сообщесту нужны статьи всякие и разные.
Total votes 1: ↑1 and ↓0+1
Вот тут про это хорошо написано: wearestac.com/blog/2011/05/organise_your_models
А я написал простенький генератор для генерации таких вот штук: github.com/ognevsky/xf-generators. Там, правда, нужно код почистить, а то выложил что сразу получилось и забил ;/

Пользоваться так: rails g xf:concern user friendship (по аналогии со статьей по ссылке выше). Создаются все необходимые файлы (в тч файл спека) с заготовленным текстом.
Total votes 1: ↑1 and ↓0+1
В ссылке на гитхаб точка лишняя :)
This comment wasn’t rated yet0
Ну это парсер же лох :) Как же без точки и конце предложения можно жить?
This comment wasn’t rated yet0
Про Concern полезный раздел, я про него сам узнал только с месяц назад.

Но все же DRY заключается не в размазывании кода по модулям.

Если имеем два класса с одинаковым кодом, то да, можно и модули применить.

Хотя даже в этом случае вижу минимум два варианта отDRYить классы:
* унаследовать от одного класса, все общее перенести туда
* общее вынести в модуль и инклюдить его
This comment wasn’t rated yet0
Наследование в случае рельсовых моделей хреново выглядит. Там STI по-умолчанию пытается включиться.
This comment wasn’t rated yet0
Sign up to leave a comment.

Articles

Show the best of all time
Unlock dark mode

Your account

Sections

Information

Services

Support
© 2006–2024, Habr