[볼링 점수판] 1단계 - 질문 삭제하기 기능 리팩토링 by logantect · Pull Request #807 · next-step/java-bowling · GitHub
Skip to content

[볼링 점수판] 1단계 - 질문 삭제하기 기능 리팩토링#807

Merged
ydh6226 merged 1 commit into
next-step:jeongjaeeomfrom
logantect:step1
May 18, 2022
Merged

[볼링 점수판] 1단계 - 질문 삭제하기 기능 리팩토링#807
ydh6226 merged 1 commit into
next-step:jeongjaeeomfrom
logantect:step1

Conversation

@logantect

Copy link
Copy Markdown

안녕하세요, 새로운 리뷰어님!

[볼링 점수판] 1단계 - 질문 삭제하기 기능 리팩토링 완료하여 PR드립니다!

리뷰 잘 부탁드릴게요!! :)
감사합니다!

1단계 - 질문 삭제하기 기능 리팩토링
- Answers 일급컬렉션 구현
- Answer 객체 삭제 및 삭제 히스토리 객체 변환 메서드 추가
- Question 객체 삭제 및 삭제 히스토리 객체 변환 메서드 추가
- Answer, Question 단위 테스트 코드 작성

@ydh6226 ydh6226 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

안녕하세요 정재님
볼링 미션을 함께 하게된 양동혁입니다 :)
미션 중 궁금하신 점이 있다면 편하게 DM이나 코멘트로 질문남겨주세요 🔥

1단계 미션 잘 진행해주셔서 바로 머지하겠습니다.
몇 가지 코멘트 남겨드렸으니 확인해보시고
다음 단계에서 같이 반영부탁드려요!

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getter을 사용하지 않고도 가능할까요?

public Question setContents(String contents) {
this.contents = contents;
return this;
private void validateDeleteQuestion(User loginUser) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

개인적으로 구체적으로 어떤 검증을 하는지 네이밍으로 표현하면 좋을 것 같아요 :)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

새롭게 생성된 public메소드에 대한 테스트도 작성해볼까요?

@ydh6226 ydh6226 merged commit 8d1ef40 into next-step:jeongjaeeom May 18, 2022
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

Successfully merging this pull request may close these issues.

2 participants