用1个月重构了同事写的烂代码，我总结出了15条重写烂代码的经验！

青旅半醒 2023-09-23 15:29 144阅读 0赞

## 1.注释尽可能全面,写有意义的注释 ##

接口方法、类、复杂的业务逻辑，都应该添加有意义的注释

*  对于接口方法的注释，应该包含详细的入参和结果说明，有异常抛出的情况也要详细叙述
 *  类的注释应该包含类的功能说明、作者和修改者。
 *  如果是业务逻辑很复杂的代码，真的非常有必要写清楚注释。

清楚的注释，更有利于后面的维护。

![77a6d7ca3ddbb5f9dbe1387b2f68a772.png][]

## 2.项目拆分合理的目录结构 ##

记得读大学那会，刚学做各种各样的管理系统，都是用`MVC`模式，也就是`controller、service、mapper、entity`。如果未来业务扩展，你没有拆分业务结构的话，很可能就会发现，一个`service`包下，有上百个服务。。。

正确的做法，如果服务过多，应该根据不同的业务进行划分，比如订单、登陆、积分等等

![a3ec1300e0ef97f4aebc7a78e55dc3c3.png][]

当然，你也可以根据不同的业务划分模块，比如建一个`moudles`包，然后按订单、登陆等业务划分，每个业务都有自己的`controller、service、mapper、entity`。

我们拆分的目的，就是**让项目结构更清晰，可读性更强，更容易维护**而已。

## 3. 不在循环里远程调用、或者数据库操作，优先考虑批量进行。 ##

远程操作或者数据库操作都是**比较耗网络、IO资源**的，所以尽量不在循环里远程调用、不在循环里操作数据库，能**批量一次性查回来尽量不要循环多次去查**。（但是呢，如果是操作数据库，也不要一次性查太多数据哈，可以分批500一次酱紫）。

正例：

remoteBatchQuery(param);

反例：

for(int i=0;i<n;i++){
      remoteSingleQuery(param)
    }

![bfe18257bf24a24a6e9f2658e0a9bf59.png][]

## 4. 封装方法形参 ##

如果你的方法参数过多，要封装一个对象出来。反例如下：

public void getUserInfo（String name,String age,String sex,String mobile,String idNo){
      // do something ...
    }

如果参数很多，做新老接口兼容处理也比较麻烦。建议写个对象出来，如下：

public void getUserInfo（UserInfoParamDTO userInfoParamDTO){
      // do something ...
    }
    
    class UserInfoParamDTO{
      private String name;
      private String age; 
      private String sex;
      private String mobile;
      private String idNo;
    }

## 5. 封装通用模板 ##

一个优秀的后端开发，应该具备**封装通用模板**的编码能力。

我们来看一个业务需求：假设我们有这么一个业务场景：内部系统不同商户，调用我们系统接口，去跟外部第三方系统交互（http方式）。走类似这么一个流程，如下：

![33bdd7325b7b3b020514ee7f8f1c81d2.png][]

一个请求都会经历这几个流程：

*  查询商户信息
 *  对请求报文加签
 *  发送http请求出去
 *  对返回的报文验签

通过HTTP发请求出去时，有的商户可能是**走代理**的，有的是走直连。假设当前有A，B商户接入，不少伙伴可能这么实现，伪代码如下：

// 商户A处理句柄
    CompanyAHandler implements RequestHandler {
       Resp hander(req){
       //查询商户信息
       queryMerchantInfo();
       //加签
       signature();
       //http请求（A商户假设走的是代理）
       httpRequestbyProxy()
       //验签
       verify();
       }
    }
    // 商户B处理句柄
    CompanyBHandler implements RequestHandler {
       Resp hander(Rreq){
       //查询商户信息
       queryMerchantInfo();
       //加签
       signature();
       // http请求（B商户不走代理，直连）
       httpRequestbyDirect();
       // 验签
       verify(); 
       }
    }

假设新加一个C商户接入，你需要再实现一套这样的代码。显然，这样代码就重复了。这时候我们可以**封装一个通用模板**！我们就可以定义一个抽象类，包含请求流程的几个方法，伪代码如下：

