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

重构干燥控制器代码

dry ruby-on-rails

Gav · 技术社区 · 15 年前

在控制器的创建和更新操作中,我有许多这样的语句:

@company.set_preference(:api_username, params[:company]['api_username']) if params[:company]['api_username']
@company.set_preference(:api_password, params[:company]['api_password']) if params[:company]['api_password']

我想知道如何重构这些代码以使代码更加干燥。我很清楚它们可能属于这个模型(即胖模型和瘦控制器),但我不知道该怎么做。有人能启发我吗?

提前谢谢你的时间,

伽伐文

1 回复 | 直到 15 年前

mikej heading_to_tahiti 15 年前

是重复的吗 @company.set_preference 你想要避免的?或者重复 params[:company].. ?

给你的 Company 型号:

def update_preferences(prefs)
  prefs.each_pair do |pref_name, value|
    set_preference(pref_name.to_sym, value)
  end
end

然后打电话给

@company.update_preferences(params[:company])

您还可以添加一个检查以确保只设置有效的首选项,例如。

VALID_PREFERENCES = ['api_username', 'api_password']

def update_preferences(prefs)
  prefs.each_pair do |pref_name, value|
    set_preference(pref_name.to_sym, value) if VALID_PREFERENCES.include?(pref_name)
  end
end

推荐文章

thesunneversets · 如何使用Nikkou(新手Ruby问题!)

2 年前

Animesh Arya · 关联模型的Rspec

2 年前

yellowreign · Rails:将created\u at转换为PST

2 年前

Kellen · 查看$卷展栏功能列表

2 年前

Akshit Thakur Ak · 我怎样才能把铁轨停下来?

2 年前

solidsnake99 · Rails db:如何绕过验证

2 年前

Spyros · 从数组中删除nil值,并从引用数组中删除相应的项

2 年前

Pauli-E · 当div打开并滚动到屏幕上时,向div添加类

2 年前

Raghuram Murthy P · 在ActiveRecord::Relation delete_by方法上不调用after_commit和after_destroy回调

2 年前

ejoka · 如何使用Rails 7模型属性作为CSS标记值

2 年前