7주차: 10장 조건부 로직 간소화

[choisohyun]

	질문	답변
1	3 @suojae	1 @choisohyun
2	4 @BEMELON	2 @RAINBOWJYP
3	1 @choisohyun	5 @hamo-o
4	5 @hamo-o	3 @suojae
5	2 @RAINBOWJYP	4 @BEMELON

[suojae]

질문 1

중첩된 조건문은 어떻게 리팩토링할 수 있을까요?

function checkAccess(user) {
  if (user) {
    if (user.isAdmin) {
      return "Access granted";
    }
  }
  return "Access denied";
}

위 코드를 리팩토링 해주세요!

assert 문과 guard 문의 차이와 각각 어떤상황에서 쓰는게 적합한가요?

[choisohyun]

1. 삼항 연산자를 사용하거나 보호 구문으로 바꿔서 리팩토링해 보았습니다.

// isAdmin 값이 falsy할 경우 리턴값이 다름 
function checkAccess(user) {
  return user?.isAdmin ? "Access granted" : "Access denied";
}
// 동작이 완전 같음
function checkAccess(user) {
  if (!user) return "Access denied"

  if (user.isAdmin) {
    return "Access granted";
  }
}

assert문은 항상 참이라고 가정하는 조건부 문장으로, assert가 실패했다는 것은 프로그래머가 잘못했다는 뜻입니다.
guard문은 예상 가능한 예외 상황을 처리할 때 사용합니다. 예외 처리로 대응할 때 사용합니다.
assert문은 디버깅할 경우, guard문은 외부 값에 대한 예외 처리를 할 때 사용하면 적합할 것 같습니다.

[BEMELON]

질문 4

조건부 로직의 대부분은 if/else 구문과 switch/case로 이루어지는데, 왜 이러한 구문으로 이루어지는 것이 문제가 되는걸까요?

순수하게 의견을 묻습니다..

다형성을 이용한 리팩토링은 언제즈음 진행하는게 좋을까요?

최근에 다형성을 이용해서 리팩토링은 진행한 적이 있었는데, 그 때 하면서 궁금했었던 내용이였습니다 ㅎㅎ;
뭔가 논리적으로 말할 수 없는 답답함이 느껴질때 진행하는거 같은데, 다른 분들의 의견이 궁금해지더라구요

[RAINBOWJYP]

A1.
조건부 로직은 프로그램의 힘을 강화하는 데 크게 기여하지만, 프로그램을 복잡하게 만드는 주요 원흉이기도 하다.
다양한 조건, 그에 따라 동작도 다양한 코드를 작성하면 순식간에 꽤 긴 함수가 탄생한다. 긴 함수는 그 자체로 읽기가 어렵지만, 조건문은 그 어려움을 한층 가중시킨다. 조건을 검사하고 그 결과에 따른 동작을 표현한 코드는 무슨 일이 일어나는지는 이야기해주지만 왜! 일어나는지는 제대로 말해주지 않을 때가 많은 것이 문제다.
라고 마틴 파울러가 말했습니다. 😀

A2.
경험이 없어 식견이 부족합니다...
다형성 리팩터링은

똑같은 분기 로직(주로 Swtich문)이 여러 곳에 등장할 때

한다고 하는데 남용하기 쉬워서 모든 조건부 로직을 다형성으로 대체하는 것은 옳지 않다고 했네요.

개발중에 다형성으로 바꾸는건 좋지만 그게 아니라 개발이 끝난 후에 모든 조건부를 다형성으로 대체하려는게 유지보수하기 어려워서 그런걸까요...

그게 아니면 성능에 영향이 미칠 수 있어서 그런걸까요 (그렇다면 이건 어떻게 판단하는지)

[RAINBOWJYP]

질문 5

Q1. 여기 리팩터링 전 코드가 있습니다. 10장에 나온 기법을 활용하여 리팩터링을 해주세요! (code made by gpt)

class Customer {
    constructor(name, isVerified) {
        this.name = name;
        this.isVerified = isVerified;
    }

    getDiscount() {
        if (this.isVerified) {
            return 0.1; // 10% 할인
        } else {
            return 0; // 할인 없음
        }
    }
}

[BEMELON]

첫번째 들었던 생각은.. 다형성을 이용해서 분리할 수 있겠다 생각이 들었습니다