abstract class AbstractMerchantService  { 
    
         //模板方法流程
         Resp handlerTempPlate(req){
               //查询商户信息
               queryMerchantInfo();
               //加签
               signature();
               //http 请求
               httpRequest();
               // 验签
               verifySinature();
         }
          // Http是否走代理（提供给子类实现）
          abstract boolean isRequestByProxy();
    }

然后所有商户接入，都做这个流程。如果这个通用模板是你抽取的，别的小伙伴接到开发任务，都是接入你的模板，是不是会有点自豪呀，哈哈~

**封装通用模板**，就是抽个模板模式嘛？其实不仅仅是，而是自己对**需求、代码的思考与总结**，一种**编程思想的升华**。

## 6. 封装复杂的逻辑判断条件 ##

我们来看下这段代码：

public void test(UserStatus userStatus){
        if (userStatus != UserStatus.BANNED && userStatus != UserStatus.DELETED && userStatus != UserStatus.FROZEN) {
            //doSomeThing
            return
        }
    }

这段代码有什么问题呢？是的，**逻辑判断条件太复杂啦，我们可以封装一下它**。如下：

public void test(UserStatus userStatus){
        if (isUserActive(userStatus)) {
            //doSomeThing
        }
    }
    
    private boolean isUserActive(UserStatus userStatus) {
        return userStatus != UserStatus.BANNED && userStatus != UserStatus.DELETED && userStatus != UserStatus.FROZEN;
    }

## 7. 保持优化性能的嗅觉 ##

优秀的后端开发，应该保持优化性能的嗅觉。比如`避免创建比必要的对象、异步处理、使用缓冲流，减少IO操作`等等。

比如，我们设计一个`APP`首页的接口，它需要`查用户信息、需要查banner信息、需要查弹窗信息`等等。假设耗时如下：

![148c2044f37634f1973b1ba321a640c9.png][]

查用户信息`200`ms，查banner信息`100`ms、查弹窗信息`50`ms，那一共就耗时`350`ms了。如果还查其他信息，那耗时就更大了。如何优化它呢？可以并行发起，耗时可以降为`200`ms。如下：

![6b3d1a868969332d6a0811e59591e4e7.png][]

之前我写过一篇后端思维的文章，手把手教大家如何抽并行调用框架，大家可以看下：[后端思维篇：手把手教你写一个并行调用模板][Link 1]

## 8. 可变参数的配置化处理 ##

日常开发中，我们经常会遇到一些可变参数，比如`用户多少天没登录注销`、`运营活动，不同节日红包皮肤切换、订单多久没付款就删除`等等。对于这些可变的参数，不用该直接写死在代码。优秀的后端，要做配置化处理，你可以把这些可变参数，放到数据库一个配置表里面，也可以放到项目的配置文件或者`apollo`上。

比如产品经理提了个红包需求，圣诞节的时候，红包皮肤为圣诞节相关的，春节的时候，为春节红包皮肤等。如果在代码写死控制，可有类似以下代码：

if(duringChristmas){
       img = redPacketChristmasSkin;
    }else if(duringSpringFestival){
       img =  redSpringFestivalSkin;
    }

如果到了元宵节的时候，运营小姐姐突然又有想法，红包皮肤换成灯笼相关的，这时候，是不是要去修改代码了，重新发布了？

> 从一开始接口设计时，可以实现一张**红包皮肤的配置表**，将红包皮肤做成配置化呢？更换红包皮肤，只需修改一下表数据就好了。当然，还有一些场景适合一些配置化的参数：一个分页多少数量控制、某个抢红包多久时间过期这些，都可以搞到参数配置化表里面。这也是扩展性思想的一种体现。

## 9. 会总结并使用工具类。 ##

很多小伙伴，判断一个`list`是否为空，会这么写：

if (list == null || list.size() == 0) {
      return null;
    }

这样写呢，逻辑是没什么问题的。但是更建议用工具类，比如：

if (CollectionUtils.isEmpty(list)) {
       return null;
    }

日常开发中，我们既要会用工具类，更要学会自己去总结工具类。比如去文件处理工具类、日期处理工具类等等。这些都是优秀后端开发的一些好习惯。

