Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[정한샘] Spring 8 #113

Conversation

koseha
Copy link
Collaborator

@koseha koseha commented Feb 27, 2025

요구사항

https://5-sprint-mission-fe-two.vercel.app/community

기본

  • [x]
  • []
  • []

심화

  • [x]
  • []

주요 변경사항

스크린샷

멘토에게

  • 셀프 코드 리뷰를 통해 질문 이어가겠습니다.

Copy link
Collaborator

@dennis2311 dennis2311 left a comment

Choose a reason for hiding this comment

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

잘 하셨습니다

  • 주석 주석 주석 주석주석 활용해주세요. JSDoc을 사용하는 것을 강력히 권장해 드립니다.
    • 꼭 상수와 함수에 대한 주석이 아니어도, HTML 구조가 복잡해지면 중간중간 주석을 이용하여 코드 섹션을 시각적으로 분리하여 주는 것이 좋습니다. 혹은, 섹션별 함수를 새로 정의하여 나누는 방법을 사용하는 것도 좋습니다.
  • 거의 모든 함수를 const 로 정의하고 계신데, 저 같은 경우 정확한 쓰임이 있는 것이 아니라면 함수에 const를 사용하지 않습니다. 가독성 측면에서도 그렇고, function으로 선언하면 hoisting이 되기 때문에 혹시라도 후선언 하였을 때 에러 가능성을 줄일 수 있기 때문입니다.

@dennis2311 dennis2311 merged commit 9d815f4 into codeit-sprint-fullstack:next-정한샘 Mar 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants