[BE] 이주애 👥 오늘의 짝꿍은? by leejuae · Pull Request #7 · Leets-Official/leets-mate-java-1

leejuae · 2024-04-10T06:47:48Z

미션 실행 결과

기능 명세서

기능

참여자의 이름을 입력할 시, 팀이 랜덤배정됨.
다시 랜덤으로 입력을 받기 원하면, y를 입력하면 인원이 재배열됨.
n을 입력 시에 프로그램 종료

ERROR SATUS

참여자 이름 입력 시 영어가 입력되면 오류 발생
최대 짝수가 입력된 인원보다 많을 시 오류발생
y/n 이외의 문자가 입력 시 오류 발생

taeseokyang

수고하셨습니다!!

텍스트 프린트도 구현 해주시면 감사하겠습니다!

taeseokyang · 2024-04-10T12:04:41Z

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

    public void run() {
+        Scanner sc = new Scanner(System.in);
+        String members = sc.nextLine();
+        int ret = 1;


딱 보고 의미를 알 수 있도록 변수명을 적어주시면 좋을거 같습니다!

taeseokyang · 2024-04-10T12:06:34Z

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

    }

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


return Arrays.asList(members.split(","));
위 와 같이 작성하시면 쉽게 분리 할 수 있습니다!

ChatGPT를 이용하여 코드를 붙여 넣고 "더 좋은 방법이 있을까? 리팩토링해줘' 와 같이 질문하시면 좋은 피드백을 받을 수 있습니다!

그렇게도 지피티를 사용해볼 수 있겠네요!! 참고해보겠습니다 ㅎㅎ

taeseokyang · 2024-04-10T12:09:59Z

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

+            printResult(generateRandomGroups(memArray, maximumGroupSize));
+
+            System.out.print("다시 구성하시겠습니까? (y or n): ");
+            String re = sc.nextLine();


쉽게 이해 할 수 있는 변수명 부탁드립니다!

taeseokyang · 2024-04-10T12:12:07Z

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

+    }
+
+    // 다시 추첨을 진행하는 함수입니다.
+    public int reGeneration(String re) {


함수명이 함수의 기능을 잘 설명해주지 않는것 같습니다!

getRetryDecision() 같은건 어떨까요?
'다시 추첨할지 말지 입력 받는 함수 이름 뭐라고 할까?' 라고 GPT에게 물어본 결과입니다!

ay-eonii

첫 미션 고생하셨어요!

미션 예시와 매우 유사하게 구현해주셨네요👍

ay-eonii · 2024-04-10T14:38:01Z

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

+        int memberCount = memberNumber(memArray);
+        // 이름 개수보다 짝 수가 더 많으면 예외발생
+        int maximumGroupSize = Integer.parseInt(sc.nextLine());
+        checkDataValidity(memberCount, maximumGroupSize);


하나의 메서드에 적절하게 책임을 분리했네요 👍

ay-eonii · 2024-04-10T14:38:49Z

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

+
+        while (ret == 1){
+            // 랜덤 짝궁 추천
+            printResult(generateRandomGroups(memArray, maximumGroupSize));


핵심 로직이 숨어있어요~ 😶‍🌫️

ay-eonii · 2024-04-10T14:40:25Z

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

+        Collections.shuffle(memberList);
+        List<List<String>> groups = new ArrayList<>();
+        for (int i = 0; i < memberList.size(); i += maximumGroupSize) {
+            groups.add(new ArrayList<>(memberList.subList(i, Math.min(i + maximumGroupSize, memberList.size()))));


다른 사람이 쉽게 이해할 수 있도록 inline으로 사용한 로직을 지역변수를 사용해 분리해보는 건 어떨까요?

ay-eonii · 2024-04-10T14:41:18Z

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

+        for (List<String> group : result) {
+            System.out.print("[ ");
+            for (int i = 0; i < group.size(); i++) {
+                System.out.print(group.get(i));
+                if (i < group.size() - 1) {
+                    System.out.print(" | ");
+                }
+            }
+            System.out.println(" ]");
+        }


StringJoiner 또는 String.join()에 대해 알아보면 도움이 될 것 같아요 😆

ay-eonii · 2024-04-10T14:42:28Z

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

+
+    // 다시 추첨을 진행하는 함수입니다.
+    public int reGeneration(String re) {
+        if(Objects.equals(re, "n")) {


String의 equals가 아니라 Objects의 equals를 사용한 이유가 궁금해요!

해당부분 제가 String 클래스에 대한 이해도가 부족해서 그랬던거 같아요!! String.equals(re, "n")이 맞는 문법이라 생각했는데 다시 찾아보니 re.equals("n")로 처리를 했어야 했네요..ㅎㅎ 그냥 넘어갈뻔 했던 부분인데 덕분에 다시 짚고 넘어갈 수 있었어요!

Object.equals()는 주소값을 비교하는데, String.equals()는 문자 하나하나를 비교해요! 같은 "n" 이더라도 Object.equals()를 사용하면 false가 나올 수 있는거죠! 이런 문제가 있어 문자열을 비교할때는 String.equals()를 사용하는게 좋아요!

추가적으로, re.equals("n")의 경우 re가 null이라면 문제가 있을 거 같은데요🤔

ay-eonii · 2024-04-10T14:42:58Z

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

+            return 0;
+        } else if (Objects.equals(re, "y")) {
+            return 1;


0 또는 1을 반환하는 듯한데 boolean을 사용하지 않은 이유가 있을까요?

그렇게 처리하는것도 좋을거 같네요! bool 리턴보단 0,1 리턴이 좀 더 익숙해서 이렇게 처리했던거 같아요 ㅎㅎ

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

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

    // 동작 함수입니다.
    public void run() {
+        Scanner input = new Scanner(System.in); // Scanner 클래스 생성
+        int ret = 1;    // loop 돌리기 위한 변수 설정


boolean 타입으로 변수를 선언해주면 좋았을 것 같아요!

jwnnoh · 2024-04-11T10:42:11Z

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

+        while(st.hasMoreTokens())
+            memArray.add(st.nextToken());
+
+        return memArray;


StringTokenizer도 좋지만, split()메서드도 현재 로직에서 상당히 괜찮아 보입니다!

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

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


    // 멤버 문자열에 영어가 있는지 검사합니다. 영어가 있다면 예외 출력
    public void checkHasNoEnglish(String members) {
+        if (!members.matches("[가-힣,]+")) {


메서드명과 메서드가 하는 행위가 일치하면 좋을 것 같습니다!

mission1: 1차 제출코드

59c2af1

leejuae self-assigned this Apr 10, 2024

taeseokyang reviewed Apr 10, 2024

View reviewed changes

ay-eonii reviewed Apr 10, 2024

View reviewed changes

leejuae changed the title ~~[BE] {이주애} {leets-mate-java-1}~~ [BE] 이주애 👥 오늘의 짝꿍은? Apr 11, 2024

fix: print문 수정, Object.equals 수정

a813dc6

jwnnoh reviewed Apr 11, 2024

View reviewed changes

Comments

Conversation

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

Uh oh!

미션 실행 결과

기능 명세서

기능

ERROR SATUS

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!

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

leejuae commented Apr 10, 2024 •

edited

Loading