## 10. 控制方法函数复杂度 ##

你的方法不要**写得太复杂，逻辑不要混乱，也不要太长**。一个函数不能超过80行。写代码不仅仅是能跑就行，而是为了以后更好的维护。

反例如下：

public class Test {
        private String name;
        private Vector<Order> orders = new Vector<Order>();
    
        public void printOwing() {
            //print banner
            System.out.println("****************");
            System.out.println("*****customer Owes *****");
            System.out.println("****************");
    
            //calculate totalAmount
            Enumeration env = orders.elements();
            double totalAmount = 0.0;
            while (env.hasMoreElements()) {
                Order order = (Order) env.nextElement();
                totalAmount += order.getAmout();
            }
    
            //print details
            System.out.println("name:" + name);
            System.out.println("amount:" + totalAmount);
            ......
        }
    }

其实可以使用`Extract Method`，抽取功能单一的代码段，组成命名清晰的小函数，去解决长函数问题，正例如下：

public class Test {
        private String name;
        private Vector<Order> orders = new Vector<Order>();
    
        public void printOwing() {
    
            //print banner
            printBanner();
            //calculate totalAmount
            double totalAmount = getTotalAmount();
            //print details
            printDetail(totalAmount);
        }
    
        void printBanner(){
            System.out.println("****************");
            System.out.println("*****customer Owes *****");
            System.out.println("****************");
        }
    
        double getTotalAmount(){
            Enumeration env = orders.elements();
            double totalAmount = 0.0;
            while (env.hasMoreElements()) {
                Order order = (Order) env.nextElement();
                totalAmount += order.getAmout();
            }
            return totalAmount;
        }
    
        void printDetail(double totalAmount){
            System.out.println("name:" + name);
            System.out.println("amount:" + totalAmount);
        }   
    }

## 11. 在finally块中对资源进行释放 ##

应该大家都有过这样的经历，`windows`系统桌面如果打开太多文件或者系统软件，就会觉得电脑很卡。当然，我们`linux`服务器也一样，平时操作文件，或者数据库连接，`IO`资源流如果没关闭，那么这个`IO`资源就会被它占着，这样别人就没有办法用了，这就造成资源浪费。

我们操作完文件资源，需要在在finally块中对资源进行释放。

FileInputStream fdIn = null;
    try {
        fdIn = new FileInputStream(new File("/公众号_捡田螺的小男孩.txt"));
    } catch (FileNotFoundException e) {
        log.error(e);
    } catch (IOException e) {
        log.error(e);
    }finally {
        try {
            if (fdIn != null) {
                fdIn.close();
            }
        } catch (IOException e) {
            log.error(e);
        }
    }

## 12.把日志打印好 ##

日常开发中，一定需要把日志打印好。比如：你实现转账业务，转个几百万，然后转失败了，接着客户投诉，然后你还没有打印到日志，想想那种水深火热的困境下，你却毫无办法。。。

一般情况，方法入参、出参需要打印日志，异常的时候，也要打印日志等等，如下：

public void transfer(TransferDTO transferDTO){
        log.info("invoke tranfer begin");
        //打印入参
        log.info("invoke tranfer,paramters:{}",transferDTO);
        try {
          res=  transferService.transfer(transferDTO);
        }catch(Exception e){
         log.error("transfer fail,account：{}",
         transferDTO.getAccount（）)
         log.error("transfer fail,exception:{}",e);
        }
        log.info("invoke tranfer end");
        }

之前写过一篇打印日志的15个建议，大家可以看看哈：[工作总结！日志打印的15个建议][15]

## 13. 考虑异常，处理好异常 ##

优秀的后端开发，应当考虑到异常，并做好异常处理。田螺哥给大家提了10个异常处理的建议：

