多应用+插件架构,代码干净,二开方便,首家独创一键云编译技术,文档视频完善,免费商用码云13.8K 广告
### Remove Assignments to Parameters(移除对参数的赋值动作) 你的代码对一个参数进行赋值动作。 以一个临时变量取代该参数的位置。 ~~~ int discount (int inputVal, int quantity, int yearToDate) { if (inputVal > 50) inputVal -= 2; ~~~ => ~~~ int discount (int inputVal, int quantity, int yearToDate) { int result = inputVal; if (inputVal > 50) result -= 2; ~~~ **动机(Motivation)** 首先,我要确定大家都清楚「对参数赋值」这个说法的意思。如果你把一个名为foo 的对象作为参数传给某个函数,那么「对参数赋值」意味改变foo,使它引用(参考、指涉、指向)另一个对象。如果你在「被传入对象」身上进行什么操作,那没问题,我也总是这样干。我只针对「foo被改而指向(引用)完全不同的另一个对象」这种情况来讨论: ~~~ void aMethod(Object foo) { foo.modifyInSomeWay(); // that's OK foo = anotherObject; // trouble and despair will follow you ~~~ 我之所以不喜欢这样的作法,因为它降低了代码的清晰度,而且混淆了 pass by value(传值〕和 pass by reference (传址)这两种参数传递方式。Java只采用 pass by value传递方式(稍后讨论),我们的讨论也正是基于这一点。 在 pass by value情况下,对参数的任何修改,都不会对调用端造成任何影响。那些用过 pass by reference的人可能会在这一点上犯糊涂。 另一个让人糊涂的地方是函数本体内。如果你只以参数表示「被传递进来的东西」,那么代码会清晰得多,因为这种用法在所有语言中都表现出相同语义。 在Java中,不要对参数赋值;如果你看到手上的代码已经这样做了,请使用Remove Assignments to Parameters。 当然,面对那些使用「输出式参数」( output parameters)的语言,你不必遵循这条规则。不过在那些语言中我会尽量少用输出式参数。 **作法(Mechanics)** - 建立一个临时变量,把待处理的参数值赋予它。 - 以「对参数的赋值动作」为界,将其后所有对此参数的引用点,全部替换为「对此临时变量的引用动作」。 - 修改赋值语句,使其改为对新建之临时变量赋值。 - 编译,测试。 - 如果代码的语义是 pass by reference,请在调用端检查调用后是否还使用了这个参数。也要检查有多少个 pass by reference参数「被赋值后又被使用」。请尽量只以return方式返回一个值。如果需要返回的值不只一个,看看可否把需返回的大堆数据变成单一对象,或千脆为每个返回值设计对应的一个独立函数。 **范例(Example)** 我从下列这段简单代码开始: ~~~ int discount (int inputVal, int quantity, int yearToDate) { if (inputVal > 50) inputVal -= 2; if (quantity > 100) inputVal -= 1; if (yearToDate > 10000) inputVal -= 4; return inputVal; } ~~~ 以临时变量取代对参数的赋值动作,得到下列代码: ~~~ int discount (int inputVal, int quantity, int yearToDate) { int result = inputVal; if (inputVal > 50) result -= 2; if (quantity > 100) result -= 1; if (yearToDate > 10000) result -= 4; return result; } ~~~ 还可以为参数加上关键词final,从而强制它遵循「不对参数赋值」这一惯例: ~~~ int discount (final int inputVal, final int quantity, final int yearToDate) { int result = inputVal; if (inputVal > 50) result -= 2; if (quantity > 100) result -= 1; if (yearToDate > 10000) result -= 4; return result; } ~~~ 不过我得承认,我并不经常使用final来修饰参数,因为我发现,对于提高短函数的清晰度,这个办法并无太大帮助。我通常会在较长的函数中使用它,让它帮助我检查参数是否被做了修改。 **Java的pass by value(传值)** Java使用"pass by value"「函数调用」方式,这常常造成许多人迷惑。在所有地点,Java都严格釆用pass by value,所以下列程序: ~~~ class Param { public static void main(String[] args) { int x = 5; triple(x); System.out.println ("x after triple: " + x); } private static void triple(int arg) { arg = arg * 3; System.out.println ("arg in triple: " + arg); } } ~~~ 会产生这样的输出: ~~~ arg in triple: 15 x after triple: 5 ~~~ 这段代码还不至于让人糊涂。但如果参数中传递的是对象,就可能把人弄迷糊了。如果我在程序中以Date对象表示日期,那么下列程序: ~~~ class Param { public static void main(String[] args) { Date d1 = new Date ("1 Apr 98"); nextDateUpdate(d1); System.out.println ("d1 after nextDay: " + d1); Date d2 = new Date ("1 Apr 98"); nextDateReplace(d2); System.out.println ("d2 after nextDay: " + d2); } private static void nextDateUpdate (Date arg) { arg.setDate(arg.getDate() + 1); System.out.println ("arg in nextDay: " + arg); } private static void nextDateReplace (Date arg) { arg = new Date (arg.getYear(), arg.getMonth(), arg.getDate() + 1); System.out.println ("arg in nextDay: " + arg); } } ~~~ 产生的输出是: ~~~ arg in nextDay: Thu Apr 02 00:00:00 EST 1998 d1 after nextDay: Thu Apr 02 00:00:00 EST 1998 arg in nextDay: Thu Apr 02 00:00:00 EST 1998 d2 after nextDay: Wed Apr 01 00:00:00 EST 1998 ~~~ 从本质上说,object reference是按值传递的(passed by value)。因此我可以修改参数对象的内部状态,但对参数对象重新赋值,没有意义。 Java1.1及其后版本,允许你将参数标示为final,从而避免函数中对参数赋值。 即使某个参数被标示为final,你仍然可以修改它所指向的对象。我总是把参数视为final,但是我得承认,我很少在参数列(parameter list)中这样标示它们。