refactor: createLoggerを少しリファクタリング (#2489)

VOICEVOX · Jan 17, 2025 · 8e4eb03 · 8e4eb03
1 parent 4dcc415
commit 8e4eb03
Showing 1 changed file with 25 additions and 16 deletions.
diff --git a/src/domain/frontend/log.ts b/src/domain/frontend/log.ts
@@ -1,22 +1,31 @@
+type LogLevel = "info" | "warn" | "error";
+type LogFunction = (...args: unknown[]) => void;
+
 /** ログ出力用の関数を生成する。ブラウザ専用。 */
 // TODO: window.backendをDIできるようにする
-export function createLogger(scope: string) {
-  const createInner =
-    (
-      method: "logInfo" | "logWarn" | "logError",
-      fallbackMethod: "info" | "warn" | "error",
-    ) =>
-    (...args: unknown[]) => {
-      if (window.backend == undefined) {
-        // eslint-disable-next-line no-console
-        console[fallbackMethod](...args);
+export function createLogger(scope: string): Record<LogLevel, LogFunction> {
+  return {
+    info: createLogFunction("info"),
+    warn: createLogFunction("warn"),
+    error: createLogFunction("error"),
+  };
+
+  function createLogFunction(logType: LogLevel): LogFunction {
+    return (...args: unknown[]) => {
+      if (window.backend != undefined) {
+        const method = (
+          {
+            info: "logInfo",
+            warn: "logWarn",
+            error: "logError",
+          } as const
+        )[logType];
+        window.backend[method](`[${scope}]`, ...args);
         return;
       }
-      window.backend[method](`[${scope}]`, ...args);
+
+      // eslint-disable-next-line no-console
+      console[logType](...args);
     };
-  return {
-    info: createInner("logInfo", "info"),
-    warn: createInner("logWarn", "warn"),
-    error: createInner("logError", "error"),
-  };
+  }
 }