阿里P8架構師教你kill祖傳石山代碼重複/大量ifelse 1 crud 工程師之“痛” 2 工廠+模板方法模式,消除多if和重複代碼 3 註解+反射消除重複代碼 4 屬性拷貝 5 總結

文章收錄在我的 GitHub 倉庫,歡迎Star/fork:

Java-Interview-Tutorial

https://github.com/Wasabi1234/Java-Interview-Tutorial

本文就教你如何優雅消除重複代碼並改變你對業務代碼沒技術含量的觀念。

1 crud 工程師之“痛”

很多 crud 工程師抱怨業務開發沒有技術含量,什麼設計模式、高併發都用不到,就是堆CRUD。每次面試被問到“講講常用設計模式?”,都只能把單例講到精通,其他設計模式即使聽過也只會簡單說說,因爲根本沒實際用過。
對於反射、註解,也只是知道在框架中用的很多,但自己又不寫框架,更不知道該如何使用。

  • 設計模式是世界級軟件大師在大型項目的經驗所得,是被證實利於維護大型項目的。
  • 反射、註解、泛型等高級特性在框架被大量使用,是因爲框架往往需要以同一套算法應對不同數據結構,而這些特性可以幫助減少重複代碼,也是利於維護。

提升項目的可維護性是每個 coder 必須注意的,非常重要的一個手段就是減少代碼重複,因爲重複過多會導致:

  • 容易修改一處忘記修改另一處,造成Bug
  • 有一些代碼並非完全重複,而是相似度高,修改這些類似的代碼容易改(cv)錯,把原本有區別的地方改成一樣

2 工廠+模板方法模式,消除多if和重複代碼

2.1 需求

開發購物車下單,對不同用戶不同處理:

  • 普通用戶需要收取運費,運費是商品價格的10%,無商品折扣
  • VIP用戶同樣需要收取商品價格10%的快遞費,但購買兩件以上相同商品時,第三件開始享受一定折扣
  • 內部用戶可以免運費,無商品折扣

實現三種類型的購物車業務邏輯,把入參Map對象(K:商品ID,V:商品數量),轉換爲出參購物車類型Cart。

2.2 菜鳥實現

  • 購物車


  • 購物車中的商品


2.2.1 普通用戶

2.2.2 VIP用戶

VIP用戶能享受同類商品多買的折扣。只需額外處理多買折扣部分。


2.2.3 內部用戶

免運費、無折扣,只處理商品折扣和運費時的邏輯差異。


三種購物車超過一半代碼重複。
雖然不同類型用戶計算運費和優惠的方式不同,但整個購物車的初始化、統計總價、總運費、總優惠和支付價格邏輯都一樣。

代碼重複本身不可怕,可怕的是漏改或改錯
比如,寫VIP用戶購物車的同學發現商品總價計算有Bug,不應該是把所有Item的price加在一起,而是應該把所有Item的price*quantity相加。
他可能只修VIP用戶購物車的代碼,漏了普通用戶、內部用戶的購物車中重複邏輯實現的相同Bug。

有三個購物車,就需根據不同用戶類型使用不同購物車。

  • 使用多if實現不同類型用戶調用不同購物車process


就只能不斷增加更多的購物車類,寫重複的購物車邏輯、寫更多if邏輯嗎?
當然不是,相同的代碼應該只在一處出現

2.3 重構祕技 - 模板方法模式

可以把重複邏輯定義在抽象類,三個購物車只要分別實現不同部分的邏輯。
這其實就是模板方法模式
在父類中實現購物車處理的流程模板,然後把需要特殊處理的留抽象方法定義,讓子類去實現。由於父類邏輯無法單獨工作,因此需要定義爲抽象類。

如下代碼所示,AbstractCart抽象類實現了購物車通用的邏輯,額外定義了兩個抽象方法讓子類去實現。其中,processCouponPrice方法用於計算商品折扣,processDeliveryPrice方法用於計算運費。


有抽象類,三個子類的實現就簡單了。

  • 普通用戶的購物車NormalUserCart,實現0優惠和10%運費


  • VIP用戶的購物車VipUserCart,直接繼承NormalUserCart,只需修改多買優惠策略


  • 內部用戶購物車InternalUserCart最簡單,直接設置0運費、0折扣


抽象類和三個子類的實現關係圖

2.4 重構祕技之工廠模式 - 消除多if

既然三個購物車都叫XXXUserCart,可將用戶類型字符串拼接UserCart構成購物車Bean的名稱,然後利用IoC容器,通過Bean的名稱直接獲取到AbstractCart,調用其process方法即可實現通用。

這就是工廠模式,藉助Spring容器實現:


