使用Action表驱动代替switch…case语句
2011-11-22 22:12 imyang 阅读(1168) 评论(3) 编辑 收藏 举报大家都知道,代码中出现过多的if…else嵌套语句时很难读,所以常用switch…case语句进行代替,代码的可读性就好了很多。
举个简单的例子:
public void DoSomeThing(string str) { if (str == "A") { DoSomethingForA(str); } else if (str == "B") { DoSomethingForB(str); } else if (str == "C") { DoSomethingForC(str); } }
我们一眼就看出,这段代码可以使用switch语句进行重构,重构后的代码如下:
public void DoSomeThing(string str) { switch (str) { case "A": DoSomethingForA(str); break; case "B": DoSomethingForB(str); break; case "C": DoSomethingForC(str); break; default: break; } }
这样看下去,代码确实清晰了很多。这种重构方法我们很多人都会这么做,而且效果还不错。上面的例子中代码行还很少,实际当中有可能会出现很多的case语句,
即使我们这样重构,在阅读的时候还是需要拉很长的滚动条才可以看到需要找的case语句对应的代码。所以,针对这种情况,我们还可以用更简洁的方法进行重构。
实现起来也挺简单的,即:建立全局的字典类型的变量,保存不同的case语句条件以及对应的方法,这样,在需要调用的时候直接查找到该方法,invoke即可,其实
也是用了委托来实现。看以下代码:
private static Dictionary<string, Action<string>> dict = new Dictionary<string, Action<string>>(); public BuildDictionary() { dict.Add("A", DoSomethingForA); dict.Add("B", DoSomethingForB); dict.Add("C", DoSomethingForC); }
上面是声明变量及填充字典,做好这部分的工作之后,调用起来就方便多了,同样的DoSomeThing方法,代码就少了很多
public void DoSomeThing(string str) { if (dict.ContainsKey(str)) { Action<string> action = dict[str]; action.Invoke(str); } }
重构之后,是不是简洁了很多,即使再多的case语句,我们也只需要在BuildDictionary方法中把它添加进去就可以了。
那么,是不是所有的switch…case语句都可以这样重构呢,以这样的实现方式来看,需要case条件对应的方法参数类型相同,这样才容易构造出字典来保存。
如果方法还要返回值,那我们就不能用Action,需要用Func表达式来保存,因为它可以带有返回值。
以上,希望对大家有所帮助。