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

24주차 PR답변 모음 #130

Open
ScottSung7 opened this issue Nov 19, 2024 · 7 comments
Open

24주차 PR답변 모음 #130

ScottSung7 opened this issue Nov 19, 2024 · 7 comments
Assignees

Comments

@ScottSung7
Copy link
Collaborator

No description provided.

@ScottSung7 ScottSung7 self-assigned this Nov 19, 2024
@ScottSung7
Copy link
Collaborator Author

ScottSung7 commented Nov 19, 2024

  1. 도메인의 인터페이스의 DTO 참조 해결.

답변: 이 부분은 Hotfix로 수정하도록 하겠습니다! #131

답변링크

@ScottSung7
Copy link
Collaborator Author

ScottSung7 commented Nov 19, 2024

  1. import문 제거 단축키

Optimize Imports는 정말 좋군요! 매번 하나씩 제거 했었습니다.

답변: 이 경우에는 인텔리제이에 특정 포메팅이 등록되어 있어야 하는 걸까요?? clean up을 눌러 보았지만 변경되는게 없습니다. 검색하다보니까 Optimize Imports를 쓰니 제거 되는데 혹시 이걸 말씀하신 걸까요?

답변링크

@ScottSung7
Copy link
Collaborator Author

ScottSung7 commented Nov 19, 2024

  1. Search Controller 에서 도메인 서비스 사용

답변: 감사합니다! 도메인 서비스를 관련 비즈니스 처리에 유용하게 쓸 수 있어서 좋은 것 같습니다.

답변링크

@ScottSung7
Copy link
Collaborator Author

  1. 컨트롤러에서 inner class로 request, response 사용

답변: 네! 맞습니다. inner class로 request, response를 사용하는게 훨씬 찾기도 쉽고 좋았습니다. 컨트롤러가 매우 길어지는 경우가 아니면 이런 방식을 우선 사용하려고 합니다.

답변링크

@ScottSung7
Copy link
Collaborator Author

  1. inner class 사용 관련 주의 사항

답변: 음.. 이 부분이 조금 걱정되기는 했습니다. 그래도 public하고 열어 놓아서 다른 곳에서 재사용할 수도 있기는 한데, 다른 곳에서 사용되는 경우는 캡슐화 위반이 아닌가 걱정이 되었습니다. 여러군데에서 사용하기 시작하면 따로 파일을 생성하려고 합니다!

답변링크

@ScottSung7
Copy link
Collaborator Author

  1. instance of 개선 관련

답변: 아무래도 1번이 가장 큰 이유인듯 합니다. 왜냐하면 Check In, Line Up, Etc는 쓰기때는 따로 로직을 가지지만, 조회되는 경우는 종종 helpId 라는 부모의 식별자로 조회가 되어야 했습니다.

생소하다고 하셔서 조금 걱정되기는 하는데 혹시.. 염려되는 부분이 있어서 말씀주신걸까요?

답변링크

@ScottSung7
Copy link
Collaborator Author

  1. 성공 메시지 정책

답변: 네 맞습니다! 성공시에는 성공 메시지를 주려고 합니다.

답변링크

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant