我们可以用C#中的对象替换if语句吗

Question

下面是一个方法：

  private RiskFactor calculateMotoristRiskFactor()
    {               

        if (motorist.PointsOnLicense > 3
        || motorist.Age < 25)

            return RiskFactor.HighRisk;
        if (motorist.PointsOnLicense > 0)
            return RiskFactor.ModerateRisk;

        return RiskFactor.LowRisk;
    }

我不需要那些 if 语句。

我可以使用策略模式来解决这个问题吗？ 如果是，那么我也不希望不同多态类中的每个方法都应该有一个 If 语句。

RiskFactor 是一个枚举 有

什么更好的方法可以使其更加面向对象而不是过程化？

Answer 1

好吧，你可以有一个 List, RiskFactor>：

var filters = new List<Tuple<Func<Motorist, bool>, RiskFactor>
{
    Tuple.Create(m => m.PointsOnLicense > 3, RiskFactor.HIGH_RISK),
    Tuple.Create(m => m.Age < 25, RiskFactor.HIGH_RISK),
    Tuple.Create(m => m.PointsOnLicense > 0, RiskFactor.MODERATE_RISK),
};

那么：

var risk = filters.Where(filter => filter.Item1(motorist))
                  .Select(filter => filter.Item2)
                  .DefaultIfEmpty(RiskFactor.LOW_RISK)
                  .First();

这至少可以轻松添加额外的检查，并且它只是按顺序运行它们。这有点繁琐 - 例如，我可能会创建一个自定义 Filter 类型而不是 Tuple - 但它应该可以工作......

Answer 2

我认为策略模式不应该是这里的答案 - 通常，如果您正在处理的特定类型的驾驶者影响其行为，您通常会使用策略。

在您的情况下，您只有基于驾车者属性的标准域逻辑。我认为有条件的处理是非常恰当的。

Answer 3

完全取消 RiskFactor 函数。不要使用该函数的代码，而是将需要其返回值的逻辑封装到以不同方式执行任务的单独类中。

例如，您可以定义一个抽象类Policy，它由LowRiskPolicy、ModerateRiskPolicy和HighRiskPolicy继承。这些只是返回根据适当的策略计算的值，并且使用代码不知道或关心它们是什么类型的策略。所有特定于策略的逻辑都包含在策略中。

Answer 4

我不认为你需要更加面向对象只是为了更加面向对象。
如果它仅位于您计算风险的代码中，并且我假设还有根据年龄和许可点计算的其他数据数量，我会保留您的代码不变。

除了可能将 Motorist 作为参数传递给 RiscCalculator 类的方法之外。

以防万一这里有类似的例子 用多态性替换条件

Answer 5

您将拥有更多代码，但它允许您将其分解出来。您可以为每种类型的风险因素创建一个策略类并将它们收集在一起。像这样的东西：

    public abstract class RiskPolicy
    {
        public abstract RiskFactor Risk { get; }
    }

    public class PointsPolicy : RiskPolicy
    {
        private readonly int _points;
        public PointsPolicy(int points)
        {
            _points = points;
        }

        public override RiskFactor Risk
        {
            get { return _points > 3 ? RiskFactor.HIGH_RISK : 
                         _points > 0 ? RiskFactor.MODERATE_RISK : RiskFactor.LOW_RISK; }
        }
    }

    public class AgePolicy : RiskPolicy
    {
        private readonly int _age;
        public AgePolicy(int age)
        {
            _age = age;
        }

        public override RiskFactor Risk
        {
            get { return _age < 25 ? RiskFactor.HIGH_RISK : RiskFactor.LOW_RISK; }
        }
    }