step 2-2 feat: 출력함수구현, refactor: switch문 객체로 수정#88
step 2-2 feat: 출력함수구현, refactor: switch문 객체로 수정#88honux77 merged 2 commits intocode-squad:wo0kgodfrom
Conversation
app.js - 실행 파일 calc.js - 계산 관련 함수 check.js - validate 관련 함수
글쌔요, |
crongro
left a comment
There was a problem hiding this comment.
코드 깔끔합니다.
그런데 printExecution.. 어쩌구 함수 만드는 미션을 빼먹으셨어요.
그거랑 리뷰드린 거랑 한번 수정해보세요!
| } | ||
|
|
||
| rectancleArea = function(...param){ | ||
| const [width , height] = check.checkParam(2,param); |
calc.js
Outdated
| } | ||
|
|
||
| getArea = function(name,...param){ | ||
| switch(name){ |
There was a problem hiding this comment.
음 코드가 너무 좋은데요.
음 switch문이나 없애볼까요?
(힌트 객체활용)
|
다음엔, |
sequence 변수를 만들고 getArea에서 push를 하고 나중에 출력하도록 하였습니다. 이때 sequence 변수는 객체의 속성으로 정의하였습니다. switch문 대신에 polygonAreaFunction 객체를 만들고 getArea에서 입력받은 문장을 key값으로 , value는 함수를 연결하였습니다. 출력해주는 printExecutionSequence 함수에서 값의 변경을 용이하게 하기 위해 객체로 값을 넘겨주었습니다.
1.1 printExecutionSequence 함수 구현하기sequence 변수를 만들고 getArea에서 push를 하고 나중에 출력하도록 하였습니다. 이때 sequence 변수는 객체의 속성으로 정의하였습니다. 1.2 switch문 대신 객체로 변경하기switch문 대신에 polygonAreaFunction 객체를 만들고 getArea에서 입력받은 문장을 key값으로 , value는 함수를 연결하였습니다. 1.3 sequence에 객체로 인자를 넘김출력해주는 printExecutionSequence 함수에서 값의 변경을 용이하게 하기 위해 객체로 값을 넘겨주었습니다. 궁금한점
if(name === 'circle' && param.length === 2) {
answer = getRecursiveCircle(...param);
}
else {answer = polygonAreaFunction[name](...param);}polygonAreaFunction 안에서 처리할 수 있는 방법이 없을까요?
|
|
step 2-2 로 올렸어야했는데 처음 pr을 step 2-1 브랜치로 올렸네요. |
| } | ||
| } | ||
|
|
||
| module.exports = { |
app.js - 실행 파일
calc.js - 계산 관련 함수
check.js - validate 관련 함수
이렇게 따로 함수를 만드는게 좋을지, circleArea 함수안에 재귀적으로 넓이를 계산하는 함수를 만드는게 좋을지 여쭤보고싶습니다!