如何在 switch 语句中重构以下 switch？

Question

关于如何重构以下内容的任何想法：

 private string GetErrorText(ErrorType type, int error, string id)
 {
     string errorMessage = string.Empty;

     switch (type)
     {
         case ErrorType.Generic:
             switch (error)
             {
                 case (int)ErrorType.One:
                 errorMessage = string.Format("Generic Error {0}",id);
                 break;
                 case (int)ErrorType.Two:
                 errorMessage = "Generic Error 2.";
                 break;
             }
             break;
             case ErrorType.NonGeneric:
                    switch (error)
                    {
                        case (int)NonGenericError.One:
                            errorMessage = "One";
                            break;
                        case (int)NonGenericError.Two:
                            errorMessage = "Two";
                            break;
                        case (int)NonGenericError.Three:
                            errorMessage = "Three";
                            break;
                        case (int)NonGenericError.Four:
                            errorMessage = "Four"
                            break;
                    }
                    break;
            }
            return errorMessage;
        }

我知道我可能可以只返回字符串而不是使用break关键字。听说可以把这个放进字典里就可以完全避免切换了。另外，switch语句可以组合吗？即使我需要将多个参数传递给 string.Format 方法，也会发生什么情况。现在，只需要一个？

Answer 1

将字符串值与枚举关联的一种相当常见的方法是使用自定义属性，并在运行时通过反射查找它。这是一个很好的解释。

当然，在实践中使用反射会比仅使用条件慢。

Answer 2

一个明显的观察结果是，“type”的值并不重要 - 您只需在“error”上设置一个开关即可。

string.Format 根据需要接受尽可能多的参数，例如

errorMessage = string.Format("Generic Error {0}. Other data {1}. ", id, otherData);

Answer 3

为什么不呢？

string GetGenericErrorText(ErrorType error, string id)
{
   switch (error)              
   {
       case ErrorType.One:
           return string.Format("Generic Error {0}",id);

       default:
           return "Generic Error 2.";
    } 
}

和

string GetNonGenericErrorText(NonGenericError error)
{
   case NonGenericError.One:
       return "One";

   case NonGenericError.Two:
       reutrn "Two";

   case NonGenericError.Three: 
       return "Three";

   default: 
       return "Four";
}

编辑

如果真正的代码很简单，就像你可以做的例子一样

string GetNonGenericErrorText(NonGenericError error)
{
    return error.ToString();
}