[BE] 이동훈 👥 오늘의 짝꿍은? by move-hoon · Pull Request #21 · Leets-Official/leets-mate-java-1

move-hoon · 2024-04-11T08:58:53Z

패키지 구조

미션 실행 결과

실행 결과

예외 처리

멤버 이름에 영어가 들어가는 경우

최대 짝 수가 숫자가 아닌 경우

최대 짝 수는 이름의 갯수보다 큰 경우

재시작 입력이 y,n이 아닌경우

기능 명세서

👥 미션 - 오늘의 짝꿍은?

✅ 핵심 기능

목표

나의 짝꿍을 정해줄 랜덤 짝꿍 매칭 프로그램을 작성

✅ 1주차 미션 목표

과제를 통해 생각해보기

어떻게 하면 코드를 효율적으로 짤 수 있는지
어떻게 하면 유지 보수가 쉬운 코드가 될 수 있는지
객체지향 프로그래밍이 목표하는 바가 무엇이고 이를 어떻게 코드로 구현할 수 있는지

✅ 구현할 기능 목록

[ Domain ]

Member
참가자의 데이터 구조와 관련 로직을 담당
문자열로된 멤버들을 리스트로 분리하고, 총 멤버수를 반환합니다.

[ Service ]

GroupService
참가자들을 그룹화하는 로직을 담당
랜덤 짝꿍을 추첨합니다.

[ View ]

InputView
멤버 이름을 입력받음 (,로 구분)
최대 짝 수를 입력받음
다시 구성할지 말지 입력받음(y,n)
OutputView
짝 추천 결과를 보여줌(default, y)
인사 메세지를 보여줌(n)

[ Controller ]

MatchingController
짝궁 추천 요청을 처리하는 로직을 구현

✅ 예외 처리 - exception

공통

예외 상황 시 에러 문구를 출력. 단, 에러 문구는 "[ERROR]"로 시작

이름 입력

이름이 한글이 아니면 예외처리

최대 짝수 입력

최대 짝 수가 이름의 갯수보다 크면 예외처리

retry 입력

y와 n 이외의 숫자면 예외처리

- 유효하지 않은 입력에 대한 예외 메시지 - IllegalArgumentException 객체를 제공

- controller에서 사용하기 위한 static 추가

- ArrayList의 제네릭 타입을 Member 클래스로 변환한 것에 대한 수정

- 필드 없이 그룹핑 하는 작업만 수행하기에 서비스 레이어로 옮기는 것이 더 낫다고 판단했습니다.

- group 리스트를 비우면 groups 리스트 내에 추가된 해당 리스트의 내용도 함께 삭제되기에 새로운 ArrayList 인스턴스 할당

- 기존에 memberCount를 리턴해서 리스트 크기가 고정되는 현상을 해결

- groups.add(new ArrayList<>(group)) 부분이 중요하다.

- 객체를 결합할 수는 없기에 String으로 타입을 변환해야 했다.

jwnnoh · 2024-04-11T10:34:21Z

src/main/java/leets/leets_mate/view/exception/ExceptionType.java

+    String getMessage();
+
+    T getException();
+}


jwnnoh · 2024-04-11T10:38:39Z

docs/README.md

+
+**retry 입력**
+- [x] y와 n 이외의 숫자면 예외처리
+


체크리스트를 통해 TODO를 관리하는 편이 정말 좋은 것 같아요!

taeseokyang

수고하셨습니다!

taeseokyang · 2024-04-11T11:14:47Z

src/main/java/leets/leets_mate/controller/MatchingController.java

+
+        List<Member> membersOfList = Member.parseMembers(members);
+
+        boolean retry = true;


처음 할때도 retry라고 하는데 의미가 잘 안맞는거 같습니다. isDone같은건 어떨까요? 그러면 while(!isDone) 이렇게 해야하는데 아연 누나는 조건으로 !들어가는게 싫다고 하네요. 그럼 isContinue 로 하면 된다고 해요! boolean타입이니까 변수명 앞에 is가 붙으면 더 읽기 쉬울거 같아요

taeseokyang · 2024-04-11T11:15:36Z

src/main/java/leets/leets_mate/controller/MatchingController.java

+            OutputView.printCompletionMessage();
+
+            OutputView.printCanYouRetry();
+            retry = InputView.readRetry();


readRetryOpinion 은 어떨까요? 재시도입력 이라는 뜻이 함수의 기능을 잘 설명안해주는거 같아요

taeseokyang · 2024-04-11T11:18:14Z

src/main/java/leets/leets_mate/service/GroupService.java

