我为Java程序分配了一些维护任务,并发现:
public void setActiveCode(boolean isActiveCode) {
this.isActiveCode = isActiveCode & Boolean.TRUE;
}
Run Code Online (Sandbox Code Playgroud)
类型this.isActiveCode是a boolean,对于类中的每个布尔设置器重复相同的概念.我无法弄清楚为什么这样做,我不能问原始的开发人员.
这样做有没有正当理由?
这样做有没有正当理由?
不.这只是更加冗长的代码,清晰度为零增益(可以说是清晰度的损失,因为在这里你想知道它的全部内容).
这让我想起
public boolean isTrue(boolean b) {
if(b == true) {
return true;
}
else {
return false;
}
}
Run Code Online (Sandbox Code Playgroud)
不幸的是,你会不时在野外看到.真是太伤心了.
| 归档时间: |
|
| 查看次数: |
108 次 |
| 最近记录: |