fix ClassFileContainer.getName

oraveczandrew · oraveczandrew · commit ddf39d821875 · 2025-03-09T06:32:35.000+01:00
diff --git a/src/main/java/the/bytecode/club/bytecodeviewer/gui/util/BytecodeViewPanelUpdater.java b/src/main/java/the/bytecode/club/bytecodeviewer/gui/util/BytecodeViewPanelUpdater.java
@@ -110,12 +110,18 @@ public void processDisplay()
                 else
                 {
                     final Decompiler decompiler = bytecodeViewPanel.decompiler;
-                    final String workingDecompilerName = viewer.resource.workingName + "-" + decompiler.getDecompilerName();
+                    String decompilerName = decompiler.getDecompilerName();
+                    final String workingDecompilerName = viewer.resource.workingName + "-" + decompilerName;
 
                     //perform decompiling inside of this thread
                     final String decompiledSource = decompiler.getDecompiler().decompileClassNode(viewer.resource.getResourceClassNode(), classBytes);
 
-                    ClassFileContainer container = new ClassFileContainer(workingDecompilerName, decompiledSource, viewer.resource.container);
+                    ClassFileContainer container = new ClassFileContainer(
+                        viewer.resource.workingName,
+                        decompilerName,
+                        decompiledSource,
+                        viewer.resource.container
+                    );
 
                     if (!BytecodeViewer.viewer.workPane.classFiles.containsKey(workingDecompilerName))
                     {
diff --git a/src/main/java/the/bytecode/club/bytecodeviewer/resources/classcontainer/ClassFileContainer.java b/src/main/java/the/bytecode/club/bytecodeviewer/resources/classcontainer/ClassFileContainer.java
@@ -1,6 +1,7 @@
 package the.bytecode.club.bytecodeviewer.resources.classcontainer;
 
-import com.github.javaparser.*;
+import com.github.javaparser.JavaParser;
+import com.github.javaparser.ParseResult;
 import com.github.javaparser.ast.CompilationUnit;
 import com.github.javaparser.resolution.TypeSolver;
 import com.github.javaparser.symbolsolver.JavaSymbolSolver;
@@ -12,7 +13,6 @@
 import the.bytecode.club.bytecodeviewer.resources.classcontainer.locations.*;
 import the.bytecode.club.bytecodeviewer.resources.classcontainer.parser.visitors.MyVoidVisitor;
 
-import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
 import java.util.NavigableMap;
@@ -27,6 +27,7 @@
  */
 public class ClassFileContainer
 {
+
     public transient NavigableMap<String, ArrayList<ClassFieldLocation>> fieldMembers = new TreeMap<>();
     public transient NavigableMap<String, ArrayList<ClassParameterLocation>> methodParameterMembers = new TreeMap<>();
     public transient NavigableMap<String, ArrayList<ClassLocalVariableLocation>> methodLocalMembers = new TreeMap<>();
@@ -35,13 +36,15 @@ public class ClassFileContainer
 
     public boolean hasBeenParsed = false;
     public final String className;
+    public final String decompiler;
     private final String content;
     private final String parentContainer;
     private final String path;
 
-    public ClassFileContainer(String className, String content, ResourceContainer resourceContainer)
+    public ClassFileContainer(String className, String decompiler, String content, ResourceContainer resourceContainer)
     {
         this.className = className;
+        this.decompiler = decompiler;
         this.content = content;
         this.parentContainer = resourceContainer.name;
         this.path = resourceContainer.file.getAbsolutePath();
@@ -95,15 +98,19 @@ public boolean shouldParse()
 
     public String getName()
     {
-        if (this.className.contains("/"))
-            return this.className.substring(this.className.lastIndexOf('/') + 1, this.className.lastIndexOf('.'));
-        else
-            return this.className.substring(0, this.className.lastIndexOf('.'));
+        int from = className.lastIndexOf('/') + 1;
+
+        int until = className.lastIndexOf('.');
+        if (until == -1 || until < from) {
+            until = className.length();
+        }
+
+        return className.substring(from, until);
     }
 
     public String getDecompiler()
     {
-        return this.className.substring(this.className.lastIndexOf('-') + 1);
+        return decompiler;
     }
 
     public String getParentContainer()
