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 的大作中提到】
: 【 以下文字转载自 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 的大作中提到】
: 老中的劣根性。纠结这些小问题显示自己懂的多。一辈子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 的大作中提到】
: 【 以下文字转载自 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 的大作中提到】
: 【 以下文字转载自 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 的大作中提到】
: 第一个不能算什么 “不相信自己眼睛”吧， it's a bug, ww can see how it xame
: into being, many people make such bugs
|
m*******y
发帖数: 151 | 11 看公司的工程质量
【在 k***f 的大作中提到】
: 这样的虫子如果偶然一次就算了。经常发生就是incompetent. 该炒鱿鱼。
:
: xame
|
k***f
发帖数: 264 | 12 我见到更奇葩的代码是除了GUI，其它所有的methods和变量 都是static 。就连GUI类
里面也有一大堆static 。这可不是虫子。
【在 m*******y 的大作中提到】
: 看公司的工程质量
|
f******o
发帖数: 1505 | 13 仅仅是 ||， && 搞反了吗？ 这是 Java 吧
【在 s******7 的大作中提到】
: 前两个算bug, || , &&搞反了
: 最后一个算烙印在凑行数，也算常见了，外包给烙印就这下场
|
