重构这段代码 private void automaticDispatch(JointCase jointCase) { if (EmptyUtils.isNotEmpty(jointCase) && EmptyUtils.isNotEmpty(jointCase.getCaseSmallType())) { BaseHandleTimeDetailsResponse baseHandleTimeDetailsResponse = baseHandleTimeHelper.getBaseHandleTimeDetailsResponse(jointCase.getCaseSmallType()); if (EmptyUtils.isNotEmpty(baseHandleTimeDetailsResponse) && DispatchTypeEnum.AUTOMATIC_DISPATCH_TYPE.getCode().equals(baseHandleTimeDetailsResponse.getDispatchType()) && CollectionTypeEnum.SELF_EXAMINATION_TYPE.getCode() != jointCase.getCollectionType()) { int handleTime = baseHandleTimeDetailsResponse.getHandleDays() * JointCommandConstants.HOURS * JointCommandConstants.MINUTE * JointCommandConstants.SECOND + baseHandleTimeDetailsResponse.getHandleHours() * JointCommandConstants.MINUTE * JointCommandConstants.SECOND + baseHandleTimeDetailsResponse.getHandleMinute() * JointCommandConstants.SECOND; List<String> assistDeptCodes = ClassCastUtils.toList(baseHandleTimeDetailsResponse.getAssistDeptCode(), SymbolConstants.COMMA.getSymbol()); JointCommandCaseDispatchAddRequest dispatchAddRequest = JointCommandCaseDispatchAddRequest.builder().jointCaseId(jointCase.getId()).dispatchRemarks(JointCommandConstants.DISPATCH_SLOGAN) .handleTime(handleTime).mianDeptCode(baseHandleTimeDetailsResponse.getMainDeptCode()).assistDeptCodeList(assistDeptCodes).version(GeneralConstants.NUMBER_ZERO).build(); approvalProcessService.submitFirstDispatch(dispatchAddRequest); } } }

时间: 2024-03-22 07:41:16 浏览: 55
这段代码的主要问题在于可读性较差,存在一些魔法数字和硬编码的部分,同时也有一些过长的方法调用链,影响了代码的可维护性和可扩展性。下面是一个重构后的代码示例: ```java private void automaticDispatch(JointCase jointCase) { if (jointCase == null || jointCase.getCaseSmallType() == null) { return; } BaseHandleTimeDetailsResponse handleTimeDetails = baseHandleTimeHelper.getBaseHandleTimeDetailsResponse(jointCase.getCaseSmallType()); if (handleTimeDetails == null || !DispatchTypeEnum.AUTOMATIC_DISPATCH_TYPE.getCode().equals(handleTimeDetails.getDispatchType())) { return; } if (CollectionTypeEnum.SELF_EXAMINATION_TYPE.getCode() == jointCase.getCollectionType()) { return; } int handleTimeInSeconds = convertHandleTimeToSeconds(handleTimeDetails); List<String> assistDeptCodes = convertAssistDeptCodesToList(handleTimeDetails.getAssistDeptCode()); JointCommandCaseDispatchAddRequest dispatchRequest = JointCommandCaseDispatchAddRequest.builder() .jointCaseId(jointCase.getId()) .dispatchRemarks(JointCommandConstants.DISPATCH_SLOGAN) .handleTime(handleTimeInSeconds) .mianDeptCode(handleTimeDetails.getMainDeptCode()) .assistDeptCodeList(assistDeptCodes) .version(GeneralConstants.NUMBER_ZERO) .build(); approvalProcessService.submitFirstDispatch(dispatchRequest); } private int convertHandleTimeToSeconds(BaseHandleTimeDetailsResponse handleTimeDetails) { int handleDaysInSeconds = handleTimeDetails.getHandleDays() * JointCommandConstants.HOURS * JointCommandConstants.MINUTE * JointCommandConstants.SECOND; int handleHoursInSeconds = handleTimeDetails.getHandleHours() * JointCommandConstants.MINUTE * JointCommandConstants.SECOND; int handleMinutesInSeconds = handleTimeDetails.getHandleMinute() * JointCommandConstants.SECOND; return handleDaysInSeconds + handleHoursInSeconds + handleMinutesInSeconds; } private List<String> convertAssistDeptCodesToList(String assistDeptCodes) { if (StringUtils.isBlank(assistDeptCodes)) { return Collections.emptyList(); } return Arrays.asList(assistDeptCodes.split(SymbolConstants.COMMA.getSymbol())); } ``` 重构后的代码中,我们将原来的多个 if 判断合并成了一个顺序执行的逻辑,这样可以提高代码的可读性。我们还将一些常量提取成了静态变量或方法,这样可以避免硬编码和魔法数字。此外,我们还将一些过长的方法调用链拆分成了多个方法,提高了代码的可维护性和可扩展性。
阅读全文

