说明:为了使逻辑更易懂,也为了可维护性更高,应该把代码按模块分割,而构成模块的类的方法,原则上只能有一个入口和一个出口。另外,关于嵌套的深度和模块的行数,也应该控制在一眼就能看清楚的程度上。 qH}8TC
r(>S
就Java语言来说,有如下的几点需要在编码的时候特别留意。 <k:I2LF_
I\.|\^
① switch的case,default部分原则上必须以break终止 xnY?<?J"!
*,\"}x*
如果不用break来终止的话,必须代之以编码规约中既定的注释。但是,如果swich或者if/else需要反复进行,就需要重新考虑是不是应该使用多态。 @V%\Gspv
G]'ah1W
② 处理途中只允许返回异常,返回方法中不允许把异常和return混杂在一起。 ^c\O,*:
$+*nb4
处理途中发生的错误,原则上都要使用异常来返回。例如,在对参数进行检查时发生异常时,要抛出IllegalArgumentException。在一本程序里,把错误一会儿用异常抛出,一会儿又用返回值(返回ERROR等)返回,往往是程序混乱的根源。必须在设计阶段明确方针,避免这种混乱。 |Kd#pYt%O
] 3{t}qY$A
③ finally中,不要写return。 ,n/]ALz>~
,&hv x
例: V.GM$
SyvoN,;Q
① switch中需要考虑的要点 Int dd = 0; PM\Ju]
Switch(status) { 0|P=S|%~
case ‘ A’ : FU3K?A
B
dd = 9; mTE(JZt
break; //OK:正确,用break终止 (C!p2f
case ‘ B’: V?u#WJy/
dd++; aA`eKy) \
/* FALL THROUGH */ //OK:有意不使用break文,插入了规定的注释 J2=4%#R!
case ‘ C’: l 00i2w
dd++; //ERROR:没有插入注释,明显是缺少break的Bug GcVQz[E
default: ]8p{A#1
dd = -1; #fuUAbU0X
break; //OK:default里也没有忘记break v"G1vSx)BT
} y]j.PT`Cw
② 用异常处理来返回错误 If (value == null) { 75# 8P?i
throw new IllegalArgumentException(); //OK:不使用return,用抛出异常的方式返回错误 g&$=Y7G
} 6@N,'a8r
③ 在finally块中,对返回值有影响的情况 Int value = 1; 8Qg10Yjy
Try{ 3( BL
return value; X0.H(p#s
} finally{ &6x(%o|
value = 4; '}Fe&%
return value; //ERROR:因为finally块是肯定会被执行到的,所以在try里本来想 (T%F^s5D
//返回1,而最终结果永远是4 pR
S!
} V:n0BlZ,B
//OK:如果在这里写return的话,出口就只有一个了。 a"vzC$Hxd
补充说明 多态的使用 v)5;~.+%
"V|Rq]_+%
sweitch或者if/else的多次反复,往往造成程序难懂,可维护性下降。建议声明Interface或者abstract class,通过多态来解决。 V\L;EHtc$