代码写成这样,老板把我开除了,最让人感到无可奈何的代码,你可千万别这么写!

1、耍猴的最佳手段

还记得 XX 手机的抢购活动么,抢购按钮的代码类似是这样的:

<a href="javasript:alert('抢购结束!')">立即抢购</a>

活动还没开始就结束了。。

后面有人挖出来这段神逻辑,大家真是哭晕在厕所了,还好,抢购买手机这种我从来不参加。。

就是在前端写死代码造成抢购结束的假象,其实请求压根没有发送到服务器,或者休眠几秒直接跳转到抢购结束页面,良心点的,点 10 次有一次发到服务器就不错了,这样耍猴是不是太好玩了?

众所周知,为了提高服务器的承载能力,现在各类 APP、网页等都会或多或少加入前端缓存,但上面这种耍猴的代码也太无耻了。

2、猪一样的队友

大家有没有见过这样的代码:

<!-- 原价 180 元,现在活动价 228 元,活动结束后恢复原价 -->...

这上面的活动价比原价还贵,真是哭笑不得……

类似这样的代码,一些程序员在注释中说明活动的内容,又或者是把一些生产上环境的 IP、端口、用户名、密码等敏感信息把在注释中,好随时能切换……

真是猪一样的队友,这样的程序员不开除么!?定时炸弹啊!

3、最牛逼的排序算法

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

排序算法代码大概是这样的:

这个程序员更屌啊,数字排序,用一行简单的 Arrays.sort 就搞定的东西,他竟用到了这么多概念。

1、循环

2、线程休眠

3、多线程

还好是数字小啊,还好休眠是毫秒啊,要是数字大,要是休眠时间是秒,那要等多少休眠时间才能排序完?

按道理,他的程序也没问题啊,老板为什么要开除他?应用程序中出 BUG 不是很正常的事吗?但他这种排序思维,能写出这样的隐藏 BUG 也是绝了,创造性的发明了 "休眠排序" 算法,系统里面还不知道有多少这样的坑,不开除他开除谁啊?

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

4、休眠的最高境界

/**
 * 获取未来几天的日期
 * @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();}

获取指定日期用休眠?写完,已笑晕在厕所。。。

开除的原因是因为没写注释吗?

显然不是,休眠的逻辑,大家都懂,不需要写注释,你注释写休眠 1 天也没意义啊。。。

这个程序员的思维不是一般的牛啊,获取下一天的日期,居然要休眠等到下一天再获取,欲哭无泪。。。

想离职找不到借口?

那还不容易,哥送你这段加强版,复制这段代码,提交打包测试上线,坐等开除。

5、真就是假,假就是真

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

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

还有类似这样的:

if (xx1 == true && xx2 !=  true){ ...} 
else { ...}

还有这样的:

if (xx1 > xx2){return true;} 
else {return false;}

写得太优雅了!

6、循环循环循环

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

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

for (;;){...}

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

7、魔法值的真谛

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

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

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

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

8、看花眼的参数

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

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

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

9、消失的异常

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

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

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

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

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

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

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

10、神奇的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 能搞定的非要写这么长,还缩进得这么有层次感。。

小结

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

本文分享自微信公众号 - 程序员乔戈里(CXYqiaogeli)

原文出处及转载信息见文内详细说明,如有侵权,请联系 yunjia_community@tencent.com 删除。

原始发表时间:2019-10-24

本文参与腾讯云自媒体分享计划,欢迎正在阅读的你也加入,一起分享。

发表于

我来说两句

0 条评论
登录 后参与评论

相关文章

来自专栏看那个码农

分享 | 震惊,机器学习居然有这些事

机器学习是一类算法的总称,这些算法企图从大量历史数据中挖掘出其中隐含的规律,并用于预测或者分类,更具体的说,机器学习可以看作是一个函数,输入是样本数据,输出是期...

7620
来自专栏看那个码农

干货 | 一分钟了解PyQt的绝对定位

今天我们首先来谈谈的是PyQt编程中的绝对定位,绝对定位就是每个控件按程序员自己指定的位置进行放置。

8320
来自专栏看那个码农

干货 | 一分钟带你了解PyQt的窗口布局

水平布局管理器(QHBoxLayout):这个管理器是将控件进行水平布局,在水平方向上整理排列控件。

7810
来自专栏AI掘金志

大华股份:未来 5 年 B 端市场大有潜力,智能解决方案将加速普及

近日,海富通基金、富国基金等数家机构对大华股份进行了实地调研,大华股份相关高管就相关问题进行了解答。

11520
来自专栏LINUX阅码场

赵晨雨: 从微观角度来看linux内核设计

最近总结出来学习内核有两个大的角度,一种就是从宏观角度来看,总的来说就是顺着抽象,管理,操作来看,这种角度更多的是内核中应用层面的内容,用来理解内核中是怎么运转...

6410
来自专栏新智元

一文梳理2019年腾讯广告算法大赛冠军方案

作为从本次比赛共157队伍中脱颖而出的冠军方案,评分达到87.9683,从数据清洗、模型构建、目标优化等有非常多值得学习的地方。

9320
来自专栏LINUX阅码场

Barrier-enabled IO stack for Flash storage

本期和大家聊的是刚刚在2月份拿到FAST 2018最佳论文奖的一篇文章。它讨论了实现一种支持保序IO操作的设备会带来哪些好处,目前的Linux内核里处理FLUS...

11310
来自专栏看那个码农

PyQt十讲 | Qt Designer工具的使用方法

Qt Designer是PyQt程序UI界面的实现工具,Qt Designer工具使用简单,可以通过拖拽和点击完成复杂界面设计,并且设计完成的.ui程序可以转换...

15320
来自专栏LINUX阅码场

宋宝华: 迭代螺旋法——关于Linux学习方法的血泪建议

在下是一个码农,也号称是一个老湿,平生阅码农无数(吹牛的 ^-^)。经由大量的案例,我能够理解了为什么很多码农学了很多年Linux,还是感觉没有掌握要领,仍然内...

5920
来自专栏看那个码农

实战 | 一行代码让你的电脑可以看图说话

Image Caption 任务是一个需要综合计算机视觉和自然语言处理的任务,需要使用计算机建立某种映射方式,将处于视觉模态当中的数据映射到文本模态当中,即让视...

10910

扫码关注云+社区

领取腾讯云代金券

年度创作总结 领取年终奖励