优化下这段代码 Boolean tspCheckEnabled = ThreadLocalVariable.getApp().getTspCheckEnabled(); String appKey = ThreadLocalVariable.getApp().getAppKey(); if (tspCheckEnabled) { // 校验vin+sn if (ArrayUtil.isNotEmpty(checkAppKey)) { if (Arrays.stream(checkAppKey).anyMatch(item -> StrUtil.equalsAnyIgnoreCase(appKey, item))) { itspService.checkVin(vin); } else { itspService.checkVINAndSN(vin, sn, null); } } else { itspService.checkVINAndSN(vin, sn, null); } }
时间: 2023-05-16 15:06:30 浏览: 55
可以尝试使用以下代码进行优化:
Boolean tspCheckEnabled = ThreadLocalVariable.getApp().getTspCheckEnabled();
String appKey = ThreadLocalVariable.getApp().getAppKey();
if (tspCheckEnabled && ArrayUtil.isNotEmpty(checkAppKey) && Arrays.stream(checkAppKey).anyMatch(item -> StrUtil.equalsAnyIgnoreCase(appKey, item))) {
itspService.checkVin(vin);
} else {
itspService.checkVINAndSN(vin, sn, null);
}
这样可以减少不必要的嵌套和重复代码,提高代码的可读性和执行效率。
相关问题
优化代码 public static void main(String[] args) { ZkemSDK sdk = new ZkemSDK(); boolean connFlag = sdk.connect("192.168.1.201", 4370); System.out.println("conn:" + connFlag); boolean readGeneralLogData = sdk.readGeneralLogData(); System.out.println("读取打卡机信息到缓存:" + readGeneralLogData); //如果连接打卡机和读取信息到缓存都没问题再去缓存中获取数据 if (connFlag) { //获取打卡机所有的用户信息 List<UserInfo> userInfo = sdk.getUserInfo(); //转化为JSON字符串 String usersJsonString = JSON.toJSONString(userInfo); //反序列化对象 List<UserInfo> userInfos = JSON.parseArray(usersJsonString, UserInfo.class); userInfos.forEach(System.out::println); for (UserInfo info : userInfos) { String name = info.getName(); Boolean enabled = info.getEnabled(); String pwd = info.getPassword(); Integer privilege = info.getPrivilege(); String enrollNumber = info.getEnrollNumber(); Integer userId = Integer.valueOf(enrollNumber); /** * 上面的都是打卡机的数据,可以根据自己的需求处理 / System.out.println(name+"-"+enabled+"-"+pwd+"-"+privilege+"-"+userId+""); } List<Map<String, Object>> generalLogData = sdk.getGeneralLogData(); String logDataJsonString = JSON.toJSONString(generalLogData); List<LogData> logDatas = JSON.parseArray(logDataJsonString, LogData.class); logDatas.forEach(System.out::println); for(LogData data : logDatas){ //打卡机的时间 String time = data.getTime(); String enrollNumber = data.getEnrollNumber(); /* * data有很多的数据,可以根据自己的业务逻辑拿想要的数据 * 我这里只取了打卡时间和enrollNumber * */ Map<String, Object> userInfoByNumber = getUserInfoByNumber(enrollNumber); System.out.println(time); } } }
这段代码中存在一些可以优化的地方。以下一些建议:
1 将连接打卡机读取信息到缓存代码封装成一个方法,以提高代码的可读性和重用性。
2 使用try-with-resources语自动关闭资源,ZkemSDK对象。
. 将打卡机用户信息和日志数据的处理逻辑提取成单独的方法,以提高代码的模块化。
下面是优化后的代码示例:
```java
public static void main(String[] args) {
ZkemSDK sdk = new ZkemSDK();
boolean connFlag = connectAndReadData(sdk);
if (connFlag) {
List<UserInfo> userInfo = sdk.getUserInfo();
processUserInfo(userInfo);
List<Map<String, Object>> generalLogData = sdk.getGeneralLogData();
processLogData(generalLogData);
}
}
private static boolean connectAndReadData(ZkemSDK sdk) {
try {
boolean connFlag = sdk.connect("192.168.1.201", 4370);
System.out.println("conn:" + connFlag);
boolean readGeneralLogData = sdk.readGeneralLogData();
System.out.println("读取打卡机信息到缓存:" + readGeneralLogData);
return connFlag && readGeneralLogData;
} finally {
sdk.disconnect(); // 根据实际情况调用释放资源的方法
}
}
private static void processUserInfo(List<UserInfo> userInfo) {
String usersJsonString = JSON.toJSONString(userInfo);
List<UserInfo> userInfos = JSON.parseArray(usersJsonString, UserInfo.class);
userInfos.forEach(info -> {
String name = info.getName();
Boolean enabled = info.getEnabled();
String pwd = info.getPassword();
Integer privilege = info.getPrivilege();
String enrollNumber = info.getEnrollNumber();
Integer userId = Integer.valueOf(enrollNumber);
System.out.println(name + "-" + enabled + "-" + pwd + "-" + privilege + "-" + userId);
});
}
private static void processLogData(List<Map<String, Object>> generalLogData) {
String logDataJsonString = JSON.toJSONString(generalLogData);
List<LogData> logDatas = JSON.parseArray(logDataJsonString, LogData.class);
logDatas.forEach(data -> {
String time = data.getTime();
String enrollNumber = data.getEnrollNumber();
Map<String, Object> userInfoByNumber = getUserInfoByNumber(enrollNumber);
System.out.println(time);
});
}
```
请注意,上述代码中的`getUserInfoByNumber`方法没有提供实现,你需要根据自己的业务逻辑来实现该方法。此外,你可能还需要根据实际情况调整代码中的其他部分。
优化这段代码boolean flag = permissionService.isPermitted("system:course:list"); if (flag) { return listAll(vo); } flag = permissionService.isPermitted("system:course:list:dept"); if (flag) { return listByDept(vo); } flag = permissionService.isPermitted("system:course:list:creator"); if (flag) { return listByCreator(vo); } return R.failed("对不起,您没有课程查询权限");
您可以尝试使用switch-case语句来优化这段代码,减少重复的判断:
```
String permission = "";
if(permissionService.isPermitted("system:course:list")){
permission = "listAll";
} else if(permissionService.isPermitted("system:course:list:dept")){
permission = "listByDept";
} else if(permissionService.isPermitted("system:course:list:creator")){
permission = "listByCreator";
} else {
return R.failed("对不起,您没有课程查询权限");
}
try {
Method method = this.getClass().getMethod(permission, vo.getClass());
return (R) method.invoke(this, vo);
} catch (Exception e) {
e.printStackTrace();
return R.failed("系统异常,请联系管理员");
}
```
这样可以将重复的代码进行封装,提高了代码的可读性和可维护性。