设计原则-迪米特法则

迪米特法则(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 

这样改一下，逻辑就清晰了很多，看来在工作中还是要注意下设计原则滴~