[BE] 김예찬 👥 오늘의 짝꿍은? by yechan-kim · Pull Request #14 · Leets-Official/leets-mate-java-1

yechan-kim · 2024-04-10T14:29:02Z

미션 실행 결과

[테스트 실행 터미널 캡쳐 이미지 파일 첨부]
[테스트]

[실행결과]

미션 실행 실패 케이스 존재 시, 실패한 이유 고찰해보기

정상 작동

기능 명세서

String으로 이름을 입력받아 [parseMembers] 메서드에서 ','를 기준으로 이름 리스트 생성
[memberNumber] 메서드에서 입력된 이름수를 저장
[parseMembers] 메서드에서 [checkHasNoEnglish] 메서드를 통해 이름 중에 영어가 없도록 예외 처리
최대 짝 수를 입력하고, [checkMaxGroup]를 통해 최대 짝 수가 입력된 이름 수보다 많지 않게 예외 처리
[generateRandomGroups] 메서드에서 그룹별 리스트 생성
[printResult] 메서드에서 전체 그룹 출력
[ask] 메서드에서 재구성 여부를 입력받고, y/n 이외의 입력을 받지 않도록 예외 처리
반복문을 통해 재실행 기능 구현

taeseokyang

로직을 작성한 후, GPT에게 개선을 요청 해보시는것을 추천드립니다! GPT는 좋은 코드리뷰어이고, 개선 결과에서 배우는 것이 많습니다.

랜덤 그룹 매칭 함수 로직을 개선 해보시는 걸 추천드립니다!!

수고하셨습니다!

taeseokyang · 2024-04-10T15:55:19Z

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

+        System.out.println("[Leets 오늘의 짝에게]를 시작합니다.");
+
+        System.out.println("\n참석자들의 이름을 입력해 주세요. (,로 구분)");
+        String inputStr = sc.nextLine();


어떤 입력인지 알 수 있는 변수명이면 더욱 좋을거 같습니다!

memberNamesInput 는 어떨까요?

taeseokyang · 2024-04-10T15:56:54Z

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

+        checkDataValidity(memberNumber(members), maximumGroupSize);
+        System.out.println();
+
+        boolean result = true;


변수명이 적절하지 않은 것 같습니다! 결과가 거짓, 진실은 아니기 때문입니다.

isDone 같은 변수는 어떤가요? boolean형 변수는 앞에 is를 붙이면 읽기 쉽습니다!

ay-eonii

1주차 미션 고생하셨습니다🥳
요구사항을 너무 잘 지켜주셨네요!

ay-eonii · 2024-04-11T00:59:51Z

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

+        Scanner sc = new Scanner(System.in);
+        System.out.println("[Leets 오늘의 짝에게]를 시작합니다.");
+
+        System.out.println("\n참석자들의 이름을 입력해 주세요. (,로 구분)");


개행문자를 \n로 두었을 때 어떤 문제가 있을까요?

ay-eonii · 2024-04-11T01:02:35Z

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

+    }
+
+    // 다시 추천을 할지 물어보는 함수입니다.
+    public boolean ask(String operator) throws Exception {


ask에는 재추첨 여부라는 의미가 잘 담겨있지 않은 것 같아요 🥲

ay-eonii · 2024-04-11T01:03:37Z

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

+        Collections.shuffle(memberList);
+
+        List<List<String>> result = new ArrayList<>();
+        List<String> temp = new ArrayList<>();


조금더 의미를 담아 변수명을 만들어주시면 다른 사람이 코드 읽을 때 이해가 쉬울 것 같아요😄

ay-eonii · 2024-04-11T01:04:12Z

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

+        for (List<String> group : result) {
+            sb.append("[");
+
+            for (int i = 0; i < group.size() - 1; i++) {
+                sb.append(" ").append(group.get(i)).append(" |");
+            }
+
+            sb.append(" ").append(group.get(group.size() - 1)).append(" ]\n");
+        }


StringJoiner에 대해 알아보시면 도움이 될 것 같아요 😊

ay-eonii · 2024-04-11T01:09:00Z

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

+        else if (operator.equals("y")){
+            System.out.println("--------------------------------");
+            return true;
+        }


early return 패턴이라면 else가 필요없어 보이네요😊

yechan-kim added 2 commits April 5, 2024 16:31

feat: 랜덤 뽑기 기능, 다시 추천 기능 추가

c0b3175

test: 다시 추천 기능 예외 처리를 위한 테스트 추가

bc31b2a

taeseokyang reviewed Apr 10, 2024

View reviewed changes

ay-eonii reviewed Apr 11, 2024

View reviewed changes

jwnnoh assigned yechan-kim Apr 11, 2024

yechan-kim added 3 commits April 11, 2024 17:58

refactor: 코드 리뷰 적용

17a4646

refactor: 피드백 적용

0923c0c

test: 리펙터링으로 인한 테스크 코드 수정

7e76890

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[BE] 김예찬 👥 오늘의 짝꿍은?#14

[BE] 김예찬 👥 오늘의 짝꿍은?#14
yechan-kim wants to merge 5 commits intoLeets-Official:mainfrom
yechan-kim:yechan-kim

yechan-kim commented Apr 10, 2024

Uh oh!

taeseokyang left a comment

Uh oh!

taeseokyang Apr 10, 2024

Uh oh!

taeseokyang Apr 10, 2024

Uh oh!

ay-eonii left a comment

Uh oh!

ay-eonii Apr 11, 2024

Uh oh!

ay-eonii Apr 11, 2024

Uh oh!

ay-eonii Apr 11, 2024

Uh oh!

ay-eonii Apr 11, 2024

Uh oh!

ay-eonii Apr 11, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments

Conversation

yechan-kim commented Apr 10, 2024

미션 실행 결과

미션 실행 실패 케이스 존재 시, 실패한 이유 고찰해보기

기능 명세서

Uh oh!

taeseokyang left a comment

Choose a reason for hiding this comment

Uh oh!

taeseokyang Apr 10, 2024

Choose a reason for hiding this comment

Uh oh!

taeseokyang Apr 10, 2024

Choose a reason for hiding this comment

Uh oh!

ay-eonii left a comment

Choose a reason for hiding this comment

Uh oh!

ay-eonii Apr 11, 2024

Choose a reason for hiding this comment

Uh oh!

ay-eonii Apr 11, 2024

Choose a reason for hiding this comment

Uh oh!

ay-eonii Apr 11, 2024

Choose a reason for hiding this comment

Uh oh!

ay-eonii Apr 11, 2024

Choose a reason for hiding this comment

Uh oh!

ay-eonii Apr 11, 2024

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants