Skip to content

Conversation

@taejinn
Copy link

@taejinn taejinn commented Mar 24, 2025

No description provided.

Copy link

@BaeJinho4028 BaeJinho4028 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

요구사항대로 오버엔지니어링 없이 잘 작성해주셨습니다. 커밋도 깔끔하네요 👍
이번주차 과제 고생하셨어요!

int attempts = getAttempts();
Racing racing = new Racing(cars, attempts);
racing.start();
printWinners(racing.getWinners()); // try-catch 제거

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

주석을 남겨놓으신 이유가 있을까요?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

처음엔 try-catch로 예외 처리를 했었는데, 테스트 코드 실행 후 오류가 발생하여 보았더니... 예외를 직접 던져야 한다는걸 알게되어서...😭 try-catch는 지우고 주석으로 잠시 기록해둔 후, 커밋할 때 잊어버려 지우지 못한 것 같습니다...

String[] names = Console.readLine().split(",");
return Arrays.stream(names)
.map(Car::new)
.collect(Collectors.toList());

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'collect(Collectors.toList())'와 그냥 'toList()'는 무슨 차이가 있을까요?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

toList()는 수정 불가능한 리스트를 반환하고, collect(Collectors.toList())는 수정 가능한 리스트를 반환합니다!

import java.util.List;
import java.util.stream.Collectors;

public class Racing {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

모듈화가 잘되어있네요 👍

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

감사합니다 😆

@@ -1,0 +1,29 @@
## ✅ 핵심 기능 목록 (체크리스트)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

체크리스트 좋네요. 완성되었으니 체크 표시 하면 좋을 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants