何时使用 FOR-CASE（C# 中的 Foreach/switch）？

Question

在我正在处理的项目中，我发现了与 FOR-CASE 结构类似的 C# 结构：

foreach (string param in params.Split(';'))
{
    string[] parts = param.Split('=');
    string key = parts[0].Trim().ToLower();
    string value = parts[1].Trim();
    switch (key)
    {
        case "param1": this.param1 = value; break;
        case "param2": this.param2 = value; break;
        case "param3": this.param3 = value; break;
        case "param4": this.param4 = value; break;
        default: break;
    }
}

（更改了变量名称以保护有罪者。）

您将如何实现此代码？

Answer 1

我认为您问题中的代码与您链接到的代码不同......

如果我编写了一个命令行工具，问题中的代码看起来像是我可能会做的事情。

我没有看到问题中的代码有什么问题吗？

另一种方法是使用反射来填充参数值变量。我有时也这样做。

顺便说一句：我曾经用一种脚本语言编写了一个程序，该程序将 switch 作为唯一的流控制机制，并且没有 gosub/return。我程序中的代码的结构有点像你链接的那个。一种指令指针变量的大规模开关，在每种情况下都会重新分配，并且在开关周围几乎是无限循环。它完成了工作。

Answer 2

我看到您的类中已经有多个用于保存变量的字段。在这种情况下，你正在做的很好。

否则，您可以使用 1 个 HashTable（也许添加 C# indexor 作为一个转折点）来保存所有这些，并且您的循环将像这样结束：

foreach (string param in params.Split(';'))
{
    string[] parts = param.Split('=');
    string key = parts[0].Trim().ToLower();
    string value = parts[1].Trim();
    MyHashTable[key] = value;
}

这种方法的问题是您应该只有一种类型的值。例如，如果您的参数列表可以同时包含 string 和 int 类型，它会使代码更加混乱，尤其是您需要执行错误检查和验证等。

我个人会坚持你已经拥有的。

Answer 3

您可以为此使用反射：

Type t = this.GetType();
foreach (string param in params.Split(';'))
{    
    string[] parts = param.Split('=');    
    string key = parts[0].Trim().ToLower();    
    string value = parts[1].Trim();    

    t.GetProperty(key).SetValue(this, value, null);
}

Answer 4

就其价值而言，WTF 文章是 WTF，因为它的外部循环完全没有用，正如文章中所指出的那样 - 直接设置索引变量比循环和测试更简单，更直接它。

Answer 5

我也不确定我是否理解，但听起来你让自己复杂化了。不要重新发明轮子，尽可能多地使用 BCL 类，这些类被证明可以有效地工作并为您节省大量时间。听起来您可以使用某种 Dictionary 以及 Guge 建议的 Reflection 来实现它。

Answer 6

我实际上认为 OP 的代码很好。它并不完美——可能有更简单或更简洁的方法来做到这一点，但它有效地允许成员/属性名称和输入参数名称之间的可读映射。它使您的属性成为强类型（与 hashmap/dictionary 解决方案不同，除非您的类的所有属性只有一种类型......）并为您提供一个相当明显的位置来修复或添加映射。

Answer 7

或正则表达式：

string parms = "param1=1;param2=2;param3=3";
string[] parmArr = parms.Split(';');        

string parm1 = Regex.Replace(parmArr[0], "param1=", "");
string parm2 = Regex.Replace(parmArr[1], "param2=", "");
string parm3 = Regex.Replace(parmArr[2], "param3=", "");