动机
如果某个函数只是向你提供一个值,没有任何看得到的副作用,那么这是个很有价值的东西。你可以任意调用这个函数,也可以把调用动作搬到函数的其他地方。简而言之,需要操心的事情少多了。
明确表现出“有副作用”与“无副作用”两种函数之间的差异,是个很好的想法。下面是一条好规则:任何有返回值的函数,都不应该有看得到的副作用。有些程序员甚至将此作为一条必须遵守的规则。就像对待任何东西一样,我并不绝对遵守它,不过我总是尽量遵守,而它也回报我很好的效果。
做法
- 新建一个查询函数,令它返回的值与原函数相同。
- 修改原函数,令它调用查询函数,并返回获得的结果。
2.1. 原函数中的每个return句都应该像这样:return newQuery(),而不应该返回其他东西。
2.2. 如果调用者将返回赋给了一个临时变量,你应该能够去除这个临时变量。 - 编译,测试。
- 将调用原函数的代码改为调用查询函数。然后,在调用查询函数的那一行之前,加上对原函数的调用。每次修改后,编译并测试。
- 将原函数的返回值改为void,并删除其中所有的return语句。
范例
有这样一个函数,一旦有人入侵安全系统,它会告诉我入侵者的名字,并发送一个警报。如果入侵者不止一个,也只发送一条警报:
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 "";
}
该函数被下列代码