タイプのチェックに基づいてコードの匂いに基づいて状態を判断していますか？

Question

私はその状態に基づいて動作を変更するオブジェクトの設計に取り組んでいます。 1つのプロパティにさまざまな種類のオブジェクトが含まれている場合は1つ

class Person { 

    // many types of glasses available 
    protected $glasses; 

    public function putOnBoots() 
    { 
     // does this type check smell? 
     if ($this->glasses instanceof SunGlasses) { 
      trigger_error('never wear boots with sunglasses'); 
      return false; 
     } 
     // [...] put on boots 
     return true; 
    } 

    public function lookAtStuff ($stuff) 
    { 
     // glasses provides polymorphic behavior here 
     return $this->glasses->filter($stuff); 
    } 

} 

種類が大きな目的を果たすよう$glassesは、クラス内の他の場所で独自の多型動作を提供しないことに留意すべきである：ここでは例です。しかしPersonの状態を示す方法の型によって影響を受ける可能性があるにもかかわらず、$glassesに属していないと思われるいくつかの動作（putOnBoots()など）があります。

このようにオブジェクトタイプをチェックすると何かが間違って感じられます。私は匂いを検出するのですか、それとも私は物事を想像していますか？

Answer 1

私にはちょっと臭いようです。より良い選択肢は、眼鏡のいくつかの多形性の動作を使用することです。おそらく仮想メソッド...

は、ほとんどの場合TRUEを返しますが、サングラスの場合はFALSEを返します。

利点は、他のタイプのメガネを追加することでコードを拡張でき、「ブーツと一緒に行く」かどうかを自分で決めることができ、必要があるかどうか心配する必要はありません別のタイプのメガネを追加するたびにputOnBoots（）メソッドを変更してください。