Fix the issue of a null enumeration field causing a null pointer exception.

shalousun · shalousun · commit f9f368386d5a · 2024-02-02T00:53:52.000+08:00
diff --git a/src/main/java/com/thoughtworks/qdox/model/impl/DefaultJavaClass.java b/src/main/java/com/thoughtworks/qdox/model/impl/DefaultJavaClass.java
@@ -19,13 +19,7 @@
  * under the License.
  */
 
-import java.util.Collection;
-import java.util.LinkedHashMap;
-import java.util.LinkedHashSet;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
+import java.util.*;
 
 import com.thoughtworks.qdox.library.ClassLibrary;
 import com.thoughtworks.qdox.model.BeanProperty;
@@ -589,7 +583,7 @@ public List<JavaField> getEnumConstants()
         {
             for ( JavaField field : getFields() )
             {
-                if ( field.isEnumConstant() )
+                if ( Objects.nonNull(field) && field.isEnumConstant() )
                 {
                     result.add( field );
                 }

Original file line number	Diff line number	Diff line change
`@@ -19,13 +19,7 @@`
`19`	`19`	`* under the License.`
`20`	`20`	`*/`
`21`	`21`
`22`		`-import java.util.Collection;`
`23`		`-import java.util.LinkedHashMap;`
`24`		`-import java.util.LinkedHashSet;`
`25`		`-import java.util.LinkedList;`
`26`		`-import java.util.List;`
`27`		`-import java.util.Map;`
`28`		`-import java.util.Set;`
	`22`	`+import java.util.*;`
`29`	`23`
`30`	`24`	`import com.thoughtworks.qdox.library.ClassLibrary;`
`31`	`25`	`import com.thoughtworks.qdox.model.BeanProperty;`
`@@ -589,7 +583,7 @@ public List<JavaField> getEnumConstants()`
`589`	`583`	`{`
`590`	`584`	`for ( JavaField field : getFields() )`
`591`	`585`	`{`
`592`		`- if ( field.isEnumConstant() )`
	`586`	`+ if ( Objects.nonNull(field) && field.isEnumConstant() )`
`593`	`587`	`{`
`594`	`588`	`result.add( field );`
`595`	`589`	`}`