perf[stream]: stream.toList() replaces collect(Collectors.toList())

This commit is contained in:
sun
2023-09-12 13:56:51 +08:00
parent 4783a72703
commit ac2d01e6fb
19 changed files with 34 additions and 50 deletions
@@ -35,7 +35,6 @@ import java.util.HashSet;
import java.util.Objects;
import java.util.concurrent.atomic.AtomicInteger;
import java.util.function.Predicate;
import java.util.stream.Collectors;
import static com.zfoo.protocol.util.StringUtils.TAB;
import static com.zfoo.protocol.util.StringUtils.TAB_ASCII;
@@ -85,7 +84,7 @@ public abstract class GenerateProtocolFile {
var outsideGenerateProtocols = Arrays.stream(protocols)
.filter(it -> Objects.nonNull(it))
.filter(it -> generateProtocolFilter.test(it))
.collect(Collectors.toList());
.toList();;
// 需要生成的子协议,因为外层协议的内部有其它协议
var insideGenerateProtocols = outsideGenerateProtocols.stream()
@@ -93,7 +92,7 @@ public abstract class GenerateProtocolFile {
.flatMap(it -> it.stream())
.map(it -> protocols[it])
.distinct()
.collect(Collectors.toList());
.toList();;
var allGenerateProtocols = new HashSet<IProtocolRegistration>();
allGenerateProtocols.addAll(outsideGenerateProtocols);
@@ -102,7 +101,7 @@ public abstract class GenerateProtocolFile {
// 通过协议号,从小到大排序
var allSortedGenerateProtocols = allGenerateProtocols.stream()
.sorted((a, b) -> a.protocolId() - b.protocolId())
.collect(Collectors.toList());
.toList();;
// 解析协议的文档注释
GenerateProtocolNote.initProtocolNote(allSortedGenerateProtocols);
@@ -22,7 +22,6 @@ import com.zfoo.protocol.util.AssertionUtils;
import com.zfoo.protocol.util.StringUtils;
import java.util.*;
import java.util.stream.Collectors;
/**
* EN: When generating a protocol, the final generation path of the protocol will use this class.
@@ -117,7 +116,7 @@ public abstract class GenerateProtocolPath {
for (var i = splits.length; i > 0; i--) {
builder.append("../");
var path = StringUtils.joinWith(StringUtils.PERIOD, Arrays.stream(splits).limit(i).collect(Collectors.toList()).toArray());
var path = StringUtils.joinWith(StringUtils.PERIOD, Arrays.stream(splits).limit(i).toList();.toArray());
if (relativePath.startsWith(path)) {
builder.append(StringUtils.substringAfterFirst(relativePath, path).replaceAll(StringUtils.PERIOD_REGEX, StringUtils.SLASH));
return builder.toString();
@@ -181,7 +180,7 @@ public abstract class GenerateProtocolPath {
var allChildren = protocolTreeNode.flatTreeNodes()
.stream()
.filter(it -> it.getData() != null)
.collect(Collectors.toList());
.toList();;
var pathBefore = StringUtils.substringBeforeLast(protocolTreeNode.fullName(), StringUtils.PERIOD);
for (var child : allChildren) {
var protocolSimpleName = child.getData().protocolConstructor().getDeclaringClass().getSimpleName();
@@ -30,7 +30,6 @@ import java.lang.reflect.Constructor;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Modifier;
import java.util.*;
import java.util.stream.Collectors;
/**
* 对应于ProtocolRegistration
@@ -127,7 +126,7 @@ public abstract class EnhanceUtils {
var allSubProtocolIds = ProtocolAnalysis.getAllSubProtocolIds(protocolId)
.stream()
.sorted((a, b) -> Short.compare(a, b))
.collect(Collectors.toList());
.toList();;
for (var subProtocolId : allSubProtocolIds) {
var protocolRegistrationField = new CtField(classPool.get(IProtocolRegistration.class.getCanonicalName()), getProtocolRegistrationFieldNameByProtocolId(subProtocolId), enhanceClazz);
@@ -42,7 +42,6 @@ import javassist.NotFoundException;
import java.io.IOException;
import java.lang.reflect.*;
import java.util.*;
import java.util.stream.Collectors;
import static com.zfoo.protocol.ProtocolManager.*;
@@ -108,7 +107,7 @@ public class ProtocolAnalysis {
}
// 通过指定类注册的协议,全部使用字节码增强
var enhanceList = Arrays.stream(protocols).filter(Objects::nonNull).collect(Collectors.toList());
var enhanceList = Arrays.stream(protocols).filter(Objects::nonNull).toList();;
enhance(generateOperation, enhanceList);
}
@@ -122,7 +121,7 @@ public class ProtocolAnalysis {
var relevantClassList = relevantClassSet.stream()
.sorted((a, b) -> a.getCanonicalName().compareTo(b.getCanonicalName()))
.collect(Collectors.toList());
.toList();;
// 检查协议类是否合法
var noProtocolIds = new ArrayList<Class<?>>();
@@ -149,7 +148,7 @@ public class ProtocolAnalysis {
}
// 通过指定类注册的协议,全部使用字节码增强
var enhanceList = Arrays.stream(protocols).filter(Objects::nonNull).collect(Collectors.toList());
var enhanceList = Arrays.stream(protocols).filter(Objects::nonNull).toList();;
enhance(generateOperation, enhanceList);
}
@@ -415,7 +414,7 @@ public class ProtocolAnalysis {
.stream()
.sorted((a, b) -> a.getKey() - b.getKey())
.map(Map.Entry::getValue)
.collect(Collectors.toList());
.toList();;
notCompatibleFields.addAll(compatibleFields);
return notCompatibleFields;
}
@@ -39,7 +39,6 @@ import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import static com.zfoo.protocol.util.FileUtils.LS;
import static com.zfoo.protocol.util.StringUtils.TAB;
@@ -211,7 +210,7 @@ public abstract class GenerateCppUtils {
// ValueOf()方法
var valueOfParams = filedList.stream()
.map(it -> StringUtils.format("{} {}", it.getKey(), it.getValue()))
.collect(Collectors.toList());
.toList();;
var valueOfParamsStr = StringUtils.joinWith(StringUtils.COMMA + " ", valueOfParams.toArray());
var cppBuilder = new StringBuilder();
@@ -34,7 +34,6 @@ import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import static com.zfoo.protocol.util.FileUtils.LS;
import static com.zfoo.protocol.util.StringUtils.TAB;
@@ -160,7 +159,7 @@ public abstract class GenerateCsUtils {
}
// ValueOf()方法
var valueOfParams = filedList.stream().map(it -> StringUtils.format("{} {}", it.getKey(), it.getValue())).collect(Collectors.toList());
var valueOfParams = filedList.stream().map(it -> StringUtils.format("{} {}", it.getKey(), it.getValue())).toList();
var valueOfParamsStr = StringUtils.joinWith(StringUtils.COMMA + " ", valueOfParams.toArray());
var csBuilder = new StringBuilder();
@@ -33,7 +33,6 @@ import java.util.Arrays;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import static com.zfoo.protocol.util.FileUtils.LS;
import static com.zfoo.protocol.util.StringUtils.TAB;
@@ -133,7 +132,7 @@ public abstract class GenerateJsUtils {
var protocolId = registration.getId();
var fields = registration.getFields();
var fieldValueOf = StringUtils.joinWith(", ", Arrays.stream(fields).map(it -> it.getName()).collect(Collectors.toList()).toArray());
var fieldValueOf = StringUtils.joinWith(", ", Arrays.stream(fields).map(it -> it.getName()).toList();.toArray());
var fieldDefinitionBuilder = new StringBuilder();
for (var field : fields) {
@@ -33,7 +33,6 @@ import java.util.Arrays;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.stream.Collectors;
import static com.zfoo.protocol.util.FileUtils.LS;
import static com.zfoo.protocol.util.StringUtils.TAB;
@@ -136,7 +135,7 @@ public abstract class GenerateLuaUtils {
var protocolId = registration.getId();
var fields = registration.getFields();
var valueOfParams = StringUtils.joinWith(", ", Arrays.stream(fields).map(it -> it.getName()).collect(Collectors.toList()).toArray());
var valueOfParams = StringUtils.joinWith(", ", Arrays.stream(fields).map(it -> it.getName()).toList();.toArray());
var luaBuilder = new StringBuilder();
for (var i = 0; i < fields.length; i++) {
@@ -28,7 +28,6 @@ import java.util.Collections;
import java.util.List;
import java.util.function.Consumer;
import java.util.function.Predicate;
import java.util.stream.Collectors;
/**
@@ -319,7 +318,7 @@ public abstract class ReflectionUtils {
.filter(it -> !Modifier.isStatic(it.getModifiers()))
.filter(it -> !Modifier.isTransient(it.getModifiers()))
.filter(it -> !it.isAnnotationPresent(Transient.class))
.collect(Collectors.toList());
.toList();;
}
public static String fieldToGetMethod(Class<?> clazz, Field field) {