厉害了！刚写完这段代码，就被老板开除了

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

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

 

这段代码有什么问题?

这个程序员更屌啊，数字排序，用一行简单的 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 也是绝了，创造性的发明了 "休眠排序" 算法，系统里面还不知道有多少这样的坑，不开除他开除谁啊?

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

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

示例1：

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

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

示例2：

if(...) 
 a 
 b 
 c 

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

if(...) 
 a 
b 
c 

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