电竞比分网-中国电竞赛事及体育赛事平台

分享

握草,你竟然在代碼里下毒!

 小傅哥 2021-12-13


作者:小傅哥
博客:https://

沉淀、分享、成長,讓自己和他人都能有所收獲!😄

一、前言

學過的代碼記不住?方式不對才記不住,你這么記!

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

除了有點味道以外,這回是不記住了,我們編程寫代碼的過程和我們日常生活的例子,往往都是這樣可以對應上,有了真實可以觸及的實物,再去了解編程就會更加容易,也很難忘記。但可能會寫著寫著代碼,就傻笑起來!

除了這些正能量學習的例子,我們接下來再看看哪些有毒的代碼!

二、代碼有毒!

以下代碼用好了升職加薪,用不好開除走人!

1. 方法命名

public List<UserInfo> queryBitchUserInfo(String req) {

    return null;
}

  • 指數(shù):???
  • 解毒:小哥應該是想寫批量查詢用戶的方法名,結果把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();
    }
}

  • 指數(shù):???
  • 解毒:用數(shù)字休眠時常排序,誰醒來的時間早,誰就先輸出。
  • 點評:思路清奇,要不是這次排序等了一天,老板也不能踢他!

3. 有點燒腦

@Test
public void test_idx_hashMap() {
    Map<String, String> map = new HashMap<>(64);
    map.put("alderney", "未實現(xiàn)服務");
    map.put("luminance", "未實現(xiàn)服務");
    map.put("chorology", "未實現(xiàn)服務");
    map.put("carline", "未實現(xiàn)服務");
    map.put("fluorosis", "未實現(xiàn)服務");
    map.put("angora", "未實現(xiàn)服務");
    map.put("insititious", "未實現(xiàn)服務");
    map.put("insincere", "已實現(xiàn)服務");
    
    long startTime = System.currentTimeMillis();
    for (int i = 0; i < 100000000; i++) {
        map.get("insincere");
    }
    System.out.println("耗時(initialCapacity):" + (System.currentTimeMillis() - startTime));
}

  • 指數(shù):?????

  • 解毒:這是一個定義HashMap存放業(yè)務實現(xiàn)key,通過key調用服務的功能。但這里的key,只有insincere有用,其他的都是未實現(xiàn)服務。那你看到有啥問題了嗎?

    • 這點代碼乍一看沒什么問題,看明白了就是代碼里下砒霜!它的目的就一個,要讓所有的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并沒有均勻散列;
  • 點評:能寫出這種代碼就是薪資沒給夠,等著代碼優(yōu)化提加薪呢!

4. 迷之求和

@Test
public void test_add(){
    int num = 0;
    for (int i = 0; i < 100; i++) {
        num = num++;
    }
    System.out.println(num);
}

  • 指數(shù):??
  • 解毒:最終 num 結果為 0,num++ 根本沒起啥作用。因為后++,是先用結果,在++操作,不會給賦值。正確寫法是:num = ++ num;
  • 點評:這種錯誤就跟開車闖紅燈似的,輕則扣分罰款,重則傾家蕩產(chǎn)。

5. 花里胡哨

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

  • 指數(shù):?
  • 解毒:代碼可以運行,但是可以優(yōu)化為return age > 18
  • 點評:你們公司是按照代碼行數(shù)打績效?不做格式化、不整潔、不看IDEA工具提示,代碼是寫給人看的!啥有不是!

6. 數(shù)字判斷

public boolean isNumber(String str) {
    try {
        Integer.parseInt(str);
        return true;
    } catch (Exception e) {
        return false;
    }
}

  • 指數(shù):??
  • 解毒:判斷是不是數(shù)字,不拋異常就是,拋異常就不是。這可以使用 StringUtils 工具包判斷,也可以自己寫正則判斷。
  • 點評:這代碼真燒,用異常做業(yè)務。這不是把🍄蘑菇給狗狗吃嗎!🐕狗狗沒死你到是吃蘑菇呀,你吃狗粑粑。

7. 代碼健壯

public void neverStop(){
    //一直循環(huán)
    while (true) {
        try {
            //業(yè)務處理流程
        } catch (Exception e) {
            //抓到異常,不處理、不打日志、就是不要停,繼續(xù)跑
            continue ;
        }
    }
}

  • 指數(shù):???
  • 解毒:把可能拋異常的代碼用tryCatch包起來,一直跑,遇到異常也要跑。這個時候遇到異常,要做一些流程處理,最起碼要打日志和報警。
  • 點評:業(yè)務開發(fā)很多時候都是為了解決異常流程,就像擦屁屁的紙80%的面積是保護手的。怎么滴,我看你這代碼,是非要一直摳破呀!

8. 性能優(yōu)化

// APP首頁查詢,優(yōu)化前
public void queryInitInfo(){
    Thread.sleep(3000);
}

// APP首頁查詢,優(yōu)化后
public void queryInitInfo(){
    Thread.sleep(500);
}

  • 指數(shù):???
  • 解毒:沒啥解毒的,一公斤鶴頂紅兌了一口口水!
  • 點評:點評不了啦,抓到就開了吧!

9. 無用日志

// 規(guī)則引擎校驗
public boolean ruleEngine(MatterReq req) {
    try {
        // 業(yè)務流程
    } catch (Exception e) {
        logger.error(e);  // 只打異常,不打入?yún)⑿畔?/span>
    }
}

  • 指數(shù):?
  • 解毒:日志里只打了異常,沒有入?yún)⑿畔?#xff0c;當你的方法有大量的調用時,很難快速定位問題。
  • 點評:下次記得把產(chǎn)品經(jīng)理也打日志里去,要死一起死!

10. 耗時遍歷

@Test
public void test_LinkedList() {
// 初始化100萬數(shù)據(jù)
    List<Integer> list = new LinkedList<Integer>(1000000);
    
    // 遍歷求和
    int sum = 0;
    for (int i = 0; i < list.size(); i++) {
        sum += list.get(i);
    }
    
}

  • 指數(shù):????
  • 解毒:乍一看可能覺得沒什么問題,但是這個遍歷求和會非常慢。主要因為鏈表的數(shù)據(jù)結構,每一次list.get(i)都是從鏈表的頭開始查找,與ArrayList不同,LinkedList它時間復雜度是O(n)。那如果說你不知道對方傳過來的是LinkedList還是ArrayList呢,其實可以通過list instanceof RandomAccess 進行判斷。ArrayList 有隨機訪問的實現(xiàn),LinkedList 是沒有。同時也可以使用增強的for循環(huán)或者Iterator進行遍歷。
  • 點評: 根基不牢,地動山搖!一知半解,坑了老鐵!

三、總結

好的代碼千篇一律,差的程序升值加薪!,這些有毒的代碼,淋漓盡致的展示了程序員的才華出眾,同時也嚴重懷疑就是錢給少了!

敲黑板:想在這編碼這條路上走的更遠,還是需要腳踏實地的把根基打牢。所以非常推進你閱讀以下系列專欄文章,夯實基礎、拓展能力、提升眼界;

  • Java核心突破瓶頸篇:https:///itstack/interview.html
  • 重學Java設計模式篇:https:///itstack/itstack-demo-design.html
  • Java架構設計學習篇:https:///itstack-demo-ddd/itstack-demo-ddd.html

好!,本篇文章就到這里,有意思的代碼還有很多,歡迎在評論區(qū)留下你的鬼畜代碼!

    轉藏 分享 獻花(0

    0條評論

    發(fā)表

    請遵守用戶 評論公約

    類似文章 更多