[이제창] 스프린트미션5 by Jerang2 · Pull Request #143 · codeit-bootcamp-nodejs/4-sprint-mission

Jerang2 · 2025-09-21T07:30:56Z

기본

tsconfig.json 파일을 생성하고, 필요한 옵션을 설정해 주세요. (예: outDir).
필요한 npm script를 설정해 주세요. (예: 빌드 및 개발 서버 실행 명령어)
기존 Express.js 프로젝트를 타입스크립트 프로젝트로 마이그레이션 해주세요.
필요한 타입 패키지를 설치해 주세요.
any 타입의 사용은 최소화해주세요.
복잡한 객체 구조나 배열 구조를 가진 변수에 인터페이스 또는 타입 별칭을 사용하세요.
필요한 경우, 타입 별칭 또는 유틸리티 타입을 사용해 타입 복잡성을 줄여주세요
필요한 경우, declare를 사용하여 타입을 오버라이드하거나 확장합니다. (예: req.user)

개발 환경 설정

ts-node 를 사용해 .ts 코드를 바로 실행할 수 있는 npm script를 만들어 주세요. (예: npm run dev)
nodemon을 사용해 .ts 코드가 변경될 때마다 서버가 다시 실행되는 npm script를 만들어 주세요. (예: npm run dev)

심화

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

-> 구현에 또 에러가 생겨서 해결중입니다.. ㅠㅠ

…mission_fork into sprintmission5

rdd9223

전반적으로 잘 작성해주셨지만, 중간중간 아쉬운 점이 보입니다. 특히 service는 service디렉토리 하위에서 컨트롤 할 수 있도록 수정해주세요.

rdd9223 · 2025-09-23T14:39:37Z

+  console.log(`서버가 ${PORT}번에서 실행중입니다.`);
+});
+
+export default prisma;


레포지토리/미들웨어가 사용하는 prisma를 여기서 export 하면 안됩니다. 항상 의존(import)하는 건 단방향이어야 해요. repository > service > controller > route(middleware) > index가 되도록 해주세요. 현재는 index > repository > service > controller > route(middleware) > index 여서 순환참조가 일어나고 있어요. 전용 모듈로 별도로 분리하는 것이 좋습니다.

rdd9223 · 2025-09-23T14:42:01Z

+  constructor(productService: ProductService) {
+    this.productService = productService;
+    const commentRepository = new CommentRepository();
+    this.commentService = new CommentService(commentRepository);
+    const likeRepository = new LikeRepository();
+    this.likeService = new LikeService(likeRepository);
+  }


해당 ProductsController를 여러 route에서 사용하게 된다면 여러 생성자들이 추가로 생성하게 될 수 있어요. 의존성 주입 일관성을 위해서 라우터 조립부에서 생성해서 컨트롤러 생성자에 주입하는 것이 좋습니다.

Suggested change

constructor(productService: ProductService) {

this.productService = productService;

const commentRepository = new CommentRepository();

this.commentService = new CommentService(commentRepository);

const likeRepository = new LikeRepository();

this.likeService = new LikeService(likeRepository);

}

constructor(productService: ProductService, commendService: CommentService, likeService: LikeService) {

this.productService = productService;

this.commentRepository = commentService

this.likeRepository = likeRepository

}

rdd9223 · 2025-09-23T14:43:47Z

+router.delete('/products/comments/:commentId', authMiddleware, productsController.deleteComment);
+
+// 상품 좋아요 API
+router.post('/:productId/like', authMiddleware, productsController.toggleLike);


/:productId/like가/:articleId/like와 경로가 겹칩니다. /1234/like로 요청한다면 어떤 경로로 들어갈지 생각해보세요. /product/:productid/like 같이 표현하는 것이 좋아요.

rdd9223 · 2025-09-23T14:44:44Z

+router
+  .route('/products/:productId')
+  .get(optionalAuthMiddleware, productsController.getProductById)
+  .patch(validateProduct, authMiddleware, productsController.updateProduct)


이 경로로 보면 인증검증 로직보다 입력값 검증 로직이 더 빨라요. 일반적으로는 인증로직이 우선이에요.

rdd9223 · 2025-09-23T14:45:23Z

+router
+  .route('/articles/:articleId')
+  .get(optionalAuthMiddleware, articlesController.getArticleById)
+  .patch(authMiddleware, validateArticle, articlesController.updateArticle)


이 리뷰와 동일한 내용입니다.

rdd9223 · 2025-09-23T15:06:44Z

@@ -0,0 +1,50 @@
+import { Product as PrismaProduct, Prisma } from '@prisma/client';


바로 사용하지 말고 repository를 사용하도록 해주세요.

rdd9223 · 2025-09-23T15:23:59Z

