2021.04.16 CleanCode 일지 #2. 조건문을 함수로 빼기

클린코드 일지를 적는다고 했는데, 그러고 난 다음에 4개월이나 지났다.

시간이 너무 빨리가버렸다.

현재 기존에 있던 사이트가 아닌 다른 곳에서 일하고 있어서 새로운 업무를 익히는 중이다.

 

이 업무 중에는 특정 컬럼의 값이 무엇이냐에 따라서 의미가 달라지는 부분이 있다. 

조건문 안에 들어가는 부분을 이름붙여 함수로 만들면 그 부분에 대해 조금 더 캐치하기 쉬울 것 같다는 생각이 들었다.

 

실제 평가와 그 평가를 치지 못했을 때 치뤄지는 추가적인 평가가 있다. 1이면 실제 평가, 2이면 추가평가를 의미한다.

DB 컬럼에 코멘트가 있으면 조금 더 알기 수월하겠지만, 어쨌든 이부분을 함수로 만들면 어떨까 싶었다.

 

private boolean isExtraExam(){
    return testCd.equals("2");
}

 

다음과 같이 코드를 작성하고, 조건문에서 해당 함수를 호출하여 추가시험인지, 실제평가인지를 구분하도록 했다.

사실 정말 별거 아니지만 이렇게 함수로 바꿔주니까 무슨 의미인지 모를 조건이 한층 더 이해하기 쉬워진 것 같다.

만약 testCd.equals("2") 라는 조건문을 그대로 사용하였다면, 조건문 옆에 주석이라도 달고 싶었을 것이다.

사소하지만 이런 부분에서 클린코드가 주는 효율성을 느낄 수 있었다.