查看原文
其他

炸了!刚写完这段代码,就被开除了!

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

点击上方“Python大本营”,选择“置顶公众号

python大本营  IT人的职业提升平台


栈长前些天刚写完上面这篇文章,没几天,又来一个悲剧。

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

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

这段代码有什么问题?

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

1、循环

2、线程休眠

3、多线程

下面贴上完整的代码:

/**
 * 微信公众号:Java技术栈
 */

public class ArraySort implements Runnable {

    private int number;

    public ArraySort(int number) {
        this.number = number;
    }

    public static void main(String[] args) {
        int[] numbers = new int[]{102, 338, 62, 9132, 580, 666};
        for (int number : numbers) {
            new Thread(new ArraySort(number)).start();
        }
    }

    @Override
    public void run() {
        try {
            Thread.sleep(this.number);
            System.out.println(this.number);
        } catch (InterruptedException e) {
            e.printStackTrace();
        }
    }

}

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

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

或许他应该看一下排序这篇文章:图文带你了解 8 大排序算法》。

说到这个程序员,让我想起了最近审查代码时候的几个坑,真是让人哭笑不得。

示例1:

if(flag  == false){
    return true;
} else {
    return false;
}

直接 return flag 不行?写这么绕,还把逻辑写错了。

示例2:

if(...)
    a
    b
    c

多段代码,if 不加 {},后面格式化成了:

if(...)
    a
b
c

这样导致业务逻辑有严重的 BUG,无语了吧?还有更多奇葩的,栈长审查代码真心累。

你还见过哪些更奇葩的代码?欢迎留言分享一下。


—— 推 荐 阅 读 ——


点个“在看”必脱单喔 

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

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