[BE] 한성민 👥 오늘의 짝꿍은? by anxi01 · Pull Request #12 · Leets-Official/leets-mate-java-1

anxi01 · 2024-04-10T13:17:31Z

미션 결과

테스트

실행 결과

기능 명세서

랜덤 짝꿍 추첨 로직

currentIndex를 1씩 증가하면서 그룹에 멤버를 채워 result 리스트에 넣어준다.
그룹에 가득 차지 못할 경우, 나머지 멤버들을 모아 마지막 그룹에 넣어준다.

taeseokyang

수고하셨습니다!!!

taeseokyang · 2024-04-10T14:30:11Z

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

+
+        printResult(generateRandomGroups(memberList, maximumGroupSize));
+
+        while (true) {


위에서
printResult(generateRandomGroups(memberList, maximumGroupSize));
코드가 중복됩니다! while코드를 아래와 같이 수정하면 좋을 것 같습니다!

while (true) { printResult(generateRandomGroups(memberList, maximumGroupSize)); System.out.print("다시 구성하시겠습니까? (y or n): "); String answer = sc.next(); if (answer.equals("n")) { System.out.println("자리를 이동해 서로에게 인사해주세요."); break; } }

오 좋은 방법이네요!
감사합니다 :)

taeseokyang · 2024-04-10T14:58:42Z

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

    }

    // 랜덤 짝꿍 추첨하는 함수 입니다.
    public List<List<String>> generateRandomGroups(List<String> memberList, int maximumGroupSize) {


이 함수 코드를 GPT를 통해 개선 하면
아래와 같이 개선 가능한데

Collections.shuffle(memberList); List<List<String>> result = new ArrayList<>(); int memberCount = memberList.size(); int groupCount = memberCount / maximumGroupSize; int remainder = memberCount % maximumGroupSize; int currentIndex = 0; for (int i = 0; i < groupCount; i++) { List<String> group = new ArrayList<>(memberList.subList(currentIndex, currentIndex + maximumGroupSize)); result.add(group); currentIndex += maximumGroupSize; } // 남은 멤버들을 처리합니다. List<String> lastGroup = new ArrayList<>(memberList.subList(currentIndex, currentIndex + remainder)); result.add(lastGroup);

남은 멤버 따로 처리 하기 싫어서 아래와 같은 방법을 생각해 보았습니다. 아마 더 좋은 방법이 있을 거 같은데, 제 방법 공유해 드립니다!

Collections.shuffle(memberList); List<List<String>> result = new ArrayList<>(); int memberCount = memberList.size(); int groupCount = memberCount / maximumGroupSize; int leftMemberCount = memberCount; int currentIndex = 0; for (int i = 0; i < groupCount+1; i++) { int groupSize = Math.min(maximumGroupSize, leftMemberCount); List<String> group = new ArrayList<>(memberList.subList(currentIndex, currentIndex + groupSize)); result.add(group); currentIndex += maximumGroupSize; leftMemberCount -= maximumGroupSize; }

남아있는 멤버와 최대 그룹 수를 비교해주면 되는군요 !
좋은 의견 감사합니다 !

ay-eonii

1주차 미션 고생하셨습니다 👏
요구사항을 잘 반영해서 구현해주셨네요!

ay-eonii · 2024-04-11T00:30:32Z

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

+            System.out.print("다시 구성하시겠습니까? (y or n): ");
+            String answer = sc.next();
+
+            if (answer.equals("y")) {


만약 answer이 null 이라면 어떤 문제가 있을까요?

null일 경우 종료되는 문제가 발생하네요,,
y, n 그리고 이 두 값과 다른 경우를 예외처리 해야겠습니다 !
감사해요 !

y 또는 n 인경우 예외처리도 중요합니다👍
그런데 answer이 null 이라면 NPE가 날 것 같아요.

Suggested change

if (answer.equals("y")) {

if ("y".equals(answer)) {

이렇게 변경해도 NPE가 발생할 가능성이 있을까요?

euqals 메서드는 nullable한 객체를 인자로 받기 때문에 NPE가 발생하지 않을 것 같은데 어떻게 생각하시는지 궁금합니다 !

euqals 메서드는 nullable한 객체를 인자로 받기 때문에 NPE가 발생하지 않을 것 같은데

넵 맞아요!
answer.equals("y")는 NPE가 발생할 가능성이 있지만 순서를 바꿔서 "y".equals(answer)로 작성한다면 NPE가 발생하지 않겠죠! 그래서 전 후자를 선호합니다 😊

전혀 생각해본적 없었는데 깨닫고 갑니다 !
감사해요 !

ay-eonii · 2024-04-11T00:33:45Z

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

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


StringJoiner를 알아보면 도움이 될 것 같네요 😊

추가적으로, \n 개행문자를 하드코딩했을 때의 문제점은 무엇이 있을까요?

StringJoiner은 처음 들어보는데 한번 알아볼게요 !

하드코딩하면 코드가 지저분해보이는 것도 문제점이라 할 수 있을 거 같은데
아연님이 생각하시는 문제점은 무엇인지 궁금해요 !

UNIX : \n
Windows : \r\n

운영체제마다 개행문자가 다르기 때문에 원하는 대로 줄바꿈이 되지 않을 가능성이 있어보여요!

오오,, 운영체제에 대한 생각은 전혀 해본 적 없는데,, 이런 일이 발생할 수도 있군요,,! 참고하겠습니다! 감사해요 ㅎㅎ

ay-eonii · 2024-04-11T00:37:06Z

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


    // 동작 함수입니다.
-    public void run() {
+    public void run() throws Exception {


지금은 예외를 던지고만 있지만, 오타가 날 때마다 프로그램이 종료되면 불편하지 않을까요 ?😋

예외 처리 대신 boolean으로 설정해서 종료되지 않고 재입력 받을 수 있게 해야겠네요 !
예외를 던지는 것만이 방법인줄 알았는데 알려주셔서 감사합니다 :)

anxi01 added 4 commits April 5, 2024 03:04

feat: 랜덤 추출 기능 작성

b1869f6

fix : 영어 검증 로직 수정

4289636

refactor : 정적 변수 정의 및 메소드 재사용

831502d

refactor : toList 대신 asList로 변경

370dde5

anxi01 self-assigned this Apr 10, 2024

anxi01 added 4 commits April 10, 2024 22:25

fix: 정규표현식 변경

f8a9842

fix: 랜덤 짝궁 추첨 로직 변경

2b5a685

fix: 출력 로직 변경

83a3a1f

refactor: 불필요한 변수 제거

c3ee1e0

taeseokyang reviewed Apr 10, 2024

View reviewed changes

ay-eonii reviewed Apr 11, 2024

View reviewed changes


		printResult(generateRandomGroups(memberList, maximumGroupSize));

		while (true) {

Comments

Conversation

anxi01 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!

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!

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!

3 participants

anxi01 commented Apr 10, 2024 •

edited

Loading