Step5-3 : TUNA & YM 리뷰 내용 반영 (#136)

* fix : resolve conflict

* fix : 리뷰 내용 추가 REAMD.md

* Rename : lowerCase to uppercase

* Rename : cmd to command

* Refactor : 배열을 변수에 할당 후 네이밍

* Refactor : remove result variable

* Refactor : object to destructuring

* Fix : else if to else

* Refactor : modified try block

* feat : create getLogObj method

* Refactor : class to Object

* Fix : this.utils to Utils object

* Fix : remove duplicated method

* fix : fix the indent

Author: dev-dongwon

README.md

@@ -1,2 +1,73 @@
-# todo
-레벨2
+## STEP5-3 : TUNA & YM 리뷰 내용 반영  
+
+#
+#### 1. comment : 전체코드의 들여쓰기가 너무 깊은데요? 좋은 indent 가 무엇인지 한번찾아볼래요?
+
+- 전체 코드 indent 수정 : tab 인덴트를 space * 4 로 수정
+- 원인 : pair를 위해 구름 ide에서 작업 중, local으로 옮기면서 indent 오반영
+
+원래 ide의 설정은 tab이 space * 4 의 indent로 설정되어 있습니다.
+앞으로 들여쓰기가 올바르게 되어있는지, 불편하지 않은지 신경쓰도록 하겠습니다  
+
+#
+
+#### 2. comment : 주석이 필요한.. arguments.length 로 구별하는 방식을 어떻게 개선할 수 있을까요?
+
+- 리뷰 이전  
+```javascript
+if (argument.length === 3)  // updaste
+...
+} else if (arguments.length === 2) {  // add, delete
+```
+
+- 리뷰 이후 : 의미가 불분명해 주석이 필요했던 if statement 수정  
+
+```javascript
+if (command === 'update') { 
+  ...	
+} else if (command === 'add' || command === 'delete') {  
+  ...
+}
+```  
+
+#
+  
+#### 3. utils 클래스의 deepCopy 메서드의 이해
+
+- 원래 object.assign으로 얕은 복사를 썼으나, 예상한 방식대로 동작하지 않아 깊은 복사를 지원하는 deepCopy 유틸을 만들었습니다
+- 메서드는 검색을 통해 참고했으나, 참조 레퍼런스의 순수한 값만 복사하는 deep copy의 작동 방식은 이해했습니다  
+
+#
+
+#### 4. comment : 시작하는 것은 굳이 class로 만들지 않아도 될거 같아요. 사용일 될 일이 정말 없을 거 같은 부분이기도 하니까요.
+
+```javascript
+const RunTodoApp = class {
+	constructor() {
+		this.cmdArr = ['show','delete','update','add','undo','redo'];
+		this.commandParser = new CommandParser();
+```
+
+- 프로그램 실행을 담당하는 app.js의 class를 제거한 후, 리팩토링했습니다.  
+
+# 
+
+#### 5. comment : promise패턴을 사용했는데요. then메서드를 사용했을때의 장점은 무엇이라고 생각되세요?
+
+```javascript
+ this.utils.delay(0)
+ .then(() => {if (cmdList[0] === 'update') return this.utils.delay(3500);})
+```
+
+- then메서드를 사용했을 경우, 새로운 프로미스 객체가 반환되어 여러 개의 Promise를 체이닝할 수 있습니다.
+- 따라서, 여러 개의 비동기 작업을 수행할 경우, 보다 쉽게 비동기를 제어할 수 있다는 장점이 가장 큰 장점 같습니다.
+- error 처리 또한 catch로 손쉽게 할 수 있다는 점도 장점입니다.
+
+#
+
+#### 6. comment : 이부분은 객체리터럴 vs. 클래스! 중 어떤 것이 더 어울릴까요?
+```javascript
+const CustomException = class {
+```
+- 에러 처리는 변하는 값이 아닌 특수한 상황에서만 사용되는 고정된 값입니다. 따라서 동적인 인스턴스를 지원하는 class를 사요할 필요 없이
+속도가 빠르고 메모리 자원을 낭비하지 않는 리터럴이 적합하다 생각합니다.

class/README.md

@@ -0,0 +1,73 @@
+## STEP5-3 : TUNA & YM 리뷰 내용 반영  
+
+#
+#### 1. comment : 전체코드의 들여쓰기가 너무 깊은데요? 좋은 indent 가 무엇인지 한번찾아볼래요?
+
+- 전체 코드 indent 수정 : tab 인덴트를 space * 4 로 수정
+- 원인 : pair를 위해 구름 ide에서 작업 중, local으로 옮기면서 indent 오반영
+
+원래 ide의 설정은 tab이 space * 4 의 indent로 설정되어 있습니다.
+앞으로 들여쓰기가 올바르게 되어있는지, 불편하지 않은지 신경쓰도록 하겠습니다  
+
+#
+
+#### 2. comment : 주석이 필요한.. arguments.length 로 구별하는 방식을 어떻게 개선할 수 있을까요?
+
+- 리뷰 이전  
+```javascript
+if (argument.length === 3)  // updaste
+...
+} else if (arguments.length === 2) {  // add, delete
+```
+
+- 리뷰 이후 : 의미가 불분명해 주석이 필요했던 if statement 수정  
+
+```javascript
+if (command === 'update') { 
+  ...	
+} else if (command === 'add' || command === 'delete') {  
+  ...
+}
+```  
+
+#
+  
+#### 3. utils 클래스의 deepCopy 메서드의 이해
+
+- 원래 object.assign으로 얕은 복사를 썼으나, 예상한 방식대로 동작하지 않아 깊은 복사를 지원하는 deepCopy 유틸을 만들었습니다
+- 메서드는 검색을 통해 참고했으나, 참조 레퍼런스의 순수한 값만 복사하는 deep copy의 작동 방식은 이해했습니다  
+
+#
+
+#### 4. comment : 시작하는 것은 굳이 class로 만들지 않아도 될거 같아요. 사용일 될 일이 정말 없을 거 같은 부분이기도 하니까요.
+
+```javascript
+const RunTodoApp = class {
+	constructor() {
+		this.cmdArr = ['show','delete','update','add','undo','redo'];
+		this.commandParser = new CommandParser();
+```
+
+- 프로그램 실행을 담당하는 app.js의 class를 제거한 후, 리팩토링했습니다.  
+
+# 
+
+#### 5. comment : promise패턴을 사용했는데요. then메서드를 사용했을때의 장점은 무엇이라고 생각되세요?
+
+```javascript
+ this.utils.delay(0)
+ .then(() => {if (cmdList[0] === 'update') return this.utils.delay(3500);})
+```
+
+- then메서드를 사용했을 경우, 새로운 프로미스 객체가 반환되어 여러 개의 Promise를 체이닝할 수 있습니다.
+- 따라서, 여러 개의 비동기 작업을 수행할 경우, 보다 쉽게 비동기를 제어할 수 있다는 장점이 가장 큰 장점 같습니다.
+- error 처리 또한 catch로 손쉽게 할 수 있다는 점도 장점입니다.
+
+#
+
+#### 6. comment : 이부분은 객체리터럴 vs. 클래스! 중 어떤 것이 더 어울릴까요?
+```javascript
+const CustomException = class {
+```
+- 에러 처리는 변하는 값이 아닌 특수한 상황에서만 사용되는 고정된 값입니다. 따라서 동적인 인스턴스를 지원하는 class를 사요할 필요 없이
+속도가 빠르고 메모리 자원을 낭비하지 않는 리터럴이 적합하다 생각합니다.

class/app.js

@@ -0,0 +1,43 @@
+const readline = require('readline').createInterface( {
+	input:process.stdin,
+	output:process.stdout,
+});
+
+const CommandParser = require('./commandParser.js');
+const Utils = require('./utils.js');
+const Instruction = require('./instruction.js');
+const CustomException = require('./customException.js');
+
+const commandArr = ['show','delete','update','add','undo','redo'];
+const commandParser = new CommandParser();
+const instruction = new Instruction();
+
+const run = (() => {
+    readline.setPrompt('명령하세요: ');
+    readline.prompt();
+    readline.on('line', (userInput) => {
+    
+    try {
+    CustomException.missingSeperatorException(userInput);
+	const commandList = commandParser.getCommandList(userInput);
+
+	CustomException.CommandMissingException(commandList[0], commandArr);
+	commandParser.executeCommand(commandList);
+
+	Utils.delay(0)
+	.then(() => {if (commandList[0] === 'update') return Utils.delay(3500);})
+	.then(() => {return Utils.delay(1000);})
+	.then(() => {if (commandList[0] !== 'show') instruction.show('all');})
+	.catch(function(e) {console.log(e);})
+	.then(() => {readline.prompt();});
+
+    } catch(e) {
+	console.error(e.message);
+	readline.prompt();
+    }
+
+}).on('close', () => {
+   console.log("프로그램을 종료합니다.");
+   process.exit();
+   });
+})();

class/commandParser.js

@@ -0,0 +1,42 @@
+const Instruction = require('./instruction.js');
+const CustomException = require('./customException.js');
+
+const CommandParser = class {
+    
+    constructor() {
+	    this.instruction = new Instruction();
+    }
+	
+    getCommandList(input) {
+        const regexp = /[^\$]+|undo|redo/g;
+        return input.match(regexp);
+    }
+	
+    executeCommand(command) {	    
+	try {
+
+		let [primaryCommand, firstSubCommand, secondSubCommand] = [command[0], command[1], command[2]];
+		
+		if (command.length === 1) {
+			this.instruction[primaryCommand]();
+		} else if (command.length === 2) {
+			this.instruction[primaryCommand](firstSubCommand);
+		} else if (command.length === 3) {
+			this.instruction[primaryCommand](firstSubCommand, secondSubCommand);
+		} else {
+			CustomException.CommandMissingException();
+		}
+			
+	} catch (e) {
+	    console.error(e.message);
+	    return;
+        }
+    }
+
+    isValidCommand(command, arr) {
+        if (arr.includes(command)) return true;
+        return false;
+    }
+};
+
+module.exports = CommandParser;

class/customException.js

@@ -0,0 +1,33 @@
+const CustomException  = {
+    missingSeperatorException : function(input) {
+        if (!this.isValidSeperator(input)) throw new Error("구분자 $가 존재하지 않습니다");
+    },
+    
+    notExistIdException : function() {
+        throw new Error(`찾으시는 id가 존재하지 않습니다`);
+    },
+    
+    sameStatusException : function() {
+        throw new Error(`같은 상태로 업데이트 할 수 없습니다`);
+    },
+    
+    CommandMissingException : function(command, arr) {
+        if (!this.isValidCommand(command, arr))
+        throw new Error(`올바른 명령어가 아닙니다`);
+    },
+    
+    isValidSeperator : function(input) {
+        let result = true;
+        const regexp = /\$|undo|redo/g;
+        if (input.match(regexp) == null) result = false;
+        
+        return result;
+    },
+    
+    isValidCommand : function(command, arr) {
+        if (arr.includes(command)) return true;
+        return false;
+    }
+};
+
+module.exports = CustomException;