if else系列如何清理

Question

在C#中工作，想减少if else系列，实体有两个属性 FromServiceID和ToServiceID，假设我的ServiceClass实例有波纹管information.How 清理波纹管代码？任何类型的建议都将被接受。

entity= new ServiceClass();
entity.FromServiceID=3
entity.ToServiceID=1

if (entity.FromServiceID == 1)
{
    entity.1KWithdrawal();
}
else if (entity.FromServiceID == 2)
{
    entity.10KWithdrawal();
}
else if (entity.FromServiceID == 3)
{
    entity.BTWithdrawal();
}           
if (entity.ToServiceID == 1)
{
    entity.1KDeposit();
}
else if (entity.ToServiceID == 2)
{
    entity.10KDeposit();
}
else if (entity.ToServiceID == 3)
{
    entity.BTDeposit();
}


public class ServiceClass
{ 

    public int FromServiceID { get; set; }
    public int ToServiceID { get; set; }

    public void 1KWithdrawal()
    { Console.WriteLine("One_KWithdrawal"); }

    public void 10KWithdrawal()
    { Console.WriteLine("Ten_KWithdrawal"); }

    public void BTWithdrawal()
    { Console.WriteLine("BTWithdrawal"); }

    public void 1KDeposit()
    { Console.WriteLine("One_KDeposit"); }

    public void 10KDeposit()
    { Console.WriteLine("Ten_KDeposit"); }

    public void BTDeposit()
    { Console.WriteLine("Ten_KDeposit"); }
}

Answer 1

使用 Dictionary。像这样：

Dictionary<int, ServiceClass> dictionary = new Dictionary<int, ServiceClass>()
{
    {1,  new ServiceClass()},
    {2,  new ServiceClass()},
    {3,  new BTWithdrawal()},//assume BTWithdrawal inherits from ServiceClass
};

如何使用它的示例：

ServiceClass value=new ServiceClass();
value.FromServiceId=1;
value.ToServiceId = 2;
dictionary.TryGetValue(value.FromServiceId, out value);
//or dictionary.TryGetValue(value.ToServiceId, out value);
if (value != null) MessageBox.Show(value.Id.ToString());

Answer 2

您可以使用如下的 switch case:

var entity = new ServiceClass();

entity.FromServiceID = 3;
entity.ToServiceID = 1;

switch(entity.FromServiceID)
{
    case 1:
        new 1KWithdrawal();
        break;
    case 2:
        new 10KWithdrawal();
        break;
    case 3:
        new BTWithdrawal();
        break;
}

switch(entity.ToServiceID)
{
    case 1:
        new 1KDeposit();
        break;
    case 2:
        new 10KDeposit();
        break;
    case 3:
        new BTDeposit();
        break;
}

Answer 3

也许这有点矫枉过正，但您可以为每个案例创建一个 class，它继承自一个公共接口（我们称之为 ICommon），为每个案例公开一个公共方法案例（在你的案例中是一个 Create 方法），然后在 ServiceClass.

的构造函数中注入该接口

然后当你想使用 ServiceClass 时，你必须提供 ICommon 的实际实现（你从每个案例中提取的 classes 之一）最后你只需要调用 entity.Create.

我相信这就是策略模式，总而言之，您应该在通用接口下的不同 class 中提取算法。

最后，这种重构将降低代码的分圆复杂度（这主要意味着减少代码的分支），这总是一件好事。

Answer 4

您可以做的是将所有变体放入一个枚举中，并像您想要调用的方法一样调用枚举值。（我建议不要在名称中使用数字，因为编译器不允许）

为了简单和可测试性，我将枚举和方法放在同一个 class:

public class ServiceClass
{
    public enum ServiceID
    {
        OneKWithdrawal,
        Ten_KWithdrawal,
        BTWithdrawal,
        OneKDeposit,
        Ten_KDeposit,
        BTDeposit
    }
    public ServiceID From_Ser_ID { get; set; }
    public ServiceID To_Ser_ID { get; set; }

    public void One_KWithdrawal()
    { Console.WriteLine("One_KWithdrawal"); }

    public void Ten_KWithdrawal()
    { Console.WriteLine("Ten_KWithdrawal"); }

    public void BTWithdrawal()
    { Console.WriteLine("BTWithdrawal"); }

    public void One_KDeposit()
    { Console.WriteLine("One_KDeposit"); }

    public void Ten_KDeposit()
    { Console.WriteLine("Ten_KDeposit"); }
}

这将是执行您的 if-condition 方法的方法。它使用反射来访问枚举中编码的方法。您可能必须根据您的方法所在的位置调整 Invoke(sc, null); 调用中的 object 参数。如果它们与您调用 execute 的位置相同 class，您可以使用 this.

public static void execute(ServiceClass sc)
{
    sc.GetType().GetMethod(sc.From_Ser_ID.ToString()).Invoke(sc, null);
    sc.GetType().GetMethod(sc.To_Ser_ID.ToString()).Invoke(sc, null);
}

您可以在这里测试整个代码：

public static void Main(string[] args)
{

    ServiceClass entity = new ServiceClass();
    entity.From_Ser_ID = ServiceClass.ServiceID.BTWithdrawal;
    entity.To_Ser_ID = ServiceClass.ServiceID.Ten_KDeposit;

    execute(entity);
}

所以你最终会得到一个枚举和 2 行代码。

if else系列如何清理

How to clean up if else series

c#

coding-style