public Map<String, List<String>> pastfree(List<FreezeAuthorityOut> data,Map<String, List<String>> map){ for (FreezeAuthorityOut authority : data) { List<String> list = new ArrayList<>(); for (FreezeAuthorityOut offlineLockAuthority : data) { if (authority.getLockUid().equals(offlineLockAuthority.getLockUid()) && map != null) { List<String> key = map.get(authority.getLockUid()); if (key == null) { list.add(authority.getCardNumber()); } } } if (map != null) { List<String> key = map.get(authority.getLockUid()); if (key == null) { //去重 list = list.stream().distinct().collect(Collectors.toList()); map.put(authority.getLockUid(), list); } } } return map; } 如何优化
时间: 2024-02-10 20:26:11 浏览: 70
String 优化
在您提供的代码中,有几个地方可以进行优化来提高性能和简化代码:
1. 避免重复获取`map.get(authority.getLockUid())`的值:在两个地方都有类似的代码,您可以在进入循环之前将其存储在一个变量中,以避免重复的`get`操作。
2. 使用`Set`代替`List`进行去重:在添加元素到`list`之后,可以使用`HashSet`来去除重复元素,然后再转换回`List`。
3. 使用Java 8的新特性来简化代码:使用Lambda表达式和流操作可以使代码更加简洁和易读。
下面是优化后的代码示例:
```java
public Map<String, List<String>> pastfree(List<FreezeAuthorityOut> data, Map<String, List<String>> map) {
for (FreezeAuthorityOut authority : data) {
if (map != null && !map.containsKey(authority.getLockUid())) {
List<String> list = data.stream()
.filter(a -> a.getLockUid().equals(authority.getLockUid()))
.map(FreezeAuthorityOut::getCardNumber)
.distinct()
.collect(Collectors.toList());
map.put(authority.getLockUid(), list);
}
}
return map;
}
```
在优化后的代码中,我们使用了流操作和Lambda表达式来简化循环和去重的过程。首先,我们检查`map`是否为空以及是否已经包含了当前`authority`的`LockUid`。然后,我们使用流操作来筛选出具有相同`LockUid`的元素,并提取`CardNumber`属性。最后,我们使用`distinct()`方法去除重复的元素,并将结果存储到`map`中。
请注意,这只是一种可能的优化方法,具体优化策略可能需要根据实际情况进行调整。
阅读全文