2주차 리뷰 드립니다 😊

[sonypark]

안녕하세요 마르코, 소니입니다.
2주차 미션에 대해 간단한 리뷰 남기니 참고해주세요~

1. 현재는 메서드가 호출 될 때마다 Scanner 객체를 계속 생성하는 구조인데, 한 번만 생성하고 재사용하는 방법은 없을까요?
2. Random 객체도 마찬가지로 매번 생성하지 않고 재사용할 수 있는 방법을 찾아보면 좋겠네요!
3. mycost와 opponentcost 값을 game 메서드에서 해주고 있는데 더 좋은 방법은 없을까요?
4. OddOrEven 클래스가 하는 역할이 너무 많은데, 역할에 맞게 클래스를 분리하는 것도 생각해보세요 (ex. 출력 로직 분리 등)
5. 자바의 네이밍은 CamelCase가 원칙입니다. myanswer, numstage 등 이 원칙이 지켜지지 않은 부분이 보이는데 확인 후 수정해주세요~
6. printoutOddorEven라는 네이밍은 출력만 하고 반환값은 없는 메서드에 어울리는 이름입니다. 현재는 String을 리턴하는 메서드 인데요. 더 적합한 이름을 생각해보면 좋겠네요
7. 아래와 같이 String을 선언하는 경우는 거의 없습니다. 자바 String 사용법에 대해 검색해보세요

String correct = new String();

8. 아래 로직은 Comparator로 변환할 수 있을 것 같은데, Comparator에 대해 공부해보세요

Collections.sort(id, (Id o1, Id o2)->{
            return o1.getTurn()-o2.getTurn();
        });