Add error counting for caching offline player

sandtechnology · sandtechnology · commit 4d9edd6a970b · 2023-01-04T16:09:49.000+08:00
diff --git a/src/main/java/org/maxgamer/quickshop/util/PlayerFinder.java b/src/main/java/org/maxgamer/quickshop/util/PlayerFinder.java
@@ -84,6 +84,7 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[
         quickShop.getLogger().log(Level.INFO, "Large server detected (offline player > 2000), start offline player caching...");
         useOfflineStash = true;
         int amount = 0;
+        int errorAmount = 0;
         int doneAmount = 0;
         for (OfflinePlayer offlinePlayer : quickShop.getServer().getOfflinePlayers()) {
             try {
@@ -92,15 +93,16 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[
                     string2UUIDStash.put(name, offlinePlayer.getUniqueId());
                 }
             } catch (Throwable ignored) {
+                errorAmount++;
             }
             amount++;
             if (amount == 1000) {
                 doneAmount += 1000;
                 amount = 0;
-                quickShop.getLogger().log(Level.INFO, "Caching Offline player...cached " + doneAmount + "/" + offlinePlayers.length + " players.");
+                quickShop.getLogger().log(Level.INFO, "Caching Offline player...cached " + doneAmount + "/" + offlinePlayers.length + " players, " + errorAmount + " errors got when caching players.");
             }
         }
-        quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players.");
+        quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players, " + errorAmount + " errors got when caching players.");
     }
 
     public static OfflinePlayer findOfflinePlayerByName(String name) {

Original file line number	Diff line number	Diff line change
`@@ -84,6 +84,7 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[`
`84`	`84`	`quickShop.getLogger().log(Level.INFO, "Large server detected (offline player > 2000), start offline player caching...");`
`85`	`85`	`useOfflineStash = true;`
`86`	`86`	`int amount = 0;`
	`87`	`+ int errorAmount = 0;`
`87`	`88`	`int doneAmount = 0;`
`88`	`89`	`for (OfflinePlayer offlinePlayer : quickShop.getServer().getOfflinePlayers()) {`
`89`	`90`	`try {`
`@@ -92,15 +93,16 @@ public static void doLargeOfflineCachingWork(QuickShop quickShop, OfflinePlayer[`
`92`	`93`	`string2UUIDStash.put(name, offlinePlayer.getUniqueId());`
`93`	`94`	`}`
`94`	`95`	`} catch (Throwable ignored) {`
	`96`	`+ errorAmount++;`
`95`	`97`	`}`
`96`	`98`	`amount++;`
`97`	`99`	`if (amount == 1000) {`
`98`	`100`	`doneAmount += 1000;`
`99`	`101`	`amount = 0;`
`100`		`- quickShop.getLogger().log(Level.INFO, "Caching Offline player...cached " + doneAmount + "/" + offlinePlayers.length + " players.");`
	`102`	`+ quickShop.getLogger().log(Level.INFO, "Caching Offline player...cached " + doneAmount + "/" + offlinePlayers.length + " players, " + errorAmount + " errors got when caching players.");`
`101`	`103`	`}`
`102`	`104`	`}`
`103`		`- quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players.");`
	`105`	`+ quickShop.getLogger().log(Level.INFO, "Done! cached " + offlinePlayers.length + " players, " + errorAmount + " errors got when caching players.");`
`104`	`106`	`}`
`105`	`107`
`106`	`108`	`public static OfflinePlayer findOfflinePlayerByName(String name) {`