若有新用戶類型、用戶邏輯,只要新增一個XXXUserCart類繼承AbstractCart,實現特殊的優惠和運費處理邏輯即可。

工廠+模板方法模式,消除了重複代碼,還避免修改既有代碼。這就是設計模式中的開閉原則:對修改關閉,對擴展開放。

3 註解+反射消除重複代碼

3.1 需求

銀行提供了一些API接口,對參數的序列化不使用JSON,而需要我們把參數依次拼在一起構成一個大字符串。

  • 按照銀行提供的API文檔的順序,把所有參數構成定長的數據,然後拼接在一起作爲整個字符串
  • 因爲每種參數都有固定長度,未達到長度時需填充:
    • 字符串類型的參數不滿長度部分需要以下劃線右填充,也就是字符串內容靠左
    • 數字類型的參數不滿長度部分以0左填充,也就是實際數字靠右
    • 貨幣類型的表示需要把金額向下舍入2位到分,以分爲單位,作爲數字類型同樣進行左填充。
      對所有參數做MD5操作作爲簽名(爲了方便理解,Demo中不涉及加鹽處理)。

比如,創建用戶方法和支付方法的定義是這樣的:



3.2 菜鳥實現

直接根據接口定義實現填充、加簽名、請求調用:

public class BankService {

    // 創建用戶
    public static String createUser(String name, String identity, String mobile, int age) throws IOException {
        StringBuilder stringBuilder = new StringBuilder();
        // 字符串靠左,多餘的地方填充_
        stringBuilder.append(String.format("%-10s", name).replace(' ', '_'));
        stringBuilder.append(String.format("%-18s", identity).replace(' ', '_'));
        // 數字靠右,多餘的地方用0填充
        stringBuilder.append(String.format("%05d", age));
        // 字符串靠左
        stringBuilder.append(String.format("%-11s", mobile).replace(' ', '_'));
        // MD5簽名
        stringBuilder.append(DigestUtils.md2Hex(stringBuilder.toString()));
        return Request.Post("http://localhost:45678/reflection/bank/createUser")
                .bodyString(stringBuilder.toString(), ContentType.APPLICATION_JSON)
                .execute().returnContent().asString();
    }
    
    // 支付
    public static String pay(long userId, BigDecimal amount) throws IOException {
        StringBuilder stringBuilder = new StringBuilder();
        // 數字靠右
        stringBuilder.append(String.format("%020d", userId));
        // 金額向下舍入2位到分,以分爲單位,作爲數字靠右,多餘的地方用0填充
        stringBuilder.append(String.format("%010d", amount.setScale(2, RoundingMode.DOWN).multiply(new BigDecimal("100")).longValue()));
        // MD5簽名
        stringBuilder.append(DigestUtils.md2Hex(stringBuilder.toString()));
        return Request.Post("http://localhost:45678/reflection/bank/pay")
                .bodyString(stringBuilder.toString(), ContentType.APPLICATION_JSON)
                .execute().returnContent().asString();
    }
}

這段代碼的重複粒度更細:

  • 三種標準數據類型的處理邏輯有重複
  • 處理流程中字符串拼接、加簽和發請求的邏輯,在所有方法重複
  • 實際方法的入參的參數類型和順序,不一定和接口要求一致,容易出錯
  • 代碼層面針對每一個參數硬編碼,無法清晰地進行覈對,如果參數達到幾十個、上百個,出錯的概率極大。

3.3 重構祕技之註解&反射

針對銀行請求的所有邏輯均使用一套代碼實現,不會出現任何重複。

要實現接口邏輯和邏輯實現的剝離,首先要以POJO類定義所有的接口參數。

  • 創建用戶API的參數
@Data
public class CreateUserAPI {
    private String name;
    private String identity;
    private String mobile;
    private int age;
}

有了接口參數定義,就能通過自定義註解爲接口和所有參數增加一些元數據。

  • 如下定義一個接口API的註解BankAPI,包含接口URL地址和接口說明


再定義一個自定義註解@BankAPIField,描述接口的每一個字段規範,包含參數的次序、類型和長度三個屬性:

  • 定義CreateUserAPI類描述創建用戶接口的信息,通過爲接口增加@BankAPI註解,來補充接口的URL和描述等元數據;通過爲每一個字段增加@BankAPIField註解,來補充參數的順序、類型和長度等元數據:

  • 類似的還有PayAPI類



    這2個類繼承的AbstractAPI類是一個空實現,因爲該案例中的接口無公共數據。

通過這倆類,即可在幾秒鐘內完成和API清單表格的核對。若我們的核心翻譯過程(即把註解和接口API序列化爲請求需要的字符串的過程)沒問題,只要註解和表格一致,API請求翻譯就不會有問題。

通過註解實現了對API參數的描述。看反射如何配合註解實現動態的接口參數組裝:

