重構重複代碼 - 唯一區別是<符號

Question

類測試是abstract。它包含一個基本的測試名和一個分數。該分數（雙倍）可通過名爲TestScore的屬性訪問。

三類繼承自Test。他們是不同的，每種測試都有不同的評分方式。

目標：根據評分可以在的值之間的範圍值，評分與6評分之一。

class ScoreRangeAscendingTest : Test 
{ 
    public ScoreRangeAscendingTest(double score, Enums.TestType typeName) : base(score, typeName) { } 
    public double[] ranges; 
    public Enums.Rating calculateScore(double[] range) 
    { 
     if (scoreInVeryWeakRange()) 
      return Enums.Rating.VeryWeak; 
     else if (scoreInWeakRange()) 
      return Enums.Rating.Weak; 
     else if (scoreInAveragelyWeakRange()) 
      return Enums.Rating.AveragelyWeak; 
     else if (scoreInAveragelyGoodRange()) 
      return Enums.Rating.AveragelyGood; 
     else if (scoreInGoodRange()) 
      return Enums.Rating.Good; 
     else 
      return Enums.Rating.VeryGood; 
    } 

    private bool scoreInVeryWeakRange() 
    { 
     return this.TestScore < ranges[(int)Enums.Border.P10]; 
    } 
    private bool scoreInWeakRange() 
    { 
     return TestScore >= ranges[(int)Enums.Border.P10] && TestScore < ranges[(int)Enums.Border.P25]; 
    } 
    private bool scoreInAveragelyWeakRange() 
    { 
     return TestScore >= ranges[(int)Enums.Border.P25] && TestScore < ranges[(int)Enums.Border.P50]; 
    } 
    private bool scoreInAveragelyGoodRange() 
    { 
     return TestScore >= ranges[(int)Enums.Border.P50] && TestScore < ranges[(int)Enums.Border.P75]; 
    } 
    private bool scoreInGoodRange() 
    { 
     return TestScore >= ranges[(int)Enums.Border.P75] && TestScore < ranges[(int)Enums.Border.P90]; 
    } 
    private bool scoreInVeryGoodRange() 
    { 
     return TestScore > ranges[(int)Enums.Border.P90]; 
    } 

} 

}

問題：我的測試，其相應的範圍上去測試其相應的範圍下去。因此，我需要爲第二種測試編寫完全相同的代碼，但這次所有'<'符號都必須是'>'符號，反之亦然。

問題：如何避免重複此代碼？有沒有可能將其添加到基類？

Answer 1

我不是100％確定我收到了問題，但您可能可以使用委託來執行比較。

下應該相當於你的當前代碼：

class ScoreRangeAscendingTest : Test 
{ 
    public ScoreRangeAscendingTest(double score, Enums.TestType typeName) : base(score, typeName) { } 
    public double[] ranges; 
    public Enums.Rating calculateScore(double[] range) { 
     Func<double, double, bool> cmp = (x, y) => x < y; // you can swap the comparison here, e.g. (x, y) => x > y 
     if (scoreInVeryWeakRange(cmp)) 
      return Enums.Rating.VeryWeak; 
     else if (scoreInWeakRange(cmp)) 
      return Enums.Rating.Weak; 
     else if (scoreInAveragelyWeakRange(cmp)) 
      return Enums.Rating.AveragelyWeak; 
     else if (scoreInAveragelyGoodRange(cmp)) 
      return Enums.Rating.AveragelyGood; 
     else if (scoreInGoodRange(cmp)) 
      return Enums.Rating.Good; 
     else 
      return Enums.Rating.VeryGood; 
    } 

    private bool scoreInVeryWeakRange(Func<double, double, bool> cmp) 
    { 
     return cmp(this.TestScore, ranges[(int)Enums.Border.P10]); 
    } 
    private bool scoreInWeakRange(Func<double, double, bool> cmp) 
    { 
     return !cmp(TestScore, ranges[(int)Enums.Border.P10]) && cmp(TestScore, ranges[(int)Enums.Border.P25]); 
    } 
    private bool scoreInAveragelyWeakRange(Func<double, double, bool> cmp) 
    { 
     return !cmp(TestScore, ranges[(int)Enums.Border.P25]) && cmp(TestScore, ranges[(int)Enums.Border.P50]); 
    } 
    private bool scoreInAveragelyGoodRange(Func<double, double, bool> cmp) 
    { 
     return !cmp(TestScore, ranges[(int)Enums.Border.P50]) && cmp(TestScore, ranges[(int)Enums.Border.P75]); 
    } 
    private bool scoreInGoodRange(Func<double, double, bool> cmp) 
    { 
     return !cmp(TestScore, ranges[(int)Enums.Border.P75]) && cmp(TestScore, ranges[(int)Enums.Border.P90]); 
    } 
    private bool scoreInVeryGoodRange(Func<double, double, bool> cmp) 
    { 
     return (TestScore == ranges[(int)Enums.Border.P90]) || !cmp(TestScore, ranges[(int)Enums.Border.P90]); 
    } 

} 

然而，由於在加評論指出，現在你可以使用不同的比較函數，也就是說，交換你<爲>。

Answer 2

System.Double實施IComparable<double>。所以，如果你有幾行代碼，這不同之處僅比較操作符，你可以寫這樣的事情：

abstract class ComparisonBase 
{ 
    protected bool IsSatisfiesCondition(double left, double right, params int[] conditions) 
    { 
     return conditions.Contains(left.CompareTo(right)); 
    } 

    public abstract bool Compare(double left, double right); 
} 

class GreaterThan : ComparisonBase 
{ 
    public override bool Compare(double left, double right) 
    { 
     // instead of left > right 
     return IsSatisfiesCondition(left, right, 1); 
    } 
} 

class LessThan : ComparisonBase 
{ 
    public override bool Compare(double left, double right) 
    { 
     // instead of left < right 
     return IsSatisfiesCondition(left, right, -1); 
    } 
} 

class GreaterThanOrEqual : ComparisonBase 
{ 
    public override bool Compare(double left, double right) 
    { 
     // instead of left >= right 
     return IsSatisfiesCondition(left, right, 1, 0); 
    } 
} 

這只是樣品，但它應該給你一個想法如何在實際應用它碼。