Skip to content

[심하원] part3 - mission5#130

Open
gypsophila22 wants to merge 3 commits intocodeit-bootcamp-nodejs:심하원from
gypsophila22:심하원-part3-mission5

Hidden character warning

The head ref may contain hidden characters: "\uc2ec\ud558\uc6d0-part3-mission5"
Open

[심하원] part3 - mission5#130
gypsophila22 wants to merge 3 commits intocodeit-bootcamp-nodejs:심하원from
gypsophila22:심하원-part3-mission5

Conversation

@gypsophila22
Copy link
Collaborator

요구사항

기본

  • tsconfig.json 파일을 생성하고, 필요한 옵션을 설정해 주세요.
  • 필요한 npm script를 설정해 주세요.
  • 기존 Express.js 프로젝트를 타입스크립트 프로젝트로 마이그레이션 해주세요.
  • 필요한 타입 패키지를 설치해 주세요.
  • any 타입의 사용은 최소화해주세요.
  • 복잡한 객체 구조나 배열 구조를 가진 변수에 인터페이스 또는 타입 별칭을 사용하세요.
  • 필요한 경우, 타입 별칭 또는 유틸리티 타입을 사용해 타입 복잡성을 줄여주세요.
  • 필요한 경우, declare를 사용하여 타입을 오버라이드하거나 확장합니다.
  • ts-node 를 사용해 .ts 코드를 바로 실행할 수 있는 npm script를 만들어 주세요.
  • nodemon을 사용해 .ts 코드가 변경될 때마다 서버가 다시 실행되는 npm script를 만들어 주세요.

심화

  • Controller, Service, Repository로 나누어 코드를 리팩토링해 주세요.
  • 필요하다면, 계층 사이에서 데이터를 주고 받을 때 DTO를 활용해 주세요

주요 변경사항

  • ts-node & nodemon이 아닌 tsx watch를 사용함. ("dev": "tsx watch src/app.ts")

멘토에게

  • 하다 보니 파일이 엄청 여러 개로 나뉜 거 같은데 이 방식에 문제가 없진 않을지 걱정됩니다.

@gypsophila22 gypsophila22 force-pushed the 심하원-part3-mission5 branch from 13a46e4 to 8660f06 Compare September 22, 2025 06:09
@rdd9223 rdd9223 self-requested a review September 23, 2025 15:30
Copy link
Collaborator

@rdd9223 rdd9223 left a comment

Choose a reason for hiding this comment

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

전반적으로 typescript로 잘 작성해주셔서 좋습니다. 일관성이 유효한지는 확인해봐야겠지만, !같은 non-null assertion이나 any를 사용하지 않도록 한다면 더 좋을 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants