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

refactor: API에서 가짜 중복된 타입들 분리 #424

Closed
Tekiter opened this issue Feb 2, 2023 · 1 comment
Closed

refactor: API에서 가짜 중복된 타입들 분리 #424

Tekiter opened this issue Feb 2, 2023 · 1 comment
Labels
🛠 Refactor 리팩토링

Comments

@Tekiter
Copy link
Member

Tekiter commented Feb 2, 2023

✨ 설명하기

현재 API 타입들의 많은 부분에서 가짜 중복을 제거한 코드들이 있어요. 가짜 중복이란 현재는 서로 모양이 비슷해 보여 중복처럼 보이지만, 서로 다른 속도와 이유로 각자 다르게 변화할 수 있는 코드에요. 가짜 중복을 제거해버리면 나중에 해당 코드에 변화가 생길 때, 생각했던 곳과 다른 부분에서 예상치 못하게 문제가 발생할 수 있어요.

하나의 예시가 #423, #382 에서 문제가 된 상황이에요. 프로젝트 조회할 때 필드를 추가했더니, 수정과 등록에서 타입 오류가 발생해요. 왜냐하면 조회 타입과 수정, 등록 타입을 공유하고 있는데, 조회할 때의 필드와 수정, 등록의 필드가 달라졌기 때문이에요.

이렇게 프로젝트가 발전하면서 문제가 될 수 있는 상황을 미연에 방지하고자, 가짜 중복을 분리하고자 해요.

@Tekiter
Copy link
Member Author

Tekiter commented Jul 12, 2023

#674 에서 차차 해결해나기로 함

@Tekiter Tekiter closed this as completed Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Refactor 리팩토링
Projects
None yet
Development

No branches or pull requests

1 participant