From: Kai Moritz <kai@juplo.de>
Date: Tue, 20 Feb 2024 16:32:54 +0000 (+0100)
Subject: refactor: DRY for logging-category from io.projectreactor
X-Git-Tag: rebase--2024-02-23--16-28~17
X-Git-Url: http://juplo.de/gitweb/?a=commitdiff_plain;ds=sidebyside;h=7107fd0b4795d95b6c7f49cf4e765b82c16d016e;p=demos%2Fkafka%2Fchat

refactor: DRY for logging-category from io.projectreactor
---

diff --git a/src/main/java/de/juplo/kafka/chat/backend/api/ChatBackendController.java b/src/main/java/de/juplo/kafka/chat/backend/api/ChatBackendController.java
index 47ae6a5d..6c7cff7d 100644
--- a/src/main/java/de/juplo/kafka/chat/backend/api/ChatBackendController.java
+++ b/src/main/java/de/juplo/kafka/chat/backend/api/ChatBackendController.java
@@ -18,6 +18,7 @@ public class ChatBackendController
 {
   private final ChatHomeService chatHomeService;
   private final StorageStrategy storageStrategy;
+  private final String loggingCategory = ChatBackendController.class.getSimpleName();
   private final Level loggingLevel;
   private final boolean showOperatorLine;
 
@@ -133,7 +134,7 @@ public class ChatBackendController
     return chatRoomData
         .listen()
         .log(
-            ChatBackendController.class.getSimpleName(),
+            loggingCategory,
             loggingLevel,
             showOperatorLine)
         .map(message -> MessageTo.from(message))
diff --git a/src/main/java/de/juplo/kafka/chat/backend/storage/files/FilesStorageStrategy.java b/src/main/java/de/juplo/kafka/chat/backend/storage/files/FilesStorageStrategy.java
index aaa61598..7c1fef02 100644
--- a/src/main/java/de/juplo/kafka/chat/backend/storage/files/FilesStorageStrategy.java
+++ b/src/main/java/de/juplo/kafka/chat/backend/storage/files/FilesStorageStrategy.java
@@ -33,6 +33,7 @@ public class FilesStorageStrategy implements StorageStrategy
   private final Path storagePath;
   private final ShardingStrategy shardingStrategy;
   private final ObjectMapper mapper;
+  private final String loggingCategory = FilesStorageStrategy.class.getSimpleName();
   private final Level loggingLevel;
   private final boolean showOperatorLine;
 
@@ -53,7 +54,7 @@ public class FilesStorageStrategy implements StorageStrategy
 
       return chatRoomInfoFlux
           .log(
-              FilesStorageStrategy.class.getSimpleName(),
+              loggingCategory,
               loggingLevel,
               showOperatorLine)
           .doFirst(() ->
@@ -107,7 +108,7 @@ public class FilesStorageStrategy implements StorageStrategy
     return Flux
         .from(new JsonFilePublisher<ChatRoomInfoTo>(chatroomsPath(), mapper, type))
         .log(
-            FilesStorageStrategy.class.getSimpleName(),
+            loggingCategory,
             loggingLevel,
             showOperatorLine)
         .map(chatRoomInfoTo ->
@@ -146,7 +147,7 @@ public class FilesStorageStrategy implements StorageStrategy
 
       return messageFlux
           .log(
-              FilesStorageStrategy.class.getSimpleName(),
+              loggingCategory,
               loggingLevel,
               showOperatorLine)
           .doFirst(() ->
@@ -200,7 +201,7 @@ public class FilesStorageStrategy implements StorageStrategy
     return Flux
         .from(new JsonFilePublisher<MessageTo>(chatroomPath(chatRoomId), mapper, type))
         .log(
-            FilesStorageStrategy.class.getSimpleName(),
+            loggingCategory,
             loggingLevel,
             showOperatorLine)
         .map(MessageTo::toMessage);