日韩性视频-久久久蜜桃-www中文字幕-在线中文字幕av-亚洲欧美一区二区三区四区-撸久久-香蕉视频一区-久久无码精品丰满人妻-国产高潮av-激情福利社-日韩av网址大全-国产精品久久999-日本五十路在线-性欧美在线-久久99精品波多结衣一区-男女午夜免费视频-黑人极品ⅴideos精品欧美棵-人人妻人人澡人人爽精品欧美一区-日韩一区在线看-欧美a级在线免费观看

歡迎訪問 生活随笔!

生活随笔

當前位置: 首頁 > 编程资源 > 编程问答 >内容正文

编程问答

这代码写的跟狗屎一样!怎么优化?

發布時間:2024/4/11 编程问答 29 豆豆
生活随笔 收集整理的這篇文章主要介紹了 这代码写的跟狗屎一样!怎么优化? 小編覺得挺不錯的,現在分享給大家,幫大家做個參考.

點擊上方“朱小廝的博客”,選擇“設為星標”

回復”1024“獲取獨家整理的學習資料

1.

導讀


隨著軟件項目代碼的日積月累,系統維護成本變得越來越高,是所有軟件團隊面臨的共同問題。持續地優化代碼,提高代碼的質量,是提升系統生命力的有效手段之一。軟件系統思維有句話“Less coding, more thinking(少編碼、多思考)”,也有這么一句俚語“Think more, code less(思考越多,編碼越少)”。所以,我們在編碼中多思考多總結,努力提升自己的編碼水平,才能編寫出更優雅、更高質、更高效的代碼。

本文總結了一套與Java函數相關的編碼規則,旨在給廣大Java程序員一些編碼建議,有助于大家編寫出更優雅、更高質、更高效的代碼。這套編碼規則,通過在高德采集部門的實踐,已經取得了不錯的成效。

2.

使用通用工具函數


2.1 案例一

現象描述:

不完善的寫法:

thisName != null && thisName.equals(name);

更完善的寫法:

(thisName == name) || (thisName != null && thisName.equals(name));

建議方案:

Objects.equals(name, thisName);

2.2 案例

現象描述:

!(list == null || list.isEmpty());

建議方案:

import org.apache.commons.collections4.CollectionUtils; CollectionUtils.isNotEmpty(list);

2.3?主要收益

  • 函數式編程,業務代碼減少,邏輯一目了然;

  • 通用工具函數,邏輯考慮周全,出問題概率低。

3.

拆分超大函數


當一個函數超過80行后,就屬于超大函數,需要進行拆分。

3.1 案例一:每一個代碼塊都可以封裝為一個函

每一個代碼塊必然有一個注釋,用于解釋這個代碼塊的功能。

如果代碼塊前方有一行注釋,就是在提醒你——可以將這段代碼替換成一個函數,而且可以在注釋的基礎上給這個函數命名。如果函數有一個描述恰當的名字,就不需要去看內部代碼究竟是如何實現的。

現象描述:

建議方案:

// 每日生活函數 public void liveDaily() {// 吃飯eat();// 編碼code();// 睡覺sleep(); }// 吃飯函數 private void eat() {// 吃飯相關代碼 }// 編碼函數 private void code() {// 編碼相關代碼 }// 睡覺函數 private void sleep() {// 睡覺相關代碼 }

3.2 案例二:每一個循環體都可以封裝為一個函

現象描述:

建議方案:

// 生活函數 public void live() {while (isAlive) {// 每日生活liveDaily();} }// 每日生活函數 private void liveDaily() {// 吃飯eat();// 編碼code();// 睡覺sleep(); }

3.3 案例三:每一個條件體都可以封裝為一個函

現象描述:

建議方案:

// 外出函數 public void goOut() {// 判斷是否周末// 判斷是否周末: 是周末則游玩if (isWeekday()) {play();}// 判斷是否周末: 非周末則工作else {work();} }// 游玩函數 private void play() {// 游玩代碼幾十行 }// 工作函數 private void work() {// 工作代碼幾十行 }

