迪米特法则(Law of Demeter)，又称最少知识原则，简单说来就是 一个对象应该对其他对象有最少的了解。迪米特法则用于降低类与类之间的耦合，那么当其中某一个模块发生修改时，就会尽量少地影响其他模块，扩展会相对容易。
一句话概括下来，就是“只和朋友打交道，朋友的朋友不是朋友”，对于一个对象，其朋友包括以下几类：
(1) 当前对象本身(self)；
(2) 以参数形式传入到当前对象方法中的对象；
(3) 当前对象的成员对象(依rails来说，assocation的属于成员对象)；
(4) 如果当前对象的成员对象是一个集合，那么集合中的元素也都是朋友；
(5) 当前对象所创建的对象。
举一个例子
发布批时会更新产品信息价，在价格网quotation_batch.rb定义方法大致如下：
class QuotationBatch
def publish_prices_no_transaction(options={})
sql, arr = options[:publish_type] == :full ?
['', self.id] : [' and products.id in (?)',[self.id, options[:ids]]]
products_ids = Product.all :conditions => "some",:select => "id"
return 'true' if products_ids.blank?
products_ids.each_slice(1000) do |id_arr|
products = Product.all(:conditions => ['id in (?)', id_arr],
:include => [:all_product_prices])
products.each do |product|
raise ActiveRecord::Rollback unless product.publish_price(options[:user], self)
end
end
'true'
end
end
简要说明一下: quotation_batch 是保存批的类;product_price是保存报价的类,字段batch_id保存批号,字段product_id保存产品号,last字段用于声明是最新报价还是历史报价(1为最新报价，3是历史报价);product是产品类。代码大致流程如下：
publish_prices_no_transaction 遍历product,调用product的publish_price发布报价，而product再根据条件修改product_price状态
代码不太符合迪米特法则，product和quotation_batch之间不应该有关联，初看代码很是一头雾水。
首先，真正发布报价的方法不应该定义在product中，而应该定义在product_price中，因为只有product_price和product和batch是朋友关系，那么，在publish_prices_no_transaction中应该遍历product_price，而不是product
伪代码如下
class ProductPrice
belongs_to :product
belongs_to :batch
def publish_price
#更新上一版报价状态
self.prev_price.update_status
#使本次报价为最新报价
self.make_newst_price
...
end
end
class Product
has_many :product_prices
end
class QuotationBatch
has_many :product_prices
def publish_batch
self.product_prices.each do |product_price|
product_price.publish_price
end
#更新企业状态等其他代码
end
end
这样改一下，逻辑就清晰了很多，看来在工作中还是要注意下设计原则滴~
