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.

Sign up for GitHub

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

Jump to bottom

[Study] Naming Convention 규칙 설정 정하자 #39

Open

wintiger98 opened this issue Apr 7, 2024 · 7 comments

Assignees

Labels

study

Contributor

wintiger98 commented Apr 7, 2024 •

edited

Loading

DTO 객체에 대해 뒤에 DTO 붙이는 것에 대해 어떻게 생각하는지
Interface의 구현체에 대해 네이밍 규칙에 Impl를 둬야하는지 말아야하는지
우리 프로젝트의 경우, Service를 인터페이스와 구현체로 나누는게 나을지 아니면 클래스만 둬도 될지
- 인터페이스가 필요한 도메인이 있을지

참고자료

DTO 네이밍 규칙
DTO 네이밍 규칙 by 김영한
interface 구현체 네이밍 규칙 관련1
interface 구현체 네이밍 규칙 관련2
인터페이스와 구현체를 나눠야할까?

wintiger98 self-assigned this

Contributor Author

wintiger98 commented Apr 7, 2024 •

edited

Loading

@kkho9654 @chanmin97

네이밍 규칙상 붙이지 말라는 글이 좀 보여서 어떻게 생각하는지 궁금 왜냐하면 이 네이밍 규칙도 우리 팀원끼리 공통적으로 적용해야될거같아서
1과 마찬가지
네번째 참고자료에서 보면 우리 프로젝트 상 필요할까라는 생각이들어서 함 질문해봅니다.

wintiger98 added the study label

wintiger98 added this to 공부목록

Contributor

kkho9654 commented Apr 7, 2024

1. 저거 보니까 dto랑 Impl을 붙일 이유가 없긴 한 것 같네. 난 안붙이는 거에 한표!
굳이 인터페이스를 사용안해도 되는데 우리는 공부하는 목적이니까 최대한 다양한 방법을 경험해보는 것을 목적으로 하는 거니까 이것 저것 할 수 있는 거는 다 해보는게 좋을 것 같은데 어때?

Contributor

kkho9654 commented Apr 7, 2024

우리 정하는김에 #40 이것도 정하는게 어때
이미 한거는 말고 이제부터

Contributor

kkho9654 commented Apr 7, 2024

다시보니까 3번 인터페이스도 그냥 하지말자
You aren't going to need it"(YAGNI) 라는 원칙을 믿어보는 것도 좋을 것 같아.
...우리는 AGILE이니까. ㅋ

Contributor

chanmin97 commented Apr 7, 2024

인터페이스 일단 우린 필요없는거 같긴하네 ㅇㅈ

Contributor Author

wintiger98 commented Apr 7, 2024

이렇게 컨벤션 정한건 어디에 따로 적을까?

Contributor

kkho9654 commented Apr 7, 2024

음 위키?

wintiger98 mentioned this issue

[0408] 회의록 #46

Open

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