[Review] 2022/04/10 2차 PR

 

1. 코드 리뷰

 

1. 기존 순서와 겹치는 경우

순서가 겹치는 경우 해당 1번이 뒤로 밀리고, 그 사이에 삽입 되도록 하였다.

원래의 순서 1번은 2번이 되게 될 것 이다!

2. 초기화 메서드 추출하기

초기화에 해당되는 부분이다.

위 코드는 생성자에서 한번 호출되기 때문에 사실 따로 분리시키지는 않았었다.

리뷰어님의 말씀처럼 메서드로 뽑을 수 도 있지만, 그냥 명시적으로 생성자 안에서 초기화 하고 있음을 보이는 것 또한 좋다 생각한다!

 

3. 로직을 함수로 더 분리하기

함수 안에 여러 로직들이 퍼져 있다. 관련된 하나의 덩어리 들 끼리 메서드로 뽑아 주어야 할것 같다.