feat: use logger

sysnote8main · sysnote8main · commit 795a2bfda294 · 2025-06-03T20:45:43.000+09:00
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/discord/DiscordHandler.java b/src/main/java/net/azisaba/ryuzupluginchat/discord/DiscordHandler.java
@@ -89,7 +89,7 @@ private void registerDiscordToGlobal(Snowflake discordChannelId) {
 
                 discordMessageDeliverer.sendToGlobal(event);
               } catch (Exception e) {
-                e.printStackTrace();
+                plugin.getSLF4JLogger().error("Failed to register discord channel for global chat", e);
               }
             });
   }
@@ -111,7 +111,7 @@ private void registerDiscordToChannels(Snowflake discordChannelId, ChannelChatSy
 
                 discordMessageDeliverer.sendToChannel(event, syncData);
               } catch (Exception e) {
-                e.printStackTrace();
+                plugin.getSLF4JLogger().error("Failed to register discord channel for specific channels", e);
               }
             });
   }
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/message/JsonDataConverter.java b/src/main/java/net/azisaba/ryuzupluginchat/message/JsonDataConverter.java
@@ -44,7 +44,7 @@ public SystemMessageData convertIntoSystemMessageData(String data) {
           (Map<String, Object>)
               mapper.readValue(data, new TypeReference<Map<String, Object>>() {}).get("map"));
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to processing json", e);
     }
     return null;
   }
@@ -53,7 +53,7 @@ private Object convertInto(String data, Class<?> clazz) {
     try {
       return mapper.readValue(data, clazz);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to data conversion", e);
     }
     return null;
   }
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/redis/MessagePublisher.java b/src/main/java/net/azisaba/ryuzupluginchat/redis/MessagePublisher.java
@@ -33,7 +33,7 @@ public boolean publishGlobalMessage(GlobalMessageData data) {
     try {
       jsonMessage = converter.convertIntoString(data);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to serialize data for publishing global message", e);
       return false;
     }
 
@@ -50,7 +50,7 @@ public boolean publishPrivateMessage(PrivateMessageData data) {
     try {
       jsonMessage = converter.convertIntoString(data);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to serialize data for publishing private message", e);
       return false;
     }
 
@@ -67,7 +67,7 @@ public boolean publishChannelChatMessage(ChannelChatMessageData data) {
     try {
       jsonMessage = converter.convertIntoString(data);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to serialize data for publishing channel chat message", e);
       return false;
     }
 
@@ -81,7 +81,7 @@ public boolean publishSystemMessage(SystemMessageData data) {
     try {
       jsonMessage = converter.convertIntoString(data);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to serialize data for publishing system message", e);
       return false;
     }
 
@@ -96,7 +96,7 @@ public void notifyPrivateChatReached(PrivateMessageData data) {
     try {
       jsonMessage = mapper.writeValueAsString(data);
     } catch (JsonProcessingException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to serialize data for notifying private chat reached", e);
       return;
     }
 
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/redis/MessageSubscriber.java b/src/main/java/net/azisaba/ryuzupluginchat/redis/MessageSubscriber.java
@@ -59,7 +59,7 @@ public void onPMessage(String pattern, String channel, String message) {
                             try {
                               c.accept(data);
                             } catch (Exception e) {
-                              e.printStackTrace();
+                              plugin.getSLF4JLogger().error("Failed to execute consumer for global-chat", e);
                             }
                           });
                     }
@@ -75,7 +75,7 @@ public void onPMessage(String pattern, String channel, String message) {
                             try {
                               c.accept(data);
                             } catch (Exception e) {
-                              e.printStackTrace();
+                              plugin.getSLF4JLogger().error("Failed to execute consumer for private-chat", e);
                             }
                           });
                     }
@@ -92,7 +92,7 @@ public void onPMessage(String pattern, String channel, String message) {
                             try {
                               c.accept(data);
                             } catch (Exception e) {
-                              e.printStackTrace();
+                              plugin.getSLF4JLogger().error("Failed to execute consumer for channel-chat", e);
                             }
                           });
                     }
@@ -108,7 +108,7 @@ public void onPMessage(String pattern, String channel, String message) {
                             try {
                               c.accept(data);
                             } catch (Exception e) {
-                              e.printStackTrace();
+                              plugin.getSLF4JLogger().error("Failed to execute consumer for onPMessage", e);
                             }
                           });
                     }
@@ -123,7 +123,7 @@ public void onPong(String pattern) {
               try {
                 consumer.accept(pattern);
               } catch (Exception e) {
-                e.printStackTrace();
+                plugin.getSLF4JLogger().error("Failed to execute consumer for onPong", e);
               }
             }
           }
@@ -136,7 +136,7 @@ public void onPong(String pattern) {
           try (Jedis jedis = jedisPool.getResource()) {
             jedis.psubscribe(subscriber, "rpc:" + groupName + ":*");
           } catch (JedisConnectionException e) {
-            e.printStackTrace();
+              plugin.getSLF4JLogger().error("Failed to subscribe for {}", groupName, e);
           }
         });
 
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/redis/PrivateChatReachedSubscriber.java b/src/main/java/net/azisaba/ryuzupluginchat/redis/PrivateChatReachedSubscriber.java
@@ -43,7 +43,7 @@ public void onMessage(String channel, String message) {
 
               plugin.getPrivateChatResponseWaiter().reached(data);
             } catch (JsonProcessingException e) {
-              e.printStackTrace();
+              plugin.getSLF4JLogger().error("Failed to process on message", e);
             }
           }
 
@@ -54,7 +54,7 @@ public void onPong(String pattern) {
               try {
                 consumer.accept(pattern);
               } catch (Exception e) {
-                e.printStackTrace();
+                plugin.getSLF4JLogger().error("Failed to process on pong", e);
               }
             }
           }
diff --git a/src/main/java/net/azisaba/ryuzupluginchat/updater/GitHubPluginUpdater.java b/src/main/java/net/azisaba/ryuzupluginchat/updater/GitHubPluginUpdater.java
@@ -59,7 +59,7 @@ public boolean checkUpdate() {
       String response = readUrlAsString(url);
       data = mapper.readValue(response, new TypeReference<Map<String, Object>>() {});
     } catch (IOException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to read data from update check url content", e);
       return false;
     }
 
@@ -152,7 +152,7 @@ public boolean executeDownloadLatestJar(File file) {
       status = UpdateStatus.ALREADY_UPDATED;
       return true;
     } catch (IOException e) {
-      e.printStackTrace();
+      plugin.getSLF4JLogger().error("Failed to download latest jar file", e);
       return false;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ private void registerDiscordToGlobal(Snowflake discordChannelId) {`
`89`	`89`
`90`	`90`	`discordMessageDeliverer.sendToGlobal(event);`
`91`	`91`	`} catch (Exception e) {`
`92`		`- e.printStackTrace();`
	`92`	`+ plugin.getSLF4JLogger().error("Failed to register discord channel for global chat", e);`
`93`	`93`	`}`
`94`	`94`	`});`
`95`	`95`	`}`
`@@ -111,7 +111,7 @@ private void registerDiscordToChannels(Snowflake discordChannelId, ChannelChatSy`
`111`	`111`
`112`	`112`	`discordMessageDeliverer.sendToChannel(event, syncData);`
`113`	`113`	`} catch (Exception e) {`
`114`		`- e.printStackTrace();`
	`114`	`+ plugin.getSLF4JLogger().error("Failed to register discord channel for specific channels", e);`
`115`	`115`	`}`
`116`	`116`	`});`
`117`	`117`	`}`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ public SystemMessageData convertIntoSystemMessageData(String data) {`
`44`	`44`	`(Map<String, Object>)`
`45`	`45`	`mapper.readValue(data, new TypeReference<Map<String, Object>>() {}).get("map"));`
`46`	`46`	`} catch (JsonProcessingException e) {`
`47`		`- e.printStackTrace();`
	`47`	`+ plugin.getSLF4JLogger().error("Failed to processing json", e);`
`48`	`48`	`}`
`49`	`49`	`return null;`
`50`	`50`	`}`
`@@ -53,7 +53,7 @@ private Object convertInto(String data, Class<?> clazz) {`
`53`	`53`	`try {`
`54`	`54`	`return mapper.readValue(data, clazz);`
`55`	`55`	`} catch (JsonProcessingException e) {`
`56`		`- e.printStackTrace();`
	`56`	`+ plugin.getSLF4JLogger().error("Failed to data conversion", e);`