+  "main": "index.js",
+  "scripts": {
+    "build": "tsc",
+    "dev": "ts-node-dev --respawn --transpile-only src/index.ts",


--transpile-only은 타입에러를 런타임까지 가져갈 수 있어요. 타입스크립트의 장점을 살리기 위해 제거해주세요!

rdd9223 · 2025-09-23T15:25:23Z

+        return res.status(403).json({ message: '상품 수정 권한이 없습니다.' });
+      }
+
+      const updatedProduct = await this.productService.updateProduct(parseInt(productId), {


이런 비즈니스 로직은 Service에서 구현해주세요! Controller는 얇게 유지하는 것이 좋습니다.

rdd9223 · 2025-09-23T15:26:21Z

@@ -0,0 +1,20 @@
+{


프로젝트 전반적으로 상대경로가 많습니다. baseUrl, paths옵션을 이용해서 절대 경로로 사용할 수 있도록 구성해보세요.

rdd9223 · 2025-09-23T15:27:03Z

dist 하위의 빌드 파일은 커밋할 필요 없습니다. .gitignore에 해당 디렉토리를 추가해주세요.

rdd9223 · 2025-09-23T15:28:56Z

@Jerang2 target branch 가 잘못된 것 같네요. 개인 브랜치로 변경해주세요.

Jerang2 and others added 30 commits July 30, 2025 00:05

[이제창] sprint2 (codeit-bootcamp-nodejs#43)

f8faf87

feat: sprintmission3

2564887

feat: image resize

1439346

Merge branch '이제창' into main

f9a48f6

prisma.schema 수정

39cde28

prisma migrate

ac96417

bcrypt install

59e7d84

UserService.js 생성

ae377b5

users.router.js 생성

3f4b18c

ESM 방식 -> CommonJs 방식으로 수정

37122cf

회원가입 API 완료?

33c6e2b

오타 수정

dab8731

UserService에 로그인 로직 추가

d19e9be

users.router.js 로그인 API 추가

48dceef

articles.router.js 인증 미들웨어, 소유자 확인 로직 추가

b6b9d10

products.router.js 인증, 로그인 로직 추가

f8b9b45

prisma.schema modify T.T

e1cf603

바뀐 prisma.schema에 맞춰서 수정

9b8d94e

변경된 prisma.schema model로 products.router.js 수정

48fcfe4

user.router.js 내 정보 수정 API 구현

cb17c18

users.router.js 비밀번호 변경 API 추가

71ca614

users.router.js 작성 상품 목록 조회 API 추가

f8efc37

products.router.js에 좋아요 API 추가

df167d3

오타 수정

5a421b0

articles.router.js 좋아요 기능 추가

f2f75e7

optionalAuth.middleware.js 파일 생성

23a0531

articles.router.js에 선택적 인증 미들웨어 추가

641ca6d

products.router.js에 상품목록조회 isLiked 필드 추가

c4996fd

articles.router.js에 isLiked 필드 추가

a70d66d

UseService.js의 로그인 로직을 access, refresh token를 받는 형식으로 변경

3cc2400

Jerang2 added 23 commits September 21, 2025 15:47

서버 구현 o

d1bff8f

src로 이동후 repository add

2032cc2

Service 리팩토링

29c9305

Controller 구현

60f5d7f

roter 리팩토링

b29cb20

DTO 생성

fa86962

dto 추가, 사용하도록 수정

b7f019b

Service DTO 사용하도록 수정

9bb1d8d

CommentService, LikeService 추가

4555a12

Comment, LikeService 로 리팩토링

6d7c540

prisma와 관계 연결

9c3506a

import문 수정

823e702

controller - select문 수정

9b6f392

controller - select문 수정 2

9261f17

findlikes 추가

72f1bf1

signUp 메서드 수정

e460353

UserService import 수정

b60f8e1

import문 수정

ba427c8

cursor, data 객체 수정

64f1a15

Prisma.Product -> Product로 수정

b73fd07

Prisma.Like -> Like

a1b0d28

Prisma.xxx -> xxx 로 수정

6c306f9

README.md

3d9c8ec

Jerang2 changed the title ~~Sprintmission5~~ [이제창] 스프린트미션5 Sep 21, 2025

Jerang2 added 2 commits September 21, 2025 16:34

prettier 추가 및 실행

d24de79

Merge branch 'sprintmission5' of https://github.com/Jerang2/4-sprint-…

4c89673

…mission_fork into sprintmission5

Jerang2 requested a review from rdd9223 September 22, 2025 00:02

Jerang2 added the 순한맛🐑 마음이 많이 여립니다.. label Sep 22, 2025

rdd9223 approved these changes Sep 23, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[이제창] 스프린트미션5#143

[이제창] 스프린트미션5#143
Jerang2 wants to merge 65 commits into
codeit-bootcamp-nodejs:mainfrom
Jerang2:sprintmission5

Jerang2 commented Sep 21, 2025 •

edited

Loading

Uh oh!

rdd9223 left a comment

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 Sep 23, 2025

Uh oh!

rdd9223 commented Sep 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		@@ -0,0 +1,50 @@
		import { Product as PrismaProduct, Prisma } from '@prisma/client';

Conversation

Jerang2 commented Sep 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

기본

개발 환경 설정

심화

Uh oh!

rdd9223 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rdd9223 commented Sep 23, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Jerang2 commented Sep 21, 2025 •

edited

Loading