相关推荐

public AjaxResult importData(MultipartFile file, SysDept sysDept) throws Exception { ExcelUtil<SysDept> util = new ExcelUtil<>(SysDept.class); List<SysDept> sysDeptList = util.importExcel(file.getInputStream(), 1); sysDept.setDeptType(Constants.DEPT_BANK); sysDept.setDelFlag(Constants.STATUS_VALID); List<SysDept> depts = deptService.selectDeptList(sysDept); // 创建机构名称集合 List<String> deptNames = new ArrayList<>(); // 创建机构编号集合 List<String> deptNum = new ArrayList<>(); // 创建父部门编号map Map<String, SysDept> parentNum = new HashMap<>(); for (SysDept dept : depts) { deptNames.add(dept.getDeptName()); deptNum.add(dept.getDeptNum()); parentNum.put(dept.getDeptNum(), dept); } for (SysDept dept : sysDeptList) { if (deptNames.contains(dept.getDeptName()) || deptNum.contains(dept.getDeptNum())) { throw new ServiceException("机构已存在!"); } // 添加父部门id if (parentNum.get(dept.getParentNum()) != null) { dept.setParentId(parentNum.get(dept.getParentNum()).getDeptId()); deptNames.add(dept.getDeptName()); parentNum.put(dept.getDeptNum(), dept); deptNum.add(dept.getDeptNum()); } else { throw new ServiceException("添加" + dept.getDeptName() + "失败!经办机构不存在!"); } dept.setDeptType(Constants.DEPT_BANK); dept.setCreateBy(getUserId()); dept.setStatus(Constants.STATUS_VALID); dept.setDelFlag(Constants.STATUS_VALID); //存储用户信息 SysUser user = new SysUser(); user.setUserName(dept.getUserName()); user.setPassword(SecurityUtils.encryptPassword(dept.getPassword())); user.setPhonenumber(dept.getPhonenumber()); user.setRoleIds(dept.getRoleIds()); user.setUserType(Constants.USER_TYPE_BANK); user.setCreateBy(getUserId()); user.setStatus(Constants.STATUS_VALID); user.setDelFlag(Constants.STATUS_VALID); if (!userService.checkUserNameUnique(user)) { throw new ServiceException(ADD_USER + user.getUserName() + ACCOUNT_ALREADY_EXISTS); } else if (StringUtils.isNotEmpty(user.getPhonenumber()) && !userService.checkPhoneUnique(user)) { throw new ServiceException(ADD_USER + user.getUserName() + PHONE_EXISTS); } userService.insertUser(user); //存储部门信息 dept.setUserId(user.getUserId()); deptService.insertDept(dept); //更新用户信息 user.setDeptId(dept.getDeptId()); userService.updateUser(user); } return success(); }重构这段代码 将其认知复杂度从16降低到允许的15。

String singleSend = (String) map.get("singleSend"); String expressProductCode = (String) map.get("expressProductCode"); LinkedHashMap<String,String> senderInformation = (LinkedHashMap<String,String>)map.get("senderInformation"); String senderName = senderInformation.get("label"); String senderPhone = senderInformation.get("value"); String productName = (String) map.get("productName"); if("1".equals(singleSend)) { for(Map<String, String> oo : list ) { String receiverName = oo.get("receiverName"); String receiverPhone = oo.get("receiverPhone"); String contactAddress = oo.get("contactAddress"); String orderNos = oo.get("orderNo"); OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,"1", orderNos); String ret = apiUtil.createOrder(orderApiCreate); LoginUser loginUser = SecurityUtils.getLoginUser(); OrderInfo orderinfo = new OrderInfo(); orderinfo.setUpdateBy(loginUser.getUsername()); orderinfo.setApiOrderNo(orderApiCreate.getOrderNumber()); if(StringUtils.isNotEmpty(ret)) { orderinfo.setSendError(ret); orderinfo.setSendState("3"); orderinfo.setSendCount(0); } else { orderinfo.setSendState("2"); } orderService.updateApiSendInfo(new String[]{String.valueOf(oo.get("uuid"))}, orderinfo); } } else { String receiverName = (String) map.get("receiverName"); String receiverPhone = (String) map.get("receiverPhone"); String contactAddress = (String) map.get("contactAddress"); String productNumber = String.valueOf(uuids.length); String orderNos = list.stream().map(item->String.valueOf(item.get("orderNo"))).collect(Collectors.joining(",")); OrderApiCreate orderApiCreate = genParam(expressProductCode,productName,senderName,senderPhone,receiverName,receiverPhone,contactAddress,productNumber, orderNos); String ret = apiUtil.createOrder(orderApiCreate); LoginUser loginUser = SecurityUtils.getLoginUser(); OrderInfo orderinfo = new OrderInfo(); orderinfo.setUpdateBy(loginUser.getUsername()); orderinfo.setApiOrderNo(orderApiCreate.getOrderNumber()); if(StringUtils.isNotEmpty(ret)) { orderinfo.setSendError(ret); orderinfo.setSendState("3"); orderinfo.setSendCount(0); } else { orderinfo.setSendState("2"); } orderService.updateApiSendInfo(uuids, orderinfo); } apiUtil.getOrderSendState(); return AjaxResult.success(result);优化这段代码

