我需要用更紧凑的东西来代替C#交换机

本文关键字：交换机 | 更新日期: 2023-09-27 18:27:03

我有以下代码：

switch (pk.Substring(2, 2))
{
    case "00":
        ViewBag.Type = _reference.Get("14", model.Type).Value;
        break;
    case "01":
        ViewBag.Type = _reference.Get("18", model.Type).Value;
        break;
}

它完成了任务，但在我看来不是很干净。有没有办法让这个代码小一点。我想把数字14或18作为一个变量，但我不确定最好的编码方式是使用if else还是其他方式。

我需要用更紧凑的东西来代替C#交换机

您可以使用静态字典作为映射，而不是switch语句。

 static readonly Dictionary<string, string> map = new Dictionary<string, string> {
     { "00", "14" },
     { "01", "18" },
     // ... more ...
 };
 // ... in your method ...
 string str = pk.Substring(2, 2);
 string val;
 if (!map.TryGetValue(str, out val))
 {
     // Handle error, like in the "default:" case of the switch statement
 }
 else
 {
     ViewBag.Type = _reference.Get(val, model.Type).Value;
 }

然而，我只会这样做，如果真的有很多映射，甚至可以从外部源（如配置文件）"读取"。

还要注意的是，如果"键"实际上是一个从0开始的连续整数序列，那么您可以使用一个数组，其中"键"只是其中的索引。

 static readonly string[] map = new string[] {
    "14", "18", ...
 };
 int index = Int32.Parse(pk.Substring(2, 2)); // Error handling elided.
 if (index < 0 || index > map.Length)
 {
     // Handle error, like in the "default:" case of the switch statement
 }
 else
 {
     ViewBag.Type = _reference.Get(map[index], model.Type).Value;
 }

否则，请使用显式的switch语句（可能会考虑到更简洁的代码的赋值）：

 string val;
 switch (pk.Substring(2, 2))
 {
    case "00":
      val = "14";
      break;
    case "01":
      val = "18";
      break;
    // ... more ...
    default:
      // Error handling for unknown switch-value.
      break;
 }
 ViewBag.Type = _reference.Get(val, model.Type).Value;

"00"->"14"answers"01"->"18"之间似乎存在某种关系。我相信这种关系源于商业逻辑。您应该包装逻辑并使控制器中的代码清晰。最后，控制器中的代码应该看起来像：

public ActionResult MyAction()
{
    //some code
    ViewBag.Type = TypeProvider.GetType(pk, model.Type);
    //return something
}
class TypeProvider
{
    Dictionary<string, string> relations = ...
         //a dictionary stores "00"->"14" logics
    public static SomeType GetType(string pk, Type modelType)
    {
        return _reference.Get(relations[pk.SubString(2,2)], modelType).Value;
    }
}

var data = pk.Substring(2, 2);
var choice = data == "00" ? "14" : (data=="01"?"18":"");
if (choice != string.Empty) ViewBag.Type = _reference.Get(choice, model.Type).Value;

我对这类代码使用映射扩展：

ViewBag.Type = pk.Substring(2, 2)
.Map("00", x => GetViewBagValue("14"))
.Map("01", x => GetViewBagValue("18"))

在你的情况下，这种方法：

private ViewBagValue GetViewBagValue(string value)
{
    return _reference.Get(value, model.Type).Value; 
}

我使用这个。您可以很容易地将其更改为泛型或使用例如object[]。不是超高效，但非常紧凑：

public static class Util {
    public static string Switch(string value, params string[] nameValues) {
        for (int x = 0; x < nameValues.Length; x += 2) {
            if (nameValues[x] == value) {
                return nameValues[x + 1];
            }
        }
        return string.Empty;
    }
}

然后这样称呼它：

var res = Util.Switch("test2", "test1", "res1", "test2", "res2");

祝你好运！

相关文章：