正确封装数组集合

Question

您好，我的任务是构建一个具有以下简介的应用程序。

实现一个接口，根据促销规则计算价格。

co = Checkout.new(promotional_rules)
co.scan(item)
co.scan(item)
price = co.total

本质上，根据制定的促销规则，某些商品会相应折扣。

我对我的代码有一些反馈，指出我已经封装了 Promotion_rules 组，然后将规则公开为数组 - 糟糕的 OO

我最初创建了一个 Promotion_rules 对象，其中包含一组规则。

  def initialize
    @rules = []
  end

 def addrule(rule)
   @rules.push(rule)
 end

然后，在我的结帐对象中，我有已设置并传递到初始值设定项的 Promotion_rules 对象。我循环遍历促销规则对象中包含的规则数组，并将它们应用到结帐对象扫描的项目。

def initialize(promotionalrules=Promotionalrules.new)
  @promotionalrules = promotionalrules
end

....Other code

for rule in @promotionalrules.getrules
  for item in @items
    ##Execute rule on current item.
  end
end

我对我的代码不太满意......循环与循环等。但我只是在寻找一些封装方面的帮助，因为我不确定我哪里出了问题。

任何关于应用于简报的良好设计模式的建议也会是有益的，因为我对我所采取的方法不太有信心。谢谢

Answer 1

我猜他们是在抱怨这种“暴露”：

for rule in @promotionalrules.getrules

这会将促销规则（可能应该称为促销规则）中的内部规则泄露给调用者。解决方法是稍微扭转你的逻辑：

class Promotionalrules
  #...
  # and possibly remove the getrules method completely
  def apply_to_item(item)
    # Apply @rules to item
  end
  #...
end

然后：

# I'm not sure how the rules and item interact so this "each" might
# be a different iterator in reality
@items.each { |i| @promotional_rules.apply_to_item(i) }

本质的改变是你将规则集作为一个整体应用于每个项目。这隐藏了规则集实现细节，并且作为额外的好处，允许您轻松支持相互依赖的规则（“除非您使用优惠券 Y，否则您可以获得折扣 X”之类的东西）。