[BE] 이근표 👥 오늘의 짝꿍은? by rootTiket · Pull Request #15 · Leets-Official/leets-mate-java-1

rootTiket · 2024-04-10T15:31:06Z

테스트 결과

실행결과

기능 명세서

✅ 입력받은 이름을 , 기준으로 이름 리스트 생성
✅ 입력된 이름 수를 저장하는 메서드 구현
✅ 짝수를 입력 받고 리스트의 크기 보다 크면 예외를 처리
✅ 다시 뽑기 실행시 y,n 을 제외한 문자가 입력 될 경우 예외 처리
✅ 짝을 뽑고 주어진 형식에 맞게 출력하는 메서드 구현

taeseokyang

수고하셨습니다!!!!

taeseokyang · 2024-04-10T16:03:53Z

    public void run() {
+        System.out.println("멤버의 이름을 입력해 주세요. (, 로 구분)");
+        Scanner sc = new Scanner(System.in);
+        String input = sc.nextLine();


무슨 입력인지 나타내는 변수명이면 좋을 것 같습니다!

taeseokyang · 2024-04-10T16:04:22Z

+        System.out.println("최대 짝 수를 입력해 주세요.");
+        int maxNum = Integer.parseInt(sc.nextLine());
+        checkDataValidity(memberList.size(), maxNum);
+        String flag ="true";


무슨 플래그 인지 알려주면 좋을거 같습니다!
변수명만 보고 바로 이해 하면 코드를 읽기 더욱 편할 거 같습니다!

boolean 형 변경하는 것도 좋다 생각합니다! 만약 그렇다면 isDone은 어떨까요?
현재 String 형으로 true, exception을 반환 받고 있는데, isContinue() 함수에서는 계속 할지 안할지 true, false 둘중 하나만 반환하면 된다 생각합니다!

taeseokyang · 2024-04-10T16:05:25Z

    // 문자열로된 멤버들을 리스트로 분리하는 함수입니다.
    public List<String> parseMembers(String members) {
-        return new ArrayList<>();
+        List<String> memberList = new ArrayList<>();


return Arrays.asList(members.split(","));
위와 같이 작성 할 수 있습니다!

ay-eonii

1주차 미션 고생하셨습니다
char로 한글을 판단한게 인상깊네요! 👍

여유가 있다면 클래스가 적절한 책임을 가지도록 분리해봐도 좋겠어요!

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

+        int maxNum = Integer.parseInt(sc.nextLine());
+        checkDataValidity(memberList.size(), maxNum);
+        String flag ="true";
+        while (flag.equals("true") || flag.equals("exception")) {


만약 flag 가 null 이라면 어떤 문제가 있을까요?
그리고 이를 방지하기 위해 어떻게 해야할까요?

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

    public void checkHasNoEnglish(String members) {
+        try {
+            for (char c : members.toCharArray()) {
+                if (!(c >= '가' && c <= '힣') && !(Character.toString(c).equals(","))) {


복잡한 조건문은 메서드로 따로 빼면 가독성이 죠아져요 👍

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

+            }
+        } catch (Exception e) {
+            System.out.println(e.getMessage());
+            run();


숫자 실수하면 이름까지 다시 입력받는 하드코어 프로그램이 되었네요! 👍

ay-eonii · 2024-04-11T01:15:26Z

+        for (List<String> line : result) {
+            StringBuilder sb = new StringBuilder();
+            sb.append("[ ");
+            for (String s : line) {
+                sb.append(s).append(" | ");
+            }
+            sb.replace(sb.length()-2,sb.length(),"]");
+            System.out.println(sb);
+        }


StringJoiner에 대해 알아보면 편해질거에요!

ay-eonii · 2024-04-11T01:15:49Z

+        if (q.equals("n")) {
+            System.out.println("자리를 이동해 서로에게 인사해주세요.");
+            return "false";
+        } else if (q.equals("y")) {
+            System.out.println("--------------------------------");
+            return "true";
+        } else {
+            System.out.println("[ERROR] y 혹은 n만 입력해주세요.");
+            return "exception";
+        }


else 는 없어도 되겠네요!

ay-eonii · 2024-04-11T01:16:06Z

+    public static String isContinue() {
+        System.out.print("다시 구성하시겠습니까? (y or n): ");
+        Scanner sc = new Scanner(System.in);
+        String q = sc.nextLine();


q 는 무엇을 의미하나요?

-다시 뽑기를 위한 while 문의 flag 변수 타입을 Boolean으로 변경 -flag 변수의 이름을 isDone으로 수정 -isContinue의 반환값을 Boolean으로 수정

-for문 대신 asList를 이용하여 멤버를 분리하도록 변경

jiixon

고생하셨습니다~

jiixon · 2024-04-12T07:54:17Z

+        Boolean isDone = true;
+        while (isDone .equals(true)) {
+            printResult(generateRandomGroups(memberList,maxNum));
+            isDone = isContinue();


while (isDone) { 이렇게 구현해도 되지않을까요?

[feat]: 짝 뽑기 기능 구현

351e291

rootTiket self-assigned this Apr 10, 2024

taeseokyang reviewed Apr 10, 2024

View reviewed changes

ay-eonii reviewed Apr 11, 2024

View reviewed changes

rootTiket added 3 commits April 11, 2024 15:23

[refactor]: 멤버 입력 변수 명 수정

4031fdc

[refactor]: 다시 뽑기 기능 수정

198ce44

-다시 뽑기를 위한 while 문의 flag 변수 타입을 Boolean으로 변경 -flag 변수의 이름을 isDone으로 수정 -isContinue의 반환값을 Boolean으로 수정

[refactor]: 멤버 리스트 분리 하는 함수 수정

99dcf41

-for문 대신 asList를 이용하여 멤버를 분리하도록 변경

rootTiket changed the title ~~[BE] 이근표 오늘의 짝꿍은?~~ [BE] 이근표 👥 오늘의 짝꿍은? Apr 11, 2024

jiixon reviewed Apr 12, 2024

View reviewed changes

Uh oh!

Conversation

rootTiket commented Apr 10, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

테스트 결과

실행결과

기능 명세서

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!

ay-eonii left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

jiixon left a comment

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

rootTiket commented Apr 10, 2024 •

edited

Loading

ay-eonii left a comment •

edited

Loading