fix: fix die loop

Author: linyimin0812

src/main/java/io/github/linyimin/plugin/compile/MybatisPojoCompile.java

@@ -45,7 +45,6 @@ public static void setClassLoader(Project project) {
         List<String> dependencies = getProjectDependencies(project);
 
         List<URL> urls = new ArrayList<>();
-        final List<String> list = OrderEnumerator.orderEntries(project).recursively().runtimeOnly().getPathsList().getPathList();
         for (String path : dependencies) {
             try {
                 urls.add(new File(FileUtil.toSystemIndependentName(path)).toURI().toURL());

src/main/java/io/github/linyimin/plugin/dom/Constant.java

@@ -34,6 +34,8 @@ public class Constant {
     public static final String MYBATIS_LOGGING_SLF4J = "slf4j";
     public static final String MYBATIS_LOGGING_LOG4J = "log4j";
 
+    public static final String JAR_FILE_END = "!";
+
     private static final String PATTERN = "yyyy-MM-dd HH:mm:ss";
     private static final DateFormat DF = new SimpleDateFormat(PATTERN);
     public static Map<String, Object> normalTypes = new ContainerUtil.ImmutableMapBuilder<String, Object>()

src/main/java/io/github/linyimin/plugin/service/SqlParamGenerateService.java

@@ -61,7 +61,7 @@ private boolean checkNeedCompile(String mybatisConfig, String methodQualifiedNam
 
         try {
             MybatisSqlUtils.getSql(mybatisConfig, methodQualifiedName, params);
-        } catch (Exception e) {
+        } catch (Exception | NoClassDefFoundError e) {
             return true;
         }
 
@@ -171,9 +171,9 @@ private String parseParamNameTypeList(List<ParamNameType> paramNameTypes) {
             } else {
                 Map<String, Object> classParam = getFieldFromClass(psiClass);
                 if (paramNameType.isParamAnnotation) {
-                    param.putAll(classParam);
-                } else {
                     param.put(name, classParam);
+                } else {
+                    param.putAll(classParam);
                 }
             }
 

src/main/java/io/github/linyimin/plugin/utils/JavaUtils.java

@@ -5,6 +5,7 @@
 import com.intellij.psi.*;
 import com.intellij.psi.impl.compiled.ClsFileImpl;
 import com.intellij.psi.search.GlobalSearchScope;
+import io.github.linyimin.plugin.dom.Constant;
 import org.apache.commons.lang.ArrayUtils;
 import org.apache.commons.lang.StringUtils;
 import org.jetbrains.annotations.NotNull;
@@ -169,8 +170,8 @@ public static Set<String> getAllDependenciesRecursive(Project project) {
                 visited.add(clazz);
 
                 Set<PsiClass> dependencies = getAllDependencies(clazz);
-                dependencies.remove(psiClass);
-                queue.addAll(visited);
+                dependencies.removeAll(visited);
+                queue.addAll(dependencies);
             }
 
             visited.remove(psiClass);
@@ -180,6 +181,7 @@ public static Set<String> getAllDependenciesRecursive(Project project) {
         return allDependencies.stream()
                 .map(item -> item.getContainingFile().getVirtualFile().getCanonicalPath())
                 .filter(path -> StringUtils.isNotEmpty(path) && path.contains("jar!"))
+                .map(path -> StringUtils.substring(path, 0, path.lastIndexOf(Constant.JAR_FILE_END)))
                 .collect(Collectors.toSet());
     }
 }