public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception { if (handler instanceof HandlerMethod) { HandlerMethod handlerMethod = (HandlerMethod) handler; Method method = handlerMethod.getMethod(); RepeatSubmit annotation = method.getAnnotation(RepeatSubmit.class); if (annotation != null && this.isRepeatSubmit(request, annotation)) { AjaxResult ajaxResult = AjaxResult.error(annotation.message()); ServletUtils.renderString(response, JSON.toJSONString(ajaxResult)); return false; } // if (annotation != null) { // if (this.isRepeatSubmit(request, annotation)) { // AjaxResult ajaxResult = AjaxResult.error(annotation.message()); // ServletUtils.renderString(response, JSON.toJSONString(ajaxResult)); // return false; // } // } } boolean flag = false; TokenService tokenService = SpringUtils.getBean(TokenService.class); LoginAppUser loginUser = tokenService.getLoginUser(request); if (loginUser != null && loginUser.getGuest() != null && loginUser.getGuest().getGuestNum() != null) { tokenService.verifyToken(loginUser); String guestNum = loginUser.getGuest().getGuestNum(); HttpSession httpSession = request.getSession(); RedisCache redisUtil = SpringUtils.getBean(RedisCache.class); Guest guest = redisUtil.getCacheMapValue(RedisConstants.GUEST_MAP, guestNum); SysGuest sysGuest = JSON.parseObject(JSON.toJSONString(guest), SysGuest.class); if (sysGuest == null) { flag = false; httpSession.removeAttribute(Constants.SERVER_TYPE_APP); } else { if (httpSession.getAttribute(Constants.SERVER_TYPE_APP) == null) { httpSession.setAttribute(Constants.SERVER_TYPE_APP , sysGuest); } return true; } } //获取网名 也就是/login String servletPath = request.getServletPath(); if(servletPath.contains("/app/card/isCard") || servletPath.contains("/app/bank/signingBankCard") || servletPath.contains("/app/bank/sendMessage") || servletPath.contains("/app/bank/sendSysMessage") || servletPath.contains("/app/bank/login") || servletPath.contains("/app/card/guestIdAndPhone") || servletPath.contains("/app/family/isDevice") || servletPath.contains("/websocket/") || servletPath.contains("/upload/") || servletPath.contains("/app/deviceSocket/toSocket")) { flag = true; } if (!flag) { noLogin(response); } return flag; } 重构这段代码

最新推荐

recommend-type

StringUtils工具包中字符串非空判断isNotEmpty和isNotBlank的区别

这意味着,只要字符串不为空,哪怕它只包含一个或多个空格,`isNotEmpty()` 都会认为这个字符串是非空的。例如: ```java StringUtils.isNotEmpty(null) = false StringUtils.isNotEmpty("") = false StringUtils....
recommend-type

StringUtils中的isEmpty、isNotEmpty、isBlank和isNotBlank的区别详解

在Java开发中,Apache Commons Lang库提供了一个非常实用的工具类`StringUtils`,它包含了一系列用于处理字符串的方法,包括`isEmpty()`, `isNotEmpty()`, `isBlank()`和`isNotBlank()`。这些方法在处理字符串时能...
recommend-type

WordPress作为新闻管理面板的实现指南

资源摘要信息: "使用WordPress作为管理面板" WordPress,作为当今最流行的开源内容管理系统(CMS),除了用于搭建网站、博客外,还可以作为一个功能强大的后台管理面板。本示例展示了如何利用WordPress的后端功能来管理新闻或帖子,将WordPress用作组织和发布内容的管理面板。 首先,需要了解WordPress的基本架构,包括它的数据库结构和如何通过主题和插件进行扩展。WordPress的核心功能已经包括文章(帖子)、页面、评论、分类和标签的管理,这些都可以通过其自带的仪表板进行管理。 在本示例中,WordPress被用作一个独立的后台管理面板来管理新闻或帖子。这种方法的好处是,WordPress的用户界面(UI)友好且功能全面,能够帮助不熟悉技术的用户轻松管理内容。WordPress的主题系统允许用户更改外观,而插件架构则可以扩展额外的功能,比如表单生成、数据分析等。 实施该方法的步骤可能包括: 1. 安装WordPress:按照标准流程在指定目录下安装WordPress。 2. 数据库配置:需要修改WordPress的配置文件(wp-config.php),将数据库连接信息替换为当前系统的数据库信息。 3. 插件选择与定制:可能需要安装特定插件来增强内容管理的功能,或者对现有的插件进行定制以满足特定需求。 4. 主题定制:选择一个适合的WordPress主题或者对现有主题进行定制,以实现所需的视觉和布局效果。 5. 后端访问安全:由于将WordPress用于管理面板,需要考虑安全性设置,如设置强密码、使用安全插件等。 值得一提的是,虽然WordPress已经内置了丰富的管理功能,但在企业级应用中,还需要考虑性能优化、安全性增强、用户权限管理等方面。此外,由于WordPress主要是作为内容发布平台设计的,将其作为管理面板可能需要一定的定制工作以确保满足特定的业务需求。 【PHP】标签意味着在实现该示例时,需要使用PHP编程语言。WordPress本身是由PHP语言开发的,因此开发者可能需要具备PHP开发能力,或至少能够理解PHP代码基础,以便对WordPress进行定制和扩展。 最后,【压缩包子文件的文件名称列表】中的"dctb-wp-as-admin-panel-master"暗示了这是一个存放所有相关文件的主目录,包括源代码、配置文件、模板文件、资源文件等,这些文件是安装和运行WordPress管理面板所必需的。在实际部署时,开发者需要上传这些文件到服务器,按照文档说明进行配置和安装。 总之,本示例提供了一个通过WordPress来搭建一个简单新闻或帖子管理系统的思路,展示了如何利用现有的开源工具来实现特定的业务需求。开发者需要有相应的PHP和WordPress知识,以及对网络安全和性能优化的基本了解。
recommend-type

管理建模和仿真的文件

管理Boualem Benatallah引用此版本:布阿利姆·贝纳塔拉。管理建模和仿真。约瑟夫-傅立叶大学-格勒诺布尔第一大学,1996年。法语。NNT:电话:00345357HAL ID:电话:00345357https://theses.hal.science/tel-003453572008年12月9日提交HAL是一个多学科的开放存取档案馆,用于存放和传播科学研究论文,无论它们是否被公开。论文可以来自法国或国外的教学和研究机构,也可以来自公共或私人研究中心。L’archive ouverte pluridisciplinaire
recommend-type

函数与模块化编程宝典:J750编程高效之路

![函数与模块化编程宝典:J750编程高效之路](https://github.blog/wp-content/uploads/2021/12/python-intermediate-update.png?resize=1024%2C494) 参考资源链接:[泰瑞达J750设备编程基础教程](https://wenku.csdn.net/doc/6412b472be7fbd1778d3f9e1?spm=1055.2635.3001.10343) # 1. 函数与模块化编程基础 ## 1.1 编程中的函数基础 函数是编程的基础构件,它允许将代码块组织成可重复使用的组件。在函数中封装特定的逻辑
recommend-type

用C语言求有4个圆塔,圆心分别为(2,2),(2,-2),(-2,2),(-2,-2)圆半径为1, 这4个塔的高度为10m 塔以外无建筑物接输入任意点的坐标 求该点的建筑高度(塔外的高度为零)的程序

在C语言中,你可以编写一个简单的函数来解决这个问题。首先,你需要确定每个圆是否包含了给定的点。如果包含,则返回塔高10米,如果不包含则返回0。这里提供一个基本的伪代码思路: ```c #include <stdio.h> #include <math.h> // 定义圆的结构体 typedef struct { double x, y; // 圆心坐标 int radius; // 半径 } Circle; // 函数判断点是否在圆内 int is_point_in_circle(Circle circle, double px, double py) { d
recommend-type

NPC_Generator:使用Ruby打造的游戏角色生成器

资源摘要信息:"NPC_Generator是一个专门为角色扮演游戏(RPG)或模拟类游戏设计的角色生成工具,它允许游戏开发者或者爱好者快速创建非玩家角色(NPC)并赋予它们丰富的背景故事、外观特征以及可能的行为模式。NPC_Generator的开发使用了Ruby编程语言,Ruby以其简洁的语法和强大的编程能力在脚本编写和小型项目开发中十分受欢迎。利用Ruby编写的NPC_Generator可以集成到游戏开发流程中,实现自动化生成NPC,极大地节省了手动设计每个NPC的时间和精力,提升了游戏内容的丰富性和多样性。" 知识点详细说明: 1. NPC_Generator的用途: NPC_Generator是用于游戏角色生成的工具,它能够帮助游戏设计师和玩家创建大量的非玩家角色(Non-Player Characters,简称NPC)。在RPG或模拟类游戏中,NPC是指在游戏中由计算机控制的虚拟角色,它们与玩家角色互动,为游戏世界增添真实感。 2. NPC生成的关键要素: - 角色背景故事:每个NPC都应该有自己的故事背景,这些故事可以是关于它们的过去,它们为什么会在游戏中出现,以及它们的个性和动机等。 - 外观特征:NPC的外观包括性别、年龄、种族、服装、发型等,这些特征可以由工具随机生成或者由设计师自定义。 - 行为模式:NPC的行为模式决定了它们在游戏中的行为方式,比如友好、中立或敌对,以及它们可能会执行的任务或对话。 3. Ruby编程语言的优势: - 简洁的语法:Ruby语言的语法非常接近英语,使得编写和阅读代码都变得更加容易和直观。 - 灵活性和表达性:Ruby语言提供的大量内置函数和库使得开发者可以快速实现复杂的功能。 - 开源和社区支持:Ruby是一个开源项目,有着庞大的开发者社区和丰富的学习资源,有利于项目的开发和维护。 4. 项目集成与自动化: NPC_Generator的自动化特性意味着它可以与游戏引擎或开发环境集成,为游戏提供即时的角色生成服务。自动化不仅可以提高生成NPC的效率,还可以确保游戏中每个NPC都具备独特的特性,使游戏世界更加多元和真实。 5. 游戏开发的影响: NPC_Generator的引入对游戏开发产生以下影响: - 提高效率:通过自动化的角色生成,游戏开发团队可以节约大量时间和资源,专注于游戏设计的其他方面。 - 增加多样性:自动化的工具可以根据不同的参数生成大量不同的NPC,为游戏世界带来更多的故事线和交互可能性。 - 玩家体验:丰富的NPC角色能够提升玩家的沉浸感,使得玩家在游戏中的体验更加真实和有吸引力。 6. Ruby在游戏开发中的应用: 虽然Ruby不是游戏开发中最常用的编程语言,但其在小型项目、原型设计、脚本编写等领域有其独特的优势。一些游戏开发工具和框架支持Ruby,如Ruby on Rails可以在Web游戏开发中发挥作用,而一些游戏开发社区也在探索Ruby的更多潜力。 7. NPC_Generator的扩展性和维护: 为了确保NPC_Generator能够长期有效地工作,它需要具备良好的扩展性和维护性。这意味着工具应该支持插件或模块的添加,允许社区贡献新功能,并且代码应该易于阅读和修改,以便于未来的升级和优化。 综上所述,NPC_Generator是一款利用Ruby编程语言开发的高效角色生成工具,它不仅提高了游戏开发的效率,而且通过提供丰富多样的NPC角色增加了游戏的深度和吸引力。随着游戏开发的不断发展,此类自动化工具将变得更加重要,而Ruby作为一种支持快速开发的编程语言,在这一领域有着重要的应用前景。
recommend-type

"互动学习:行动中的多样性与论文攻读经历"

多样性她- 事实上SCI NCES你的时间表ECOLEDO C Tora SC和NCESPOUR l’Ingén学习互动,互动学习以行动为中心的强化学习学会互动,互动学习,以行动为中心的强化学习计算机科学博士论文于2021年9月28日在Villeneuve d'Asq公开支持马修·瑟林评审团主席法布里斯·勒菲弗尔阿维尼翁大学教授论文指导奥利维尔·皮耶昆谷歌研究教授:智囊团论文联合主任菲利普·普雷教授,大学。里尔/CRISTAL/因里亚报告员奥利维耶·西格德索邦大学报告员卢多维奇·德诺耶教授,Facebook /索邦大学审查员越南圣迈IMT Atlantic高级讲师邀请弗洛里安·斯特鲁布博士,Deepmind对于那些及时看到自己错误的人...3谢谢你首先,我要感谢我的两位博士生导师Olivier和Philippe。奥利维尔,"站在巨人的肩膀上"这句话对你来说完全有意义了。从科学上讲,你知道在这篇论文的(许多)错误中,你是我可以依
recommend-type

流程控制与循环结构详解:J750编程逻辑构建指南

![流程控制与循环结构详解:J750编程逻辑构建指南](https://d1g9li960vagp7.cloudfront.net/wp-content/uploads/2018/10/While-Schleife_WP_04-1024x576.png) 参考资源链接:[泰瑞达J750设备编程基础教程](https://wenku.csdn.net/doc/6412b472be7fbd1778d3f9e1?spm=1055.2635.3001.10343) # 1. 流程控制与循环结构的编程基础 编程中,流程控制和循环结构是构建有效程序逻辑的核心元素。本章将带你从基础概念出发,逐步深入理解
recommend-type

python实现生成一个窗口,其窗口题目为“二冷配水模型模型”,窗口中包含八个输入栏,三个按钮,每个按钮点击后会产生一个不同的页面

在Python中,尤其是使用Tkinter库可以方便地创建图形用户界面(GUI)。为了实现你所描述的功能,我们可以创建一个简单的窗口,并添加相应的组件。以下是一个基本的例子: ```python import tkinter as tk def button_click_1(): # 这里可以编写打开新页面的逻辑,这里仅作示例 new_window = tk.Toplevel() new_window.title("新页面1") # 添加其他元素到新窗口... def button_click_2(): new_window = tk.Toplev