z****e
发帖数: 54598 | 1 【 以下文字转载自 Java 讨论区 】
发信人: airdragon77 (仍然自由自我永远高唱我歌), 信区: Java
标 题: 最近Code Review看到的奇葩代码
发信站: BBS 未名空间站 (Fri Jul 11 17:07:58 2014, 美东)
简直不能相信自己的眼睛。嗯，这些代码是外包公司写的。
1
String perm = something;
if (perm != null || perm != "") {
String[] permArray = perm.split(";");
...
}
2
String ruleArr[] = ruleName.split(",");
if (ruleArr != null && ruleArr.length > 0) {
for (int iCount = 0; iCount < ruleArr.length; iCount++) {
if (null != ruleArr[iCount] || ruleArr[iCount].length() > 0
|| ruleArr[iCount] != "") {
ruleNames.add(ruleArr[iCount]);
}
}
}
3
String l_alphabet;
if(null!=p_filterDTO.getAlphabet()){
l_alphabet=p_filterDTO.getAlphabet();
}else{
l_alphabet=null;
} |
s*****r
发帖数: 43070 | |
q*****u
发帖数: 7 | |
s****y
发帖数: 503 | |
s*****r
发帖数: 43070 | 5 1,2里面该用&&的
3里面的if毫无意义
【在 q*****u 的大作中提到】
: 求大神comment下
|
w********s
发帖数: 1570 | 6 说明你代码看的太少。
【在 z****e 的大作中提到】
: 【 以下文字转载自 Java 讨论区 】
: 发信人: airdragon77 (仍然自由自我永远高唱我歌), 信区: Java
: 标 题: 最近Code Review看到的奇葩代码
: 发信站: BBS 未名空间站 (Fri Jul 11 17:07:58 2014, 美东)
: 简直不能相信自己的眼睛。嗯，这些代码是外包公司写的。
: 1
: String perm = something;
: if (perm != null || perm != "") {
: String[] permArray = perm.split(";");
: ...
|
C*******n
发帖数: 193 | |
c******f
发帖数: 243 | 8 里面的perm != ""有问题
很久没见过这种for loop了 |
A*****i
发帖数: 3587 | 9 尼玛的连我这java菜鸟都知道string不能用!=来比较
我就想知道老印都这水平是怎么拿到外包的？ |
j******o
发帖数: 4219 | 10 我PYTHON都是这么写的。
更烂的代码你还没见过呢，反正外包的代码只要能WORK就行了。 |
R*****i
发帖数: 2126 | 11
问题大了
【在 C*******n 的大作中提到】
: 1,2其实没什么大问题
|
