Skip to content

Conversation

@czmcm5
Copy link
Contributor

@czmcm5 czmcm5 commented Jun 28, 2025

개요

지원하기, 좋아요 동작 완료 시 alert 대신 snakbar으로 메세지를 띄우는 것으로 변경하며 해당 훅과 컴포넌트에 대한 코드 리팩토링을 진행하였습니다.

변경 사항

  • 새로운 기능 추가
  • 버그 수정
  • 리팩토링
  • 문서 수정ㅁ

구현 내용

  • ProjectLike.tsx

    • 불필요한 loading 중복검사 식 제거
  • ProjectApplyForm.tsx / useApplyFrom.ts

    • 관심사 분리 원칙을 적용하여 비지니스 로직(hook)과 인터페이스(view)를 분리
    • 모든 비지니스 로직들은 useApplyFrom.ts에서 관리 합니다.
  • useCreateProjectLike.ts

    • 기존의 react-query함수와 커스텀훅이 동시에 선언되어 있어 각각 분리 후 알맞은 파일로 이동시켰습니다. (hook/useOptimisticProjectLike.ts)
  • useOptimisticProjectLike.ts

    • 불필요한 메모제이션을 삭제하였습니다.
    • snakbarAlert를 추가하여 좋아요/ 좋아요 취소 후 알림창이 보이도록 수정하였습니다.
  • useCancelProjectApplication.ts

    • snakbarAlert를 추가하여 지원하기/ 지원 취소 후 알림창이 보이도록 수정하였습니다.

개발 후기 및 개선사항

이번 작업에서 배운 점

  • (없다면 패스)

어려웠던 점 / 에로사항

  • (없다면 패스)

다음에 개선하고 싶은 점

  • 전역변수 user나 useParams로 값을 가져오는 projectId의 경우 대부분 파일에서 인자로 넘겨주기 위해서만 사용되는 중인데 추후에 각 컴포넌트의 의존도를 낮추기 위해 로직들을 다시 검토해보는 것도 좋은 공부가 될 것 같습니다.

팀원들과 공유하고 싶은 팁

  • 현재 hook과 view의 경계가 모호한 컴포넌트가 많습니다. 당장 오늘까지가 마감이라 당연히 다 수정할 수는 없지만, 이렇게 개선해보면서 관심사 분리의 원칙에 대해 공부하고 생각해보는 시간을 가져도 좋을 것 같습니다 ! 😆 저는 이런거 할 때가 제일 재밋더라구요

Copy link
Contributor

@namee-h namee-h left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

많이 알지 못하지만 회고회의가 기대가 됩니다 !! 🙌🏻
많이 알려주세용 다빈님 ㅋㅋㅋ

@tkyoun0421 tkyoun0421 merged commit 0b96540 into amicable-development-center:develop Jun 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: refactor 코드 리팩토링

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants