Merge pull request #941 from linwumingshi/fix/list-empty

fix: 🐛 Resolve error caused by `EmptyList` being final

Author: shalousun

src/main/java/com/ly/doc/model/AbstractRpcApiDoc.java

@@ -1,16 +1,14 @@
 package com.ly.doc.model;
 
-import com.power.common.util.CollectionUtil;
-
 import java.io.Serializable;
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
 
 /**
  * Abstract API Doc.
  *
+ * @author linwumingshi
  * @param <T> the type of method doc
  */
 public abstract class AbstractRpcApiDoc<T extends IMethod>
@@ -180,8 +178,8 @@ public String getDocClass() {
 
 	@Override
 	public List<IMethod> getMethods() {
-		if (CollectionUtil.isEmpty(this.list)) {
-			return Collections.emptyList();
+		if (Objects.isNull(this.list)) {
+			return new ArrayList<>();
 		}
 		return new ArrayList<>(this.list);
 	}

src/main/java/com/ly/doc/model/ApiDoc.java

@@ -20,7 +20,6 @@
  */
 package com.ly.doc.model;
 
-import com.power.common.util.CollectionUtil;
 import com.power.common.util.StringUtil;
 
 import java.util.ArrayList;
@@ -153,8 +152,8 @@ public void setName(String name) {
 	}
 
 	public List<ApiMethodDoc> getList() {
-		if (CollectionUtil.isEmpty(this.list)) {
-			return Collections.emptyList();
+		if (Objects.isNull(this.list)) {
+			return new ArrayList<>();
 		}
 		return list;
 	}
@@ -263,8 +262,8 @@ public String getDocClass() {
 
 	@Override
 	public List<IMethod> getMethods() {
-		if (CollectionUtil.isEmpty(this.list)) {
-			return Collections.emptyList();
+		if (Objects.isNull(this.list)) {
+			return new ArrayList<>();
 		}
 
 		return new ArrayList<>(this.list);

src/main/java/com/ly/doc/model/WebSocketDoc.java

@@ -20,8 +20,9 @@
  */
 package com.ly.doc.model;
 
-import java.util.Collections;
+import java.util.ArrayList;
 import java.util.List;
+import java.util.Objects;
 
 /**
  * the webSocket doc
@@ -76,8 +77,8 @@ public void setSubProtocols(String subProtocols) {
 	}
 
 	public List<ApiParam> getPathParams() {
-		if (pathParams == null) {
-			return Collections.emptyList();
+		if (Objects.isNull(this.pathParams)) {
+			return new ArrayList<>();
 		}
 		return pathParams;
 	}
@@ -88,7 +89,7 @@ public void setPathParams(List<ApiParam> pathParams) {
 
 	public List<ApiParam> getMessageParams() {
 		if (messageParams == null) {
-			return Collections.emptyList();
+			return new ArrayList<>();
 		}
 		return messageParams;
 	}

src/main/java/com/ly/doc/model/javadoc/JavadocApiDoc.java

@@ -23,13 +23,17 @@
 import com.ly.doc.model.IDoc;
 import com.ly.doc.model.IMethod;
 import com.ly.doc.model.JavadocJavaMethod;
-import com.power.common.util.CollectionUtil;
 
 import java.util.ArrayList;
-import java.util.Collections;
 import java.util.List;
 import java.util.Objects;
 
+/**
+ * Java Doc ApiDoc
+ *
+ * @author chenchuxin
+ * @since 3.0.5
+ */
 public class JavadocApiDoc implements IDoc, Comparable<JavadocApiDoc> {
 
 	/**
@@ -181,8 +185,8 @@ public String getDocClass() {
 
 	@Override
 	public List<IMethod> getMethods() {
-		if (CollectionUtil.isEmpty(this.list)) {
-			return Collections.emptyList();
+		if (Objects.isNull(this.list)) {
+			return new ArrayList<>();
 		}
 		return new ArrayList<>(this.list);
 	}