經歷了的項目越多，看過的奇葩代碼也就越多。發現爛代碼的目的并不是抨擊和取笑，主要還是為了避免讓自己或者身邊的人寫出類似的代碼。

碼云上面就有個熱門倉庫 bullshit-codes[1] 專門收集史上最坑爹的代碼片段。下面放幾個比較有意思的給大家看看：

1.一行代碼就可以搞定的，非要寫的啰里啰嗦。不要給我扯什么可讀性，以下代碼一行的寫法可讀性也更好。

private boolean isEligible(int age) { 
    boolean result; 
    if (age > 18) { 
        result = true; 
    } else { 
        result = false; 
    } 
    return result; 
 
} 

2.前大廠程序員設計的智能 AI 客服系統

Scanner sc = new Scanner(System.in); 
String str; 
// 待優化 
while (true) { 
    str = sc.next(); 
    // 測試環境，正式環境從數據庫中獲取 
    str = str.replace("嗎", ""); 
    str = str.replace("？", "！"); 
    str = str.replace("？", "！"); 
    str = str.replace("不", "很"); 
    str = str.replace("你們", "我們"); 
    str = str.replace("有", "沒有"); 
 
    System.out.println(str); 
} 

效果展示：

在嗎？ 
在！ 
你好! 
你好! 
產品有問題啊 
產品沒有問題啊 
你們的服務態度不好 
我們的服務態度很好 

在嗎?在!你好!你好!產品有問題啊產品沒有問題啊你們的服務態度不好我們的服務態度很好

3.只要我的代碼夠多，你就你很難看出有啥問題

/** 
 * 三年前接手的一個政府網站項目，真假不知，據說是一個兩年web開發經驗的程序員寫的， 用的spring系列還是3.x版本，前端還是jsp，可能是老項目。 
 * 主要特點是：實體類中不寫注釋，數據庫表中也沒有注釋，命名經常使用英文拼音混合，而且還可能是簡寫(grzlChangeStatus:個人資料變更狀態) 
 * 
 * 挖坑不填坑，菊花塞電燈 
 */ 
@Controller 
public class DemoController { 
 /** 
  * 這個方法一共有800多行，有30多個if或else if的條件判斷,只寫代表性的幾個 
  * 
  * 具體方法名稱記不得，邏輯是這樣,前端jsp提交一個請求過來，界面中除了一些必填信息外，還有有5個下拉框，后臺判斷各種組合， 
  * 分別往request里面塞入不同的對象，返回的頁面視圖也可能不同。 
  */ 
 @RequestMapping("/submit") 
 public String template(HttpServletRequest request, HttpServletResponse response, HttpSession session) { 
  String param1 = request.getParameter("param1"); 
  String param2 = request.getParameter("param2"); 
  String param3 = request.getParameter("param3"); 
  String param4 = request.getParameter("param4"); 
  String param5 = request.getParameter("param5"); 
  // 你沒看錯，各種狀態全部存字串，而且判斷的時候不考慮空指針問題 
  if (param1.equals("已消毒")) { 
   // 邏輯 
   request.setAttribute("attr", "value"); 
   return "jsp1"; 
  } else if (param1.equals("已消毒") && param2.equals("1.8米以上")) { 
   // 邏輯 
   request.setAttribute("attr", "value"); 
   // 你沒看錯，可能又是另一個視圖 
   return "jsp2"; 
   // 你沒看錯，還可能出現多個條件，|| 和 && 
  } else if (param1.equals("已消毒") && param2.equals("2米以上") || param3.equals("緬甸")) { 
   // 邏輯 
   request.setAttribute("attr", "value"); 
   return "jsp3"; 
   // 你沒看錯，因為他的狀態或選項沒有用數字類型,所以不能用 > < 之類的來判斷，要列舉所有狀態 
  } else if (param1.equals("未進站") || param1.equals("已進站") || param1.equals("已消毒") && param2.equals("2米以上")) { 
   request.setAttribute("attr", "value"); 
   return "jsp3"; 
   //你沒看錯，還會出現 true==true 或false==false這樣的代碼 
  }else if(param4.equals("禁止入境")==true && param5.equals("農用車")) { 
   request.setAttribute("attr", "value"); 
   return "jsp4"; 
   //你沒看錯，還會出現邏輯非，而且，要先判斷什么條件，從來不用括號包起來，要理他的邏輯，還要先理條件判斷優先級 
  }else if(param1.equals("未進站") || param1.equals("已進站") || param1.equals("已消毒") && param2.equals("2米以上") && !param3.equals("緬甸")) { 
   request.setAttribute("attr", "value"); 
   return "jsp5"; 
  } 
  //省略剩下的30多個類似的判斷 
  else { 
   return "default"; 
  } 
 } 
} 

以上代碼的問題：

• 空指針異常
• 邏輯混亂，部分 else if 根本進不去

4.獲取一天后的日期：讓線程睡一會

public static Date getAfterDate(int day) { 
    try { 
        Thread.sleep(day * 24 * 60 * 60 * 1000); 
    } catch (Exception e) { 
        e.printStackTrace(); 
    } 
    return new Date(); 
} 

5.新上的項目的代碼命名,表示看不太懂

6.為了得到了一個用戶總人數，把所有的 User 對象查詢出來。

/** 
  統計用戶總數,來源于N年前某個用戶抱怨執行效率低,貼出類似邏輯的代碼. 
  @author wendal 
*/ 
@IocBean 
public class CountUserTable { 
 
    @Inject Dao dao; 
 
    @Test 
    public void test_user_count() { 
        // 取出全部用戶對象(dao.query返回List<User>),然后取list大小就可以啦,是不是很機智 
        int count = dao.query(User.class, null).size(); 
        System.out.println("用戶總數是" + count); 
    } 
} 

參考資料

[1]bullshit-codes: https://gitee.com/oschina/bullshit-codes