代码之家 › 专栏 › 技术社区 › truongnm

Rails-FeatureEnvy:ApplicationHelper#full_title指的是“page_title”,而不是self(可能将其移动到另一个类?)

feature-envy reek ruby ruby-on-rails

2

truongnm · 技术社区 · 9 年前

app/helpers/application_helper.rb

module ApplicationHelper
  def full_title page_title = ""
    base_title = t "app_name"
    page_title.empty? ? base_title : page_title + " | " + base_title
  end
end

app/helpers/application_helper.rb -- 1 warning:
  [4, 4]:FeatureEnvy: ApplicationHelper#full_title refers to 'page_title' more than self (maybe move it to another class?) [https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]

1 回复 | 直到 9 年前

1

2

Mate Solymosi 9 年前

这个 documentation of reek 功能嫉妒

功能嫉妒 当代码片段引用另一个对象的频率高于其自身时,或当多个客户端对特定类型的对象执行相同的一系列操作时发生。

self ,它是本例中的视图对象。

如何从reek的分析中排除助手:

通过将其添加到 config.reek 文件:

"app/helpers":
  FeatureEnvy:
    enabled: false