如何重构嵌套的 if else 语句

Question

假设你有一个怪物 object 的下表：

性别	年龄	类型	结果
男性	年轻的	火	1
男性	老的	冰	2
男性	老的	木头	6
女性	年轻的	火	4
女性	老的	冰	5
女性	老的	木头	8
其他	年轻的	火	7
其他	老的	冰	10
其他	老的	木头	20

如果我有一个怪物并输入它以获得结果； 如何在不进行长嵌套 if-else 语句的情况下获得怪物 object 的结果？

if(monster.Gender == Male && monster.Age == Old)
    if(monster.Type == Ice)
        return 2
    if(monster.Type == Wood)
        return 6
if(monster.Gender == Male && monster.Age == Young)
etc...

我似乎无法将我的头脑围绕使它可扩展等等：说我想在将来添加一个列或者想在另一件事之前验证某件事。

是否有任何简单的设计模式等我可以实现来简化它并使其可扩展？

Answer 1

您可以将 C# 8.0 中引入的switch 表达式与模式匹配一起使用。

假设这些声明：

public enum Gender
{
    Other,
    Female,
    Male
}

public enum Age
{
    Young,
    Old
}

public enum MonsterType
{
    Fire,
    Ice,
    Wood
}

class Monster
{
    public Gender Gender { get; set; }
    public Age Age { get; set; }
    public MonsterType Type { get; set; }
}

使用元组模式：

return (monster.Gender, monster.Age, monster.Type) switch {
    (Gender.Male, Age.Old, MonsterType.Ice) => 2,
    (Gender.Male, Age.Old, MonsterType.Wood) => 6,
    (Gender.Male, Age.Young, MonsterType.Ice) => 12,
    ... etc.
    _ => -1
};

使用属性模式：

return monster switch {
    { Gender: Gender.Male, Age: Age.Old, Type: MonsterType.Ice } => 2,
    { Gender: Gender.Male, Age: Age.Old, Type: MonsterType.Wood } => 6,
    { Gender: Gender.Male, Age: Age.Young, Type: MonsterType.Ice } => 12,
    ... etc.
    _ => -1
};

通过将此Deconstruct方法添加到Monster class...

public void Deconstruct(out Gender gender, out Age age, out MonsterType type)
{
    gender = Gender;
    age = Age;
    type = Type;
}

...我们可以使用位置模式（看起来像元组模式；但是，我们可以直接打开monster而不必先创建元组）：

return monster switch {
    (Gender.Male, Age.Old, MonsterType.Ice) => 2,
    (Gender.Male, Age.Old, MonsterType.Wood) => 6,
    (Gender.Male, Age.Young, MonsterType.Ice) => 12,
    ... etc.
    _ => -1
};

如果我们将怪物声明为记录，我们将免费获得这个 Deconstruct 方法：

record Monster (Gender Gender, Age Age,  MonsterType Type);

---

一种更面向 object 的方法是从抽象基础 class 为不同的怪物类型派生不同的怪物类，而不是具有Type属性。 怪物类将负责他们自己的Result值（无论这代表什么。我将保留名称Result ）。

然后你只需要打开Gender和Age 。

抽象基础 class（ enum MonsterType现在已过时）：

abstract class Monster
{
    public Gender Gender { get; set; }
    public Age Age { get; set; }

    public void Deconstruct(out Gender gender, out Age age)
    {
        gender = Gender;
        age = Age;
    }

    public abstract int Result { get; }
}

Fire 类型的实现

class FireMonster : Monster
{
    public override int Result => this switch {
        (Gender.Male, Age.Young) => 1,
        (Gender.Female, Age.Young) => 4,
        (Gender.Other, Age.Young) => 7,
        // etc.
        _ => -1
    };
}

注意：如果案例涵盖性别和年龄的所有组合，即，如果它们是详尽的，则不再需要默认案例_ => （如果这些属性是枚举类型。我他们是int常量 C# 编译器不能确定它们是否详尽）。

Answer 2

这看起来像是应该使用配置文件的情况。 您可以将“表”存储在 CSV 文件或类似文件（如数据库表）中，并创建一个 class 以从该文件中读取并与您正在搜索的怪物匹配。 每当您添加新怪物时，您只需将它们添加到配置文件中。

我正在使用CsvHelper读取下面的 CSV 文件：

public class Monster
{
    public string Gender { get; set; }
    public string Age { get; set; }
    public string Type { get; set; }
    public int Result { get; set; }
}

public class MonsterLookup
{
    private readonly List<Monster> _lookupList;
    
    public MonsterLookup(string configFilePath)
    {
        using var reader = new StreamReader(configFilePath);
        using var csv = new CsvReader(reader, CultureInfo.InvariantCulture);
        _lookupList = csv.GetRecords<Monster>().ToList();
    }
    
    public int GetResult(Monster monster)
    {
        var match = _lookupList.FirstOrDefault(
            m => m.Gender == monster.Gender
                && m.Age == monster.Age
                && m.Type == monster.Type);
        return match?.Result ?? -1;
    }
}

// Call it like:
var monsterLookup = new MonsterLookup(configFilePath);
var result = monsterLookup.GetResult(myMonster);

配置文件（您可以添加所有类型）：

Gender,Age,Type,Result
Male,Young,Fire,1
Male,Old,Ice,2
Female,Young,Fire,4

有关示例，请参见此小提琴。