Как обработать тривиальное “дублирование кода”, пахнут в направляющих/Ruby

Так, все мы стремимся уменьшить дублирование (DRY) и другие запахи, и сохранить наш код максимально хорошим и чистым. Для кода Ruby существует много инструментов для обнаружения запахов, например, довольно хороший сервис Калибра.

Однако кажется, что у меня есть другое определение дублирования кода, чем инструменты. Я думаю, что это могло бы быть подключено к Ruby способ сделать вещи, где Вы почти никогда не получаете доступ к переменной непосредственно, но через вызов метода вместо этого. Рассмотрите этот отрывок от контроллера направляющих:

def update_site_settings
  SiteSettings.site_name = params[:site_name]
  SiteSettings.site_theme = params[:site_theme]
  expire_fragment('layout_header')
  flash[:notice] = t(:Site_settings_updated)
  redirect_to :controller => 'application', :action => 'edit_site_settings'
end

Это отмечается с помощью предупреждения дублирования кода из-за двух вызовов к "params" методу. Таким образом, мой вопрос, был бы он действительно быть улучшением для присвоения params к локальной переменной? Я рассматриваю способ, которым это записано, чтобы быть самым ясным и кратким способом сделать это, и факт это params метод, и не переменная просто "стоимость ведения бизнеса" в Ruby.

Я вижу это неправильный путь?

Править: В этом случае более симпатичный путь мог бы состоять в том, чтобы сделать a SiteSettings.update_attributes(params) обновление стиля. Рассмотрите, если Вы будете, та же проблема в другом отрывке:

def update
  @mailing_list = MailingList.find(params[:id])

  if @mailing_list.update_attributes(params[:mailing_list])
    flash[:notice] = t:Mailing_list_updated
    redirect_to(mailing_lists_path)
    ...