*  尽量不要使用`e.printStackTrace()`,而是使用`log`打印。因为`e.printStackTrace()`语句可能会导致内存占满。
 *  `catch`住异常时，建议打印出具体的`exception`，利于更好定位问题
 *  不要用一个`Exception`捕捉所有可能的异常
 *  记得使用`finally`关闭流资源或者直接使用`try-with-resource`。
 *  捕获异常与抛出异常必须是完全匹配，或者捕获异常是抛异常的父类
 *  捕获到的异常，不能忽略它，至少打点日志吧
 *  注意异常对你的代码层次结构的侵染
 *  自定义封装异常，不要丢弃原始异常的信息`Throwable cause`
 *  运行时异常`RuntimeException` ，不应该通过`catch`的方式来处理，而是先预检查，比如：`NullPointerException`处理
 *  注意异常匹配的顺序，优先捕获具体的异常

## 14. 考虑系统、接口的兼容性 ##

优秀的后端开发，会考虑系统、接口的兼容性。

如果修改了对外旧接口，但是却不做兼容。这个问题可能比较严重，甚至会直接导致系统发版失败的。新手程序员很容易犯这个错误哦~

![9cc2d686504a2a9fa6e252667b1117b3.png][]

因此，如果你的需求是在原来接口上修改，尤其这个接口是对外提供服务的话，一定要考虑接口兼容。举个例子吧，比如`dubbo`接口，原本是只接收`A，B`参数，现在你加了一个参数`C`，就可以考虑这样处理：

//老接口
    void oldService(A,B){
      //兼容新接口，传个null代替C
      newService(A,B,null);
    }
    
    //新接口，暂时不能删掉老接口，需要做兼容。
    void newService(A,B,C){
      ...
    }

## 15. 采取措施避免运行时错误 ##

优秀的后端开发，应该在编写代码阶段，就采取措施，**避免运行时错误**，如数组边界溢出，被零整除，空指针等运行时错误。类似代码比较常见:

String name = list.get(1).getName(); //list可能越界，因为不一定有2个元素哈

所以，应该采取措施，预防一下数组边界溢出，正例如下：

if(CollectionsUtil.isNotEmpty(list)&& list.size()>1){
      String name = list.get(1).getName(); 
    }

[77a6d7ca3ddbb5f9dbe1387b2f68a772.png]: https://img-blog.csdnimg.cn/img_convert/77a6d7ca3ddbb5f9dbe1387b2f68a772.png
[a3ec1300e0ef97f4aebc7a78e55dc3c3.png]: https://img-blog.csdnimg.cn/img_convert/a3ec1300e0ef97f4aebc7a78e55dc3c3.png
[bfe18257bf24a24a6e9f2658e0a9bf59.png]: https://img-blog.csdnimg.cn/img_convert/bfe18257bf24a24a6e9f2658e0a9bf59.png
[33bdd7325b7b3b020514ee7f8f1c81d2.png]: https://img-blog.csdnimg.cn/img_convert/33bdd7325b7b3b020514ee7f8f1c81d2.png
[148c2044f37634f1973b1ba321a640c9.png]: https://img-blog.csdnimg.cn/img_convert/148c2044f37634f1973b1ba321a640c9.png
[6b3d1a868969332d6a0811e59591e4e7.png]: https://img-blog.csdnimg.cn/img_convert/6b3d1a868969332d6a0811e59591e4e7.png
[Link 1]: https://mp.weixin.qq.com/s?__biz=Mzg3NzU5NTIwNg==&mid=2247499504&idx=1&sn=bb62226e6cffeb1859efb0100c796050&chksm=cf2221d9f855a8cf23f75cb51c1a407578fb0f279e96ddae74b5b8c84f2f5dc71762425b17cb&token=1305910004&lang=zh_CN&scene=21#wechat_redirect
[15]: https://mp.weixin.qq.com/s?__biz=Mzg3NzU5NTIwNg==&mid=2247494838&idx=1&sn=cdb15fd346bddf3f8c1c99f0efbd67d8&chksm=cf22339ff855ba891616c79d4f4855e228e34a9fb45088d7acbe421ad511b8d090a90f5b019f&token=162724582&lang=zh_CN&scene=21#wechat_redirect
[9cc2d686504a2a9fa6e252667b1117b3.png]: https://img-blog.csdnimg.cn/img_convert/9cc2d686504a2a9fa6e252667b1117b3.png