规范代码之方法重用

今天在进行服务器迁移的时候需要将之前的代码用两种方式实现。

具体的业务场景为：

之前的调用方调用一个http接口获取服务，但是现在这个http接口需要废掉了。

而这个http服务分布在不同的主机上，

所以调用方的写法是类似于下面这样：

public Map<Long, Integer> getServiceUsersState(List<Long> serviceUserIds, String host)；

其中host为传入的服务所在的域名，当然，前面还有一个网关负责服务的分发。

所以我要做的就是将所有带host的接口找出来，写出一个不带host的版本。

然后将所有调用带host的接口的方法，按照逻辑分为两种调用，

即如果是调到废掉的服务器上的方法走不带host的，如果不是则继续走原先的方法。

所以我写了一个没有host的版本。不调用原先的http接口。

public Map<Long, Integer> getServiceUsersStateNoHost(List<Long> serviceUserIds)；

那么问题来了，有如下类似代码：

public Result switchService(long userId, long shopId, long curServiceUserId, long nextServiceUserId, String host, PlatformEnum type) {
   //大量代码

         splitServiceUserIds = getSplitServiceUserIds(shopId, host)；

    //大量代码 
}

private Map<String, List<Long>> getSplitServiceUserIds(long shopId, String host){

    //大量代码
            serviceStateMap = sellerServiceBiz.getServiceUsersState(totalServiceUserIds,host);
     
     //大量代码
        
}
这里switchService里面有个PlatformEnum可以判断需要调用有host的还是没有host的方法。
方法1：
switchService中加一个

           if (PlatformEnum.type1!= type) {
                splitServiceUserIds = getSplitServiceUserIds(shopId, host);
            } else {
                splitServiceUserIds = getSplitServiceUserIdsNoHost(shopId);
            }
这样的判断，然后在写一个getSplitServiceUserIdsNoHost方法，方法里面调用的是getServiceUsersStateNoHost。
这样也可以完成目标但是就有了大量的代码的重复。
而我想的是是否可以重写getSplitServiceUserIds这个方法，让他可以判断出如果是不需要host版本的就调用getServiceUsersStateNoHost
反之，则调用getServiceUsersState。这样就需要注入一个type属性，需要改写switchService里面的getSplitServiceUserIds。不符合开闭原则。
所以我就在getSplitServiceUserIds里面自己又包了一层。加了个type字段，同时写了个getSplitServiceUserIdsNoHost也加了个字段。利用这个字段区分，则不具备侵入性了。
伪代码如下：

private Map<String, List<Long>> getSplitServiceUserIdsNoHost(long shopId) {
//switchService里面判断是没有host版本调用这个，然后他出入type=true

    boolean type = true;
    String host=null;
    return getSplitServiceUserIds(type,shopId,host);

}

private Map<String, List<Long>> getSplitServiceUserIds(long shopId, String host){

//switchService里面判断是有host版本调用这个，然后他出入type=false

    boolean type = false;
    return getSplitServiceUserIds(type,shopId,host);

}


private Map<String, List<Long>> getSplitServiceUserIds(boolean type,long shopId, String host){

        //根据传入的type调用相应的host或者nohost
        if(type == true) {
            serviceStateMap = sellerServiceBiz.getServiceUsersStateNoHost(totalServiceUserIds);
        }else {
            serviceStateMap = sellerServiceBiz.getServiceUsersState(totalServiceUserIds,host);
        }

}

这样的做法减少了代码的侵入性，同时完成了业务逻辑的判断，只改了自己需要的部分，不影响别人的调用。
话说的很乱。0-0  等我组织好语言在来更新这篇博客。PS:一般是更新不了了。。。。

补一张随缘图（有缘者能看懂）：