[BE] 나아연 👥 오늘의 짝꿍은?

[ay-eonii]

미션 실행 결과

테스트 변경 有

실행 터미널

기능 명세서

입력

• 참석자들의 이름을 입력 받는다
  • , 로 구분한다.
  • 한글이 아니라면 예외가 발생한다.
  • 이름은 중복될 수 없다.
• 최대 짝(그룹) 인원 수를 입력 받는다.
  • 숫자가 아니라면 예외가 발생한다.
  • 이름 수 < 최대 짝(그룹) 인원 수인 경우 예외가 발생한다.
• 재추천 여부를 입력 받는다.
  • y 또는 n 가 아니라면 예외가 발생한다.

출력

• 짝 추천 결과를 출력한다
  • [ 이름1 | ... | 이름2 ] 형식으로 출력한다.
• 예외는 [ERROR]와 메시지를 함께 출력한다.

사람

• 이름은 5글자 이내이다.

짝

• 최대 짝 수는 사람 수보다 작거나 같아야 한다.

매칭

• 랜덤으로 매칭한다.

[taeseokyang]

어나더 레벨

[taeseokyang]

코드 멋집니다

[ay-eonii]

@taeseokyang 뭐야 저도 리뷰해줘요

[taeseokyang]

@ay-eonii 아 너무 좋은데;; 클래스가 많다는 생각이 듭니다! pairCount 와 pairCountValidator 등등의 관련 기능들을 한 클래스로 합치는 것은 어떤가요? 가독성에 오히려 안좋은 영향이 될수도 있지 않을까 생각합니다.

[ay-eonii]

좋은 지적이에요! 사실 pairCount의 초기화과정(생성자)에서 처리하고 싶었는데, 생성자 파라미터로 멤버 수(namesCount)를 함께 넘겨주는 것에 고민이 있었습니다. 생성자가 가지는 멤버변수가 아닌데 생성자 파라미터에 함께 넣어주면 오히려 코드를 이해하는데 더 많은 비용이 들지 않을까 했습니다! 그래서 검증하는 역할을 가진 pairCountValidator를 만들게 되었습니다

많은 클래스 vs 무슨 역할인지 모를 생성자 파라미터

어떤 게 더 나을까요🥲