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

docs: update reviewer guide for 6th level1 backend #25

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions reviewer/be-lv1/blackjack.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@

블랙잭 도메인을 충분히 학습하고 설계에 녹여냈는지를 중점으로 리뷰한다.

블랙잭 미션 중간에 있는 제네릭 이해하기에 대한 부분은 리뷰를 할 필요가 없다.

* 블랙잭 도메인을 잘 이해하고 그것을 코드에 반영하였는가?
* 상속과 인터페이스를 적절하게 사용하였는가?
* Controller와 View 없이도 블랙잭 게임을 할 수 있는가?
Expand Down
5 changes: 2 additions & 3 deletions reviewer/be-lv1/chess.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,12 @@
체스 미션은 레벨1의 마지막 미션으로 굉장히 높은 난이도를 가지고 있다.
추가로 데이터베이스에 대한 내용도 다루고 있기 때문에 리뷰할 내용이 많아 힘들 수 있다.

내용이 많은 만큼 데이터베이스, 상세 로직을 꼼꼼히 보는 것보다 객체 간의 설계가 잘 되었는지를 위주로 리뷰한다.
내용이 많은 만큼 상세 로직을 꼼꼼히 보는 것보다 객체 간의 설계가 잘 되었는지를 위주로 리뷰한다.
또한 데이터베이스 설계 시 기존 객체 설계와 일치하는지와 어떠한 부분을 중점으로 트레이드 오프하여 테이블을 설계하였는지 확인하면 좋겠다.

체스 미션 중간에 있는 Lambda, Stream에 대한 부분은 리뷰를 할 필요가 없다.

* 체스 도메인을 잘 이해하고 그것을 코드에 반영하였는가?
* 필요 이상으로 과한 설계를 하지 않았는가?
* 용도에 맞게 Stream API를 사용하고 있는가?
* 데이터베이스와 관련된 코드와 도메인 사이에 의존성을 잘 분리하였는가?

## 경험할 학습 목표
Expand Down
2 changes: 0 additions & 2 deletions reviewer/be-lv1/ladder.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,6 @@
이번 미션의 핵심 목표는 TDD, 리팩토링을 해보는 경험이다.
단순히 단위 테스트를 작성했는지가 아닌 TDD로 개발하는지 집중적으로 피드백 주시면 좋겠다.

사다리 타기 미션 중간에 있는 리스트 이해하기에 대한 부분은 리뷰를 할 필요가 없다.

* 함수(또는 메서드)의 길이가 10라인을 넘어가지 않도록 구현했는가?
* 클래스를 작은 단위로 잘 분리했는가?
* 도메인 객체에 대한 단위 테스트가 모두 존재하는가?
Expand Down