[자동차 경주] 오정진 미션 제출합니다. #747

ojj1123 · 2023-11-01T14:47:48Z

구현할 기능 목록

경주할 자동차 이름을 입력받는다. 입력받은 자동차 이름 각각에 대해 Car 인스턴스를 생성한다.
시도할 게임 횟수를 입력받는다.
경주할 자동차에 대해 각각 0 ~ 9사이의 정수 중 무작위 값을 구한 후 4이상인 경우 이동한다.
경주 게임 우승자는 이동 경로가 가장 긴 자동차이다. 게임 횟수가 0이 됐을 때(게임 완료) 우승한 자동차 이름을 쉼표(,)로 구분해 출력한다.
사용자 입력을 검증한다.
- 경주할 자동차 이름에 대한 검증
- 시도할 횟수에 대한 검증
- 검증 완료 후 에러 발생 시 [ERROR] 로 시작하는 에러 문구와 함께 예외를 throw하고 애플리케이션을 종료한다.

테스트 코드 목록

사용자 입력 검증 기능 테스트 코드

- Game 객체의 #count 변수 감소 - #count가 0이 되면 게임 종료

Car 클래스에 3개의 메서드 추가 - move(): 0 ~ 9사이의 정수 중 무작위 값을 골라 4 이상이면 #path에 '-' 문자 추가 - getName(): 자동차 이름 반환 - getPath(): 자동자가 이동한 path 반환

