♻️Refactor: else문 제거

prgrms-web-devcourse-final-project · Dec 5, 2024 · d2d49d5 · d2d49d5
1 parent a85bed3
commit d2d49d5
Show file tree

Hide file tree

Showing 6 changed files with 23 additions and 24 deletions.
diff --git a/.vscode/typescript.code-snippets b/.vscode/typescript.code-snippets
@@ -59,10 +59,10 @@
       "          default:",
       "            throw new Error(message || '알 수 없는 오류가 발생했습니다.')",
       "        }",
-      "      } else {",
-      "        // 요청 자체가 실패한 경우",
-      "        throw new Error('네트워크 연결을 확인해주세요')",
-      "      }",
+      "      } ",
+      "      // 요청 자체가 실패한 경우",
+      "      throw new Error('네트워크 연결을 확인해주세요')",
+      "      ",
       "    }",
       "",
       "    console.error('예상치 못한 에러:', error);",

diff --git a/src/apis/chatRoom/createChatRoom.ts b/src/apis/chatRoom/createChatRoom.ts
@@ -37,10 +37,9 @@ export const createChatRoom = async (req: CreateChatRoomRequest): Promise<APIRes
           default:
             throw new Error(message || '알 수 없는 오류가 발생했습니다.')
         }
-      } else {
-        // 요청 자체가 실패한 경우
-        throw new Error('네트워크 연결을 확인해주세요')
       }
+      // 요청 자체가 실패한 경우
+      throw new Error('네트워크 연결을 확인해주세요')
     }
 
     console.error('예상치 못한 에러:', error)

diff --git a/src/apis/chatRoom/fetchChatRoomList.ts b/src/apis/chatRoom/fetchChatRoomList.ts
@@ -26,10 +26,9 @@ export const fetchChatRoomList = async (): Promise<APIResponse<FetchChatRoomList
           default:
             throw new Error(message || '알 수 없는 오류가 발생했습니다.')
         }
-      } else {
-        // 요청 자체가 실패한 경우
-        throw new Error('네트워크 연결을 확인해주세요')
       }
+      // 요청 자체가 실패한 경우
+      throw new Error('네트워크 연결을 확인해주세요')
     }
 
     console.error('예상치 못한 에러:', error)

diff --git a/src/apis/friend/fetchFriendList.ts b/src/apis/friend/fetchFriendList.ts
@@ -26,10 +26,9 @@ export const fetchFriendList = async (): Promise<APIResponse<FetchFriendListResp
           default:
             throw new Error(message || '알 수 없는 오류가 발생했습니다.')
         }
-      } else {
-        // 요청 자체가 실패한 경우
-        throw new Error('네트워크 연결을 확인해주세요')
       }
+      // 요청 자체가 실패한 경우
+      throw new Error('네트워크 연결을 확인해주세요')
     }
 
     console.error('예상치 못한 에러:', error)

diff --git a/src/apis/member/fetchProfile.ts b/src/apis/member/fetchProfile.ts
@@ -38,10 +38,9 @@ export const fetchProfile = async ({ memberId }: FetchProfileRequest): Promise<A
           default:
             throw new Error(message || '알 수 없는 오류가 발생했습니다.')
         }
-      } else {
-        // 요청 자체가 실패한 경우
-        throw new Error('네트워크 연결을 확인해주세요')
       }
+      // 요청 자체가 실패한 경우
+      throw new Error('네트워크 연결을 확인해주세요')
     }
 
     console.error('예상치 못한 에러:', error)

diff --git a/src/components/SendMessageForm/index.tsx b/src/components/SendMessageForm/index.tsx
@@ -10,16 +10,19 @@ type SendMessageFormProps = React.FormHTMLAttributes<HTMLFormElement> & {
 export default function SendMessageForm({ chatRoomId, chatCount, ...rest }: SendMessageFormProps) {
   const onSubmit = async (e: React.FormEvent<HTMLFormElement>) => {
     e.preventDefault()
-    if (rest.onSubmit) rest.onSubmit(e)
-    else {
-      if (chatCount === 0) {
-        //* 채팅방 생성
-        await createChatRoom({ opponentMemberId: 123 })
-      }
-      //* 채팅 전송 웹소켓
+    if (rest.onSubmit) {
+      rest.onSubmit(e)
+      return
     }
+
+    if (chatCount === 0) {
+      //* 채팅방 생성
+      await createChatRoom({ opponentMemberId: 123 })
+    }
+    //* 채팅 전송 웹소켓
+    console.log('chatRoomId:', chatRoomId)
   }
-  console.log('chatRoomId:', chatRoomId)
+
   return (
     <S.SendMessageForm onSubmit={onSubmit} {...rest}>
       <S.ChatInput placeholder='채팅 내용 입력' />