視点を変えるだけでスパゲッティコードを解きほぐす

ifの巨大な混乱 ステートメントはあなたの顔を見つめ続けます。 すべきような気がします 邪魔をし続けるビジネスロジックを除いて、それを単純化することができます。

たとえば、Quoteを作成する販売プラットフォームがあるとします。 s、多くのLineItem s。ただし、広告の場合は、広告申込情報が重複している見積もりを作成できます。 、ただし、複数のウェブサイトがある場合 、価格を合計して、単一の広告申込情報として表示する必要があります。また、ウェブサイトを購入し、見積もりにすでに5つの広告がある場合は、ウェブサイトで20％の割引を提供する必要があります。

ここからずっと、窓からノートパソコンを投げているのが聞こえます。

あなたはできた たくさんのifを書く これらのルールを処理するステートメント：

class Quote
  attr_accessor :line_items
 
  ... 
 
  def add_line_item(line_item)
    if line_item.kind_of?(Ad)
      self.line_items << line_item
    elsif line_item.kind_of?(Website)
      if @line_items.select {|item| item.kind_of?(Ad) }.length >= 5
        # TODO: Put the fractions of a cent into a bank account
        # I have set up
        line_item.price *= 0.8
      end
      existing_website = self.line_items.detect { |item| item.kind_of?(Website) }
      if existing_website
        existing_website.price += line_item.price
      else
        self.line_items << line_item
      end
    end
  end
end

しかし、それはひどいことに同意できると思います。 どうすればそのようなものを解くことができますか？

メソッドを小さなメソッドの束に分解することもできますが、それはクローゼットにすべてのおもちゃを押し込んで、お母さんが部屋を掃除したと思うようにするようなものです。そして、それらのkind_of? sはまだ私を大いに悩ませます。

しかし、広告申込情報から物事を見始めたらどうなるでしょうか。 引用の代わりに、視点？ 質問の代わりに 扱っている広告申込情報の種類を見積もりに追加すると、伝えた それ自体を追加する広告申込情報 見積もりに？

メソッドを逆にします！

コードをリファクタリングする私のお気に入りの方法の1つは、呼び出し元と呼び出し先を逆にしてみることです。 上記のコードを使用した例を次に示します。

class Quote
  ...
  def add_line_item(line_item)
    line_item.add_to_quote(self)
  end
end

class Ad < LineItem
  ...
  def add_to_quote(quote)
    quote.line_items << self
  end 
end

class Website < LineItem
  def add_to_quote(quote)
    if quote.line_items.select {|item| item.kind_of?(Ad) }.length >= 5
      # TODO: Put the fractions of a cent into a bank account
      # I have set up
      self.price *= 0.8
    end
    existing_website = quote.line_items.detect { |item| item.kind_of?(Website) }
    if existing_website
      existing_website.price += self.price
    else
	  quote.line_items << self
    end
  end
end

完璧ではありません。 website.rb まだロットが必要です リファクタリングのヘルプであり、メソッドを逆にするとline_itemsのカプセル化がどのように壊れたかに満足していません。 。

しかし、あなたは複雑さの最初の層を取り除きました。これで、LineItemにコードを配置できます または Quote 、それが最も理にかなっている場所に応じて。 LineItem オブジェクトは、継承とミックスインを使用して、各LineItem間の類似点と相違点を処理できます。 サブクラス。さらに、新しいLineItemを簡単に追加できるようになりました add_line_itemを肥大化させずにサブクラス メソッド。

コードは少しすっきりしていて、はるかに柔軟です。 一般的に、私はそれを勝利と呼びます。