for문으로 변경한 이유 - 게임 횟수 값 그대로 유지(#count) - jest에서 while 문 사용시 memory leak 발생

- '모든 자동차가 이동한 후 게임 횟수를 1 감소시킨다' 기능 제거

사용자 입력 검증 함수 - 자동차 이름 입력 검증(쉼표 구분, 자동차 이름 5글자 이상) - 유효한 게임 횟수 입력 검증(0 이상의 값)

teawon

�코드가 깔끔해서 많이 배우고갑니다!!

src/Car.js

teawon · 2023-11-01T18:03:13Z

src/Game.js

+  async setCount() {
+    const input = new Input();
+
+    const value = await input.readLine('시도할 횟수는 몇 회인가요?\n', {


Input을 별도로 분리하고 예외처리를 적용한 로직은 너무 깔끔하지만 하나가 아니라 확인해야하는 예외케이스가 있을 수 있으니,
배열의 형태로 props를 받도록 수정하거나

아니면 "자동차 이름 Validator" 검증함수를 만들고 그 안에서, Validator.validCarList를 호출하도록 수정하면 새로운 검증로직을 추가할 때 더 용이하게 확장할 수 있을 것 같아요 어떻게 생각하시나요?

배열의 형태로 props를 받도록 수정하거나

저도 검증 로직이 더 많아지면 어떻게 처리하면 좋을까 고민하다가 validator 옵션으로 배열을 넘겨주는 걸 생각해봤어요!
Like this:

// use case const value = await input.readLine( '경주할 자동차 이름을 입력하세요.(이름은 쉼표(,) 기준으로 구분)\n', { validators: [Validator.validCarList, Validator.otherValidator, Validator.anotherValidator] } ); // Input.js class Input { async readLine(query, { validators }) { try { const input = await Console.readLine(query); if(Array.isArray(validators)) { validators.forEach(validator => validator(input)); } return input; } catch(error) { throw error; } } }

이런식으로 validators 를 배열로 받는걸 생각해봤습니다 ~

@teawon

아니면 "자동차 이름 Validator" 검증함수를 만들고 그 안에서, Validator.validCarList를 호출하도록 수정하면 새로운 검증로직을 추가할 때 더 용이하게 확장할 수 있을 것 같아요 어떻게 생각하시나요?

"자동차 이름 Validator" 검증 함수를 만들자는 말이 어떤 의미인지 수도코드로 작성해주실 수 있어요?
제가 이해를 잘 못했어요 ㅜㅜ

더 많아지면 어떻게 처리하면 좋을까 고민하다가 validator 옵션으로

static validateUserNumbersInput(userResponse) { ValidatorUtil.validateLength(userResponse, ANSWER_LENGTH); ValidatorUtil.validateNotDuplicate(userResponse); ValidatorUtil.validateIsNumber(userResponse); }

1주차에 작성했던 코드 방식인데,

"자동차 이름을 입력받을 때 수행하는 검증함수" 라는 별도의 함수를만들고,
해당 함수 내부에서 여러가지 Validation을 처리하는방식을 말씀드리고 싶었어요~~

src/Game.js

kimsuyeon0916 · 2023-11-02T02:12:34Z

src/Game.js

+    for (let i = 0; i < this.#count; i++) {
+      this.#cars.forEach((car) => {
+        car.move();
+        Console.print(`${car.getName()} : ${car.getPath()}`);
+      });
+      Console.print('');
+    }


이부분도 따로 함수화 했으면 좀 더 직관적으로 로직이 보일 것 같아요!

kimsuyeon0916 · 2023-11-02T02:12:57Z

src/Game.js

+    const input = new Input();
+
+    const value = await input.readLine(
+      '경주할 자동차 이름을 입력하세요.(이름은 쉼표(,) 기준으로 구분)\n',


상수 파일을 따로 빼도 좋을 듯 합니다!

kimsuyeon0916 · 2023-11-02T02:14:28Z

src/Input.js

@@ -0,0 +1,19 @@
+import { Console } from '@woowacourse/mission-utils';
+
+class Input {


따로 Input 부분을 빼서 간결하게 예외 처리한거 좋네요!! 저도 다음에 차용해봐야겠습니다!☺️

와 입력값을 따로 검증할 생각을 못했네요 ..! 👍🏻

사용자 입력 검증은 input 과 따로 분리해 input 외부에서 검증 로직을 주입할 수 있도록 설계 했습니다 :) 아무래도 사용자 입력 검증은 복잡해질 수 있어 관심사 분리를 한거죠.
Input 관심사 -> 사용자 입력
Validator 관심사 -> 입력 검증

react-hook-form 이라는 라이브러리에서 그 아이디어를 좀 따왔습니다

jinsoul75 · 2023-11-02T04:20:22Z

src/Input.js

@@ -0,0 +1,19 @@
+import { Console } from '@woowacourse/mission-utils';
+
+class Input {


와 입력값을 따로 검증할 생각을 못했네요 ..! 👍🏻

jinsoul75 · 2023-11-02T04:23:27Z

src/Input.js

+    try {
+      const input = await Console.readLineAsync(query);
+
+      if (typeof validator === 'function') {


매개변수로 받은 값의 타입을 체크하는 이유가 궁금합니다!

validator에 함수 말고 다른 값이 들어갈 수 있기 때문입니다
함수인 경우만 validator(input)으로 함수를 호출할 수 있고 그외의 값이 들어오면 무시를 하겠죠?

그리고 나중에 검증 로직이 복잡해지면 함수 뿐만 아니라 다른 값이 들어갈 수도 있어서 저렇게 해보았습니다! 만약 타입스크립트를 쓴다면 타입 좁히기(type narrow) 효과도 볼 수 있어요!

아래 코드처럼 validator 에 함수 뿐만 아니라 다른 검증 로직이 들어갈 수 있도록 설계한 것이고 추후에 Input을 아래와 같이 수정해볼 수 있습니다

// use case // 여러개의 검증 함수를 배열로 주입 const complexValue = await input.readLine('복잡한 값\n', { validator: [Validator.valid1, Validator.valid2, Validator.valid3], }); // 함수를 통해 검증 로직 주입 const number = await input.readLine('시도할 횟수는 몇 회인가요?\n', { validator: Validator.rangeOverZero, }); // 객체를 통해 검증 로직 주입 const email = await input.readLine('이메일\n', { validator: { pattern: 'email regex' }, }); // Input.ts class Input { async readLine(query, { validator }) { try { const input = await Console.readLineAsync(query); // 타입 좁히기 효과 => 개발 생산성 올라감 if (Array.isArray(validator)) { validator.forEach((valid) => valid(input)); } else if (typeof validator === 'function') { validator(input); } else { // another validator logic } return input; } catch (error) { throw error; } } }

Input을 분리하고 매개변수로 validator를 받아오는 구조가 좋네요! 혹시 여기서 validator가 function이 아니면 그냥 input을 return하는 것 같은데 의도하신 부분일까요?

today-is-first

많이 배우고 갑니다!

today-is-first · 2023-11-02T05:41:16Z

src/utils/Validator.js

+  if (Number(input) < 0) {
+    throw new Error('[ERROR] 음이 아닌 정수를 입력해주세요');
+  }


Q. 저는 게임 시도 횟수가 0일 경우도 게임 작동이 불가능하지 않을까 생각했었는데 어떻게 생각하시는지 궁금합니다!

이거는 요구사항을 어떻게 분석했느냐에 따라 다를거 같아요
저는 0을 입력한 경우 게임을 하나도 진행하지 않는 것으로 이해했습니다.(즉, 횟수가 0이니까요)
대신 음수인 경우는 횟수로 보지 않기 때문에 예외 처리를 해두었습니다

puranium235

전체적으로 깔끔하게 잘 만드신 것 같습니다! 특시 Input을 분리해서 validator를 매개변수로 받는 부분이 기억에 남네요. 고생하셨습니다! 😊

puranium235 · 2023-11-02T05:58:30Z

src/utils/Validator.js

+function rangeOverZero(input) {
+  if (!Number.isSafeInteger(Number(input))) {
+    throw new Error('[ERROR] 올바른 정수를 입력해주세요');
+  }


isSafeInteger라는 함수가 있군요! JavaScript 내장 함수를 잘 사용하신 것 같습니다. 다만 rangeOverZero라는 이름은 0 이상인지만 확인해줄 것 같고 정수인지는 확인해줄 것 같지 않다는 느낌이 듭니다. 그리고 현재는 빈 입력이 들어올 때도 0이 들어왔을 때와 같이 작동하는 것 같습니다. 게임 횟수의 상한도 정하면 어떨까요?

와 isSafeIntegar 이라는 게 있군요！ 덕분에 하나 또 알아갑니다！ ㅎㅎ

puranium235 · 2023-11-02T06:02:32Z

src/utils/Validator.js

+function validCarList(input) {
+  const list = input.split(',');
+  if (list.every((item) => validLength(item, { maxLength: 5 }))) {
+    return;
+  }
+  throw new Error('[ERROR] 자동차 이름을 1글자 이상 5글자 이하로 작성해주세요');
+}


이름을 받을 때 앞뒤 공백을 자르면 좋을 것 같습니다. 저같은 경우에는 입력할 때 습관적으로 쉼표 뒤에 공백을 넣는데, 그러면 이름이 5글자인 경우에도 에러가 발생합니다. 입력으로 공백만 들어온 경우나 중복된 이름이 있는 경우 등도 추가하면 좋을 것 같습니다.

puranium235 · 2023-11-02T06:03:57Z

src/Input.js

+    try {
+      const input = await Console.readLineAsync(query);
+
+      if (typeof validator === 'function') {


Input을 분리하고 매개변수로 validator를 받아오는 구조가 좋네요! 혹시 여기서 validator가 function이 아니면 그냥 input을 return하는 것 같은데 의도하신 부분일까요?

puranium235 · 2023-11-02T06:05:34Z

__tests__/ValidatorTest.js

+import Validator from '../src/utils/Validator';
+
+describe('Validator 함수 테스트', () => {
+  test('5글자 초과 시 예외 처리', () => {
+    const input = 'abcdefghi,abc';
+
+    expect(() => Validator.validCarList(input)).toThrow('[ERROR]');
+  });
+
+  test('음의 정수 입력 시 예외 처리', () => {
+    const input = '-1';
+
+    expect(() => Validator.rangeOverZero(input)).toThrow('[ERROR]');
+  });
+
+  test('정수 이외의 입력값에 대한 예외 처리', () => {
+    const input = 'abcd';
+
+    expect(() => Validator.rangeOverZero(input)).toThrow('[ERROR]');
+  });
+
+  test('5글자 초과 시 false 반환', () => {
+    const input = 'abcdef';
+
+    expect(Validator.validLength(input, { maxLength: 5 })).toBe(false);
+  });
+});


에러 처리 뿐만 아니라 바른 입력이 들어왔을 때 pass되는 것도 테스트에 추가하면 좋을 것 같습니다.

starcradle101

안녕하세요 정진님! 코드 리뷰 남기러 왔습니다 😆
아직 다른 분들의 코드를 읽는게 익숙하지 않아서 이해하는데만해도 시간이 걸리는데, 이미 다른 분들께서 좋은 리뷰를 많이 남겨 주신 것 같습니다. 뭔가 나름대로 인사이트를 드리고 싶었는데, 아직은 제 실력이 미숙해서 그렇지 못하는 것 같아 너무 아쉽네요 ㅠㅠ
2주차 미션도 수고 많으셨고 남은 2주동안도 화이팅입니다! 😆

starcradle101 · 2023-11-02T15:20:58Z

src/App.js

-  async play() {}
+  async play() {
+    const game = new Game();
+    await game.play();


App.play()와 명확하게 구분할 수 있도록 game.play보다는 조금 더 구체적인 이름을 사용하면 어땠을까라는 생각이 듭니다 😁

starcradle101 · 2023-11-02T15:23:23Z

src/Input.js

+
+      return input;
+    } catch (error) {
+      throw error;


저희 미션에서는 에러를 throw 하기만 하고 따로 처리를 해 주지 않고 있는데, 혹시 try-catch 구문을 사용하신 이유가 있으실까요?

ojj1123 added 12 commits November 1, 2023 19:24

docs: 구현할 기능 목록 및 테스트 코드 목록 작성

5354a73

docs: 구현할 기능 목록 수정

e2b9947

feat: 경주할 자동차 입력 받아 Car 인스턴스 생성 메서드 작성

1f9205b

feat(Game): 시도할 게임 횟수 입력 받는 기능

6b0c356

feat(Game): 남은 게임 횟수 감소

14a88f9

- Game 객체의 #count 변수 감소 - #count가 0이 되면 게임 종료

feat: 자동차 경주 기능

fa7572e

Car 클래스에 3개의 메서드 추가 - move(): 0 ~ 9사이의 정수 중 무작위 값을 골라 4 이상이면 #path에 '-' 문자 추가 - getName(): 자동차 이름 반환 - getPath(): 자동자가 이동한 path 반환

feat(Game): 경주 게임 우승자 출력 기능

f53a616

chore(Game): while문에서 for문으로 변경

34c3604

for문으로 변경한 이유 - 게임 횟수 값 그대로 유지(#count) - jest에서 while 문 사용시 memory leak 발생

docs: 구현할 기능 목록 수정

d8ceb39

- '모든 자동차가 이동한 후 게임 횟수를 1 감소시킨다' 기능 제거

feat: 사용자 입력 검증 함수 작성

ab517aa

사용자 입력 검증 함수 - 자동차 이름 입력 검증(쉼표 구분, 자동차 이름 5글자 이상) - 유효한 게임 횟수 입력 검증(0 이상의 값)

test(Validator): Validator 테스트 코드

8d81cd1

docs: 테스트 코드 목록 수정

ee1dd39

teawon reviewed Nov 1, 2023

View reviewed changes

wwwr-kim0en reviewed Nov 1, 2023

View reviewed changes

src/Game.js Show resolved Hide resolved

skiende74 reviewed Nov 1, 2023

View reviewed changes

src/Game.js Show resolved Hide resolved

kimsuyeon0916 reviewed Nov 2, 2023

View reviewed changes

jinsoul75 reviewed Nov 2, 2023

View reviewed changes

today-is-first reviewed Nov 2, 2023

View reviewed changes

puranium235 reviewed Nov 2, 2023

View reviewed changes

starcradle101 reviewed Nov 2, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[자동차 경주] 오정진 미션 제출합니다. #747

[자동차 경주] 오정진 미션 제출합니다. #747

ojj1123 commented Nov 1, 2023

teawon left a comment

teawon Nov 1, 2023

ojj1123 Nov 1, 2023 •

edited

Loading

ojj1123 Nov 1, 2023 •

edited

Loading

teawon Nov 2, 2023 •

edited

Loading

kimsuyeon0916 Nov 2, 2023

kimsuyeon0916 Nov 2, 2023

kimsuyeon0916 Nov 2, 2023

jinsoul75 Nov 2, 2023

ojj1123 Nov 2, 2023 •

edited

Loading

jinsoul75 Nov 2, 2023

jinsoul75 Nov 2, 2023

ojj1123 Nov 2, 2023 •

edited

Loading

puranium235 Nov 2, 2023

today-is-first left a comment

today-is-first Nov 2, 2023

ojj1123 Nov 2, 2023

puranium235 left a comment

puranium235 Nov 2, 2023

wwwr-kim0en Nov 2, 2023

puranium235 Nov 2, 2023

puranium235 Nov 2, 2023

puranium235 Nov 2, 2023

starcradle101 left a comment

starcradle101 Nov 2, 2023

starcradle101 Nov 2, 2023

		@@ -0,0 +1,19 @@
		import { Console } from '@woowacourse/mission-utils';

		class Input {

[자동차 경주] 오정진 미션 제출합니다. #747

Are you sure you want to change the base?

[자동차 경주] 오정진 미션 제출합니다. #747

Conversation

ojj1123 commented Nov 1, 2023

구현할 기능 목록

테스트 코드 목록

teawon left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ojj1123 Nov 1, 2023 • edited Loading

Choose a reason for hiding this comment

ojj1123 Nov 1, 2023 • edited Loading

Choose a reason for hiding this comment

teawon Nov 2, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ojj1123 Nov 2, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ojj1123 Nov 2, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

today-is-first left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

puranium235 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

starcradle101 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

ojj1123 Nov 1, 2023 •

edited

Loading

ojj1123 Nov 1, 2023 •

edited

Loading

teawon Nov 2, 2023 •

edited

Loading

ojj1123 Nov 2, 2023 •

edited

Loading

ojj1123 Nov 2, 2023 •

edited

Loading