[2주차 회고] 사다리 게임 페어 프로그래밍

우아한테크코스에서 진짜 아무것도 안 한 것 같으면서도 너무 많은 것들을 한 느낌이 드네요. 생각보다 시간이 빨리 지나가 버렸어요
우아한테크코스에서 1주일이 진짜 세상에서 제일 빨리 지나가는 것 같네요
회고를 적기로 결심했던 것은 정말 당행인 것 같네요. 1주일을 어떻게 보냈는지도 되돌아볼 수도 있는 것 같고 이번 주에 했던 실수를 쭉 되돌아볼 수도 있고, 다음 주차를 더 깔끔하게 만들려면 무조건 써야될 것 같습니다

 

이번주에는 저번주 미션 리팩토링을 했고, 2단계 페어프로그래밍을 진행했습니다

자동차 페어프로그래밍 리팩토링을 진행하면서 나왔던 부분들입니다

1. 입출력 어디까지 테스트해야 하는가?

https://be-student.tistory.com/43

java에서 입출력 어디까지 테스트 해야할까요?

이에 관해서는 따로 글을 파서 정리할만큼 내용이 많아서 정리를 해보았는데요

궁금하신 분은 가서 보셔도 좋을 것 같아요

입력에 대해서는 테스트를 작성하지 않는 방법을 앞으로는 사용하려고 합니다

2. Comparable 을 구현할 때 기능추가에 대해서 미리 신경 써야 한다

1등을 가려낼 때, Comparable 메서드를 구현해서 이를 바탕으로 정렬한 이후에, 가장 점수가 높은 사람들을 찾아서 공동 1등이라고 출력해주는 기능을 구현해보려고 했는데요

사람에 대해서 Comparable 을 구현하게 되면, 사람을 정렬할 때, 점수가 아닌 닉네임에 대해서 정렬을 하라는 요구사항이 생겼을 경우에, 되게 애매해질 수 있는데요

새로운 정렬 방식이 필요하지 않을 것이라는 어느정도 기능이 확정된 이후에야 구현을 고려하는 것이 좋아보이네요

 

3. 테스트를 위해서는 원하는 결과를 보기 편하게 입력받을 수 있도록 만드는 쪽이 좋다

public class MockRandomPicker implements NumberPicker {

    private final List<Integer> expected;
    private int pos = 0;


    public MockRandomPicker(List<Integer> expected) {
        this.expected = expected;
    }

    @Override
    public int pickNumber() {
        return expected.get(pos++);
    }
}

 이렇게 생성자를 통해서 원하는 결과를 받도록 하는 방향을 이용해서 테스트한다면 훨씬 깔끔한 코드가 나올 가능성이 높을 것 같습니다

 

4. 파일의 끝마다 줄바꿈을 무조건 해야된다

No newline at end of file

라고 하는 경고메시지가 뜨게 됨

5. assertThatThrownBy 

이것을 통해서 메시지까지 한번에 검사할 수 있다

 

 

6. 줄바꿈

Static 변수와, Instance 변수 사이에는 줄바꿈을 넣으면 좋은 경우가 많다

 

아쉬웠던 점

페어와 프로그래밍을 할때, 아직도 주장이 강했던 부분이 많았던 것 같습니다

조금 더 여유롭게 기다렸어도 괜찮은데, 그 기다리기가 힘들었던 것 같았습니다

천천히 조금만 더 여유를 가지고 진행해보려고 합니다

 

생각보다 개인공부를 했던 시간이 많이 적었던 것 같습니다

우테코도 물론 좋지만, 개인적으로 공부하는 시간을 조금 갖도록 해야 할 것 같습니다

글도 조금 자주자주 쓰고, 해야할 것 같다

 

잘했던 점

그래도 첫주차보다는 직접 코딩하는 정도를 많이 줄였고, 같이 진행하는 페어라는 느낌을 더 잘 줄 수 있었던 것 같다

그래도 일요일에는 정신을 차려서 이렇게 글도 많이 쓰고, 회고도 하고 있는 것 같아서 좀 나은 것 같다