if-else
通常是一个糟糕的选择——它导致设计复杂,代码可读性变差,并可能造成重构困难!但是,if-else
已成为事实上的代码分支解决方案,这也确实有道理。不幸的是,许多开发人员从没有前进到更合适的分支策略——他们的口头禅是:if-else
是一把锤子,一切都是钉子。
本文向大家展示一些技巧和模式——它们将终结这种可怕的做法。但是,这并不肯定或者否定哪一种写法,仅仅为大家提供一些其他的编码思路或者一些值得借鉴的点子罢了。
目录
1 删掉else块
1.1 传统写法
public void PerformOperation(int para)
{
if (para > 5)
{
//do something
}
else
{
//do something else
}
}
1.2 优化写法
public void PerformOperation(int para)
{
if (para > 5)
{
//do something
return;
}
//do something else
}
2 多值分配
2.1 传统写法
public static string DetermineGender(int para)
{
string gender = string.Empty;
if (para == 0)
{
gender = "woman";
}
else if (para == 1)
{
gender = "man";
}
else
{
gender = "unknown";
}
return gender;
}
2.2 优化写法
public static string DetermineGender(int para)
{
if (para == 0) return "woman";
if (para == 1) return "man";
return "unknown";
}
3 前提条件检查
3.1 传统写法
// Para must be 0 or 1
public static string DetermineGender(int para)
{
// Continue executing logic
}
3.2 优化写法
public static string DetermineGender(int para)
{
if (para < 0) throw new ArgumentException();
if (para > 1) throw new ArgumentException();
return para == 0 ? "woman" : "man";
}
4 将if-else转换为字典
4.1 传统写法
public void PerformOperation(string operationName)
{
if (operationName == "Op1")
{
//something
}
else if (operationName == "Op2")
{
//something else
}
else
{
//default
}
}
4.2 优化写法
// 仅出于说明目的将字典放置在方法内部,你可能希望从其他地方提供它
public void PerformOperation(string operationName)
{
Dictionary<string, Action> operations = new Dictionary<string, Action>();
operations["Op1"] = () => { /*something*/};
operations["Op2"] = () => { /*something else*/};
operations[operationName].Invoke();
}
5 扩展应用程序,完全避免if-else
研究也不深入,目前还不懂这个。
5.1 传统写法
public string PrintOrder(Order order ,string formatType)
{
// Guard clauses left out for brevity
string result = string.Empty;
if (formatType == "Json")
{
result = JsonSerializer.Serialize(order);
}
else if (formatType == "PlainText")
{
result = $"Id:{order.Id}\nSum:{order.Sum}";
}
else
{
result = "Unknown format";
}
return result;
}