private static String remoteCall(AbstractAPI api) throws IOException {
    // 從類上獲得BankAPI註解,然後拿到其URL屬性,後續進行遠程調用
    BankAPI bankAPI = api.getClass().getAnnotation(BankAPI.class);
    bankAPI.url();
    StringBuilder stringBuilder = new StringBuilder();
    // 使用stream快速實現獲取類中所有帶BankAPIField註解的字段,並把字段按order屬性排序,然後設置私有字段反射可訪問。
    Arrays.stream(api.getClass().getDeclaredFields()) //獲得所有字段
            //查找標記了註解的字段
            .filter(field -> field.isAnnotationPresent(BankAPIField.class))
            // 根據註解中的order對字段排序
            .sorted(Comparator.comparingInt(a -> a.getAnnotation(BankAPIField.class).order()))
            .peek(field -> field.setAccessible(true)) //設置可以訪問私有字段
            .forEach(field -> {
                // 實現了反射獲取註解的值,然後根據BankAPIField拿到的參數類型,按照三種標準進行格式化,將所有參數的格式化邏輯集中在了這一處
                // 獲得註解
                BankAPIField bankAPIField = field.getAnnotation(BankAPIField.class);
                Object value = "";
                try {
                    // 反射獲取字段值
                    value = field.get(api);
                } catch (IllegalAccessException e) {
                    e.printStackTrace();
                }
                // 根據字段類型以正確的填充方式格式化字符串
                switch (bankAPIField.type()) {
                    case "S": {
                        stringBuilder.append(String.format("%-" + bankAPIField.length() + "s", value.toString()).replace(' ', '_'));
                        break;
                    }
                    case "N": {
                        stringBuilder.append(String.format("%" + bankAPIField.length() + "s", value.toString()).replace(' ', '0'));
                        break;
                    }
                    case "M": {
                        if (!(value instanceof BigDecimal))
                            throw new RuntimeException(String.format("{} 的 {} 必須是BigDecimal", api, field));
                        stringBuilder.append(String.format("%0" + bankAPIField.length() + "d", ((BigDecimal) value).setScale(2, RoundingMode.DOWN).multiply(new BigDecimal("100")).longValue()));
                        break;
                    }
                    default:
                        break;
                }
            });
    // 實現參數加簽和請求調用
    // 簽名邏輯stringBuilder.append(DigestUtils.md2Hex(stringBuilder.toString()));
    String param = stringBuilder.toString();
    long begin = System.currentTimeMillis();
    //發請求
    String result = Request.Post("http://localhost:45678/reflection" + bankAPI.url())
            .bodyString(param, ContentType.APPLICATION_JSON)
            .execute().returnContent().asString();
    log.info("調用銀行API {} url:{} 參數:{} 耗時:{}ms", bankAPI.desc(), bankAPI.url(), param, System.currentTimeMillis() - begin);
    return result;
}

所有處理參數排序、填充、加簽、請求調用的核心邏輯,都匯聚在remoteCall。有這方法,BankService中每一個接口的實現就非常簡單了,只是參數的組裝,然後調用remoteCall。

涉及類結構性的通用處理,都可按照該模式減少重複代碼。

  • 反射使得我們在不知類結構時,按固定邏輯處理類成員
  • 註解給我們爲這些成員補充元數據的能力,使得我們利用反射實現通用邏輯的時候,可以從外部獲得更多我們關心的數據

4 屬性拷貝

對於三層架構系統,層間解耦及每層對數據的不同需求,每層都會有自己的POJO實體。
手動寫這些實體之間的賦值代碼,容易出錯。對於複雜業務系統,實體有幾十甚至幾百個屬性也很正常。比如ComplicatedOrderDTO,描述一個訂單中幾十個屬性。如果轉換爲一個類似的DO,複製其中大部分的字段,然後把數據入庫,勢必需要進行很多屬性映射賦值操作。就像這樣,密密麻麻的代碼是不是已經讓你頭暈了?

