From 213f27465bdf3319665b44dad6834ec7009f73ae Mon Sep 17 00:00:00 2001 From: MinseoLee <mmmm@ajou.ac.kr> Date: Fri, 8 Dec 2023 22:29:13 +0900 Subject: [PATCH] style: no `console.log()` in production --- src/common/instances/AdminSocket.ts | 5 +++-- src/common/instances/Socket.ts | 5 +++-- src/common/utils/logger.ts | 6 ++++++ src/contexts/bottom-sheet/index.tsx | 4 +++- src/contexts/cart/index.tsx | 4 +++- src/contexts/toast/index.tsx | 4 +++- 6 files changed, 21 insertions(+), 7 deletions(-) create mode 100644 src/common/utils/logger.ts diff --git a/src/common/instances/AdminSocket.ts b/src/common/instances/AdminSocket.ts index 2ba1136..4d7d4ea 100644 --- a/src/common/instances/AdminSocket.ts +++ b/src/common/instances/AdminSocket.ts @@ -1,6 +1,7 @@ import { io } from "socket.io-client"; import { SOCKET_URL } from "../utils/api"; +import logger from "../utils/logger"; class AdminSocket { @@ -8,11 +9,11 @@ class AdminSocket { constructor() { this.ioInstance.on('connect', () => { - console.log('WS: admin connected'); + logger('WS: admin connected'); }); this.ioInstance.on('auth', (e) => { - console.log('WS: admin auth', e); + logger('WS: admin auth', e); }); } diff --git a/src/common/instances/Socket.ts b/src/common/instances/Socket.ts index 4d450a1..8c22bc8 100644 --- a/src/common/instances/Socket.ts +++ b/src/common/instances/Socket.ts @@ -1,6 +1,7 @@ import { io } from "socket.io-client"; import { SOCKET_URL } from "../utils/api"; +import logger from "../utils/logger"; class Socket { @@ -8,11 +9,11 @@ class Socket { constructor() { this.ioInstance.on('connect', () => { - // console.log('Connected to Socket.IO server'); + logger('Connected to Socket.IO server'); }); this.ioInstance.on('auth', (e) => { - // console.log('auth', e); + logger('auth', e); }); } diff --git a/src/common/utils/logger.ts b/src/common/utils/logger.ts new file mode 100644 index 0000000..83c095a --- /dev/null +++ b/src/common/utils/logger.ts @@ -0,0 +1,6 @@ +function logger(message?: any, ...optionalParams: any[]): string|void { + if (process.env.NODE_ENV === 'production') return; + else console.log(message, ...optionalParams); +} + +export default logger; diff --git a/src/contexts/bottom-sheet/index.tsx b/src/contexts/bottom-sheet/index.tsx index b785c9b..f636107 100644 --- a/src/contexts/bottom-sheet/index.tsx +++ b/src/contexts/bottom-sheet/index.tsx @@ -1,5 +1,7 @@ import React, { createContext, useEffect, useState } from "react"; +import logger from "../../common/utils/logger"; + import type { FC, ReactNode, JSX } from "react"; interface BSContextType { @@ -19,7 +21,7 @@ const BSProvider: FC<Props> = ({ children }) => { const [item, setItem] = useState<BSContextType['bSElement']>(null); useEffect(() => { - // console.log('GLOBAL BSProvider: ', item); + logger('GLOBAL BSProvider: ', item); } ,[item]); const setBSElement = (el: ReactNode | null) => { diff --git a/src/contexts/cart/index.tsx b/src/contexts/cart/index.tsx index 9e95448..bcb80dc 100644 --- a/src/contexts/cart/index.tsx +++ b/src/contexts/cart/index.tsx @@ -1,5 +1,7 @@ import { createContext, useEffect, useState } from "react"; +import logger from "../../common/utils/logger"; + import type { CartItemData } from "../../pages/cart-page/config/type"; import type { FC, ReactNode } from "react"; @@ -29,7 +31,7 @@ const CartProvider: FC<Props> = ({ children }) => { flushCartItem(); } - // console.log('GLOBAL CartProvider: ', item); + logger('GLOBAL CartProvider: ', item); }, [item]); diff --git a/src/contexts/toast/index.tsx b/src/contexts/toast/index.tsx index 0a91a83..a571504 100644 --- a/src/contexts/toast/index.tsx +++ b/src/contexts/toast/index.tsx @@ -1,5 +1,7 @@ import { createContext, useEffect, useState } from "react"; +import logger from "../../common/utils/logger"; + import type { ToastData } from "../../components/toast/config"; import type { FC, ReactNode } from "react"; @@ -19,7 +21,7 @@ const ToastProvider: FC<Props> = ({ children }) => { useEffect(() => { - console.log('GLOBAL ToastProvider: ', item); + logger('GLOBAL ToastProvider: ', item); }, [item]); -- GitLab