刚写完排序算法，就被开除了…

点击上方“程序人生”，选择“置顶公众号”

第一时间关注程序猿（媛）身边的故事

作者

栈长

如需转载，请联系原作者。

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

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

这段代码有什么问题？

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

1、循环

2、线程休眠

3、多线程

下面贴上完整的代码：

/**
 * 
 */
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 也是绝了，创造性的发明了 "休眠排序" 算法，系统里面还不知道有多少这样的坑，不开除他开除谁啊？

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

示例1：

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

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

示例2：

if(...)
    a
    b
    c

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

if(...)
    a
b
c

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

☆

2018 中国大数据技术大会将于 12 月 6 -8 日在新云南皇冠假日酒店举行。汇聚超百位国内外实力讲师从学界翘楚到技术权威，全方位立体解读大数据时代的技术进程，为众技术爱好者奉上一场优质干货盛宴。

「若你有原创文章想与大家分享，欢迎投稿。」

加编辑微信ID，备注#投稿#：

程序 丨 druidlost  

小七 丨 duoshangshuang

点文末阅读全文，看『程序人生』其他精彩文章推荐。

推荐阅读：

print_r('点个赞吧')；
var_dump('点个赞吧')；
NSLog(@"点个赞吧！")
System.out.println("点个赞吧!");
console.log("点个赞吧!");
print("点个赞吧!");
printf("点个赞吧!\n");
cout << "点个赞吧!" << endl;
Console.WriteLine("点个赞吧!");
fmt.Println("点个赞吧!")
Response.Write("点个赞吧")；
alert（’点个赞吧’）