[박수훈] sprint5 리팩토링 및 sprint6 제출 by mdeeno · Pull Request #9 · codeit-sprint-fullstack/11-sprint-mission-be

mdeeno · 2026-01-20T01:33:59Z

1. 스프린트 미션 요구사항

기본 요구사항

중고마켓

mongoDB에서 PostgreSQL을 사용하도록 코드를 마이그레이션 해주세요.

공통

PostgreSQL를 이용해 주세요.
데이터 모델 간의 관계를 고려하여 onDelete를 설정해 주세요.
데이터베이스 시딩 코드를 작성해 주세요.
각 API에 적절한 에러 처리를 해 주세요.
각 API 응답에 적절한 상태 코드를 리턴하도록 해 주세요.

자유게시판

2. 주요 변경사항

sprint5 기능별 구조로 리팩토링(레포지토리 패턴) 및 sprint6 요구사항 구현

3. 멘토님에게 남길 메시지

기존에 계층형 아키텍쳐로 sprint5를 구현하였으나, 해당 아키텍쳐에 이해가 부족하여... 기능별 계층구조로 sprint5 리팩토링하고 sprint6 진행하였습니다.
리팩토링하면서 npm run dev 시 각 파일에 import 했던 요소 중 Alias(#)를 지정하여 사용한 절대경로들이 에러가 떠서 대부분 상대경로로 바꿨습니다. 이에 꽤 많은 시간이 걸렸는데 혹시 한번에 수정/변경 하는 방법이 있을까요?

reach0908 · 2026-01-28T06:59:50Z

기존에 계층형 아키텍쳐로 sprint5를 구현하였으나, 해당 아키텍쳐에 이해가 부족하여... 기능별 계층구조로 sprint5 리팩토링하고 sprint6 진행하였습니다.

-> 이번 멘토링에서 설명을 드리긴했지만 아직은 어렵다면 지금 프로젝트에서 진행중인 수준의 아키텍처만으로도 아직은 충분할 것 같습니다. 하지만 코드가 길어지거나 반복되는 코드가 많아진다면 항상 이걸 분리해서 간결하게 만든다 는 생각을 해보시면 좋을 것 같아요

리팩토링하면서 npm run dev 시 각 파일에 import 했던 요소 중 Alias(#)를 지정하여 사용한 절대경로들이 에러가 떠서 대부분 상대경로로 바꿨습니다. 이에 꽤 많은 시간이 걸렸는데 혹시 한번에 수정/변경 하는 방법이 있을까요?

vscode의 find and replace라는 기능을 사용하면 좋습니다. 우선은 alias 별칭 설정에 오류가 왜 생겼는지 파악하고 이를 수정하는 방법을 찾는 것이 더 좋을 수도 있습니다. 우선 전체 변경기능을 이용해서 기존의 별칭들을 상대경로가 아닌 src/ 와 같이 절대경로로 변경을 먼저하시면 좋을 것 같아요 상대경로 변경은 AI를 활용하는게 빠를 것 같습니다.

reach0908

Code Review Summary

전체적으로 PostgreSQL 마이그레이션과 Repository 패턴 적용이 잘 되어있습니다! 다만 몇 가지 오타와 누락된 부분이 있어 코멘트 남깁니다.

상세 리뷰는 아래 인라인 코멘트들을 확인해주세요.

reach0908

Draft Review

reach0908 · 2026-01-28T07:37:56Z

src/exceptions/HttpException.js

+    this.name = this.constructor.name;
+    Error.captureStackTrace(this, this.constructor);
+  }
+}


[Good] HttpException 계층 구조가 잘 설계되어 있습니다!

Error.captureStackTrace로 스택 트레이스를 깔끔하게 처리하고, statusCode, details 등 필요한 정보를 담을 수 있게 확장한 점이 좋습니다.

reach0908 · 2026-01-28T07:38:05Z

src/constants/http-status.js

+  CONFLICT: 409,
+
+  INTERNAL_SERVER_ERROR: 500,
+};


[Good] HTTP 상태 코드를 상수로 관리하는 좋은 패턴입니다!

매직 넘버 대신 HTTP_STATUS.CREATED, HTTP_STATUS.NOT_FOUND 등으로 사용하면 코드 가독성이 높아지고 오타로 인한 버그를 방지할 수 있습니다.

reach0908 · 2026-01-28T07:38:15Z

src/repository/comment.repository.js

+  return prisma.comment.findMany({
+    where,
+    take: Number(take) || 10,
+    // Cursor 방식 페이지네이션 적용


[Good] Cursor 기반 페이지네이션을 잘 구현했습니다!

skip: 1, cursor: { id: cursor }로 커서 이후 데이터를 가져오는 방식이 올바르게 적용되어 있습니다. 대량의 데이터에서 offset 방식보다 성능이 좋습니다.

reach0908 · 2026-01-28T07:38:24Z

scripts/seed.js

+const pool = new Pool({ connectionString: process.env.DATABASE_URL });
+const adapter = new PrismaPg(pool);
+const prisma = new PrismaClient({ adapter });
+


[Good] 시딩 스크립트가 잘 작성되어 있습니다!

환경 체크로 프로덕션 실수 방지

트랜잭션으로 기존 데이터 초기화

faker.js로 현실적인 더미 데이터 생성

적절한 에러 핸들링과 연결 해제

개발 환경 테스트에 매우 유용합니다.

reach0908 · 2026-01-28T07:38:33Z

src/db/prisma.js

+
+const adapter = new PrismaPg(pool);
+
+export const prisma = new PrismaClient({ adapter });


[Good] Prisma 설정이 깔끔합니다!

Pg Pool과 PrismaPg 어댑터를 활용한 PostgreSQL 연결 구성이 잘 되어 있습니다. 싱글톤 패턴으로 prisma 인스턴스를 export하여 앱 전체에서 재사용하는 것도 좋습니다.

reach0908 · 2026-01-28T07:38:43Z

prisma/schema.prisma

+  article   Article? @relation(fields: [articleId], references: [id], onDelete: Cascade)
+
+  productId String?
+  product   Product? @relation(fields: [productId], references: [id], onDelete: Cascade)


[Good] onDelete: Cascade 설정이 잘 되어 있습니다!

부모(Article/Product) 삭제 시 관련 댓글들이 자동으로 삭제되어 데이터 무결성이 유지됩니다. 고아 레코드 문제를 방지하는 좋은 설계입니다.

reach0908 · 2026-01-28T07:38:52Z

src/repository/article.repository.js

+
+// 게시글 등록
+function createArticle(data) {
+  return prisma.article.create({ data });


[Good] Repository 패턴이 잘 적용되어 있습니다!

데이터베이스 접근 로직을 Repository로 분리하여:

라우터는 HTTP 요청/응답만 담당

Repository는 데이터 조회/조작만 담당

이렇게 관심사를 분리하면 테스트와 유지보수가 쉬워집니다.

reach0908 · 2026-01-28T07:39:02Z

src/config/config.js

+  PORT: z.coerce.number().min(1000).max(65535).default(5001),
+
+  DATABASE_URL: z.string().url(),
+});


[Good] Zod로 환경변수를 검증하는 좋은 패턴입니다!

서버 시작 시점에 환경변수 유효성을 체크하여, 런타임 에러를 방지할 수 있습니다. z.coerce.number()로 문자열을 숫자로 변환하는 것도 깔끔합니다.

reach0908

추가 칭찬 코멘트

reach0908 · 2026-01-28T07:39:02Z

src/repository/article.repository.js

@@ -0,0 +1,53 @@
+import { prisma } from '#db/prisma.js';


기존의 다른 파일들처럼 article.repository.ts와 같은 파일 네이밍을 유지하면 좋을 것 같습니다!

전체적으로 router, middleware, repo 들의 네이밍이 모두 컨벤션이 달라서 맞추면 좋을듯 합니다

reach0908 · 2026-01-28T07:40:46Z

src/middleware/validation.middleware.js

+
+  // 통과
+  next();
+};


[Suggestion] Zod를 활용한 유효성 검사 미들웨어로 개선할 수 있습니다!

현재 수동으로 각 필드를 검사하고 있는데, Zod 스키마를 활용하면 더 간결하고 재사용 가능합니다.

1. 스키마 정의:

// schemas/product.schema.js import { z } from 'zod'; export const createProductSchema = z.object({ name: z.string().min(1, '이름은 필수입니다'), description: z.string().min(1, '설명은 필수입니다'), price: z.number().positive('가격은 0보다 커야 합니다'), tags: z.array(z.string()).optional(), }); // update는 모든 필드가 optional export const updateProductSchema = createProductSchema.partial();

