TIL 리팩토링

Shin Woohyun·2022년 11월 22일
0

🌐주요 리팩토링 참고

node-schedule로 1분마다 작동하는 함수를 app.js 최상위에서 활용중이었습니다.

콜백함수를 따로 빼내어서 작동 중이라는 의미를 극대화시키고 싶었습니다.

🍀함수 쪼개기

스케줄러 콜백함수에선 많은 일이 일어나고 있었습니다.

적어둔 주석을 보니 엄청 고민한 거 같습니다. 과거의 나…!

풀어져있는 코드는 해석하기 어려워서 의미별로 나누었습니다.

요약하면, 찾아서 열고 닫기 입니다.

🍀let → const

습관적으로 let으로 선언하고 있었습니다. 그러다 리팩토링 책을 읽고 다시 수정중에 있습니다.

let은 변수를 선언할 때 사용하고, const 는 상수를 선언할 때 사용합니다.

let 과 const 를 변수와 상수의 의미를 잘 나누어서 사용하면 코드를 보자마자 ‘아 이 친구는 변수구나’하고 알 수 있겠습니다.

리팩토링을 하면서 나쁜 습관도 깨닫게 됩니다.

🍀매개변수 순서 변경하기

재사용성에 대한 고민을 하다보니 리팩토링 완료한 함수 두 가지가 매우 비슷해보였습니다.

다른 하나는 매개변수 중 2가지가 필요 없는 함수였습니다. 순서를 바꾸어 두 가지를 함께 사용할 수 있게끔 정리하였습니다.

//변경 전
export function publishMessage(topic, command, error_code, detail, obj = {}){...}
//변경 후
export function publishMessage(topic, command, obj = {}, error_code, detail){...}

리팩토링을 하다 보니 코드에 대한 파악도가 더 높아져서 중요성을 더 깨닫게 됩니다.

0개의 댓글