인프런 커뮤니티 질문&답변

복돌님의 프로필 이미지

작성한 질문수

[코드팩토리] [초급] NestJS REST API 백엔드 완전 정복 마스터 클래스 - NestJS Core

Delete 함수로 데이터 삭제하기

getPostById를 다른 함수에서 사용

23.10.28 21:02 작성

·

358

0

안녕하세요 강사님 CRUD를 위한 API를 생성하는 과정에서 의문이 생겨 질문 드립니다.

 

Post부분에서 getPostById 함수를 구현을 했는데, 다른 함수들에서 getPostById와 동일한 로직이 필요한 것을 발견했습니다!

 

예를 들어, update 또는 delete 함수를 구현할 때, id에 맞는 post를 검색하고, 만약 존재하지 않는다면 에러를, 존재한다면 post를 반환하는 로직이 필요합니다.

 

근데 getPostById 함수가 그 로직을 그대로 갖고 있기 때문에 다른 함수들에서 getPostById 함수를 가져다 사용해도 되나 해서 여쭤봅니다.

Screenshot 2023-10-28 at 9.00.40 PM.png

이런식으로요!

코드 가독성 측면에서는 더 좋아보이긴 하지만, 이게 안티패턴인지 아닌지를 모르겠기도 하고, 제가 모르는 문제가 있나 싶어서 여쭤봅니다!

감사합니다!

답변 2

0

heeseok yang님의 프로필 이미지

2024. 03. 13. 09:33

Wow.. 저도 같은 생각을 해봤는데^^ ㅎ

thanks..

0

코드팩토리님의 프로필 이미지
코드팩토리
지식공유자

2023. 10. 30. 10:19

안녕하세요!

하셔도 괜찮고 안하셔도 괜찮습니다.

개인적으론 패턴은 행동에 대한 정당화가 되기만 한다면 문제가 없다고 생각합니다.

하지만 보통은 같은 레이어끼리는 액세스를 안하는걸 좋아하는 개발자들이 많긴 한걸로 알고 있습니다 (예-service끼리 service를 액세스 하지 않음)

저는 개인적으로 크게 신경쓰지 않습니다. 특히나 일반화나 가독성 측면에서 장점이 있다면요.

감사합니다!

복돌님의 프로필 이미지

작성한 질문수

질문하기