优化这段代码if("1".equals(userFlag)){ commonUse.setPlanterOnlyKey(UUID.randomUUID().toString().replaceAll("-","")); int res = planterDao.insert(commonUse); if(res > 0){ CommonUse commonUse1 = planterDao.viewByPlanterId(commonUse); sdk(commonUse1.getPlanterOnlyKey(),commonUse.getUserId()); } return res; }else if("2".equals(userFlag)){ result = processDao.insert(commonUse); if(result>0){ sdk(key,userId); } return result; }else if("3".equals(userFlag)){ result = logisticsDao.insert(commonUse); if(result>0){ sdk(key,userId); } return result; }else if ("4".equals(userFlag)){ result = saleDao.insert(commonUse); if(result>0){ sdk(key,userId); } return result; }
时间: 2024-02-14 15:11:29 浏览: 27
根据这段代码的结构,我们可以尝试将其拆分成多个方法,以提高代码的可读性和可维护性。具体优化方式如下:
1. 将 `if` 判断的部分抽取成多个方法,每个方法负责处理一个业务逻辑,这样可以减少代码的嵌套层次,提高代码的可读性。
2. 将一些重复的代码进行提取,比如 `sdk` 方法可以抽取出来,避免代码的重复。
3. 将 `UUID.randomUUID().toString().replaceAll("-","")` 这段代码提取出来,避免多次重复调用。
通过以上优化,可以得到如下的代码示例:
```
public int insertCommonUse(CommonUse commonUse, String userFlag, String userId) {
int result = 0;
String key = UUID.randomUUID().toString().replaceAll("-","");
if ("1".equals(userFlag)) {
result = insertPlanter(commonUse, key);
} else if ("2".equals(userFlag)) {
result = processDao.insert(commonUse);
} else if ("3".equals(userFlag)) {
result = logisticsDao.insert(commonUse);
} else if ("4".equals(userFlag)) {
result = saleDao.insert(commonUse);
}
if (result > 0) {
sdk(key, userId);
}
return result;
}
private int insertPlanter(CommonUse commonUse, String key) {
commonUse.setPlanterOnlyKey(key);
int res = planterDao.insert(commonUse);
if (res > 0) {
CommonUse commonUse1 = planterDao.viewByPlanterId(commonUse);
sdk(commonUse1.getPlanterOnlyKey(), commonUse.getUserId());
}
return res;
}
private void sdk(String key, String userId) {
// 处理sdk逻辑
}
```
通过以上优化,代码结构更加清晰,可读性和可维护性也得到了提高。
相关推荐
![pdf](https://img-home.csdnimg.cn/images/20210720083512.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)
![](https://csdnimg.cn/download_wenku/file_type_ask_c1.png)