Комментарии к записи: Рефакторинг с использованием rails_best_practices

Автор: niquola

niquola — Tue, 04 Oct 2011 20:43:25 +0000

Мы в нашем проекте написали простенький guard, который прогоняет rails_best_practices
и reek (https://github.com/kevinrutherford/reek/wiki). И постепенно рефакторим.

rails_best_practices не особо многословен, но его почти всегда стоит послушаться.
reek => делает ваш код еще лучше ( тока стоит предварительно познакомиться с каталогом его запашков)

если знаете еще интересные метрики => посоветуйте

Автор: Ouch!

Ouch! — Fri, 30 Sep 2011 05:57:03 +0000

Потому что форма выводится в другом контроллере. Это форма добавления комментария, и она выводится при просмотре поста. И мне хочется, чтобы ошибки валидации комментария показывались именно на этой странице.

Автор: Michail

Michail — Fri, 30 Sep 2011 05:56:16 +0000

Всё правильно, что в гем файл.
Добавляете в группу дев и всё, все разработчики в команде счастливы!
group :development
gem “rails_best_practices”
end

Автор: Michail

Michail — Fri, 30 Sep 2011 05:53:53 +0000

А зачем вообще делать редирект после ошибки?!
Нужно просто отрендерить форму через render :new

Автор: Ouch!

Ouch! — Fri, 30 Sep 2011 05:31:54 +0000

Про фильтр согласен - именно там ему и место.

Про объект во флэше - а как мне еще передать объект в другой экшен, в случае, если мне нужно отобразить ошибки валидации? Flash-объект тут подходит как нельзя кстати, так как он “живет” до следующего экшена.

Автор: Ouch!

Ouch! — Fri, 30 Sep 2011 05:26:04 +0000

Мне так удобнее, т.к. разрабатываю на 2-х разных машинах. Тупо делаю bundle install и не парюсь.

Автор: alvir

alvir — Thu, 29 Sep 2011 14:39:39 +0000

Зачем добавлять его в Gemfile проекта, его надо просто поставить через gem install не более

Автор: Roman V. Babenko

Roman V. Babenko — Thu, 29 Sep 2011 13:21:54 +0000

У тебя код станет еще чище, если ты вынесеш проверку бота в фильтр.
А вот передавать целый объект во флеше - это это жесть.
Тут уже бестпрактики не помогут ;-)

Автор: Ouch!

Ouch! — Thu, 29 Sep 2011 10:41:33 +0000

Спасибо, исправил эту злую ошибку :)

Автор: none

none — Thu, 29 Sep 2011 09:38:46 +0000

Для получения html надо выполнить:
rails_best_practices -f html .

Но в любом случае - огромное спасибо! Мне очень понравилось, потестил у себя - красота!