2. 범용 validate 미들웨어:

// middleware/validate.js import { z } from 'zod'; import { BadRequestException } from '../exceptions/BadRequsetException.js'; export const validate = (schema) => (req, res, next) => { try { req.body = schema.parse(req.body); next(); } catch (error) { if (error instanceof z.ZodError) { throw new BadRequestException( 'Validation failed', error.flatten().fieldErrors ); } next(error); } };

3. 라우터에서 사용:

import { validate } from '../middleware/validate.js'; import { createProductSchema, updateProductSchema } from '../schemas/product.schema.js'; router.post('/', validate(createProductSchema), async (req, res) => { // req.body는 이미 검증 및 타입 변환됨 const product = await productRepository.createProduct(req.body); res.status(HTTP_STATUS.CREATED).json(product); }); router.patch('/:id', validate(updateProductSchema), async (req, res) => { // partial() 덕분에 일부 필드만 있어도 OK });

장점:

스키마 재사용 (.partial(), .pick(), .extend() 활용)

타입 변환 자동화 (z.coerce.number() 등)

상세한 에러 메시지 자동 생성

코드 중복 대폭 감소

reach0908

Zod validation 제안

reach0908 · 2026-01-28T07:41:09Z

src/routes/product/product.Router.js

+    if (!exists) {
+      throw new NotFoundException(ERROR_MESSAGE.RESOURCE_NOT_FOUND);
+    }


인라인으로 if문을 작성하실때와 괄호를 사용하실때가 나뉘는 것 같은데 통일하면 코드 가독성이 더 오를 것 같습니다!

저는 항상 사용하는 편이 취향이긴합니다!

reach0908 · 2026-01-28T07:41:54Z

eslint.config.js

@@ -0,0 +1,20 @@
+export default [
+  js.configs.recommended,


[Critical] js가 import되지 않아 ReferenceError가 발생합니다.

// 상단에 추가 import js from '@eslint/js';

reach0908 · 2026-01-28T07:42:01Z

prisma/prisma.config.js

+import { defineConfig, env } from 'prisma/config';
+
+export default defineConfig({
+  schema: 'prisma/shema.prisma',


[Minor] 경로 오타

'prisma/shema.prisma' → 'prisma/schema.prisma'

reach0908 · 2026-01-28T07:42:10Z

prisma/schema.prisma

+}
+
+// 자유게시판
+model Article {


[Major] Article 스키마 필드 누락

체크리스트 요구사항에 따르면 image와 likeCount 필드가 필요합니다.

model Article { id String @id @default(uuid()) title String content String image String? // 추가 필요 likeCount Int @default(0) // 좋아요순 정렬용 createdAt DateTime @default(now()) updatedAt DateTime @updatedAt comments Comment[] }

reach0908 · 2026-01-28T07:42:18Z

server.js

+  console.log(`\n${signal} 신호를 받았습니다. 서버를 종료합니다.`);
+  server.close(async () => {
+    console.log('HTTP 서버가 닫혔습니다.');
+    await disconnectDB();


[Critical] disconnectDB() 함수가 정의되지 않았습니다.

// 상단에 추가 import { prisma } from '#db/prisma.js'; // 여기서 수정 await prisma.$disconnect();

reach0908 · 2026-01-28T07:42:28Z

src/config/config.js

+    });
+  } catch (error) {
+    if (error instanceof z.ZodError) {
+      const { fieldErrors } = flateenError(error);


[Critical] 함수명/변수명 오타

flateenError → Zod에는 이런 함수가 없습니다

filedErrors → fieldErrors

if (error instanceof z.ZodError) { const { fieldErrors } = error.flatten(); console.error('환경 변수 검증 실패:', fieldErrors); }

reach0908 · 2026-01-28T07:42:37Z

src/middleware/errorHandler.middleware.js

+  if (error instanceof HttpException) {
+    return res.status(error.statusCode).json({
+      message: error.message,
+      stack: config.ENV === 'development' ? error.stack : undefined,


[Critical] config가 import되지 않아 ReferenceError가 발생합니다.

// 상단에 추가 import { config } from '../config/config.js';

그리고 config.ENV가 아니라 config.NODE_ENV입니다.

reach0908 · 2026-01-28T07:42:48Z

src/repository/product.repository.js

+
+  return prisma.product.findMany({
+    where,
+    orderBy: {


[Critical] 오타 2개

'dest' → 'desc' (Prisma는 desc만 인식)

Line 33: take: Number(take) || 0 → 기본값 0이면 아무것도 반환 안됨

return prisma.product.findMany({ where, orderBy: { createdAt: orderBy === 'recent' ? 'desc' : 'asc', }, skip: Number(skip) || 0, take: Number(take) || 10, // 0 → 10 });

reach0908 · 2026-01-28T07:42:58Z

src/routes/index.js

+// 라우터 연결
+router.use('/products', productRouter);
+router.use('/articles', articleRouter);
+router.use('/articles', commentRouter);


[Major] 댓글 라우터 경로 구조 문제

현재 중고마켓 댓글 경로가 /api/articles/products/:productId가 됩니다.
의도한 경로는 /api/products/:productId/comments일 것 같습니다.

현재:

router.use('/products', productRouter); router.use('/articles', articleRouter); router.use('/articles', commentRouter); // 모든 댓글이 /articles 하위

해결 방안:

댓글 라우터를 별도 경로로 분리

또는 각 리소스 라우터 내에서 댓글 처리

reach0908 · 2026-01-28T07:43:10Z

src/middleware/validation.middleware.js

@@ -0,0 +1,31 @@
+// 유효성 검사
+
+import { BadRequestException } from '../errors/httpException.js';


[Critical] import 경로가 잘못되었습니다.

../errors/httpException.js 경로는 존재하지 않습니다.

// 수정 import { BadRequestException } from '../exceptions/BadRequsetException.js';

mdeeno added 3 commits January 16, 2026 17:02

feat: 백엔드 리팩토링 및 API 구현 완료 (계층형 아키텍처 적용)

3908b10

fix: 서버 실행 오류 수정 및 라우터 연결 완료

850d975

fix: 컨트롤러 오탈자 수정

e89e879

mdeeno requested a review from reach0908 January 20, 2026 01:34

mdeeno self-assigned this Jan 20, 2026

mdeeno added the 진행중 아직 스프린트 미션 제출일이 아닙니다. 새로 커밋된 내용에 대해 코드리뷰 해주세요! (2주 이상의 스프린트 미션 중간점검 PR 제출시 사용합니다.) label Jan 20, 2026

mdeeno added 4 commits January 23, 2026 13:48

refactor: 서버 이름 변경(server->src) 및 package.json 알리아스 설정

0f76616

refactor: server.js와 app.js 병합

9db6457

refactor: 기능적 계층 구조로 리팩토링(repository 패턴)

448fb0d

fix: import 파일 에러 수정

be323d4

mdeeno changed the title ~~[박수훈] sprint6 제출~~ [박수훈] sprint5 리팩토링 및 sprint6 제출 Jan 24, 2026

refactor: 에러 반환 코드 리팩토링 및 불필요한 파일 삭제

3c15016

reach0908 reviewed Jan 28, 2026

View reviewed changes

mdeeno added 4 commits February 19, 2026 09:24

refactor: 코드리뷰 반영

19700a4

feat: cors 미들웨어 whiteList 수정

dcefec2

feat: 레포지토리 내보내기 방식 수정

5900781

docs: gitignore 레포믹스 파일 추가

2087a89


		const adapter = new PrismaPg(pool);

		export const prisma = new PrismaClient({ adapter });

		@@ -0,0 +1,31 @@
		// 유효성 검사

		import { BadRequestException } from '../errors/httpException.js';

Conversation

mdeeno commented Jan 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

1. 스프린트 미션 요구사항

기본 요구사항

중고마켓

공통

자유게시판

댓글

2. 주요 변경사항

3. 멘토님에게 남길 메시지

Uh oh!

reach0908 commented Jan 28, 2026

Uh oh!

reach0908 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Code Review Summary

Uh oh!

reach0908 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Draft Review

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!

reach0908 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!

reach0908 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

mdeeno commented Jan 20, 2026 •

edited

Loading

reach0908 left a comment •

edited

Loading

reach0908 left a comment •

edited

Loading