你的程序有某个临时变量被赋值超过一次,它既不是循环变量,也不是被用于收集计算结果。
针对每次赋值,创建一个独立、对应的临时变量。
double temp=2*(_height+_width);
System.out.println(temp);
temp=_height*_width;
System.out.println(temp);
重构后:
finall double perimeter=2*(_height+_width);
System.out.println(perimeter);
final double area=_height*_width;
System.out.println(area);
动机:
临时变量有各种不同的用途,其中有些用途会很自然的导致临时变量被多次赋值。“循环变量”和“结果收集变量”就是两个典型的例子:循环变量(loop variable)会随循环的每次运行而改变,结果收集变量(collecting temporary variable) 通过整个函数的运算而构成的某个值收集起来。
除了这两种情况,还有很多临时变量用于保存一段冗长代码的运算结果,以便稍后使用。这种临时变量应该只被赋值一次。如果它被赋值超过一次,就意味着它在函数中承担了一个以上的责任。如果临时变量承担多个责任,它就应该被替换(分解)为多哥临时变量,每个变量只承担一个责任。同一个临时变量承担两件不同的事情,会令代码阅读者糊涂。