目錄
一、前言
二、代碼有毒！
1. 方法命名
2. 最佳排序
3. 有點燒腦
4. 迷之求和
5. 花裏胡哨
6. 數字判斷
7. 代碼健壯
8. 性能優化
9. 無用日誌
10. 耗時遍歷
三、總結
一、前言
學過的代碼記不住？方式不對才記不住，你這麼記！
- Git：上廁所不叫上廁所，叫拉分支！
- Socket：廁所就是服務器，坑就是端口！
- 隊列：上廁所 叫入隊列，先進先出！
- 棧：去廚房🥣叫進棧，後進先出！
- 架構：三居的格局叫MVC，四居的格局叫DDD！
- 理論：系統結構設計定的好，有點bug沒問題，能改。這就是茅坑跟坐便的區別。
除了有點味道以外，這回是不記住了，我們編程寫代碼的過程和我們日常生活的例子，往往都是這樣可以對應上，有了真實可以觸及的實物，再去了解編程就會更加容易，也很難忘記。但可能會寫着寫着代碼，就傻笑起來！
除了這些正能量學習的例子，我們接下來再看看哪些有毒的代碼！
二、代碼有毒！
以下代碼用好了升職加薪，用不好開除走人！
1. 方法命名
public List<UserInfo> queryBitchUserInfo(String req) {
return null;
}
- 指數：⭐⭐⭐
- 解毒：小哥應該是想寫批量查詢用戶的方法名，結果把
batch(批量)，寫成了bitch(婊子) - 點評：接口是上午寫的，人是下午走的！
2. 最佳排序
public static void main(String[] args) {
int[] numbers = new int[]{2, 30000000, 1, 6, 40000000, 5};
for (final int number : numbers) {
new Thread(new Runnable() {
@Override
public void run() {
try {
Thread.sleep(number);
System.out.println(number);
} catch (InterruptedException ignore) {
}
}
}).start();
}
}
- 指數：⭐⭐⭐
- 解毒：用數字休眠時常排序，誰醒來的時間早，誰就先輸出。
- 點評：思路清奇，要不是這次排序等了一天，老闆也不能踢他！
3. 有點燒腦
@Test
public void test_idx_hashMap() {
Map<String, String> map = new HashMap<>(64);
map.put("alderney", "未實現服務");
map.put("luminance", "未實現服務");
map.put("chorology", "未實現服務");
map.put("carline", "未實現服務");
map.put("fluorosis", "未實現服務");
map.put("angora", "未實現服務");
map.put("insititious", "未實現服務");
map.put("insincere", "已實現服務");
long startTime = System.currentTimeMillis();
for (int i = 0; i < 100000000; i++) {
map.get("insincere");
}
System.out.println("耗時(initialCapacity)：" + (System.currentTimeMillis() - startTime));
}
指數：⭐⭐⭐⭐⭐
解毒：這是一個定義
HashMap存放業務實現key，通過key調用服務的功能。但這裏的key，只有insincere有用，其他的都是未實現服務。那你看到有啥問題了嗎？這點代碼乍一看沒什麼問題，看明白了就是代碼裏下砒霜！它的目的就一個，要讓所有的key成一個鏈表放到HashMap中，而且把有用的key放到鏈表的最後，增加get時的耗時！
首先，
new HashMap<>(64);爲啥默認初始化64個長度？因爲默認長度是8，插入元素時，當鏈表長度爲8時候會進行擴容和鏈表樹化判斷，此時就會把原有的key散列了，不能讓所有key構成一個時間複雜度較高的鏈表。其次，所有的
key都是刻意選出來的，因爲他們在HashMap計算下標時，下標值都爲0，idx =(size - 1) & (key.hashCode() ^ (key.hashCode() >>> 16))，這樣就能讓所有key都散列到同一個位置進行碰撞。而且單詞insincere的意思是；不誠懇的、不真誠的！最後，前7個key其實都是廢
key，不起任何作用，只有最後一個 key 有服務。那麼這樣就可以在HashMap中建出來很多這樣耗時的碰撞鏈表，當然要滿足0.75的負載因子，不要讓HashMap擴容。-
整體的效果如下圖，key並沒有均勻散列；
點評：能寫出這種代碼就是薪資沒給夠，等着代碼優化提加薪呢！
4. 迷之求和
@Test
public void test_add(){
int num = 0;
for (int i = 0; i < 100; i++) {
num = num++;
}
System.out.println(num);
}
- 指數：⭐⭐
- 解毒：最終
num結果爲 0，num++根本沒起啥作用。因爲後++，是先用結果，在++操作，不會給賦值。正確寫法是：num = ++ num； - 點評：這種錯誤就跟開車闖紅燈似的，輕則扣分罰款，重則傾家蕩產。
5. 花裏胡哨
private boolean checkAge(int age ) {
boolean result;
if (age >18)
{
result=true;
} else {
result=false;
}
return result;
}
- 指數：⭐
- 解毒：代碼可以運行，但是可以優化爲
return age > 18。 - 點評：你們公司是按照代碼行數打績效？不做格式化、不整潔、不看IDEA工具提示，代碼是寫給人看的！啥有不是！
6. 數字判斷
public boolean isNumber(String str) {
try {
Integer.parseInt(str);
return true;
} catch (Exception e) {
return false;
}
}
- 指數：⭐⭐
- 解毒：判斷是不是數字，不拋異常就是，拋異常就不是。這可以使用
StringUtils工具包判斷，也可以自己寫正則判斷。 - 點評：這代碼真燒，用異常做業務。這不是把🍄蘑菇給狗狗喫嗎！🐕狗狗沒死你到是喫蘑菇呀，你喫狗粑粑。
7. 代碼健壯
public void neverStop(){
//一直循環
while (true) {
try {
//業務處理流程
} catch (Exception e) {
//抓到異常，不處理、不打日誌、就是不要停，繼續跑
continue ;
}
}
}
- 指數：⭐⭐⭐
- 解毒：把可能拋異常的代碼用tryCatch包起來，一直跑，遇到異常也要跑。這個時候遇到異常，要做一些流程處理，最起碼要打日誌和報警。
- 點評：業務開發很多時候都是爲了解決異常流程，就像
擦屁屁的紙80%的面積是保護手的。怎麼滴，我看你這代碼，是非要一直摳破呀！
8. 性能優化
// APP首頁查詢，優化前
public void queryInitInfo(){
Thread.sleep(3000);
}
// APP首頁查詢，優化後
public void queryInitInfo(){
Thread.sleep(500);
}
- 指數：⭐⭐⭐
- 解毒：沒啥解毒的，一公斤鶴頂紅兌了一口口水！
- 點評：點評不了啦，抓到就開了吧！
9. 無用日誌
// 規則引擎校驗
public boolean ruleEngine(MatterReq req) {
try {
// 業務流程
} catch (Exception e) {
logger.error(e); // 只打異常，不打入參信息
}
}
- 指數：⭐
- 解毒：日誌裏只打了異常，沒有入參信息，當你的方法有大量的調用時，很難快速定位問題。
- 點評：下次記得把
產品經理也打日誌裏去，要死一起死！
10. 耗時遍歷
@Test
public void test_LinkedList() {
// 初始化100萬數據
List<Integer> list = new LinkedList<Integer>(1000000);
// 遍歷求和
int sum = 0;
for (int i = 0; i < list.size(); i++) {
sum += list.get(i);
}
}
- 指數：⭐⭐⭐⭐
- 解毒：乍一看可能覺得沒什麼問題，但是這個遍歷求和會非常慢。主要因爲鏈表的數據結構，每一次
list.get(i)都是從鏈表的頭開始查找，與ArrayList不同，LinkedList它時間複雜度是O(n)。那如果說你不知道對方傳過來的是LinkedList還是ArrayList呢，其實可以通過list instanceof RandomAccess進行判斷。ArrayList有隨機訪問的實現，LinkedList是沒有。同時也可以使用增強的for循環或者Iterator進行遍歷。 - 點評：根基不牢，地動山搖！一知半解，坑了老鐵！
三、總結
好的代碼千篇一律，差的程序升值加薪！，這些有毒的代碼，淋漓盡致的展示了程序員的才華出衆，同時也嚴重懷疑就是錢給少了！