0414-Refactoring-and-Commit

Author Avatar
John Doe Apr 15, 2018

TIL

til 늦게라도 매일 꼭 하고 있는 것은 칭찬해주고 싶다.
커밋메시지 -> 나눠서 한 것
리뷰 고치면서 피드백 자체가 커밋 -> 안 나누면 안 받을 것이다라는 얘기를 들어서 확실히… 이번엔 의식하고 업데이트
생각 나는데로 하긴 했다. 이 참에 습관화 하자
근데 얼른 자자 …

반성

유튜브랑 웹툰… 조금만 보자 … =: 망했…

  • 토요일은 늦잠데이 잠 실컷 자야지 하고 맘 먹고 자면 … 반나절을 자버린다.
    조금 더 성실할 필요가 있다!!!

ArrayParser

내일까지는 다 해야지 목표

다형성으로 커버하려다가.. 오히려 꼬였다. 조그마한 함수로 특정한 함수가 많을 수록 더 보기도 좋고 직관적인 것 같다.
한 꺼번에 커버하려는 부분은 지금은 우선 적게 하는 방향으로 하자

for if if 였던 부분을 함수 하나에서 if 를 처리해주게 위에 빼고 집어 넣었는데 …
피드백 리뷰가 이렇게 처리 하는 것이 맞나? if if가 좀 빠지니까 좀 더 깔끔해 보이기는 하는데 뭔가 석연찮다 

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
const updateCloseState = (singleCharacter, arrayClose, objClose) => {
    updatedArrayClose = arrayClose
    updatedObjClose = objClose
    if(brackets[singleCharacter]){
        updatedArrayClose = brackets[singleCharacter](arrayClose)
        updatedObjClose = brackets[singleCharacter](objClose)
    }
    return {updatedArrayClose, updatedObjClose}
}

  _each(str, function(singleCharacter){
        if(isStringItemEnd(singleCharacter, arrayClose, objClose)){
            splitItemList.push(splitItem)
            splitItem = ''
        }
        else {
            const {updatedArrayClose, updatedObjClose} = updateCloseState(singleCharacter, arrayClose, objClose) 
            arrayClose = updatedArrayClose
            objClose = updatedObjClose
            splitItem += singleCharacter
        }
    })

Test

테스트에 toBe 왜 다른지를 찾았다… 알고보니 배열 값 비교여서 값을 비교할 수가 없었다.
그래서 ArrayToBe 라는 메소드를 하나 더 추가했다. 여기서 값을 비교할 떄 쓰는 eqaul은 message는
없도록 하게 다시 비슷한 작업을 해서 추가했는데 음…이거는 어쩔수 없나?

네이밍 Error 네이밍 너무 어렵다
중복 되는 부분이 많아서 저 함수에서 저 동작만 빼고 이런게 있을 것 같은데 없나? 찾아봐야겠다.

1
2
3
4
5
6
7
8
9
10
const equal = (a,b)=> {
    if(a!== b) throw Error(`: FAIL  targetValue is ${a}, expectValue is ${b}`)
    console.log(`: OK  target Value : ${a} expectedValue : ${b}`)
    return true;
}

const equalToNotMsg = (a,b)=>{
    if(a!== b) throw Error(`: FAIL  targetValue is ${a}, expectValue is ${b}`)
    return true;
}

알고리즘 …

알고리즘 1시간 꾸준히 한다는 약속은 이번주 어디로…

프로젝트 …

프로젝트 할 시간….