Dev.
[CleanCode] Guard clause
인쥭
2022. 1. 25. 11:46
반응형
참고
Replace Nested Conditional with Guard Clauses
Replace Nested Conditional with Guard Clauses function getPayAmount() { let result; if (isDead) result = deadAmount(); else { if (isSeparated) result = separatedAmount(); else { if (isRetired) result = retiredAmount(); else result = normalPayAmount(); } }
www.refactoring.com
- if - else가 과도하게 중첩되는 경우 가독성과 유지보수성이 떨어진다.
- guard clause는 아래와 같이 적용되어 가독성을 향상시키고, 불필요한 조건 검증을 미연에 방지한다.
- 반드시 if - else if - else 구조가 필요한 경우가 아니라면 guard clause를 적용하는 것이 바람직하다.
(() => {
console.log('!!!bad function start!!!');
bad(107);
console.log('!!!bad function done!!!\n');
console.log('!!!good function start!!!');
good(107);
console.log('!!!good function done!!!\n');
})();
function bad(num) {
if(num > 0) {
console.log('[1-1] condition passed');
if(num > 3) {
console.log('[2-1] condition passed');
if(num < 100) {
console.log('[3-1] condition passed');
console.log('result: 3 < num < 100');
} else if(num < 120) {
console.log('[3-2] condition passed');
console.log('result: 3 < num < 120');
} else {
console.log('[3-3] condition passed');
console.log('result: num >= 120');
}
} else {
console.log('[2-2] condition passed');
console.log('result: num <= 3');
}
} else {
console.log('[1-2] condition passed');
console.log('result: num <= 0');
}
}
// guard clause 적용
function good(num) {
if(num <= 0) {
console.log('[1-a] condition passed');
console.log('result: num <= 0');
return;
}
if(num <= 3) {
console.log('[2-a] condition passed');
console.log('result: num <= 3');
return;
}
if(num < 100) {
console.log('[3-a] condition passed');
console.log('result: 3 < num < 100');
return;
}
if(num < 120) {
console.log('[4-a] condition passed');
console.log('result: 3 < num < 120');
return;
}
console.log('[5-a] condition passed');
console.log('result: num >= 120');
return;
}
/* 실행 결과
!!!bad function start!!!
[1-1] condition passed
[2-1] condition passed
[3-2] condition passed
result: 3 < num < 120
!!!bad function done!!!
!!!good function start!!!
[4-a] condition passed
result: 3 < num < 120
!!!good function done!!!
*/
- console.log를 남발하여 코드가 더 지저분해진 것 같지만, guard clause의 개념을 이해한 것으로 만족하려고 한다.