abstract class AbstractCustomer {
    name: String;
    isVerified: boolean;

    constructor(name: String, isVerified: boolean) {
        this.name = name;
        this.isVerified = isVerified;
    }

    abstract getDiscount(): number;
}


class Customer extends AbstractCustomer {
    getDiscount(): number {
        return 0.1;
    }
}

class VerifiedCustomer extends AbstractCustomer {
    getDiscount(): number {
        return 0;
    }
}

생성하고 나니, Factory를 통해서 verified 또한 제거할 수 있을 것 같더라구요!
VerifiedCustomer 자체는 인증된 고객의 로직만 작성될 것 같아서 분리할 수 있다고 생각했습니다

abstract class AbstractCustomer {
    name: String;

    constructor(name: String) {
        this.name = name;
    }

    abstract getDiscount(): number;
}


class Customer extends AbstractCustomer {
    getDiscount(): number {
        return 0.1;
    }
}

class VerifiedCustomer extends AbstractCustomer {
    getDiscount(): number {
        return 0;
    }
}

class CustomerFactory {
    static getCustomer(name: String, isVerified: boolean): AbstractCustomer {
        if (isVerified) {
            return new VerifiedCustomer(name);
        } else {
            return new Customer(name);
        }
    }
}

[choisohyun]

3번 질문

Q. 아래 함수에는 이름에 'And'가 들어 있어 함수가 하는 일이 모호합니다. 어떻게 리팩터링할 수 있을까요? (382쪽 - 10.4)

function checkNameAndAgeAndAddress(user) {
  if (user.name.length >= 2 && user.name.length <= 10) {
    if (user.age >= 14 && user.age <= 120) {
      if (user.address && user.address.trim().length > 0) {
        return true;
      }
    }
  }
  return false;
}

[hamo-o]

"And"는 함수가 독립된 여러 일을 동시에 수행하고 있음을 알리므로 조건문을 함수로 추출했습니다. 각 검증 함수가 다른 로직에서도 사용된다면 함수 밖으로 따로 분리해내도 좋을 것 같습니다.

function checkUser(user) {
  if (checkName(user.name) 
    && checkAge(user.age) 
    && checkAddress(user.address)) return true;
  return false;

  function checkName(name) {
    if (name.length >= 2 && name.length <= 10) return true;
    return false;
  }
  
  function checkAge(age) {
    if (age >= 14 && age <= 120) return true;
    return false;
  }
  
  function checkAddress(address) {
    if (address && address.trim().length > 0) return true;
    return false;
  }
}

[hamo-o]

질문 4

아래 코드는 switch ~ case 문으로 작성되어 있습니다. 분기별로 다르게 체크하고 있는 변형 동작과 기본 동작을 분리하도록 어떻게 리팩터링 해볼 수 있을까요? (p.367 ~)

switch (bird.type) {
  case '유럽 제비':
    return '보통이다';
  case '아프리카 제비':
       return (bird.numberOfCoconuts > 2) ? '지쳤다' : '보통이다';
  case '노르웨이 파랑 앵무':
    return (bird.voltage > 100) ? '그을렸다' : '예쁘다';
  default:
    return '알 수 없다';
}

[suojae]

type BirdBehavior = {
  type: string;
  describe: (bird: { numberOfCoconuts?: number; voltage?: number }) => string;
};

const EuropeanSwallow: BirdBehavior = {
  type: '유럽 제비',
  describe: () => '보통이다',
};

const AfricanSwallow: BirdBehavior = {
  type: '아프리카 제비',
  describe: (bird) =>
    bird.numberOfCoconuts && bird.numberOfCoconuts > 2 ? '지쳤다' : '보통이다',
};

const NorwegianBlueParrot: BirdBehavior = {
  type: '노르웨이 파랑 앵무',
  describe: (bird) =>
    bird.voltage && bird.voltage > 100 ? '그을렸다' : '예쁘다',
};

const DefaultBird: BirdBehavior = {
  type: '기본',
  describe: () => '알 수 없다',
};

• 저는 개인적으로 상속을 안좋아해서.. 타입을 통해 다형성을 확보해보았습니다!
• 간단한 동작 캡슐화에는 객체 리터럴이 효율적이라 생각해서 작성해보았습니다!