-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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.
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
차한솔_11월20일_스터디용 코드 #2384
base: main
Are you sure you want to change the base?
차한솔_11월20일_스터디용 코드 #2384
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
# 최종 코테 연습용 | ||
|
||
## 기능 | ||
- [x] 가이드 메세지 노출 | ||
- [x] 각 자동차에 이름을 부여할 수 있다. | ||
- [x] 자동차 이름은 쉼표(,)를 기준으로 구분하며 이름은 5자 이하만 가능하다. | ||
- [x] 사용자는 몇 번의 이동을 할 것인지를 입력할 수 있어야 한다. | ||
- [x] 사용자가 잘못된 값을 입력할 경우 IllegalArgumentException을 발생시킨 후 애플리케이션은 종료되어야 한다. | ||
- [x] 입력값 없는 경우(전체 ,split ",") | ||
- [x] 숫자로 변경할 수 없는 경우 | ||
- [x] 주어진 횟수 동안 n대의 자동차는 전진 또는 멈출 수 있다. | ||
- [x] 전진하는 조건은 0에서 9 사이에서 무작위 값을 구한 후 무작위 값이 4 이상일 경우이다. | ||
- [x] 전진하는 자동차를 출력할 때 자동차 이름을 같이 출력한다. | ||
- [x] 자동차 경주 게임을 완료한 후 누가 우승했는지를 알려준다. 우승자는 한 명 이상일 수 있다. | ||
- [x] 우승자가 여러 명일 경우 쉼표(,)를 이용하여 구분한다. | ||
|
||
## 추가 요구사항 | ||
- [x] JDK에서 제공하는 Random 및 Scanner API 대신 camp.nextstep.edu.missionutils에서 제공하는 Randoms 및 Console API를 사용하여 구현해야 한다. | ||
- [x] Random 값 추출은 camp.nextstep.edu.missionutils.Randoms의 pickNumberInRange()를 활용한다. | ||
- [x] 사용자가 입력하는 값은 camp.nextstep.edu.missionutils.Console의 readLine()을 활용한다. | ||
- [x] indent(인덴트, 들여쓰기) depth를 3이 넘지 않도록 구현한다. 2까지만 허용한다. | ||
예를 들어 while문 안에 if문이 있으면 들여쓰기는 2이다. | ||
힌트: indent(인덴트, 들여쓰기) depth를 줄이는 좋은 방법은 함수(또는 메서드)를 분리하면 된다. | ||
- [x] 3항 연산자를 쓰지 않는다. | ||
- [ ] 함수(또는 메서드)가 한 가지 일만 하도록 최대한 작게 만들어라. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
package racingcar.domain; | ||
|
||
|
||
import static racingcar.domain.CarRule.*; | ||
import static racingcar.message.ErrorMessage.CAR_NAME_ERROR; | ||
|
||
public class Car { | ||
private final String name; | ||
private int location; | ||
|
||
public Car(String name) { | ||
validateName(name); | ||
this.name = name; | ||
this.location = CAR_LOCATION_INIT.getNumber(); | ||
} | ||
|
||
public void move(int number) { | ||
if (number >= CAR_MOVE_MINIMUM.getNumber()) { | ||
this.location += CAR_LOCATION_STEP.getNumber(); | ||
} | ||
} | ||
|
||
private void validateName(String newCarName) { | ||
if (!(newCarName.length() <= CAR_NAME_MIN_SIZE.getNumber())) { | ||
throw new IllegalArgumentException(CAR_NAME_ERROR.getMessage()); | ||
} | ||
} | ||
|
||
public String getName() { | ||
return name; | ||
} | ||
|
||
public int getLocation() { | ||
return location; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
package racingcar.domain; | ||
|
||
public enum CarRule { | ||
CAR_NAME_MIN_SIZE(5), | ||
CAR_LOCATION_INIT(0), | ||
CAR_LOCATION_STEP(1), | ||
CAR_MOVE_MINIMUM(4); | ||
|
||
private final int number; | ||
|
||
CarRule(int number) { | ||
this.number = number; | ||
} | ||
|
||
public int getNumber() { | ||
return number; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
package racingcar.domain; | ||
|
||
import racingcar.util.NumberGenerator; | ||
|
||
import java.util.ArrayList; | ||
import java.util.List; | ||
|
||
import static racingcar.domain.CarRule.CAR_LOCATION_INIT; | ||
|
||
public class Judge { | ||
|
||
public List<Car> getWinners(List<Car> racingCars) { | ||
List<Car> winners = new ArrayList<>(); | ||
int maxLocation = findMaxLocation(racingCars); | ||
for (Car thisCar : racingCars) { | ||
if (thisCar.getLocation() == maxLocation) { | ||
winners.add(thisCar); | ||
} | ||
} | ||
return winners; | ||
} | ||
Comment on lines
+12
to
+21
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. getLocation을 통해 maxLocation과 비교하는 것은 3주차 미션 피드백에 맞지 않아보여요! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 좋은 포스팅까지 주시다니 감사합니다 ! 꼭 포스팅내용 공부한 후에 리펙토링 해볼게요 감사합니다 |
||
|
||
private int findMaxLocation(List<Car> racingCars) { | ||
int maxLocation = CAR_LOCATION_INIT.getNumber(); | ||
NumberGenerator ng = new NumberGenerator(); | ||
for (Car thisCar : racingCars) { | ||
maxLocation = ng.findMaxNumber(maxLocation, thisCar.getLocation()); | ||
} | ||
return maxLocation; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
package racingcar.domain; | ||
|
||
import racingcar.util.NumberGenerator; | ||
|
||
import java.util.ArrayList; | ||
import java.util.List; | ||
|
||
public class RacingCars { | ||
private List<Car> cars; | ||
public RacingCars(List<String> newCars) { | ||
this.cars = new ArrayList<>(); | ||
setRacingCars(newCars); | ||
} | ||
|
||
private void setRacingCars(List<String> newCars) { | ||
for (String carName : newCars) { | ||
this.cars.add(new Car(carName)); | ||
} | ||
} | ||
|
||
public void racingStepOne() { | ||
NumberGenerator ng = new NumberGenerator(); | ||
for (Car thisCar : this.cars) { | ||
thisCar.move(ng.getRandomNumber()); | ||
} | ||
} | ||
|
||
public List<Car> getCars() { | ||
return this.cars; | ||
} | ||
Comment on lines
+28
to
+30
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이 getCars는 객체의 필드의 cars의 참조를 그대로 반환합니다. 이는 클래스 외부에서 cars 리스트를 수정할 수 있어 무결성을 위협할 수 있어 캡슐화에 위반 될 수 있습니다. |
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
package racingcar.message; | ||
|
||
public enum ErrorMessage { | ||
CAR_NAME_ERROR("[ERROR] 잘못된 이름입니다."), | ||
INCLUDE_NULL_ERROR("[ERROR] 입력에 공백이 있습니다."), | ||
NOT_NUMBERS_ERROR("[ERROR] 잘못된 숫자입니다."); | ||
private final String message; | ||
|
||
ErrorMessage(String message) { | ||
this.message = message; | ||
} | ||
|
||
public String getMessage() { | ||
return message; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
package racingcar.message; | ||
|
||
public enum GuideMessage { | ||
INPUT_NAME_GUIDE("경주할 자동차 이름을 입력하세요.(이름은 쉼표(,) 기준으로 구분)"), | ||
INPUT_RACE_NUM_GUIDE("시도할 회수는 몇회인가요?"); | ||
|
||
private final String message; | ||
|
||
GuideMessage(String message) { | ||
this.message = message; | ||
} | ||
|
||
public String getMessage() { | ||
return message; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
package racingcar.message; | ||
|
||
public enum OutputFormat { | ||
RACING_NOW_FORMAT("%s : %s%n"), | ||
RACING_WINNER_FORMAT("최종 우승자 : %s%n"), | ||
CAR_NAME_STEP_FORMAT(", "), | ||
LOCATION_STEP_FORMAT("-"); | ||
private final String format; | ||
|
||
OutputFormat(String format) { | ||
this.format = format; | ||
} | ||
|
||
public String getFormat() { | ||
return format; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,52 @@ | ||
package racingcar; | ||
|
||
import racingcar.domain.Judge; | ||
import racingcar.domain.RacingCars; | ||
import racingcar.view.InputView; | ||
import racingcar.view.OutputView; | ||
|
||
|
||
public class racingCarController { | ||
private final InputView inputView; | ||
private final OutputView outputView; | ||
private RacingCars racingCars; | ||
|
||
public racingCarController() { | ||
this.inputView = new InputView(); | ||
this.outputView = new OutputView(); | ||
} | ||
|
||
public void mainRun() { | ||
//차량이름 입력받아 준비 | ||
createCar(); | ||
|
||
//레이싱 반복 횟수 입력받아 반복실행 | ||
runRacing(); | ||
|
||
//결과 출력 | ||
result(); | ||
|
||
} | ||
|
||
private void createCar() { | ||
this.racingCars = new RacingCars(inputView.inputCarName()); | ||
} | ||
|
||
private void runRacing() { | ||
int racingNum = inputView.inputRacingNum(); | ||
racingLoop(racingNum); | ||
} | ||
|
||
private void racingLoop(int number) { | ||
for (int i = 0; i < number; i++) { | ||
racingCars.racingStepOne(); | ||
outputView.racingNow(racingCars); | ||
} | ||
} | ||
|
||
private void result() { | ||
Judge judge = new Judge(); | ||
outputView.printWinners(judge.getWinners(racingCars.getCars())); | ||
} | ||
|
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
package racingcar.util; | ||
|
||
import camp.nextstep.edu.missionutils.Randoms; | ||
|
||
public class NumberGenerator { | ||
public static final int MIN_NUMBER = 1; | ||
public static final int MAX_NUMBER = 9; | ||
public int getRandomNumber () { | ||
return Randoms.pickNumberInRange(MIN_NUMBER,MAX_NUMBER); | ||
} | ||
|
||
public int findMaxNumber (int number1, int number2) { | ||
if(number1>number2) { | ||
return number1; | ||
} | ||
return number2; | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
package racingcar.view; | ||
|
||
import java.util.List; | ||
|
||
public class InputValidator { | ||
|
||
public boolean nameSplitValid(List<String> names) { | ||
//전체가 빈 경우 | ||
if (names.isEmpty()) { | ||
return false; | ||
} | ||
//split 된 이름이 빈경우 | ||
for (String name : names) { | ||
if (name.isEmpty()) { | ||
return false; | ||
} | ||
} | ||
return true; | ||
} | ||
|
||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
package racingcar.view; | ||
|
||
import camp.nextstep.edu.missionutils.Console; | ||
import racingcar.message.OutputFormat; | ||
|
||
import java.util.Arrays; | ||
import java.util.List; | ||
|
||
import static racingcar.message.ErrorMessage.NOT_NUMBERS_ERROR; | ||
import static racingcar.message.ErrorMessage.INCLUDE_NULL_ERROR; | ||
import static racingcar.message.GuideMessage.INPUT_NAME_GUIDE; | ||
import static racingcar.message.GuideMessage.INPUT_RACE_NUM_GUIDE; | ||
|
||
public class InputView { | ||
private final InputValidator inputValidator; | ||
private static final String NAME_SPLIT_FORMAT = ","; | ||
|
||
public InputView() { | ||
this.inputValidator = new InputValidator(); | ||
} | ||
|
||
public List<String> inputCarName() { | ||
System.out.println(INPUT_NAME_GUIDE.getMessage()); | ||
List<String> cars = Arrays.asList(Console.readLine().trim().split(NAME_SPLIT_FORMAT)); | ||
if (!inputValidator.nameSplitValid(cars)) { | ||
System.out.println(INCLUDE_NULL_ERROR.getMessage()); | ||
throw new IllegalArgumentException(); | ||
} | ||
return cars; | ||
} | ||
|
||
public int inputRacingNum() { | ||
System.out.println(INPUT_RACE_NUM_GUIDE.getMessage()); | ||
try { | ||
return Integer.parseInt(Console.readLine().trim()); | ||
} catch (NumberFormatException e) { | ||
System.out.println(NOT_NUMBERS_ERROR.getMessage()); | ||
throw new IllegalArgumentException(); | ||
} | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,43 @@ | ||
package racingcar.view; | ||
|
||
import racingcar.domain.Car; | ||
import racingcar.domain.RacingCars; | ||
|
||
import java.util.Iterator; | ||
import java.util.List; | ||
|
||
import static racingcar.message.OutputFormat.*; | ||
|
||
public class OutputView { | ||
public void racingNow(RacingCars racingCars) { | ||
for (Car thisCar : racingCars.getCars()) { | ||
System.out.printf(RACING_NOW_FORMAT.getFormat(), thisCar.getName(), makeLocationPrinter(thisCar.getLocation())); | ||
} | ||
System.out.println(); | ||
} | ||
|
||
public void printWinners(List<Car> cars) { | ||
System.out.printf(RACING_WINNER_FORMAT.getFormat(), makeWinnerPrinter(cars)); | ||
} | ||
|
||
private String makeLocationPrinter(int location) { | ||
StringBuilder locationPrint = new StringBuilder(); | ||
for (int i = 0; i < location; i++) { | ||
locationPrint.append(LOCATION_STEP_FORMAT.getFormat()); | ||
} | ||
return locationPrint.toString(); | ||
} | ||
|
||
private String makeWinnerPrinter(List<Car> cars) { | ||
StringBuilder winnerPrinter = new StringBuilder(); | ||
Iterator<Car> Cars = cars.iterator(); | ||
while (Cars.hasNext()) { | ||
winnerPrinter.append(Cars.next().getName()); | ||
if (Cars.hasNext()) { | ||
winnerPrinter.append(CAR_NAME_STEP_FORMAT.getFormat()); | ||
} | ||
} | ||
return winnerPrinter.toString(); | ||
} | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
차 이름이 중복일 경우에 대한 예외처리 코드가 따로 보이지 않네요!
혹시 이 부분에 대해선 어떻게 생각하시나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그렇네요! 포인트 감사합니다.
컨트롤러 단에서 일괄 예외처리를 하려 했는데 누락했네요;;
제 코딩시 습관이나 루틴 때문에 누락된 원인이 있는지 검토해봐야 겠어요 ~