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[iC... 阅读全帖 |
|
l**********r
发帖数: 4612 | 2 【 以下文字转载自 JobHunting 讨论区 】
发信人: zhaoce (米高蜥蜴), 信区: JobHunting
标 题: 最近Code Review看到的奇葩代码 (转载)
发信站: BBS 未名空间站 (Sun Jul 13 08:34:09 2014, 美东)
发信人: 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;... 阅读全帖 |
|
l**********r
发帖数: 4612 | 3 【 以下文字转载自 JobHunting 讨论区 】
发信人: zhaoce (米高蜥蜴), 信区: JobHunting
标 题: 最近Code Review看到的奇葩代码 (转载)
发信站: BBS 未名空间站 (Sun Jul 13 08:34:09 2014, 美东)
发信人: 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;... 阅读全帖 |
|
a*********7
发帖数: 30080 | 4 简直不能相信自己的眼睛。嗯，这些代码是外包公司写的。
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_alphab... 阅读全帖 |
|
a*********7
发帖数: 30080 | 5 不是我挑剔吧。。。错误大家前面都讨论了，都是programing fundamentals
比如说第一个例子，if (perm != null || perm != "") ，一句简单的话两个错误，该
用&& 他用了||，该用!equals()他用!=。
第二个例子，也是简单一句话，
if (null != ruleArr[iCount] || ruleArr[iCount].length() > 0
|| ruleArr[iCount] != "")
三个错，1）该用&& 他用了|| 2） 该用!equals()他用!= 3）length() > 0 和 != ""
是同样意思，无用代码。
第三个例子是典型的脱裤子放屁。
编程风格这么高深的东西我还真没顾上。。。
关键这些都是最最基本的东西啊。现在印度马工找工作完全没要求的吗。。。 |
|