`57`	`57`	`}`
`58`	`58`	`return null;`
`59`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public void onPMessage(String pattern, String channel, String message) {`
`59`	`59`	`try {`
`60`	`60`	`c.accept(data);`
`61`	`61`	`} catch (Exception e) {`
`62`		`- e.printStackTrace();`
	`62`	`+ plugin.getSLF4JLogger().error("Failed to execute consumer for global-chat", e);`
`63`	`63`	`}`
`64`	`64`	`});`
`65`	`65`	`}`
`@@ -75,7 +75,7 @@ public void onPMessage(String pattern, String channel, String message) {`
`75`	`75`	`try {`
`76`	`76`	`c.accept(data);`
`77`	`77`	`} catch (Exception e) {`
`78`		`- e.printStackTrace();`
	`78`	`+ plugin.getSLF4JLogger().error("Failed to execute consumer for private-chat", e);`
`79`	`79`	`}`
`80`	`80`	`});`
`81`	`81`	`}`
`@@ -92,7 +92,7 @@ public void onPMessage(String pattern, String channel, String message) {`
`92`	`92`	`try {`
`93`	`93`	`c.accept(data);`
`94`	`94`	`} catch (Exception e) {`
`95`		`- e.printStackTrace();`
	`95`	`+ plugin.getSLF4JLogger().error("Failed to execute consumer for channel-chat", e);`
`96`	`96`	`}`
`97`	`97`	`});`
`98`	`98`	`}`
`@@ -108,7 +108,7 @@ public void onPMessage(String pattern, String channel, String message) {`
`108`	`108`	`try {`
`109`	`109`	`c.accept(data);`
`110`	`110`	`} catch (Exception e) {`
`111`		`- e.printStackTrace();`
	`111`	`+ plugin.getSLF4JLogger().error("Failed to execute consumer for onPMessage", e);`
`112`	`112`	`}`
`113`	`113`	`});`
`114`	`114`	`}`
`@@ -123,7 +123,7 @@ public void onPong(String pattern) {`
`123`	`123`	`try {`
`124`	`124`	`consumer.accept(pattern);`
`125`	`125`	`} catch (Exception e) {`
`126`		`- e.printStackTrace();`
	`126`	`+ plugin.getSLF4JLogger().error("Failed to execute consumer for onPong", e);`
`127`	`127`	`}`
`128`	`128`	`}`
`129`	`129`	`}`
`@@ -136,7 +136,7 @@ public void onPong(String pattern) {`
`136`	`136`	`try (Jedis jedis = jedisPool.getResource()) {`
`137`	`137`	`jedis.psubscribe(subscriber, "rpc:" + groupName + ":*");`
`138`	`138`	`} catch (JedisConnectionException e) {`
`139`		`- e.printStackTrace();`
	`139`	`+ plugin.getSLF4JLogger().error("Failed to subscribe for {}", groupName, e);`
`140`	`140`	`}`
`141`	`141`	`});`
`142`	`142`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public void onMessage(String channel, String message) {`
`43`	`43`
`44`	`44`	`plugin.getPrivateChatResponseWaiter().reached(data);`
`45`	`45`	`} catch (JsonProcessingException e) {`
`46`		`- e.printStackTrace();`
	`46`	`+ plugin.getSLF4JLogger().error("Failed to process on message", e);`
`47`	`47`	`}`
`48`	`48`	`}`
`49`	`49`
`@@ -54,7 +54,7 @@ public void onPong(String pattern) {`
`54`	`54`	`try {`
`55`	`55`	`consumer.accept(pattern);`
`56`	`56`	`} catch (Exception e) {`
`57`		`- e.printStackTrace();`
	`57`	`+ plugin.getSLF4JLogger().error("Failed to process on pong", e);`
`58`	`58`	`}`
`59`	`59`	`}`
`60`	`60`	`}`
Original file line number	Diff line number	Diff line change
`@@ -59,7 +59,7 @@ public boolean checkUpdate() {`
`59`	`59`	`String response = readUrlAsString(url);`
`60`	`60`	`data = mapper.readValue(response, new TypeReference<Map<String, Object>>() {});`
`61`	`61`	`} catch (IOException e) {`
`62`		`- e.printStackTrace();`
	`62`	`+ plugin.getSLF4JLogger().error("Failed to read data from update check url content", e);`
`63`	`63`	`return false;`
`64`	`64`	`}`
`65`	`65`
`@@ -152,7 +152,7 @@ public boolean executeDownloadLatestJar(File file) {`
`152`	`152`	`status = UpdateStatus.ALREADY_UPDATED;`
`153`	`153`	`return true;`
`154`	`154`	`} catch (IOException e) {`
`155`		`- e.printStackTrace();`
	`155`	`+ plugin.getSLF4JLogger().error("Failed to download latest jar file", e);`
`156`	`156`	`return false;`
`157`	`157`	`}`
`158`	`158`	`}`