+import java.util.List;
+
+public class GroupService {
+    public List<List<Member>> generateRandomGroups(List<Member> memberList, int maximumGroupSize) {


GPT가 개선한 코드입니다. 참고하시면 좋을거 같습니다!

Collections.shuffle(memberList); List<List<Member>> groups = new ArrayList<>(); int totalMembers = memberList.size(); int groupCount = (int) Math.ceil((double) totalMembers / maximumGroupSize); for (int i = 0; i < groupCount; i++) { int fromIndex = i * maximumGroupSize; int toIndex = Math.min((i + 1) * maximumGroupSize, totalMembers); List<Member> subList = memberList.subList(fromIndex, toIndex); groups.add(new ArrayList<>(subList)); } return groups;

taeseokyang · 2024-04-11T11:18:58Z

src/main/java/leets/leets_mate/view/InputView.java

+        try {
+            String input = kb.nextLine();
+            String[] members = input.split(",");
+            for (String member : members) {


문자열 분리하기 전에 검사하면 반복 필요없이 한번만 할 수 있을거 같습니다

taeseokyang · 2024-04-11T11:19:53Z

src/main/java/leets/leets_mate/view/InputView.java

+        }
+    }
+
+    public static int readMaxGroup(String[] members) {


읽는다는 함수인데, 안에서 출력도 하네요! 분리 할 수는 없었을까요?

ay-eonii

1주차 미션 고생하셨습니다🙌
MVC 패턴이 인상깊네요 😊👍

ay-eonii · 2024-04-12T04:20:13Z

src/main/java/leets/leets_mate/controller/MatchingController.java

+import java.util.List;
+
+public class MatchingController {
+    GroupService groupService = new GroupService();


default로 둔 이유가 있을까요?

ay-eonii · 2024-04-12T04:26:27Z

src/main/java/leets/leets_mate/view/InputView.java

+import java.util.Scanner;
+import java.util.regex.Pattern;
+
+public class InputView {


모든 메서드가 static이네요. 그럼 InputView는 객체스럽게 일하고 있나요?

ay-eonii · 2024-04-12T04:27:55Z

src/main/java/leets/leets_mate/view/InputView.java

+    }
+
+    public static String[] readMember() {
+        OutputView.printStartMessage();


OutputView.printStartMessage()가 멤버를 읽는 과정인가요?
추가적으로 InputView가 OutputView를 아는것에 대해 어떻게 생각하세요?

ay-eonii · 2024-04-12T04:31:41Z

src/main/java/leets/leets_mate/LeetsMateApplication.java

+        MatchingController controller = new MatchingController();
+        controller.run();
    }
 }


EOL 문제가 있어요

ay-eonii · 2024-04-12T04:34:04Z

src/main/java/leets/leets_mate/controller/MatchingController.java

+            OutputView.printCanYouRetry();
+            retry = InputView.readRetry();
+
+            if (retry == true) {


Suggested change

if (retry == true) {

if (retry) {

비교한 이유가 있나요?

ay-eonii · 2024-04-12T04:34:48Z

src/main/java/leets/leets_mate/domain/Member.java

+        return members;
+    }
+
+    public static int memberNumber(List<Member> members) {


static 인 이유가 궁금하네요

ay-eonii · 2024-04-12T04:39:27Z

src/main/java/leets/leets_mate/view/InputView.java

+            if (c == 'y') {
+                return true;
+            } else if (c == 'n') {
+                return false;


yellow를 입력해도 재매칭해줄 것 같아요

ay-eonii · 2024-04-12T04:57:22Z

src/main/java/leets/leets_mate/view/InputView.java

+            int maximumGroupSize = kb.nextInt();
+            kb.nextLine();
+
+            checkDataValidity(memberCount, maximumGroupSize);


최대 짝 수는 멤버수보다 작거나 같아야한다

View가 최대 짝 수에 대한 조건과 멤버 수를 알고 있는 거 같아요. InputView는 검증을 하는 객체인가요?

move-hoon added 30 commits April 11, 2024 09:34

docs: 구현할 기능 목록 작성

75e5b9a

refactor(LeetsMateApplication): 선언된 메서드 분리

c4c7968

feat(Member): 문자열로된 멤버들을 리스트로 분리하고, 총 멤버수를 반환하는 도메인 생성

2babb5e

feat(Group): 랜덤 짝꿍 추천을 담당하는 도메인 생성

d137178

feat(InputVIew): 멤버 이름, 최대 짝 수를 입력받는 뷰 생성

4cf6b38

feat(OutputVIew): 결과를 프린트하는 뷰 생성

1a91a77

feat(MatchingController): 짝궁 추천 요청을 처리하는 로직을 담당하는 컨트롤러 생성

a9efde8

test(LeetsMateApplicationTests): 테스트 분리

7f4274b

feat(InputView): 정적 팩토리 메서드 추가

79a2f14

test(InputViewTests): 테스트 로직 분리

a10611d

test(MemberTests): 테스트 로직 분리

5c91465

feat(ExceptionType): 예외 타입들을 표준화된 방식으로 처리하는 인터페이스 생성

12c659e

feat(IllegalArgumentExceptionType): ExceptionType 인터페이스를 구현한 열거형 생성

f168091

- 유효하지 않은 입력에 대한 예외 메시지 - IllegalArgumentException 객체를 제공

feat(OutputView): 시작 메시지를 출력하는 메서드 구현

e67a4a2

feat(IllegalArgumentExceptionType): 상수 추가

91094ac

chore(LeetsMateApplication): 사용되지 않는 import문 삭제

880f008

refactor(OutputView): static 키워드 추가

1ed6286

- controller에서 사용하기 위한 static 추가

feat(MatchingController): 출력메시지, 멤버의 이름 입력받는 기능 컨트롤러에 구현

1d5e516

feat(InputView): 멤버 이름 입력, 최대 짝 수 입력 기능 및 검증 로직 구현

0bf22e9

feat(OutputView): 최대 짝수 입력 멘트를 출력하는 메서드 구현

f665a22

refactor(InputView): 최대 짝수를 입력할 때 숫자가 아니면 예외처리

e42ab3d

refactor(MatchingController): InputView 변동사항에 대한 리팩토링

c3929f0

feat(Member): 도메인 구현

da4b742

test(MemberTests): 타입 변환에 의한에 테스트 코드 수정

615d27d

- ArrayList의 제네릭 타입을 Member 클래스로 변환한 것에 대한 수정

remove(Group): Group 도메인을 제거

ec960d5

- 필드 없이 그룹핑 하는 작업만 수행하기에 서비스 레이어로 옮기는 것이 더 낫다고 판단했습니다.

feat(GroupService): GroupService 생성

a4bb9fd

feat(GroupService): 최대 짝수만큼 Grouping하는 메서드 구현

d31a0a8

refactor(InputView): 문제 예시에 맞게 개행 추가

55c2bce

fix(GroupService): groups에 add된 group의 내용이 모두 지워지는 문제 해결

5ce6071

- group 리스트를 비우면 groups 리스트 내에 추가된 해당 리스트의 내용도 함께 삭제되기에 새로운 ArrayList 인스턴스 할당

refactor(InputView): 개행 삭제

35cbe6a

move-hoon added 13 commits April 11, 2024 16:42

fix(InputView): maximumGroupSize를 리턴하도록 변경

1df36fd

- 기존에 memberCount를 리턴해서 리스트 크기가 고정되는 현상을 해결

refactor(GroupService): clear()로 group 리스트 비우기

a4a0235

- groups.add(new ArrayList<>(group)) 부분이 중요하다.

feat(OutputView): Member 객체를 문자열로 변환하여 String.join 도입

539d294

- 객체를 결합할 수는 없기에 String으로 타입을 변환해야 했다.

feat(OutputView): 추첨 완료 메세지를 출력하는 메서드 구현

21ace85

docs: README 업데이트

3ac6f4e

feat(IllegalArgumentExceptionType): retry 시 잘못된 입력 에러 상수 추가

b6f2158

feat(InputView): retry를 입력받는 메서드 구현

7199cf8

feat(OutputView): retry 여부를 묻는 메시지 출력

3e116b7

feat(OutputView): retry를 하지 않는 경우에 해당하는 메시지 출력 메서드 구현

f4cd2e5

chore(GroupService, InputView, Member): 주석 삭제

31a6e40

feat(OutputView): 라인을 출력하는 메서드 생성

2d883e5

feat(IllegalArgumentExceptionType): retry 입력 시 y,n 제외 입력 에러 상수 추가

fcedf0c

feat(MatchingController): 컨트롤러 구현

df1c4f9

move-hoon requested review from MinhoJJang, ay-eonii and taeseokyang April 11, 2024 08:58

move-hoon self-assigned this Apr 11, 2024

move-hoon requested review from leejuae and zzu-yaaa April 11, 2024 08:59

move-hoon added 2 commits April 11, 2024 18:05

refactor(InputView): 영어+한글 혼합을 잡지 못하는 부분 수정

e22276c

docs: README 업데이트

7356307

jwnnoh reviewed Apr 11, 2024

View reviewed changes

taeseokyang reviewed Apr 11, 2024

View reviewed changes

ay-eonii reviewed Apr 12, 2024

View reviewed changes


		retry 입력
		- [x] y와 n 이외의 숫자면 예외처리


		List<Member> membersOfList = Member.parseMembers(members);

		boolean retry = true;

Comments

Conversation

move-hoon commented Apr 11, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

패키지 구조

미션 실행 결과

실행 결과

예외 처리

멤버 이름에 영어가 들어가는 경우

최대 짝 수가 숫자가 아닌 경우

최대 짝 수는 이름의 갯수보다 큰 경우

재시작 입력이 y,n이 아닌경우

기능 명세서

👥 미션 - 오늘의 짝꿍은?

✅ 핵심 기능

✅ 1주차 미션 목표

과제를 통해 생각해보기

✅ 구현할 기능 목록

[ Domain ]

[ Service ]

[ View ]

[ Controller ]

✅ 예외 처리 - exception

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

taeseokyang left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

ay-eonii left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

move-hoon commented Apr 11, 2024 •

edited

Loading