조건문 줄이기(feat. 클린 코드)

 

오늘 회사에서 코드 리뷰? 유지보수 코드 개선 방안 회의를 했는데 신입이라서 나도 참관했다!

 

그 중에 기억에 남는 부분은 조건문을 줄이는 방안에 관한 것이다.

개선해야 하는 부분은 매번 같은 로직을 타지만 케이스 별로 값만 다르게 바뀌는 상태여서

케이스별로 switch-case 또는 if 문으로 작성되어서 그 안에서 값만 바꿔주고 있었다.

 

코드가 이렇게 작성되어 있으면 혹여나 다른 케이스가 추가되는 경우 조건문도 계속 늘어나기 때문에 효율면에서 좋지가 않다.

해당 부분에서 switch-case 문을 없애는 방안에 대해 이야기 했는데,

나도 그와 비슷한 고민을 해본적은 있지만 항상 해결책을 찾지는 않고 끝났던 것 같다.

그래서 그 말을 들어도 바로 딱 해결 방안을 얘기할 수가 없었다 ...

switch-case 문 예시

 

결론은 같은 로직을 타지만 값만 바뀌는 경우이므로 const 로 해당 값들을 key:value 로 박아두고

키 별로 value 를 꺼내오면 된다는 거였다.

조건문 없앤 예시

 

이 생각을 왜 못했을까 싶다가도.. 나는 그동안 빠르게 문제 해결하는데 주로 집중하고

효율적으로 해결하는 방법은 생각할 여유가 없었다는 생각이 들었다.

내 역량이 부족해서 그런 것이겠지 ,,, ㅠㅠ 더 열심히 해야겠다는 생각이 들었다!!

 

 

회사에서 주로 유지보수를 하다보니

요즘 부쩍 클린 코드의 중요성을 깨닫고 있다..!

 

 

 

 

"이 포스팅은 쿠팡 파트너스 활동의 일환으로, 이에 따른 일정액의 수수료를 제공받습니다."