查看原文
其他

代码写成这样,老夫无可奈何!

程序人生 2019-11-27

以下文章来源于Java技术栈 ,作者栈长


作者 | 栈长
本文经授权转载自Java技术栈(ID:javastack)
你见过最无可奈何的代码是什么?
今天,栈长就来总结一下我遇到过的一些神逻辑代码,不一定很全,但我真心写不出,真心让我自叹不如啊!

耍猴的最佳手段


还记得 XX 手机的抢购活动么,抢购按钮的代码类似是这样的:
<a href="javasript:alert('抢购结束!')">立即抢购</a>
活动还没开始就结束了。
后面有人挖出来这段神逻辑,大家真是哭晕在厕所了,还好,抢购买手机这种我从来不参加。
就是在前端写死代码造成抢购结束的假象,其实请求压根没有发送到服务器,或者休眠几秒直接跳转到抢购结束页面,良心点的,点 10 次有一次发到服务器就不错了,这样耍猴是不是太好玩了?
众所周知,为了提高服务器的承载能力,现在各类 APP、网页等都会或多或少加入前端缓存,但上面这种耍猴的代码也太无耻了。

猪一样的队友


大家有没有见过这样的代码:
<!-- 原价 180 元,现在活动价 228 元,活动结束后恢复原价 -->
...
这上面的活动价比原价还贵,真是哭笑不得……
类似这样的代码,一些程序员在注释中说明活动的内容,又或者是把一些生产上环境的 IP、端口、用户名、密码等敏感信息把在注释中,好随时能切换……
真是猪一样的队友,这样的程序员不开除么!定时炸弹啊!

最牛逼的排序算法


据说是一个月薪 9K 的 Java 程序员,因老板让他写一个排序算法,然后他就写了一段屌炸天的休眠排序算法,接着他就被老板开除了……
排序算法代码大概是这样的:

这段代码的完整版解释请看这篇文章:刚写完排序算法,就被开除了…

排序排成这样,不开除你,开除谁?


休眠的最高境界


/**
 * 获取未来几天的日期
 * @param days 指定的天数
 * @author 微信公众号:Java技术栈
 * @return
 */

public static Date getNextDays(int days){
    try {
        // 休眠指定的天数
        Thread.sleep(days * 24 * 60 * 60 * 1000);
    } catch (InterruptedException e) {
        e.printStackTrace();
    }
    // 休眠结束后返回日期
    return new Date();
}
获取指定日期用休眠?写完,已笑晕在厕所。
完整版的解释请看这篇文章:刚写完这段代码,就被开除了……
想离职找不到借口?
那还不容易,哥送你这段加强版,复制这段代码,提交打包测试上线,坐等开除。

真就是假,假就是真


boolean isBoy = user.isBoy() == true ? truefalse;
if (isBoy){
    ...
else {
    ...
}

布尔值可以直接拿来用,你硬要写个三元表达式?年底绩效考核了,代码量有 KPI 么?

还有类似这样的:

if (xx1 == true && xx2 !=  true){
    ...
else {
    ...
}
还有这样的:
if (xx1 > xx2){
    return true;
else {
    return false;
}

写得太优雅了!


循环循环循环


for (int i = 0; i < 1; i++){
    ...
}

一个循环也用循环,曾经就遇到过这么一段,不知所云了,要改这段代码,还要把他叫在旁边坐着,看着改才行。

for (;;){
    ...
}

死循环这么写,我们就看不出来了么?


魔法值的真谛


曾经检查代码时,发现代码中很多魔法值,叫同事要定义成公共常量,结果来了这么一段:

private final static int ZERO = 0;
private final static int ONE = 1;
private final static int TWO = 2;
...

看到上面的常量定义,我真想打人,虽然魔法值是没了,但又定义了一些没有意义的常量,完全没有理解修改魔法值的真谛,这和脱裤子放屁有什么区别?

上面还只是一个简单的示例,实际审查情况远比你想象的糟糕。


看花眼的参数


Order order = new Order(orderNo, 1678.13"CNY"13"wx"5"1"0"8"01new Date());

看到这样的代码,我也是醉醉的了。

我就见过同事写出这样的代码,是赶时间呢,还是真的不会设计模式,一个构造函数参数能写这么多,且不说魔法值,这么长,看一眼都觉得头大。


消失的异常


try {
    ...
catch (Exception e) {}

程序捕捉了异常,却不打印。

try {
    ...
catch (Exception e) {
    e.printStackTrace();
}

错误是输出来了,但没有输出到日志文件。

try {
    ...
catch (Exception e) {
    logger.info("...", e);
}

日志级别用的不对,去错误日志文件里面死活找不着错误日志。

上面的种种不规范的写法,在生产环境出现问题时,一方面导致排查问题艰难,另一方面,监控系统也不能及时地监控到异常。


神奇的if


最后一个了,压轴了。

来,我们一起赏析下这段神奇的 if 代码:

if(user.getFirstName() != null)
    if(user.getMiddleName() != null)
        if(user.getLastName() != null)
            if(user.getNickName() != null)
                if(user.getShortName() != null)
                    if(user.getFullName() != null)
                        if(user.getFamilyName() != null)
                            if(user.getEnglishName() != null)
                                ...
                                return true;
return false;                                

这段代码够奇葩了吧,一个 if 能搞定的非要写这么长,还缩进得这么有层次感。

小结

写神逻辑代码的人才这么多,找问题、解决问题真是太难了。

福利彩蛋


参与方式:主要在本文留言参与话题“你见过最烂的代码写成什么样”即可有机会获得专属定制马克杯一个,有3个名额哦~


抽奖时间:10月29日18:00




 热 文 推 荐 20 行 Python 代码说清量子霸权!为何Google、微软、华为将亿级源代码放一个仓库?为什么我会重回到Windows的怀抱?

☞上海梦醒互联网

漫画:要跳槽?这道缓存设计题你有必要看看!

深度学习可解释性问题如何解决?图灵奖得主Bengio有一个解

 人民网:“三种眼光”读懂区块链的今天和明天

点击阅读原文,精彩继续。

你点的每个“在看”,我都认真当成了喜欢

    您可能也对以下帖子感兴趣

    文章有问题?点此查看未经处理的缓存