ComplicatedOrderDTO orderDTO = new ComplicatedOrderDTO();
ComplicatedOrderDO orderDO = new ComplicatedOrderDO();
orderDO.setAcceptDate(orderDTO.getAcceptDate());
orderDO.setAddress(orderDTO.getAddress());
orderDO.setAddressId(orderDTO.getAddressId());
orderDO.setCancelable(orderDTO.isCancelable());
orderDO.setCommentable(orderDTO.isComplainable()); //屬性錯誤
orderDO.setComplainable(orderDTO.isCommentable()); //屬性錯誤
orderDO.setCancelable(orderDTO.isCancelable());
orderDO.setCouponAmount(orderDTO.getCouponAmount());
orderDO.setCouponId(orderDTO.getCouponId());
orderDO.setCreateDate(orderDTO.getCreateDate());
orderDO.setDirectCancelable(orderDTO.isDirectCancelable());
orderDO.setDeliverDate(orderDTO.getDeliverDate());
orderDO.setDeliverGroup(orderDTO.getDeliverGroup());
orderDO.setDeliverGroupOrderStatus(orderDTO.getDeliverGroupOrderStatus());
orderDO.setDeliverMethod(orderDTO.getDeliverMethod());
orderDO.setDeliverPrice(orderDTO.getDeliverPrice());
orderDO.setDeliveryManId(orderDTO.getDeliveryManId());
orderDO.setDeliveryManMobile(orderDO.getDeliveryManMobile()); //對象錯誤
orderDO.setDeliveryManName(orderDTO.getDeliveryManName());
orderDO.setDistance(orderDTO.getDistance());
orderDO.setExpectDate(orderDTO.getExpectDate());
orderDO.setFirstDeal(orderDTO.isFirstDeal());
orderDO.setHasPaid(orderDTO.isHasPaid());
orderDO.setHeadPic(orderDTO.getHeadPic());
orderDO.setLongitude(orderDTO.getLongitude());
orderDO.setLatitude(orderDTO.getLongitude()); //屬性賦值錯誤
orderDO.setMerchantAddress(orderDTO.getMerchantAddress());
orderDO.setMerchantHeadPic(orderDTO.getMerchantHeadPic());
orderDO.setMerchantId(orderDTO.getMerchantId());
orderDO.setMerchantAddress(orderDTO.getMerchantAddress());
orderDO.setMerchantName(orderDTO.getMerchantName());
orderDO.setMerchantPhone(orderDTO.getMerchantPhone());
orderDO.setOrderNo(orderDTO.getOrderNo());
orderDO.setOutDate(orderDTO.getOutDate());
orderDO.setPayable(orderDTO.isPayable());
orderDO.setPaymentAmount(orderDTO.getPaymentAmount());
orderDO.setPaymentDate(orderDTO.getPaymentDate());
orderDO.setPaymentMethod(orderDTO.getPaymentMethod());
orderDO.setPaymentTimeLimit(orderDTO.getPaymentTimeLimit());
orderDO.setPhone(orderDTO.getPhone());
orderDO.setRefundable(orderDTO.isRefundable());
orderDO.setRemark(orderDTO.getRemark());
orderDO.setStatus(orderDTO.getStatus());
orderDO.setTotalQuantity(orderDTO.getTotalQuantity());
orderDO.setUpdateTime(orderDTO.getUpdateTime());
orderDO.setName(orderDTO.getName());
orderDO.setUid(orderDTO.getUid());

如果原始的DTO有100個字段,我們需要複製90個字段到DO中,保留10個不賦值,最後應該如何校驗正確性呢?

  • 數數嗎?即使數出有90行代碼,也不一定正確,因爲屬性可能重複賦值
  • 有時字段名相近,比如complainable和commentable,容易搞反
  • 對兩個目標字段重複賦值相同的來源字段
  • 明明要把DTO的值賦值到DO中,卻在set的時候從DO自己取值,導致賦值無效

使用類似BeanUtils這種Mapping工具來做Bean的轉換,copyProperties方法還允許我們提供需要忽略的屬性:

5 總結

重複代碼多了總有一天會出錯。

  • 有多個並行的類實現相似的代碼邏輯
    考慮提取相同邏輯在父類中實現,差異邏輯通過抽象方法留給子類實現。使用類似的模板方法把相同的流程和邏輯固定成模板,保留差異的同時儘可能避免代碼重複。同時,可以使用Spring的IoC特性注入相應的子類,來避免實例化子類時的大量if…else代碼。

  • 使用硬編碼的方式重複實現相同的數據處理算法
    考慮把規則轉換爲自定義註解,作爲元數據對類或對字段、方法進行描述,然後通過反射動態讀取這些元數據、字段或調用方法,實現規則參數和規則定義的分離。也就是說,把變化的部分也就是規則的參數放入註解,規則的定義統一處理。

  • 業務代碼中常見的DO、DTO、VO轉換時大量字段的手動賦值,遇到有上百個屬性的複雜類型,非常非常容易出錯
    不要手動進行賦值,考慮使用Bean映射工具進行。此外,還可以考慮採用單元測試對所有字段進行賦值正確性校驗。

代碼重複度是評估一個項目質量的重要指標,如果一個項目幾乎沒有任何重複代碼,那麼它內部抽象一定非常好。重構時,首要任務是消除重複。

參考

發表評論
所有評論
還沒有人評論,想成為第一個評論的人麼? 請在上方評論欄輸入並且點擊發布.
相關文章