Merge pull request #65 from KERT-core/hotfix/api-response

hotfix: 코드 문법 오류 및 비효율적인 로직 핫픽스
KERT-core · Nov 3, 2024 · 3d2571b · 3d2571b
2 parents 3167776 + b712ca3
commit 3d2571b
Show file tree

Hide file tree

Showing 2 changed files with 66 additions and 74 deletions.
diff --git a/src/pages/MyPage.jsx b/src/pages/MyPage.jsx
@@ -157,6 +157,7 @@ const InputGroupLong = styled.div`
     &:focus {
       border: 1px solid #4a90e2;
       box-shadow: none;
+    }
   }
 `;
 
@@ -169,7 +170,7 @@ const EditButton = styled.button`
   cursor: pointer;
   width: 200px;
   margin-left: auto;
-  matgin-top: auto;
+  margin-top: auto;
 `;
 
 const WarningMessage = styled.p`
@@ -302,7 +303,7 @@ export default function MyPage() {
     }
   };
 
-  const onSubmit = async (data) => {
+  const onSubmit = (data) => {
     setPasswordError('');
 
     if (data.newPassword !== data.confirmPassword) {
@@ -312,70 +313,64 @@ export default function MyPage() {
 
     const token = localStorage.getItem('accessToken');
 
-    try {
-      // 서버로 비밀번호 정보를 전송
-      const response = await API.POST(`/users/${user.student_id}`, {
-        body: data,
-        headers: {
-          'Content-Type': 'multipart/form-data',
-          Authorization: token,
-        },
-      });
-      // console.log('서버로 전송:', response.data);
-      setTimeout(() => {
+    // 서버로 비밀번호 정보를 전송
+    API.POST(`/users/${user.student_id}`, {
+      body: data,
+      headers: {
+        'Content-Type': 'multipart/form-data',
+        Authorization: token,
+      },
+    })
+      .then(() => {
         openAlert({
           title: '비밀번호 변경',
           content: <Text>비밀번호가 성공적으로 변경되었습니다.</Text>,
           onClose: () => closeAlert(),
         });
-      }, 100);
-    } catch (error) {
-      console.error('오류 발생:', error);
-      setTimeout(() => {
+      })
+      .catch((error) => {
+        console.error('오류 발생:', error);
         openAlert({
           title: '비밀번호 재설정 실패',
           content: (
             <Text>비밀번호 재설정에 실패했습니다. 다시 시도해주세요.</Text>
           ),
           onClose: () => closeAlert(),
         });
-      }, 100);
-    }
+      });
   };
 
-  const handleDeleteAccount = async () => {
+  const handleDeleteAccount = () => {
     const confirmDelete = window.confirm(
       '계정을 삭제하면 복구할 수 없습니다. 정말로 삭제하시겠습니까?',
     );
-    if (confirmDelete) {
-      try {
-        const token = localStorage.getItem('accessToken');
-        await API.DELETE(`/users/${user.student_id}`, {
-          headers: {
-            Authorization: token,
+    if (!confirmDelete) return;
+
+    const token = localStorage.getItem('accessToken');
+    API.DELETE(`/users/${user.student_id}`, {
+      headers: {
+        Authorization: token,
+      },
+    })
+      .then(() => {
+        openAlert({
+          title: '계정 삭제',
+          content: <Text>계정이 성공적으로 삭제되었습니다.</Text>,
+          onClose: () => {
+            closeAlert();
+            logout();
+            navigate('/login');
           },
         });
-        // 100ms 후 새로운 Alert 열기
-        setTimeout(() => {
-          openAlert({
-            title: '계정 삭제',
-            content: <Text>계정이 성공적으로 삭제되었습니다.</Text>,
-            onClose: () => closeAlert(),
-          });
-        }, 100);
-        logout();
-        navigate('/login');
-      } catch (error) {
+      })
+      .catch((error) => {
         console.error('계정 삭제 실패:', error);
-        setTimeout(() => {
-          openAlert({
-            title: '계정 삭제 실패',
-            content: <Text>계정 삭제에 실패했습니다. 다시 시도해주세요.</Text>,
-            onClose: () => closeAlert(),
-          });
-        }, 100);
-      }
-    }
+        openAlert({
+          title: '계정 삭제 실패',
+          content: <Text>계정 삭제에 실패했습니다. 다시 시도해주세요.</Text>,
+          onClose: () => closeAlert(),
+        });
+      });
   };
 
   return (

diff --git a/src/pages/SignUp.jsx b/src/pages/SignUp.jsx
@@ -130,45 +130,42 @@ export default function SignUp() {
   const { openAlert, closeAlert, isOpen } = useAlert();
 
   const onSubmit = async (data) => {
-    try {
-      const formData = {
-        student_id: parseInt(data.student),
-        name: data.username,
-        email: data.mail,
-        profile_picture: '',
-        generation: data.generation,
-        major: data.major,
-        password: data.password,
-      };
+    const formData = {
+      student_id: parseInt(data.student),
+      name: data.username,
+      email: data.mail,
+      profile_picture: '',
+      generation: data.generation,
+      major: data.major,
+      password: data.password,
+    };
 
-      const response = await API.POST('/users/signup', { body: formData });
-      // console.log('서버로 전송:', response.data.user);
-      setTimeout(() => {
+    API.POST('/users/signup', { body: formData })
+      .then(() => {
         openAlert({
           title: '회원가입 요청 완료',
           content: <Text>회원가입 요청이 완료되었습니다!</Text>,
-          onClose: () => closeAlert(),
+          onClose: () => {
+            closeAlert();
+            navigate('/');
+          },
         });
-      }, 100);
-      navigate('/');
-    } catch (error) {
-      console.error('Error:', error);
-      setValue('username', '');
-      setValue('student', '');
-      setValue('password', '');
-      setValue('mail', '');
-      setValue('generation', '');
-      setValue('major', '');
+      })
+      .catch((error) => {
+        console.error('Error:', error);
+        setValue('username', '');
+        setValue('student', '');
+        setValue('password', '');
+        setValue('mail', '');
+        setValue('generation', '');
+        setValue('major', '');
 
-      // 100ms 후 새로운 Alert 열기
-      setTimeout(() => {
         openAlert({
           title: '회원가입 실패',
           content: <Text>회원가입에 실패했습니다. 다시 시도해주세요.</Text>,
           onClose: () => closeAlert(),
         });
-      }, 100);
-    }
+      });
   };
 
   return (