Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[둘리] 2단계 자동차 경주 제출합니다. #68

Merged
merged 12 commits into from
Feb 15, 2023
Merged

[둘리] 2단계 자동차 경주 제출합니다. #68

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
3e97c71
refactor(CarsHelper): companion object를 object로 수정
hyemdooly Feb 12, 2023
cb95bad
refactor(CarsInfo): CarsInfo 삭제, Cars/Car 클래스만 사용
hyemdooly Feb 12, 2023
89d4dd8
refactor(InputView, RaceGame): 반복을 RaceGame이 아닌 InputView에서 하도록 수정
hyemdooly Feb 12, 2023
e37d339
refactor(RaceGame): 간접 호출 삭제
hyemdooly Feb 13, 2023
affbefb
refactor(Car,Cars): backing property 수정
hyemdooly Feb 13, 2023
b6834c8
refactor(Cars,RaceGame): 랜덤값 생성 로직을 RaceGame으로 이동
hyemdooly Feb 13, 2023
1b39386
refactor(CarsHelper): listOf() -> emptyList()로 변경
hyemdooly Feb 13, 2023
e818230
refactor(CarsHelper): CarsHelper -> WinnersFinder로 이름 변경
hyemdooly Feb 13, 2023
18e68de
refactor(CarTest): initTest -> checkCarCreateTest 이름 변경
hyemdooly Feb 13, 2023
f282093
refactor(InputView,RaceGame): 잘못된 값에 대한 함수 반복 여부 RaceGame에서 결정하도록 수정
hyemdooly Feb 13, 2023
4e8ad55
refactor(Constants, InputView, OutputView, RaceGame): 네이밍 변경
hyemdooly Feb 14, 2023
90c2b1d
refactor(Cars): backing property 삭제
hyemdooly Feb 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor(CarsHelper): listOf() -> emptyList()로 변경
  • Loading branch information
@hyemdooly
hyemdooly committed Feb 13, 2023
commit 1b39386fdd83229a1c75b92d88a8d58867f9c213
2 changes: 1 addition & 1 deletion src/main/kotlin/util/CarsHelper.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,6 @@ import model.Cars
object CarsHelper {
Copy link

Choose a reason for hiding this comment

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

객체 이름만 봤을 때는 "자동차들의 도우미"로 해석이 되는데요.
여기서 Helper라는 단어는 포괄적인 의미로 사용돼요.
클래스에 정의된 함수의 역할에 맞게 클래스명을 작성해보면 어떨까요?

fun findWinners(cars: Cars): List<String> {
val equalCars = cars.cars.groupBy({ it.position }, { it.name })
return equalCars[equalCars.keys.max()]?.toList() ?: listOf()
return equalCars[equalCars.keys.max()]?.toList() ?: emptyList()
}
}