[英]How can I follow dry method in rails
我是新来的铁路。 所以不知道如何遵循DRY方法。 让我们深入探讨这个问题。
如果我遵循相同的代码,我的 controller 中有两种方法。 那么如何减少我的代码。 在这两种方法中,只有 model 名称有所不同。 请帮助我
我需要这样的东西
array = ['Userservice', 'Userschedule' ]
array.each do |el|
draft_info = el.find_by(user_id: 39, save_option: "publish")
draft_info.delete if !draft_info.nil?
@user_info = el.find_by(user_id: 39, save_option: "draft")
el.create(@user_info.attributes.merge({:save_option => "publish", :id => el.maximum(:id).to_i.next }))
end
def create_publish_user
publish_user_schedule
publish_user_service
end
def publish_user_service
draft_service = Userservice.where(user_id: current_user.id, save_option: "publish").delete_all
user_service = Userservice.where(user_id: current_user.id, save_option: "draft")
user_service.each { |ser|
Userservice.create(ser.attributes.merge({:save_option => "publish", :id => Userservice.maximum(:id).to_i.next }))
}
end
def publish_user_schedule
draft_schedule = Userschedule.where(user_id: current_user.id, save_option: "publish").delete_all
user_schedule = Userschedule.where(user_id: current_user.id, save_option: "draft")
user_schedule.each { |sch|
Userschedule.create(sch.attributes.merge({:save_option => "publish", :id => Userschedule.maximum(:id).to_i.next }))
}
end
看来您应该能够执行以下操作:
['Userservice', 'Userschedule'].each do |el|
el_class = el.constantize
if draft_info = el_class.find_by(user_id: 39, save_option: "publish")
draft_info.delete
end
@user_info = el_class.find_by(user_id: 39, save_option: "draft")
el_class.create(
@user_info.attributes.merge({
save_option: :publish,
id: el_class.maximum(:id).to_i.next
})
)
end
我不知道您为什么在执行create
时指定id
。 您为什么要这样做,而不是让ActiveRecord
为您处理这些事情?
您可以将参数传递给您的方法
def publish(type)
draft = type.where(....)
data = type.where(....)
data.each ....
# And so on
end
# Somewhere in your code you use it then
publish(UserService)
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.