團隊里的妹子讓阿粉跟她說說怎樣寫出好的代碼
本文轉載自微信公眾號「Java極客技術」,作者鴨血粉絲。轉載本文請聯系Java極客技術公眾號。
昨天,團隊里的妹子很突然地就讓阿粉跟她說說怎么才能寫出一手好的代碼
阿粉本著負責任的態度,專門寫了一篇文章出來,妹子嘛,就是要寵的嘛
寫好注釋
寫代碼這件事情,大概就是一時編程一時爽,過后調試火葬場
阿粉有過一次經歷,就是當時開發任務緊,把功能實現之后就完事兒了,也沒有好好寫注釋。結果過了兩三個月之后,那個功能因為需求變動出 bug 了,出了 bug 就去解決唄
但是!!!因為阿粉沒有寫注釋,當時的實現邏輯,為什么這樣寫全忘了,所以相當于又重頭捋了一遍邏輯,那個味道,真是酸爽
圖片
所以阿粉要把這個建議放在第一個!
寫注釋的時候,如果邏輯比較復雜,可以先寫一下大體思路,然后下面具體實現時再詳細寫,比如阿粉曾經寫過的代碼(做了脫敏處理,大家不要太較真啦,重點是阿粉寫的注釋
- /**
- * 進行 XX 操作時調用此方法,用來向數據庫插入數據
- * @param saveModel 要插入的數據
- * @return 是否保存成功
- *
- * @author a'fen
- * @date 2020-10-17 23:50:45
- */
- public Boolean isSaveSuccess(SaveModel saveModel) {
- /*
- 整體思路:
- 1 接收傳入的所有 id/name 的信息
- 2 插入相關信息
- 2.1 XX 信息插入成功之后,再去 YY 表進行相關信息插入
- 3 對 YY 表進行插入時,先判斷是否有相關信息,如果有則進行更新,如果沒有則直接插入
- 4 是否添加 AA ,如果業務上配置了 AA ,則需要將相關內容更新到 ZZ 表中
- 注: 在 BB 界面點擊下一步時,已經將相關信息插入到了 ZZ 表中
- 所以在這里只做更新即可
- */
- // 1 接收傳入的所有 id/name 的信息
- // 創建 xxList 用來接收所有的 id/name 的信息
- List<XXModel> xxList = new ArrayList<>();
- // 對傳入的信息按照 id 進行排序,方便接下來的操作
- saveModel.getIdList().stream.collect(Collectors.groupingBy(SaveModel::getId))
- .forEach((String mapKey, List<SaveModel> save) -> {
- xxList.addAll(save);
- });
- // 2 插入相關信息
- // 循環 xxList
- for (XXModel model : xxList) {
- // 向 XX 表插入信息
- 實現代碼
- // 2.1 插入成功,去 YY 表做相關信息插入
- 實現代碼
- }
- // 3 對 YY 表進行插入時,先判斷是否有相關信息,如果有則進行更新,如果沒有則直接插入
- 實現代碼
- // 4 是否添加 AA ,如果業務上配置了 AA ,則需要將相關內容更新到 ZZ 表中
- 實現代碼
- }
在寫代碼之前,先想想可能的應用場景
有時候拿到需求之后,如果著急去做,后期可能會耗費不少的精力
阿粉曾經做過一個業務,類似用戶充值的場景,當時阿粉拿到這個需求之后,就想著不就是充值錢嘛,那就定義一個 Integer 類型不就完了,然后一頓操作猛如虎,上線之后也沒啥問題
結果有一次公司運營和其他一些第三方合作,用的就是充值這個接口,然后因為運營大批量的持續充值,直接超出了 Integer 的允許范圍,導致整條業務線都沒辦法跑通
然后排查問題排查到最后,發現是阿粉造成的,而且還是因為一個數據類型引發的...阿粉當時真的是想鉆到地下
代碼是寫給人看的,所以讓自己的代碼干凈整潔一些
代碼是寫給人看的,不是寫給機器看的。你想嘛,如果想要讓機器看,直接寫 0110001 這種編碼方式不香嗎!畢竟機器它只認 0 或者 1
所以代碼是寫給人看的,那么寫代碼的時候就讓它干凈整潔一些
比如:該縮進的縮進,該空一行的就空一行,比如下面這樣:
- public static void sleep(long millis, int nanos)
- throws InterruptedException {
- if (millis < 0) {
- throw new IllegalArgumentException("timeout value is negative");
- }
- if (nanos < 0 || nanos > 999999) {
- throw new IllegalArgumentException(
- "nanosecond timeout value out of range");
- }
- if (nanos >= 500000 || (nanos != 0 && millis == 0)) {
- millis++;
- }
- sleep(millis);
- }
要縮進有縮進,該空一行就空一行,不用你們夸,我自己都覺得這代碼寫的,真是好!(Thread 源碼誰敢說不好?!
沒事兒就和阿粉多聊聊源碼
其實想要寫一手好的代碼,最重要的就是要多看,自己多寫
看什么呢?阿粉覺得看源碼就夠了。JDK 源碼這么多人都在用,足以說明 JDK 源碼的代碼寫的有多優秀
所以呢,沒事兒就和阿粉多聊聊源碼,沒事兒就約約阿粉去圖書館一起研究研究源碼,平時自己寫代碼的時候,多注意借鑒從源碼中學到的實現思路,學到的編程技巧,日積月累下來肯定能在編程上有一個很好的提升