複数の注文依存条件をどのように扱うべきですか？

Question

私は、条件が条件に依存する複数の同時条件を持っています。私はこの問題に対処するための複数のアプローチを考え出しました。私は準備したいくつかのソリューションの中から最良のソリューションを選択するのに役立つ必要があります。考慮すべき事項は、パフォーマンス、再利用性、可読性、メモリー消費量です。

アプローチ1：オブジェクトレジストリに格納タイプおよびパラメータ試験

detect2: function(subject, target){ 
     // object registry and place switch 
     var shapes = {}; 
     shapes[subject.type] = subject; 
     shapes[target.type] = target; 

     var shape1 = subject; 
     var shape2 = target; 

     var reverseShapeOrder = function() { 
      shape2 = target; 
      shape1 = subject; 
     }; 

     if (shapes.rectangle && shapes.ellipse) { 
      if (subject.type === 'ellipse') { 
       reverseShapeOrder(); 
       return this.rectWithEllipse(shape1, shape2); 
      } 
     } 
    }, 

アプローチ3に基づいて順序を切り替える：

var collision = { 
    detect1: function(subject, target){ 
     // multiple switch cases 
     var shapes = {}; 
     shapes[subject.type] = subject; 
     shapes[target.type] = target; 

     switch(subject.type) { 
      case 'rectangle': 
       switch(target.type) { 
        case 'ellipse': 
         return this.rectWithEllipse(subject, target); 
       } 
       break; 
      case 'ellipse': 
       switch(target.type) { 
        case 'rectangle': 
         return this.rectWithEllipse(target, subject); 
       } 
     } 
    }, 

アプローチ2の複数のネストされたスイッチケースを定義する文字列にタイプを連結orderOfテストに基づいて注文を切り替えます。

detect3: function(subject, target) { 
     // string concat and decoding with place switch 
     var shapeString = subject.type + target.type; 

     var rectIndex = shapeString.indexOf('rectangle'); 
     var ellipseIndex = shapeString.indexOf('ellipse'); 
     var pointIndex = shapeString.indexOf('point'); 

     var shape1 = subject; 
     var shape2 = target; 

     var reverseShapeOrder = function() { 
      shape2 = target; 
      shape1 = subject; 
     }; 

     if (rectIndex && ellipseIndex) { 
      if (ellipseIndex < rectIndex) { 
       reverseShapeOrder(); 
      } 
      return this.rectWithEllipse(shape1, shape2); 
     } 
    }, 

アプローチ4：標準的な従来のif-else文

// traditional logic 
    detect4: function(subject, target) { 
     if (subject.type === 'rectangle' && target.type === 'ellipse') { 
      return this.rectWithEllipse(subject, target); 
     } 
     else if (subject.type ==='ellipse' && target.type === 'rectangle') { 
      return this.rectWithEllipse(target, subject); 
     } 
    }, 
    rectWithEllipse: function(rect, ellipse) { 
     return false; 
    } 
}; 

アプローチ5：参照機能を有するフライセレクタを

（おかげオンザフライセレクタ提案@Bergi用）

detect5: function(subject, target) { 
     return this[subject.type + '_with_' + target.type](subject, target); 
    }, 
    rect_with_ellipse: function(rect, ellipse) { 
     return false; 
    }, 
    ellipse_with_rect: function(rect, ellipse) { 
     this.rect_with_ellipse(ellipse, rect); 
    } 
}; 

最高のソリューションを選択して、なぜそれが最適であるか理解してください。次のように

rectWithPoint: function(rect, point) { 
    return false; 
}, 
rectWithEllipse: function(rect, ellipse) { 
    return false; 
}, 
rectWithRect: function(rect, rect) { 
    return false; 
}, 
ellipseWithPoint: function(ellipse, point) { 
    return false; 
}, 
ellipseWithEllipse: function(ellipse, ellipse) { 
    return false; 
} 

Answer 1

私は拡張することが容易になりますこれは、マッピングを定義します：

targets: { 
    rectangle: { 
    point: function rectWithPoint(rect, point) { 
     return false; 
    }, 
    ellipse: function rectWithEllipse(rect, ellipse) { 
     return false; 
    }, 
    rectangle: function rectWithRect(rectLhs, rectRhs) { 
     return false; 
    } 
    }, 
    ellipse: { 
    point: function ellipseWithPoint(ellipse, point) { 
     return false; 
    }, 
    ellipse: function ellipseWithEllipse(ellipseLhs, ellipseRhs) { 
     return false; 
    } 
    } 
}, 

detect5: function (subject, target) { 
    var tmp, candidates = this.targets[subject]; 

    if (!candidates) { 
    tmp = subject; 
    subject = target; 
    target = tmp; 
    candidates = this.targets[subject]; 
    } 

    if (candidates && candidates.hasOwnProperty(target)) { 
    return candidates[target].call(this, subject, target); 
    } 

    return false; 
}