rails将类常量重构到数据库对应的表中之三
经过博文之一和之二的重构,貌似代码表现的还不错,正常运行和test都通过鸟,但是,感觉告诉我们还是有什么地方不对劲啊!究竟是哪里不对劲呢?我们再来好好看一下.
我们把数据库表中的支付方式集合直接放在实现的地方,貌似很唐突.无论是Order model中,Order控制器中还是new.html.erb中都是如此:
#in Order.rb
validates :pay_type,inclusion:PaymentType.all.map {|type|type.pay_type}
validates :pay_type,inclusion:OrdersController.payment_types
#in OrdersController.rb
@@payment_types = PaymentType.all.map {|type|type.pay_type}
def self.payment_types
@@payment_types
#PaymentType.all.map {|type|type.pay_type}
end
def self.payment_types=(types)
@@payment_types = types
end
(是滴!为了test我又蠢蠢的加了self.payment_types=方法! :( )
人家本来是类PaymentType的东东好吧!?那么应该让其回归本源啊,于是乎有了第一步:在PaymentType中添加方法:
class PaymentType < ActiveRecord::Base
def self.types
all.map {|type|type.pay_type}
end
end
然后第二步:修改Order的检查方法:
validates :pay_type,inclusion:PaymentType.types
接着第三步:修改new.html.erb的select元素的构成:
<%= f.select :pay_type,PaymentType.types,prompt:'Select a payment method' %>多余的东东都可以删除鸟,再测试下,一切OK哦!到这里重构正式告一段落喽(还来啊!) :)
郑重声明：本站内容如果来自互联网及其他传播媒体，其版权均属原媒体及文章作者所有。转载目的在于传递更多信息及用于网络分享，并不代表本站赞同其观点和对其真实性负责，也不构成任何其他建议。
