[우아한테크코스] 프리코스 숫자야구 미션 회고

// Generator.jsclass Generator { constructor() { this.computerNumber = this.generateRandomNumber(); } generateNewCorrectNumber() { this.computerNumber = this.generateRandomNumber(); } generateRandomNumber() { // ... 생략 } export default Generator;

// BaseballGame.js async startGame() { await this.getUserInput(); } async getUserInput() { const input = await Console.readLineAsync(LOG_MESSAGE.INPUT_NUMBER); this.handleUserInputDuringGame(input); } // ... 생략 handleUserInputDuringGame(input) { if (!isValidGameInputDuringGame(input)) { throwError(ERROR_MESSAGE.INCORRECT_VALUE); }

// Node.js에서 예전부터 사용되고 있는 CommonJS의 키워드const express = require("express"); const app = express(); // ES6(ES2015)에서 새롭게 도입되어 현재 자바스크립트 생태계에서 표준이 되어가고 있는 키워드import express from "express"; const app = express();

// App.jsimport BaseballGame from './BaseballGame.js'; import { Console } from '@woowacourse/mission-utils'; import { LOG_MESSAGE } from './constants.js'; import { printMessage } from './utils.js' class App { constructor() { this.game = new BaseballGame(); } async play() { printMessage(LOG_MESSAGE.START_GAME); await this.game.startGame();// 수정된 게임 시작 메서드 호출 } } const app = new App(); app.play(); export default App;

// BaseballGame.js// ... 생략class BaseballGame { constructor() { this.computer = new Generator(); } async startGame() { await this.getUserInput(); }// 수정된 게임 시작 메서드async getUserInput() { const input = await Console.readLineAsync(LOG_MESSAGE.INPUT_NUMBER); this.handleUserInputDuringGame(input); }// 수정된 사용자 입력 메서드async recommendRestart() { await printMessage(LOG_MESSAGE.CORRECT_END); const input = await Console.readLineAsync(`${LOG_MESSAGE.RESTART_INPUT}\n`); this.handleUserInputEndGame(input); }// 수정된 게임 재시작 입력 메서드 handleUserInputDuringGame(input) { // ... 생략 } async handleUserInputEndGame(input) { const isValidGameInputEndGame = [GAME_SELECT.RESTART, GAME_SELECT.END]; // ... 생략 } restartGame() { this.computer.generateNewCorrectNumber(); this.startGame(); } } export default BaseballGame;

const convertNumberToString = (strikeNumber, ballNumber) => { let hintMessage = ''; if (ballNumber > 0) { hintMessage += `${ballNumber}${HINT_MESSAGE.BALL} `;// 공백 추가 } if (strikeNumber > 0) { hintMessage += `${strikeNumber}${HINT_MESSAGE.STRIKE}`; } if (hintMessage.length === 0) { hintMessage += HINT_MESSAGE.NOTING; } return hintMessage; };

"scripts": { "test": "jest --detectOpenHandles" }

const LOG_MESSAGE = Object.freeze({ START_GAME: '숫자 야구 게임을 시작합니다.', INPUT_NUMBER: '숫자를 입력해주세요 : ', CORRECT_END: '3개의 숫자를 모두 맞히셨습니다! 게임 종료', END_GAME: '게임 종료', RESTART_INPUT: '게임을 새로 시작하려면 1, 종료하려면 2를 입력하세요.', }); // ... 이하는 생략

// 수정 전 : includes & push 사용 generateRandomNumber() { const computerArray = []; while (computerArray.length < 3) { const newComputerNumber = Random.pickNumberInRange(1, 9); if (!computerArray.includes(newComputerNumber)) { computerArray.push(newComputerNumber); } } return computerArray.join(''); } // 수정 후 : set & add 사용 generateRandomNumber() { const generatedNumber = new Set(); while (generatedNumber.size < 3) { const newComputerNumber = generateNumberInRange(1, 9); generatedNumber.add(newComputerNumber); } return Array.from(generatedNumber).join(''); } }

// 1. 게임 진행중 입력 값이 유효한지 판단하는 메서드 이름 수정// 보통 boolean을 return하는 함수는 is ~ 이런 식으로 쓰기 때문 checkValidNumberDuringGame → isValidGameInput // 2. 입력 값의 유효성을 검사하는 파일 이름 수정// 동사도 좋지만 '역할'을 분명히 드러내면 더 좋다. checkValid.js → validator.js // 3. 컴퓨터가 랜덤한 숫자를 생성하는 클래스 이름 수정// 컴퓨터 라고 이름만 써놓는 것보다 컴퓨터가 하는 '역할'을 분명히 드러내면 더 좋다. Computer.js → Generator.js

// utils.js// 사용자에게 메시지를 출력하는 함수const printMessage = (message) => Console.print(message); // 컴퓨터가 랜덤한 숫자를 생성하는 함수const generateNumberInRange = (min, max) => Random.pickNumberInRange(min, max); // 에러를 출력하는 함수const throwError = (message, condition = true) => { if (condition) throw new Error(message); }; // ... 생략export { printMessage, generateNumberInRange, throwError };

// 수정 전 async handleUserInputEndGame(input) { const isValidGameInputEndGame = [GAME_SELECT.RESTART, GAME_SELECT.END]; if (!isValidGameInputEndGame.includes(input)) { throwError(ERROR_MESSAGE.INCORRECT_VALUE); } switch (input) {// switch문 사용case GAME_SELECT.RESTART: this.restartGame(); break; case GAME_SELECT.END: printMessage(LOG_MESSAGE.END_GAME); break; } } // 수정 후 async handleUserInputEndGame(input) { const isValidGameInputEndGame = [GAME_SELECT.RESTART, GAME_SELECT.END]; if (!isValidGameInputEndGame.includes(input)) { throwError(ERROR_MESSAGE.INCORRECT_VALUE); return;// early return } if (input === GAME_SELECT.RESTART) { this.restartGame(); return;// early return } if (input === GAME_SELECT.END) { printMessage(LOG_MESSAGE.END_GAME); } }