[英]Java concurrency, stream behavior
方法同时调用此类的getFirst()
和getSecond()
。 它是Web应用程序的一部分。
内部地图也填充,没有并发。
public class MyClass {
private Map<String, List<List<String>>> first;
private Map<String, List<List<String>>> second;
public MyClass() {
first = new ConcurrentHashMap<>();
second = new ConcurrentHashMap<>();
}
public Set<String> getFirst(String key, String token, int a, int b) {
return get(first, key, token, a, b);
}
public Set<String> getSecond(String key, String token, int a, int b) {
return get(second, key, token, a, b);
}
private Set<String> get(final Map<String, List<List<String>>> map, final String key, final String token,
final int a, final int b) {
Set<String> result = new TreeSet<>();
map.get(key).stream().filter(i -> i.size() <= b && i.size() >= a).forEach(
s -> result
.addAll(s.stream().filter(p -> StringUtils.containsIgnoreCase(p, token)).collect(Collectors.toList())));
return result;
}
}
我用ab -n 10000 -c 100
(Apache的实用程序)测试了它。 我记录下来了。 我总是得到同样的一套。 但是如果我将map.get(key).stream()
更改为map.get(key).parallelStream()
并执行相同的步骤,我有时会得到不同的结果大小(总是更小)。
它是什么?
您正在并行流的forEach
中使用TreeSet.addAll()
。 对于不同的元素, forEach
主体可以在不同的线程中同时执行多次,并且TreeSet
不是线程安全的。 要快速解决问题,您可以同步修改result
或使用forEachOrdered
。 但是,如果没有forEach
那么flatMap
你的流并立即收集它会更干净,更forEach
。 试试这个版本:
return map.get(key).stream()
.filter(i -> i.size() <= b && i.size() >= a)
.flatMap(List::stream).filter(p -> StringUtils.containsIgnoreCase(p, token))
.collect(Collectors.toCollection(TreeSet::new));
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.