3.4?主要收益

  • 函數越短小精悍,功能就越單一,往往生命周期較長;

  • 一個函數越長,就越不容易理解和維護,維護人員不敢輕易修改;

  • 在過長函數中,往往含有難以發現的重復代碼。

4.

同一函數內代碼塊級別盡量一致


4.1?案例一

現象描述:

很明顯,睡覺這塊代碼塊,跟eat(吃飯)和code(編碼)不在同一級別上,顯得比較突兀。如果把寫代碼比作寫文章,eat(吃飯)和code(編碼)是段落大意,而睡覺這塊代碼塊屬于一個詳細段落。而在liveDaily(每日生活)這個函數上,只需要寫出主要流程(段落大意)即可。

建議方案:

public void liveDaily() {// 吃飯eat();// 編碼code();// 睡覺sleep(); }// 睡覺 private void sleep() {// 睡覺相關代碼 }

4.2?主要收益

  • 函數調用表明用途,函數實現表達邏輯,層次分明便于理解;

  • 不用層次的代碼塊放在一個函數中,容易讓人覺得代碼頭重腳輕。

5.

封裝相同功能代碼為函數


5.1 案例一:封裝相同代碼為函數

現象描述:

建議方案:

// 禁用用戶函數 public void disableUser() {// 禁用黑名單用戶List<Long> userIdList = queryBlackUser();for (Long userId : userIdList) {disableUser(userId);}// 禁用過期用戶userIdList = queryExpiredUser();for (Long userId : userIdList) {disableUser(userId);} }// 禁用用戶函數 private void disableUser(Long userId) {User userUpdate = new User();userUpdate.setId(userId);userUpdate.setEnable(Boolean.FALSE);userDAO.update(userUpdate); }

5.2 案例二:封裝相似代碼為函數

封裝相似代碼為函數,差異性通過函數參數控制。

現象描述:

建議方案:

// 通過工單函數 public void adoptOrder(Long orderId) {auditOrder(orderId, OrderStatus.ADOPTED); }// 駁回工單函數 public void rejectOrder(Long orderId) {auditOrder(orderId, OrderStatus.REJECTED); }// 審核工單函數 private void auditOrder(Long orderId, OrderStatus orderStatus) {Order orderUpdate = new Order();orderUpdate.setId(orderId);orderUpdate.setStatus(orderStatus);orderUpdate.setAuditTime(new Date());orderDAO.update(orderUpdate); }

5.3?主要收益

  • 封裝公共函數,減少代碼行數,提高代碼質量;

  • 封裝公共函數,使業務代碼更精煉,可讀性可維護性更強。

6.

封裝獲取參數值函數


6.1?案例一

現象描述:

建議方案:

// 是否通過函數 public boolean isPassed(Long userId) {// 獲取通過閾值double thisPassThreshold = getPassThreshold();// 獲取通過率double passRate = getPassRate(userId);// 判讀是否通過return passRate >= thisPassThreshold; }// 獲取通過閾值函數 private double getPassThreshold() {if (Objects.nonNull(passThreshold)) {return passThreshold;}return PASS_THRESHOLD; }

6.2?主要收益

  • 把獲取參數值從業務函數中獨立,使業務邏輯更清晰;

  • 封裝的獲取參數值為獨立函數,可以在代碼中重復使用。

7.

通過接口參數化封裝相同邏輯


7.1?案例一

現象描述:

建議方案:

// 發送審核員結算數據函數 public void sendAuditorSettleData() {sendWorkerSettleData(WorkerPushDataType.AUDITOR, () -> auditTaskDAO.statAuditorSettleData()); }// 發送驗收員結算數據函數 public void sendCheckerSettleData() {sendWorkerSettleData(WorkerPushDataType.CHECKER, () -> auditTaskDAO.statCheckerSettleData()); }// 發送作業員結算數據函數 public void sendWorkerSettleData(WorkerPushDataType dataType, WorkerSettleDataProvider dataProvider) {List<WorkerSettleData> settleDataList = dataProvider.statWorkerSettleData();for (WorkerSettleData settleData : settleDataList) {WorkerPushData pushData = new WorkerPushData();pushData.setId(settleData.getWorkerId());pushData.setType(dataType);pushData.setData(settleData);pushService.push(pushData);} }// 作業員結算數據提供者接口 private interface WorkerSettleDataProvider {// 統計作業員結算數據public List<WorkerSettleData> statWorkerSettleData(); }

7.2?主要收益

  • 把核心邏輯從各個業務函數中抽析,使業務代碼更清晰更易維護;

  • 避免重復性代碼多次編寫,精簡重復函數越多收益越大。

8.

減少函數代碼層級


如果要使函數優美,建議函數代碼層級在1-4之間,過多的縮進會讓函數難以閱讀。

8.1 案例一:利用return提前返回函數

現象描述:

建議方案:

// 獲取用戶余額函數 public Double getUserBalance(Long userId) {// 獲取用戶信息User user = getUser(userId);if (Objects.isNull(user)) {return null;}// 獲取用戶賬戶UserAccount account = user.getAccount();if (Objects.isNull(account)) {return null;}// 返回賬戶余額return account.getBalance(); }

8.2 案例二:利用continue提前結束循環

現象描述:

建議方案:

// 獲取合計余額函數 public double getTotalBalance(List<User> userList) {// 初始合計余額double totalBalance = 0.0D;// 依次累加余額for (User user : userList) {// 獲取用戶賬戶UserAccount account = user.getAccount();if (Objects.isNull(account)) {continue;}// 累加用戶余額Double balance = account.getBalance();if (Objects.nonNull(balance)) {totalBalance += balance;}}// 返回合計余額return totalBalance; }

特殊說明

其它方式:在循環體中,先調用案例1的函數getUserBalance(獲取用戶余額),再進行對余額進行累加。

在循環體中,建議最多使用一次continue。如果需要有使用多次continue的需求,建議把循環體封裝為一個函數。

8.3 案例三:利用條件表達式函數減少層級

請參考下一章的"案例2: 把復雜條件表達式封裝為函數"

8.4?主要收益

  • 代碼層級減少,代碼縮進減少;

  • 模塊劃分清晰,方便閱讀維護。

9.

封裝條件表達式函數


9.1 案例一:把簡單條件表達式封裝為函數

現象描述:

建議方案:

// 獲取門票價格函數 public double getTicketPrice(Date currDate) {if (isDiscountDate(currDate)) {return TICKET_PRICE * DISCOUNT_RATE;}return TICKET_PRICE; }// 是否折扣日期函數 private static boolean isDiscountDate(Date currDate) {return Objects.nonNull(currDate) && currDate.after(DISCOUNT_BEGIN_DATE)&& currDate.before(DISCOUNT_END_DATE); }

9.2 案例二:把復雜條件表達式封裝為函數

現象描述:

建議方案:

// 獲取土豪用戶列表 public List<User> getRichUserList(List<User> userList) {// 初始土豪用戶列表List<User> richUserList = new ArrayList<>();// 依次查找土豪用戶for (User user : userList) {// 判斷土豪用戶if (isRichUser(user)) {// 添加土豪用戶richUserList.add(user);}}// 返回土豪用戶列表return richUserList; }// 是否土豪用戶 private boolean isRichUser(User user) {// 獲取用戶賬戶UserAccount account = user.getAccount();if (Objects.isNull(account)) {return false;}// 獲取用戶余額Double balance = account.getBalance();if (Objects.isNull(balance)) {return false;}// 比較用戶余額return balance.compareTo(RICH_THRESHOLD) >= 0; }

以上代碼也可以用采用流式(Stream)編程的過濾來實現。

9.3?主要收益

  • 把條件表達式從業務函數中獨立,使業務邏輯更清晰;

  • 封裝的條件表達式為獨立函數,可以在代碼中重復使用。

10.

盡量避免不必要的空指針判斷


本章只適用于項目內部代碼,并且是自己了解的代碼,才能夠盡量避免不必要的空指針判斷。對于第三方中間件和系統接口,必須做好空指針判斷,以保證代碼的健壯性。

10.1 案例一:調用函數保證參數不為空,被調用函數盡量避免不必要的空指針判斷

現象描述:

建議方案:

// 創建用戶信息 User user = new User(); ... // 賦值用戶相關信息 createUser(user);// 創建用戶函數 private void createUser(User user){// 創建用戶信息userDAO.insert(user);userRedis.save(user); }

10.2 案例二:被調用函數保證返回不為空,調用函數盡量避免不必要的空指針判斷

現象描述:

建議方案:

// 保存用戶函數 public void saveUser(Long id, String name) {// 構建用戶信息User user = buildUser(id, name);// 保存用戶信息userDAO.insert(user);userRedis.save(user); }// 構建用戶函數 private User buildUser(Long id, String name) {User user = new User();user.setId(id);user.setName(name);return user; }

10.3 案例三:賦值邏輯保證列表數據項不為空,處理邏輯盡量避免不必要的空指針判斷

現象描述:

建議方案:

// 查詢用戶列表 List<UserDO> userList = userDAO.queryAll(); if (CollectionUtils.isEmpty(userList)) {return; }// 轉化用戶列表 List<UserVO> userVoList = new ArrayList<>(userList.size()); for (UserDO user : userList) {UserVO userVo = new UserVO();userVo.setId(user.getId());userVo.setName(user.getName());userVoList.add(userVo); }// 依次處理用戶 for (UserVO userVo : userVoList) {// 處理相關邏輯... }

10.4 案例四:MyBatis查詢函數返回列表和數據項不為空,可以不用空指針判斷

MyBatis是一款優秀的持久層框架,是在項目中使用的最廣泛的數據庫中間件之一。通過對MyBatis源碼進行分析,查詢函數返回的列表和數據項都不為空,在代碼中可以不用進行空指針判斷。

現象描述:

這種寫法沒有問題,只是過于保守了。

建議方案:

// 查詢用戶函數 public List<UserVO> queryUser(Long id, String name) {// 查詢用戶列表List<UserDO> userList = userDAO.query(id, name);// 轉化用戶列表List<UserVO> voList = new ArrayList<>(userList.size());for (UserDO user : userList) {UserVO vo = new UserVO();BeanUtils.copyProperties(user, vo);voList.add(vo);}// 返回用戶列表return voList; }

10.5?主要收益

  • 避免不必要的空指針判斷,精簡業務代碼處理邏輯,提高業務代碼運行效率;

  • 這些不必要的空指針判斷,基本屬于永遠不執行的Death代碼,刪除有助于代碼維護

11.

內部函數參數盡量使用基礎類型


11.1 案例一:內部函數參數盡量使用基礎類型

現象描述:

建議方案:

// 調用代碼 double price = 5.1D; int number = 9; double total = calculate(price, number);// 計算金額函數 private double calculate(double price, int number) {return price * number; }

11.2 案例二:內部函數返回值盡量使用基礎類型

現象描述:

建議方案:

// 獲取訂單總額函數 public double getOrderAmount(List<Product> productList) {double amount = 0.0D;for (Product product : productList) {if (Objects.isNull(product) || Objects.isNull(product.getPrice())|| Objects.isNull(product.getNumber())) {continue;}amount += calculate(product.getPrice(), product.getNumber());}return amount; }// 計算金額函數 private double calculate(double price, double number) {return price * number; }

此處只是舉例說明這種現象,更好的方式是采用流式(Stream)編程。

11.3?主要收益

  • 內部函數盡量使用基礎類型,避免了隱式封裝類型的打包和拆包;

  • 內部函數參數使用基礎類型,用語法上避免了內部函數的參數空指針判斷;

  • 內部函數返回值使用基礎類型,用語法上避免了調用函數的返回值空指針判斷。

12.

盡量避免返回的數組和列表為null


12.1 案例一:盡量避免返回的數組為null,引起不必要的空指針判斷

現象描述:

建議方案:

// 調用代碼 UserVO[] users = queryUser(); for (UserVO user : users) {// 處理用戶信息 }// 查詢用戶函數 private UserVO[] queryUser() {// 查詢用戶列表List<UserDO> userList = userDAO.queryAll();if (CollectionUtils.isEmpty(userList)) {return new UserVO[0];}// 轉化用戶數組UserVO[] users = new UserVO[userList.size()];for (int i = 0; i < userList.size(); i++) {UserDO user = userList.get(i);users[i] = new UserVO();users[i].setId(user.getId());users[i].setName(user.getName());}// 返回用戶數組return users; }

12.2 案例二:盡量避免返回的列表為null,引起不必要的空指針判斷

現象描述:

建議方案:

// 調用代碼 List<UserVO> userList = queryUser(); for (UserVO user : userList) {// 處理用戶信息}// 查詢用戶函數 private List<UserVO> queryUser(){// 查詢用戶列表List<UserDO> userList = userDAO.queryAll();if(CollectionUtils.isEmpty(userList)) {return Collections.emptyList();}// 轉化用戶列表List<UserVO> userVoList = new ArrayList<>(userList.size());for(UserDO user : userList) {UserVO userVo = new UserVO();userVo.setId(user.getId());userVo.setName(user.getName());userVoList.add(userVo);}// 返回用戶列表return userVoList; }

12.3?主要收益

  • 保證返回的數組和列表不為null, 避免調用函數的空指針判斷。

13.

封裝函數傳入參數


13.1 案例一:當傳入參數過多時,應封裝為參數類

Java規范不允許函數參數太多,不便于維護也不便于擴展。

現象描述:

建議方案:

// 修改用戶函數 public void modifyUser(User user) {// 具體實現內容 }// 用戶類 @Getter @Setter @ToString private class User{private Long id;private String name;private String phone;private Integer age;private Integer sex;private String address;private String description; }

13.2 案例二:當傳入成組參數時,應封裝為參數類

既然參數成組出現,就需要封裝一個類去描述這種現象。

現象描述:

// 獲取距離函數 public double getDistance(double x1, double y1, double x2, double y2) {// 具體實現邏輯 }

建議方案:

// 獲取距離函數 public double getDistance(Point point1, Point point2) {// 具體實現邏輯 }// 點類 @Getter @Setter @ToString private class Point{private double x;private double y; }

13.3?主要收益

  • 封裝過多函數參數為類,使函數更便于擴展和維護;

  • 封裝成組函數參數為類,使業務概念更明確更清晰。

14.

盡量用函數替換匿名內部類的實現


Java匿名內部類的優缺點:

首先推薦用Lambda表達式簡化匿名內部類,其次推薦用函數替換復雜的Lambda表達式的實現。

14.1 案例一:盡量用函數替換匿名內部類(包括Lambda表達式)的實現

現象描述:

// 發送結算數據 sendWorkerSettleData(WorkerPushDataType.CHECKER, () -> {Date beginDate = DateUtils.addDays(currDate, -aheadDays);Date endDate = DateUtils.addDays(currDate, 1);return auditTaskDAO.statCheckerSettleData(beginDate, endDate); });

建議方案:

// 發送結算數據 sendWorkerSettleData(WorkerPushDataType.CHECKER, () -> statCheckerSettleData(currDate, aheadDays));// 統計驗收員結算數據函數 private List<WorkerSettleData> statCheckerSettleData(Date currDate, int aheadDays) {Date beginDate = DateUtils.addDays(currDate, -aheadDays);Date endDate = DateUtils.addDays(currDate, 1);return auditTaskDAO.statCheckerSettleData(beginDate, endDate); }

其實,還有一個更簡單的辦法。在調用函數sendWorkerSettleData(發送作業員結算數據)之前計算開始日期、結束日期,就直接可以用函數auditTaskDAO.statCheckerSettleData(beginDate, endDate)代替匿名內部類實現。

14.2 案例二:拆分復雜匿名內部類實現接口為多個函數類接口

如果一個匿名內部類實現的接口幾個函數間關聯性不大,可以把這個接口拆分為幾個函數式接口,便于使用Lambda表達式。

現象描述:

// 清除過期數據 cleanExpiredData("用戶日志表", new CleanExpiredDataOperator() {@Overridepublic List<Date> queryExpiredDate(Integer remainDays) {return userDAO.queryExpiredDate(remainDays);}@Overridepublic void cleanExpiredData(Date expiredDate) {userDAO.cleanExpiredData(expiredDate);} });// 清除過期數據函數 private void cleanExpiredData(String tableName, CleanExpiredDataOperator , cleanExpiredDataOperator) {// 功能實現代碼 }// 清除過期操作接口 interface CleanExpiredDataOperator {// 查詢過期日期public List<Date> queryExpiredDate(Integer remainDays);// 清除過期數據public void cleanExpiredData(Date expiredDate); }

建議方案:

// 清除過期數據 cleanExpiredData("用戶日志表", userDAO::queryExpiredDate,userDAO::cleanExpiredData);// 清除過期數據函數 private void cleanExpiredData(String tableName, QueryExpiredDateOperator queryExpiredDateOperator, CleanExpiredDataOperator cleanExpiredDataOperator) {// 功能實現代碼 }// 查詢過期日期接口 interface QueryExpiredDateOperator {// 查詢過期日期public List<Date> queryExpiredDate(Integer remainDays); }// 清除過期操作接口 interface CleanExpiredDataOperator {// 清除過期數據public void cleanExpiredData(Date expiredDate); }

14.3?主要收益

  • 定義函數并指定參數,明確規定了匿名內部類的代碼邊界;

  • 利用Lambda表達式簡化匿名內部類實現,使代碼更簡潔。

15.

利用return精簡不必要的代碼


15.1 案例一:刪除不必要的if

現象描述:

建議方案:

// 是否通過函數 public boolean isPassed(Double passRate) {return Objects.nonNull(passRate) && passRate.compareTo(PASS_THRESHOLD) >= 0; }

15.2 案例二:刪除不必要的else

現象描述:

建議方案:

// 結算工資函數 public double settleSalary(Long workId, int workDays) {// 根據是否合格處理if (isQualified(workId)) {return settleQualifiedSalary(workDays);}return settleUnqualifiedSalary(workDays); }

15.3 案例三:刪除不必要的變量

現象描述:

// 查詢用戶函數 public List<UserDO> queryUser(Long id, String name) {UserQuery userQuery = new UserQuery();userQuery.setId(id);userQuery.setName(name);List<UserDO> userList = userDAO.query(userQuery);return userList; }

建議方案:

// 查詢用戶函數 public List<UserDO> queryUser(Long id, String name) {UserQuery userQuery = new UserQuery();userQuery.setId(id);userQuery.setName(name);return userDAO.query(userQuery); }

15.4?主要收益

  • 精簡不必要的代碼,讓代碼看起來更清爽

16.

利用臨時變量優化代碼


在一些代碼中,經常會看到a.getB().getC()...getN()的寫法,姑且叫做“函數的級聯調用”,代碼健壯性和可讀性太差。建議:杜絕函數的級聯調用,利用臨時變量進行拆分,并做好對象空指針檢查。

16.1 案例一:利用臨時變量厘清邏輯

現象描述:

// 是否土豪用戶函數 private boolean isRichUser(User user) {return Objects.nonNull(user.getAccount())&& Objects.nonNull(user.getAccount().getBalance())&& user.getAccount().getBalance().compareTo(RICH_THRESHOLD) >= 0; }

這是精簡代碼控的最愛,但是可讀性實在太差。

建議方案:

// 是否土豪用戶函數 private boolean isRichUser(User user) {// 獲取用戶賬戶UserAccount account = user.getAccount();if (Objects.isNull(account)) {return false;}// 獲取用戶余額Double balance = account.getBalance();if (Objects.isNull(balance)) {return false;}// 比較用戶余額return balance.compareTo(RICH_THRESHOLD) >= 0; }

這個方案,增加了代碼行數,但是邏輯更清晰。

16.2 案例二:利用臨時變量精簡代碼

現象描述:

// 構建用戶函數 public UserVO buildUser(UserDO user) {UserVO vo = new UserVO();vo.setId(user.getId());vo.setName(user.getName());if (Objects.nonNull(user.getAccount())) {vo.setBalance(user.getAccount().getBalance());vo.setDebt(user.getAccount().getDebt());}return vo; }

這么寫,大約是為了節約一個臨時變量把。

建議方案:

// 構建用戶函數 public UserVO buildUser1(UserDO user) {UserVO vo = new UserVO();vo.setId(user.getId());vo.setName(user.getName());UserAccount account = user.getAccount();if (Objects.nonNull(account)) {vo.setBalance(account.getBalance());vo.setDebt(account.getDebt());}return vo; }

16.3?主要收益

  • 利用臨時變量厘清邏輯,顯得業務邏輯更清晰;

  • 利用臨時變量精簡代碼,看變量名稱即知其義,減少了大量無用代碼;

  • 如果獲取函數比較復雜耗時,利用臨時變量可以提高運行效率;

  • 利用臨時變量避免函數的級聯調用,可有效預防空指針異常。

17.

僅保留函數需要的參數


在一些代碼中,經常會看到a.getB().getC()...getN()的寫法,姑且叫做“函數的級聯調用”,代碼健壯性和可讀性太差。建議:杜絕函數的級聯調用,利用臨時變量進行拆分,并做好對象空指針檢查。

17.1 案例一:刪除多余的參數

現象描述:

// 修改用戶狀態函數 private void modifyUserStatus(Long userId, Integer status, String unused) {userCache.modifyStatus(userId, status);userDAO.modifyStatus(userId, status); }

其中,unused參數是無用參數。

建議方案:

// 修改用戶狀態函數 private void modifyUserStatus(Long userId, Integer status) {userCache.modifyStatus(userId, status);userDAO.modifyStatus(userId, status); }

17.2 案例二:用屬性取代對象

現象描述:

// 刪除用戶函數 private void deleteUser(User user) {userCache.delete(user.getId());userDAO.delete(user.getId()); }

建議方案:

// 刪除用戶函數 private void deleteUser(Long userId) {userCache.delete(userId);userDAO.delete(userId); }

建議方案:

調用函數時,參數對象不需要專門構建,而函數使用其屬性超過3個,可以不必使用該規則。

17.3?主要收益

  • 僅保留函數需要的參數,明確了調用時需要賦值的參數,避免了調用時還要去構造些無用參數。

18.

后記


"眾人拾柴火焰高"。如果有更多更好的觀點,亦或有更好的代碼案例,歡迎大家進行補充說明。

筆者希望以此文拋磚引玉,如果最終形成一套完善的Java編碼規范,善莫大焉。

想知道更多?描下面的二維碼關注我


加技術群入口(備注:Tech):>>>Learn More<<

免費資料入口(備注:2048):>>>Learn More<<

免費星球入口:>>>Free<<<

內推通道>>>>


今天開始到9月7日,當當開學季促銷,每滿100-50,實付300還可以用我的優惠碼再減50,相當于250買600的書,支持全品類。結算的時候用優惠碼?TMWCP4?即可。

點個"在看"唄^_^

總結

以上是生活随笔為你收集整理的这代码写的跟狗屎一样!怎么优化?的全部內容,希望文章能夠幫你解決所遇到的問題。

如果覺得生活随笔網站內容還不錯,歡迎將生活随笔推薦給好友。