在一系列布尔表达式中，某个变量带有“控制标记’的作用。以break或return语句取代控制标记。
动机：在一系列条件表达式中，常常会看到用以判断何时停止条件检查的控制标记。这样的标记带来的麻烦超过了它所带来的便利。人们之所以会使用这样的控制标记，因为结构化编程原则告诉他们：每个子程序只能有一个入口和出口。“单一出口“原则会让你在代码中加入让人讨厌的控制标记，大大降低条件表达式的可读性。这就是编程语言提供break和continue语句的原因：用它们跳出复杂的条件语句。去掉控制标记所产生的效果往往让你大吃一惊：条件语句真正的用途会清晰得多。
做法：1、对控制标记的处理，最显而易见的办法就是使用break或continue语句。
2、找出让你跳出这段逻辑的控制标记值。
3、找出对标记变量赋值的语句，代以恰当的break或continue语句。
4、每次替换后，编译并测试
范例：以break 取代简单的控制标记
void checkSecurity(String[] people) { boolean found = false; for (int i = 0; i < people.length; i++) { if (! found) { if (people[i].equals ("Don")){ sendAlert(); found = true; } if (people[i].equals ("John")){ sendAlert(); found = true; } } } }
这种情况下很容易找出控制标记：当变量found 被赋予true 时，搜索就结束。我可以逐一引入break 语句：
void checkSecurity(String[] people) { boolean found = false; for (int i = 0; i < people.length; i++) { if (! found) { if (people[i].equals ("Don")){ sendAlert(); break; } if (people[i].equals ("John")){ sendAlert(); found = true; } } } }
处理后的代码
void checkSecurity(String[] people) { boolean found = false; for (int i = 0; i < people.length; i++) { if (! found) { if (people[i].equals ("Don")){ sendAlert(); break; } if (people[i].equals ("John")){ sendAlert(); break; } } }
最后可以把对控制标记的所有引用去掉：
void checkSecurity(String[] people) { for (int i = 0; i < people.length; i++) { if (people[i].equals ("Don")){ sendAlert(); break; } if (people[i].equals ("John")){ sendAlert(); break; } } }
范例：以return 返回控制标记
本项重构的另一种形式将使用return 语句。为了阐述这种用法，我把前面的例子稍加修改，以控制标记记录搜索结果：
void checkSecurity(String[] people) { String found = ""; for (int i = 0; i < people.length; i++) { if (found.equals("")) { if (people[i].equals ("Don")){ sendAlert(); found = "Don"; } if (people[i].equals ("John")){ sendAlert(); found = "John"; } } } someLaterCode(found); }
在这里，变量found 做了两件事：它既是控制标记，也是运算结果。遇到这种情况，我喜欢先把计算found 变量的代码提炼到一个独立函数中：
void checkSecurity(String[] people) { String found = foundMiscreant(people); someLaterCode(found); } String foundMiscreant(String[] people){ String found = ""; for (int i = 0; i < people.length; i++) { if (found.equals("")) { if (people[i].equals ("Don")){ sendAlert(); found = "Don"; } if (people[i].equals ("John")){ sendAlert(); found = "John"; } } } return found; }
然后以return 语句取代控制标记：
String foundMiscreant(String[] people){ String found = ""; for (int i = 0; i < people.length; i++) { if (found.equals("")) { if (people[i].equals ("Don")){ sendAlert(); return "Don"; } if (people[i].equals ("John")){ sendAlert(); found = "John"; } } } return found; }
最后完全去掉控制标记：
String foundMiscreant(String[] people){ for (int i = 0; i < people.length; i++) { if (people[i].equals ("Don")){ sendAlert(); return "Don"; } if (people[i].equals ("John")){ sendAlert(); return "John"; } } return ""; }
即使不需要返回某值，你也可以使用语句来取代控制标记。这时候你只需 要一个空的return 语句就行了。
当然，如果以此办法去处理带有副作用（连带影响）的函数，会有一些问题。所以我需要先以 Separate Query from Modifier 将函数副作用分离出去。