github-actions[bot]
commented
3 days ago Test Results
57 files 57 suites 8s :stopwatch: 173 tests 169 :white_check_mark: 4 :zzz: 0 :x: 180 runs 176 :white_check_mark: 4 :zzz: 0 :x:
Results for commit 88bc02cd.
:recycle: This comment has been updated with latest results.
📌 관련 이슈
✨ PR 세부 내용
받은 피드백 조회 시 내가 작성하지 않은 사람으로부터 받은 피드백이 있는 경우 이를 빈 문자열로 반환하는 기능입니다.
p4 - isWrited 이름에 대해서
MatchResultResponse 에서 비슷한 의미로 isWrited 를 사용해서, 이번에도 같은 네이밍을 가져갔습니다. 또한 프론트에게도 DTO 가 바뀌는 것에 대해 전달했구요. 다만, 이게 적절한 이름인지는 논의해보고 싶네요. '내가 상대방에게 피드백을 작성했다' 라는 의미에서 isWrited 인데, 이는 쓰여졌는지를 말하는 변수인 것 같아 다른 적절한 네이밍이 있다면 추천해주세요!
p3 - SocialFeedbackReader, DevelopFeedbackRader의 existsByDeliverAndReceiver 메서드에 대해서
지금 해당 메서드는 정말 repository 의 메서드를 중계하는 역할밖에 하지 않습니다. 근데 그렇다고 이 메서드가 쓰이는 로직을 reader 로 내리자니, 좀 안맞는 느낌입니다. 이 메서드의 존재와 이름에 대해서 다들 어떻게 생각하시나요?
Reader, Writer 와 Service 를 분리하는 차원에서 제 구현의 방향성이 다를수도 있을 것 같아요. 다르다면 편하게 말씀주세요, 바로 수정점 반영하겠습니다!
아래는 잡담입니다.