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

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

如果非要说一个原因，我感觉，这哥们是故意这么写的，造成查询速度较慢，之后下个迭代优化，查询速度瞬间提上来了，这可是为公司做出大贡献了，年底了，奖励个优秀个人奖.....

或许他应该看一下排序这篇文章：《改变计算技术的9个伟大算法》。

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

示例1：

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

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

示例2：

if(...)
     a
     b
     c

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

if(...)
     a
 b
 c

这样导致业务逻辑有严重的 BUG，无语了吧？还有更多奇葩的.

原文发布时间为：2018-11-24

本文来自云栖社区合作伙伴“互联网架构师”，了解相关信息可以关注“互联网架构师”。