优化这段代码,复杂度降低到15以下:public AjaxResult selectDeviceByCode(HttpServletRequest request,@RequestParam Map<String, Object> form) { List<HashMap<String, Object>> list = facilityService.selectDeviceByCode(form); // 判断list是否为空 if (list == null || list.isEmpty()) { return AjaxResult.error("此设备不存在,请联系管理员"); } else { if (list.get(0).get(guestIdDetail) != null) { Attention attention = new Attention(); //设备绑定人 long beGuestId = Long.valueOf(list.get(0).get(guestIdDetail).toString()); attention.setBeGuestId(beGuestId); // 当前登入人 Long guestId = getAPPUserId(request); attention.setGuestId(guestId); //判断当前登入人与设备绑定人是否为同一人 if (attention.getGuestId().equals(beGuestId)) { return error("无法绑定自己的设备"); } // 判断是否关注亲友 List<Attention> attentionList = attentionService.selAttention(attention); if (attentionList != null && !attentionList.isEmpty()) { Attention attentionDetail = attentionList.get(0); if (attentionDetail.getOverTime() != null) { return error("该设备已被亲友绑定,且您已关注亲友"); } return error("已提交关注申请"); } } return success(list.get(0)); } }

时间: 2024-02-14 15:33:13 浏览: 30
这段代码的复杂度较高,需要进行一些优化,可以采用以下方法: 1. 将 if-else 结构中的 if 语句提取出来,避免嵌套过深; 2. 对于一些重复的变量和表达式,进行提取和复用; 3. 将一些判断条件合并,避免重复判断。 以下是优化后的代码: public AjaxResult selectDeviceByCode(HttpServletRequest request, @RequestParam Map<String, Object> form) { List<HashMap<String, Object>> list = facilityService.selectDeviceByCode(form); // 判断list是否为空 if (list == null || list.isEmpty()) { return AjaxResult.error("此设备不存在,请联系管理员"); } // 获取设备绑定人 Object guestIdDetail = list.get(0).get("guestIdDetail"); if (guestIdDetail != null) { Attention attention = new Attention(); //设备绑定人 long beGuestId = Long.valueOf(guestIdDetail.toString()); attention.setBeGuestId(beGuestId); // 当前登入人 Long guestId = getAPPUserId(request); attention.setGuestId(guestId); //判断当前登入人与设备绑定人是否为同一人 if (attention.getGuestId().equals(beGuestId)) { return error("无法绑定自己的设备"); } // 判断是否关注亲友 List<Attention> attentionList = attentionService.selAttention(attention); if (attentionList != null && !attentionList.isEmpty()) { Attention attentionDetail = attentionList.get(0); if (attentionDetail.getOverTime() != null) { return error("该设备已被亲友绑定,且您已关注亲友"); } return error("已提交关注申请"); } } return success(list.get(0)); } 通过将 if-else 结构中的 if 语句提取出来,避免嵌套过深,将一些重复的变量和表达式提取和复用,将一些判断条件合并,可以将代码的复杂度降低到15以下,使代码更加简洁和易于维护。

相关推荐

@RestController @RequestMapping("tblBasicInformationInfo") public class TblBasicInformationInfoController { /** * 服务对象 */ @Resource private TblBasicInformationInfoService tblBasicInformationInfoService; /** * 校验header */ @Value("${interfaceHeader.FskAppKey}") private String FskAppKey; @Value("${interfaceHeader.FskAppSecret}") private String FskAppSecret; @Value("${interfaceHeader.YjAppKey}") private String YjAppKey; @Value("${interfaceHeader.YjAppSecret}") private String YjAppSecret; /** * 新增数据 * * @param tblBasicInformationInfo 实体对象 * @return 新增结果 */ @PostMapping("addList") public AjaxResult addList(@RequestBody List<TblBasicInformationInfo> tblBasicInformationInfo, HttpServletRequest request) { String appKey = request.getHeader("appKey"); String appSecret = request.getHeader("appSecret"); String name = ""; if (FskAppKey.equals(appKey) && FskAppSecret.equals(appSecret)) { name = "福寿康"; } else if (YjAppKey.equals(appKey) && YjAppSecret.equals(appSecret)) { name = "颐家"; } else { return AjaxResult.error(); } //为每条数据设置创建时间和数据来源 for (TblBasicInformationInfo t : tblBasicInformationInfo) { TblBasicInformationInfo one = this.tblBasicInformationInfoService.getOne(Wrappers.<TblBasicInformationInfo>lambdaQuery().eq(TblBasicInformationInfo::getIdNumber, t.getIdNumber())); t.setCreatedTime(new DateTime()); t.setAppName(name); t.setAppId(appKey); t.setAppSecret(appSecret); if (Objects.isNull(one)) { this.tblBasicInformationInfoService.save(t); } else { t.setId(one.getId()); this.tblBasicInformationInfoService.updateById(t); } } return AjaxResult.success(200); }性能优化

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。

最新推荐

recommend-type

基于SpringMVC+Hibernate+AngularJs前后端分离的选课系统+源码+文档+界面展示(毕业设计&课程设计)

基于SpringMVC+Hibernate+AngularJs前后端分离的选课系统+源码+文档+界面展示,适合毕业设计、课程设计、项目开发。项目源码已经过严格测试,可以放心参考并在此基础上延申使用~ 基于SpringMVC+Hibernate+AngularJs前后端分离的选课系统+源码+文档+界面展示,适合毕业设计、课程设计、项目开发。项目源码已经过严格测试,可以放心参考并在此基础上延申使用~ 基于SpringMVC+Hibernate+AngularJs前后端分离的选课系统+源码+文档+界面展示,适合毕业设计、课程设计、项目开发。项目源码已经过严格测试,可以放心参考并在此基础上延申使用~ 项目简介: 本选课系统开源协议基于GPL协议,仅用作交流学习用途。 本系统采用了前后端分离的开发模式,后端采用Springmvc+Hibernate框架。 前端使用AngularJs+JQuery+Bootstrap开发,并且使用前端构建工具Gulp。
recommend-type

京瓷TASKalfa系列维修手册:安全与操作指南

"该资源是一份针对京瓷TASKalfa系列多款型号打印机的维修手册,包括TASKalfa 2020/2021/2057,TASKalfa 2220/2221,TASKalfa 2320/2321/2358,以及DP-480,DU-480,PF-480等设备。手册标注为机密,仅供授权的京瓷工程师使用,强调不得泄露内容。手册内包含了重要的安全注意事项,提醒维修人员在处理电池时要防止爆炸风险,并且应按照当地法规处理废旧电池。此外,手册还详细区分了不同型号产品的打印速度,如TASKalfa 2020/2021/2057的打印速度为20张/分钟,其他型号则分别对应不同的打印速度。手册还包括修订记录,以确保信息的最新和准确性。" 本文档详尽阐述了京瓷TASKalfa系列多功能一体机的维修指南,适用于多种型号,包括速度各异的打印设备。手册中的安全警告部分尤为重要,旨在保护维修人员、用户以及设备的安全。维修人员在操作前必须熟知这些警告,以避免潜在的危险,如不当更换电池可能导致的爆炸风险。同时,手册还强调了废旧电池的合法和安全处理方法,提醒维修人员遵守地方固体废弃物法规。 手册的结构清晰,有专门的修订记录,这表明手册会随着设备的更新和技术的改进不断得到完善。维修人员可以依靠这份手册获取最新的维修信息和操作指南,确保设备的正常运行和维护。 此外,手册中对不同型号的打印速度进行了明确的区分,这对于诊断问题和优化设备性能至关重要。例如,TASKalfa 2020/2021/2057系列的打印速度为20张/分钟,而TASKalfa 2220/2221和2320/2321/2358系列则分别具有稍快的打印速率。这些信息对于识别设备性能差异和优化工作流程非常有用。 总体而言,这份维修手册是京瓷TASKalfa系列设备维修保养的重要参考资料,不仅提供了详细的操作指导,还强调了安全性和合规性,对于授权的维修工程师来说是不可或缺的工具。
recommend-type

管理建模和仿真的文件

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

【进阶】入侵检测系统简介

![【进阶】入侵检测系统简介](http://www.csreviews.cn/wp-content/uploads/2020/04/ce5d97858653b8f239734eb28ae43f8.png) # 1. 入侵检测系统概述** 入侵检测系统(IDS)是一种网络安全工具,用于检测和预防未经授权的访问、滥用、异常或违反安全策略的行为。IDS通过监控网络流量、系统日志和系统活动来识别潜在的威胁,并向管理员发出警报。 IDS可以分为两大类:基于网络的IDS(NIDS)和基于主机的IDS(HIDS)。NIDS监控网络流量,而HIDS监控单个主机的活动。IDS通常使用签名检测、异常检测和行
recommend-type

轨道障碍物智能识别系统开发

轨道障碍物智能识别系统是一种结合了计算机视觉、人工智能和机器学习技术的系统,主要用于监控和管理铁路、航空或航天器的运行安全。它的主要任务是实时检测和分析轨道上的潜在障碍物,如行人、车辆、物体碎片等,以防止这些障碍物对飞行或行驶路径造成威胁。 开发这样的系统主要包括以下几个步骤: 1. **数据收集**:使用高分辨率摄像头、雷达或激光雷达等设备获取轨道周围的实时视频或数据。 2. **图像处理**:对收集到的图像进行预处理,包括去噪、增强和分割,以便更好地提取有用信息。 3. **特征提取**:利用深度学习模型(如卷积神经网络)提取障碍物的特征,如形状、颜色和运动模式。 4. **目标
recommend-type

小波变换在视频压缩中的应用

"多媒体通信技术视频信息压缩与处理(共17张PPT).pptx" 多媒体通信技术涉及的关键领域之一是视频信息压缩与处理,这在现代数字化社会中至关重要,尤其是在传输和存储大量视频数据时。本资料通过17张PPT详细介绍了这一主题,特别是聚焦于小波变换编码和分形编码两种新型的图像压缩技术。 4.5.1 小波变换编码是针对宽带图像数据压缩的一种高效方法。与离散余弦变换(DCT)相比,小波变换能够更好地适应具有复杂结构和高频细节的图像。DCT对于窄带图像信号效果良好,其变换系数主要集中在低频部分,但对于宽带图像,DCT的系数矩阵中的非零系数分布较广,压缩效率相对较低。小波变换则允许在频率上自由伸缩,能够更精确地捕捉图像的局部特征,因此在压缩宽带图像时表现出更高的效率。 小波变换与傅里叶变换有本质的区别。傅里叶变换依赖于一组固定频率的正弦波来表示信号,而小波分析则是通过母小波的不同移位和缩放来表示信号,这种方法对非平稳和局部特征的信号描述更为精确。小波变换的优势在于同时提供了时间和频率域的局部信息,而傅里叶变换只提供频率域信息,却丢失了时间信息的局部化。 在实际应用中,小波变换常常采用八带分解等子带编码方法,将低频部分细化,高频部分则根据需要进行不同程度的分解,以此达到理想的压缩效果。通过改变小波的平移和缩放,可以获取不同分辨率的图像,从而实现按需的图像质量与压缩率的平衡。 4.5.2 分形编码是另一种有效的图像压缩技术,特别适用于处理不规则和自相似的图像特征。分形理论源自自然界的复杂形态,如山脉、云彩和生物组织,它们在不同尺度上表现出相似的结构。通过分形编码,可以将这些复杂的形状和纹理用较少的数据来表示,从而实现高压缩比。分形编码利用了图像中的分形特性,将其转化为分形块,然后进行编码,这在处理具有丰富细节和不规则边缘的图像时尤其有效。 小波变换和分形编码都是多媒体通信技术中视频信息压缩的重要手段,它们分别以不同的方式处理图像数据,旨在减少存储和传输的需求,同时保持图像的质量。这两种技术在现代图像处理、视频编码标准(如JPEG2000)中都有广泛应用。
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

【进阶】Python高级加密库cryptography

![【进阶】Python高级加密库cryptography](https://img-blog.csdnimg.cn/20191105183454149.jpg) # 2.1 AES加密算法 ### 2.1.1 AES加密原理 AES(高级加密标准)是一种对称块密码,由美国国家标准与技术研究院(NIST)于2001年发布。它是一种分组密码,这意味着它一次处理固定大小的数据块(通常为128位)。AES使用密钥长度为128、192或256位的迭代密码,称为Rijndael密码。 Rijndael密码基于以下基本操作: - 字节替换:将每个字节替换为S盒中的另一个字节。 - 行移位:将每一行
recommend-type

linuxjar包启动脚本

Linux中的jar包通常指的是Java Archive(Java归档文件),它是一个包含Java类、资源和其他相关文件的压缩文件。启动一个Java应用的jar包通常涉及到使用Java的Runtime或JVM(Java虚拟机)。 一个简单的Linux启动jar包的脚本(例如用bash编写)可能会类似于这样: ```bash #!/bin/bash # Java启动脚本 # 设置JAVA_HOME环境变量,指向Java安装路径 export JAVA_HOME=/path/to/your/java/jdk # jar包的路径 JAR_FILE=/path/to/your/applicat
recommend-type

Microsoft OfficeXP详解:WordXP、ExcelXP和PowerPointXP

"第四章办公自动化软件应用,重点介绍了Microsoft OfficeXP中的WordXP、ExcelXP和PowerPointXP的基本功能和应用。" 在办公自动化领域,Microsoft OfficeXP是一个不可或缺的工具,尤其对于文字处理、数据管理和演示文稿制作。该软件套装包含了多个组件,如WordXP、ExcelXP和PowerPointXP,每个组件都有其独特的功能和优势。 WordXP是OfficeXP中的核心文字处理软件,它的主要特点包括: 1. **所见即所得**:这一特性确保在屏幕上的预览效果与最终打印结果一致,包括字体、字号、颜色和表格布局等视觉元素。 2. **文字编辑**:WordXP提供基础的文字编辑功能,如选定、移动、复制和删除,同时具备自动更正和自动图文集,能即时修正输入错误,并方便存储和重复使用常用文本或图形。 3. **格式编辑**:包括字符、段落和页面的格式设置,使用户可以灵活调整文档的视觉风格,以适应不同的需求。 4. **模板、向导和样式**:模板简化了创建有固定格式文档的过程,向导引导用户完成模板填充,而样式则允许用户自定义和保存可重复使用的格式组合。 5. **图文混排**:WordXP的强大之处在于其处理图像和文本的能力,使得文档中的图片、图表和文本可以自由布局,增强了文档的表现力。 接下来,ExcelXP是电子表格软件,主要用于数据管理、计算和分析。它的主要功能包括: - 创建和编辑复杂的公式,进行数学计算和数据分析。 - 使用图表功能将数据可视化,帮助理解趋势和模式。 - 数据排序、筛选和查找功能,便于信息检索和管理。 - 表格和工作簿模板,方便用户快速生成标准格式的工作表。 最后,PowerPointXP是用于制作电子演示文稿的工具,其特性如下: - 简单易用的界面,方便用户创建引人入胜的幻灯片。 - 多样化的主题、过渡和动画效果,提升演示的视觉吸引力。 - 支持嵌入多媒体内容,如视频和音频,增强演示的交互性。 - 可以预览和控制演示流程,确保在实际展示时的流畅性。 这三款软件共同构成了OfficeXP,是办公环境中提高效率和专业性的关键工具。通过熟练掌握它们,用户可以高效地完成报告编写、数据分析和演讲准备等任务。