자바스크립트 깔끔한 코딩을 위한 필수 개념

- 4월 18, 2023

하나의 function에는 하나의 역할만!!

한 사람은 하나의 역할만 한다.

식당에서 홀은 홀 담당, 주방은 주방 담당인 것처럼 코딩에서 하나의 함수는 하나의 역할만 하게 만드는 것이 중요하다. 그래야 재사용 가능하고, 수정이 용이한 코드를 만들어낼 수 있다.

예제 Example

수정 전

showPrimes(20);

function showPrimes(limit) {
  for (let num = 2; num <= limit; num++) {
    let isPrime = true;
    for (let factor = 2; factor < num; factor++) {
      if (num % factor === 0) {
        isPrime = false;
        break;
      }
    }

    if (isPrime) console.log(num);
  }
}

수정 전 showPrimes 함수에는 2부터 limit까지 숫자를 반복해서 소수이면 출력하는 역할과 들어온 숫자가 소수인지 판단하는 역할, 총 2가지의 역할이 있었다. 이는 가독성이 좋지 못한 코드다.

수정 후

showPrimes(20);

function showPrimes(limit) {
  for (let num = 2; num <= limit; num++) {
    if (isPrime(num)) console.log(num);
  }
}

function isPrime(num) {
  for (let factor = 2; factor < num; factor++) {
    if (num % factor === 0) {
      return false;
    }
    return true;
  }
}

수정된 코드에는 총 두 가지 함수가 있으며 showPrimes는 2부터 limit까지의 숫자를 반복해서 소수이면 출력하는 역할이고, isPrime함수는 들어온 숫자가 소수인지 판단하는 역할이다.

댓글