l**********r 发帖数: 4612 | 1 【 以下文字转载自 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; 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;
} |
d**e 发帖数: 75 | 2 哈哈,这是传说中的defensive programming? |
g****y 发帖数: 1172 | 3 你们公司的编译器有问题吧
【在 l**********r 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 【 以下文字转载自 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;
|
M*********n 发帖数: 4839 | 4 老中的劣根性。纠结这些小问题显示自己懂的多。一辈子code review的命。 |
k***f 发帖数: 264 | 5 你把造成程序崩溃的问题叫作小问题?Lol
【在 M*********n 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 老中的劣根性。纠结这些小问题显示自己懂的多。一辈子code review的命。
|
c******n 发帖数: 4965 | 6 第一个不能算什么 “不相信自己眼睛”吧, it's a bug, ww can see how it xame
into being, many people make such bugs
【在 l**********r 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 【 以下文字转载自 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;
|
s*****m 发帖数: 8094 | 7 算个屁啊,老子看过的阿三的极度奇葩的code比这个恶心无数倍,比文科生写的都狗血。
都没法code review,最后几个大牛和对方manager开会,让他们签免责协议,保证那鸡
巴玩意儿
只能在jailed box上用,这才lgtm的。
【在 l**********r 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 【 以下文字转载自 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;
|
g*******e 发帖数: 115 | |
s******7 发帖数: 1758 | 9 前两个算bug, || , &&搞反了
最后一个算烙印在凑行数,也算常见了,外包给烙印就这下场 |
k***f 发帖数: 264 | 10 这样的虫子如果偶然一次就算了。经常发生就是incompetent. 该炒鱿鱼。
xame
【在 c******n 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 第一个不能算什么 “不相信自己眼睛”吧, it's a bug, ww can see how it xame : into being, many people make such bugs
|
m*******y 发帖数: 151 | 11 看公司的工程质量
【在 k***f 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 这样的虫子如果偶然一次就算了。经常发生就是incompetent. 该炒鱿鱼。 : : xame
|
k***f 发帖数: 264 | 12 我见到更奇葩的代码是除了GUI,其它所有的methods和变量 都是static 。就连GUI类
里面也有一大堆static 。这可不是虫子。
【在 m*******y 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 看公司的工程质量
|
f******o 发帖数: 1505 | 13 仅仅是 ||, && 搞反了吗? 这是 Java 吧
【在 s******7 的大作中提到】![](/moin_static193/solenoid/img/up.png) : 前两个算bug, || , &&搞反了 : 最后一个算烙印在凑行数,也算常见了,外包给烙印就这下场
|