feat: improve project code

src/main/java/cn/pupperclient/PupperClient.java

@@ -16,9 +16,9 @@
 import cn.pupperclient.management.profile.ProfileManager;
 import cn.pupperclient.management.websocket.WebSocketManager;
 import cn.pupperclient.skia.font.Fonts;
-import cn.pupperclient.utils.ExternalToolManager;
-import cn.pupperclient.utils.IMinecraft;
-import cn.pupperclient.utils.Multithreading;
+import cn.pupperclient.utils.system.ExternalToolManager;
+import cn.pupperclient.utils.minecraft.interfaces.IMinecraft;
+import cn.pupperclient.utils.thread.Multithreading;
 import cn.pupperclient.utils.file.FileLocation;
 import cn.pupperclient.utils.language.*;
 import com.viaversion.viafabricplus.ViaFabricPlus;
@@ -73,6 +73,8 @@ public void onShutdown() {
         if (keybindManager != null) {
             keybindManager.cleanup();
         }
+        cn.pupperclient.skia.Skia.getImageHelper().clear();
+        cn.pupperclient.skia.font.FontHelper.clearCache();
         Multithreading.shutdown();
     }
 
@@ -141,7 +143,7 @@ private void registerTermsScreenCheck() {
         ClientTickEvents.END_CLIENT_TICK.register(client -> {
             if (firstLaunch && client.world != null && !hasAcceptedTerms) {
                 TermsScreen termsScreen = new TermsScreen();
-                client.setScreen(termsScreen.build());
+                client.setScreen(termsScreen);
             }
         });
     }

src/main/java/cn/pupperclient/PupperEventHandle.java

@@ -1,27 +1,16 @@
 package cn.pupperclient;
 
 import java.util.List;
-import java.util.Objects;
 
 import cn.pupperclient.event.EventBus;
 import cn.pupperclient.event.EventListener;
 import cn.pupperclient.event.EventType;
 import cn.pupperclient.event.client.*;
 import cn.pupperclient.event.server.impl.GameJoinEvent;
 import cn.pupperclient.management.profile.Profile;
-import cn.pupperclient.utils.ChatUtils;
-import net.fabricmc.fabric.api.client.message.v1.ClientReceiveMessageEvents;
 import net.minecraft.client.MinecraftClient;
 
 public class PupperEventHandle {
-
-    @EventListener
-    public void onMotion(EventMotion e) {
-        if (MinecraftClient.getInstance().player != null && e.getType() == EventType.PRE && MinecraftClient.getInstance().player.deathTime <= 1) {
-            EventBus.getInstance().post(new EventRespawn());
-        }
-    }
-
 	public final EventBus.EventListener<ClientTickEvent> onClientTick = event -> {
 		PupperClient.getInstance().getColorManager().onTick();
 		PupperClient.getInstance().getHypixelManager().update();

src/main/java/cn/pupperclient/event/EventBus.java

@@ -110,6 +110,9 @@ public <T extends Event> void post(T event) {
 
         if (listeners != null) {
             for (EventListener<?> listener : listeners) {
+                if (event.isCancelled()) {
+                    break;
+                }
                 try {
                     @SuppressWarnings("unchecked")
                     EventListener<T> castedListener = (EventListener<T>) listener;
@@ -178,7 +181,7 @@ private EventListener<?> getEventHandler(final Object o, final Field field) {
         try {
             fieldSubscription = (EventListener<?>) field.get(o);
         } catch (IllegalAccessException e) {
-            e.printStackTrace();
+            cn.pupperclient.PupperLogger.error("EventBus", "Failed to get event handler", e);
         } finally {
             field.setAccessible(accessible);
         }

src/main/java/cn/pupperclient/event/client/TotemEvent.java

@@ -4,7 +4,7 @@
 import net.minecraft.entity.player.PlayerEntity;
 
 public class TotemEvent extends Event {
-    private PlayerEntity player;
+    private final PlayerEntity player;
 
     public TotemEvent(PlayerEntity entity) {
         player = entity;

src/main/java/cn/pupperclient/gui/GuiResourcePackConvert.java

@@ -15,8 +15,8 @@
 import com.google.gson.JsonObject;
 import com.google.gson.JsonParser;
 import cn.pupperclient.libraries.resourcepack.ResourcePackConverter;
-import cn.pupperclient.utils.JsonUtils;
-import cn.pupperclient.utils.Multithreading;
+import cn.pupperclient.utils.misc.JsonUtils;
+import cn.pupperclient.utils.thread.Multithreading;
 import cn.pupperclient.utils.file.FileLocation;
 
 import it.unimi.dsi.fastutil.objects.ObjectObjectImmutablePair;
@@ -65,7 +65,7 @@ private ResourcePackConverter createConverter() {
 		try {
 			Files.createDirectories(cacheDir.toPath());
 		} catch (IOException e) {
-			e.printStackTrace();
+			PupperClient.LOGGER.error("Failed to create cache directory", e);
 		}
 
 		for(File f : detectPacks()) {
@@ -80,7 +80,7 @@ private ResourcePackConverter createConverter() {
 
 				packs.add(ObjectObjectImmutablePair.of(targetFile, outputFile));
 			} catch (Exception e) {
-				e.printStackTrace();
+				PupperClient.LOGGER.error("Failed to move resource pack", e);
 			}
 		}
 
@@ -121,7 +121,7 @@ private List<File> detectPacks() {
 					zipIn.closeEntry();
 				}
 			} catch (IOException e) {
-				e.printStackTrace();
+				PupperClient.LOGGER.error("Failed to detect resource packs", e);
 			}
 		}