握草，你居然在代碼裏下毒！

時間 2020-09-09

標籤居然代碼下毒欄目興趣愛好简体版

原文原文鏈接

做者：小傅哥
博客：https://bugstack.cnhtml

沉澱、分享、成長，讓本身和他人都能有所收穫！😄

1、前言

學過的代碼記不住？方式不對才記不住，你這麼記！java

Git：上廁所不叫上廁所，叫拉分支！
Socket：廁所就是服務器，坑就是端口！
隊列：上廁所🚽叫入隊列，先進先出！
棧：去廚房🥣叫進棧，後進先出！
架構：三居的格局叫MVC，四居的格局叫DDD！
理論：系統結構設計定的好，有點bug沒問題，能改。這就是茅坑跟坐便的區別。

除了有點味道之外，這回是不記住了，咱們編程寫代碼的過程和咱們平常生活的例子，每每都是這樣能夠對應上，有了真實能夠觸及的實物，再去了解編程就會更加容易，也很難忘記。但可能會寫着寫着代碼，就傻笑起來！程序員

除了這些正能量學習的例子，咱們接下來再看看哪些有毒的代碼！編程

2、代碼有毒！

如下代碼用好了升職加薪，用很差開除走人！設計模式

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進行遍歷。
點評：根基不牢，地動山搖！只知其一;不知其二，坑了老鐵！