1. 상속과 중복 코드

1. DRY(Don’t Repeat Yourself) 원칙

<aside> 💡

중복 코드는 변경을 방해한다

</aside>

• 프로그램의 본질은 비즈니스와 관련된 지식을 코드로 변환하는 것
  • 이 지식은 항상 변화, 이에 맞춰 지식을 표현하는 코드 역시 변경해야 함
• 중복 코드는 코드를 수정하는 데 필요한 노력을 몇 배로 증가시킴
  • 어떤 코드가 중복인지를 찾아야 함
  • 찾아낸 모든 코드를 일관되게 수정해야 함
  • 모든 중복 코드를 개별적으로 테스트해서 동일한 결과를 내놓는지 확인해야함
• 중복 여부를 판단하는 기준은 변경
  • 요구사항이 변경됐을 때 두 코드를 함께 수정해야 한다면 중복
  • 중복 여부를 결정하는 기준은 코드가 변경에 반응하는 방식(코드의 모양이 아님)

2. 중복과 변경

1. 중복 코드 살펴보기

• 한 달에 한 번씩 가입자별로 전화 요금을 계산하는 애플리케이션
  • 요금 = 통화 시간 / 단위 시간 당 요금
  • 10초 당 5원 → 100초 통화 시 100 / 10 * 5 = 50원

class Call
{
public:
	Call(const FLocalDateTime& InFrom, const FLocalDateTime& InTo)
		: From(InFrom)
		, To(InTo)
	{}

	FDuration GetDuration() const { return FDuration::Between(From, To); }
	FLocalDateTime GetFrom() const { return From; }

private:
	FLocalDateTime From;
	FLocalDateTime To;
};

• 개별 통화 기간을 저장하는 클래스

class FPhone
{
public:
	FPhone(FMoney* InAmount, const FDuration& InSeconds)
		: Amount(InAmount)
		, Seconds(InSeconds)
	{}

	void Call(FCall* InCall) { Calls.push_back(InCall); }
	std::vector<FCall*> GetCalls() const { return Calls; }
	FMoney* GetAmount() const { return Amount; }
	FDuration GetSeconds() const { return Seconds; }

	FMoney* CalculateFee() const
	{
		FMoney* Result = const_cast<FMoney*>(FMoney::ZERO);
		for (FCall* Call : Calls)
		{
			Result = Result->Plus(Amount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
		}

		return Result;
	}

private:
	FMoney* Amount;
	FDuration Seconds;
	std::vector<FCall*> Calls;
};

• 통화 요금 계산 객체
  • 전체 통화 목록에 대해 알고 있는 정보 전문가에게 일임
  • 통화 목록은 전화기에 보관: Phone
• 필드: 단위 요금 Amount, 단위 시간 Seconds, 통화 목록 리스트 Calls
  • 10초 당 5원 → amount = 5, seconds = 10

FPhone* Phone = new FPhone(FMoney::Wons(5), FDuration::OfSeconds(10));
Phone->Call(new FCall(FLocalDateTime(2018, 1, 1, 12, 10, 0), FLocalDateTime(2018, 1, 1, 12, 11, 0)));
Phone->Call(new FCall(FLocalDateTime(2018, 1, 2, 12, 10, 0), FLocalDateTime(2018, 1, 2, 12, 11, 0)));

FMoney* Fee = Phone->CalculateFee();  // FMoney::Wons(60)

• ‘10초당 5원’씩 부과되는 요금제에 가입한 사용자가 각각 1분 동안 두 번 통화
• ‘심야 할인 요금제’ 추가 요구사항 접수: 밤 10시 이후의 통화에 대해 요금을 할인

class FNightlyDiscountPhone
{
public:
	FNightlyDiscountPhone(FMoney* InNightlyAmount, FMoney* InRegularAmount, const FDuration& InSeconds)
		: NightlyAmount(InNightlyAmount)
		, RegularAmount(InRegularAmount)
		, Seconds(InSeconds)
	{
	}

	void Call(FCall* InCall) { Calls.push_back(InCall); }
	std::vector<FCall*> GetCalls() const { return Calls; }
	FMoney* GetAmount() const { return NightlyAmount; }
	FDuration GetSeconds() const { return Seconds; }

	FMoney* CalculateFee() const
	{
		FMoney* Result = const_cast<FMoney*>(FMoney::ZERO);
		for (FCall* Call : Calls)
		{
			if (Call->GetFrom().GetHour() >= LATE_NIGHT_HOUR)
			{
				Result = Result->Plus(NightlyAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
			}
			else
			{
				Result = Result->Plus(RegularAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
			}
		}

		return Result;
	}

private:
	constexpr int32_t LATE_NIGHT_HOUR = 22;

	FMoney* NightlyAmount;
	FMoney* RegularAmount;
	FDuration Seconds;
	std::vector<FCall*> Calls;
};

• 쉽고도 가장 빠른 방법은 Phone의 코드를 복사해서 NightlyDiscountPhone이라는 새로운 클래스를 만든 후 수정하는 것
• 밤 10시 전후로의 요금을 구분
• 밤 10시를 기준으로 regularAmount와 nightlyAmount 중에서 기준 요금을 결정한다는 점을 제외하고는 Phone과 거의 유사

2. 중복 코드 수정하기

• 새로운 요구사항: 통화 요금에 부과할 세금 계산
  • 가입자 핸드폰마다 세율 상이
  • 통화 요금 계산 로직이 Phone, NightlyDiscountPhone 모두 구현돼있어 두 클래스 함께 수정

class FPhone
{
public:
	FPhone(FMoney* InAmount, const FDuration& InSeconds, double InTaxRate)
		: Amount(InAmount)
		, Seconds(InSeconds)
		, TaxRate(InTaxRate)
	{}

	void Call(FCall* InCall) { Calls.push_back(InCall); }
	std::vector<FCall*> GetCalls() const { return Calls; }
	FMoney* GetAmount() const { return Amount; }
	FDuration GetSeconds() const { return Seconds; }

	FMoney* CalculateFee() const
	{
		FMoney* Result = const_cast<FMoney*>(FMoney::ZERO);
		for (FCall* Call : Calls)
		{
			Result = Result->Plus(Amount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
		}

		return Result->Plus(Result->Times(TaxRate));
	}

private:
	FMoney* Amount;
	FDuration Seconds;
	std::vector<FCall*> Calls;

	double TaxRate;
};

class FNightlyDiscountPhone
{
public:
	FNightlyDiscountPhone(FMoney* InNightlyAmount, FMoney* InRegularAmount, const FDuration& InSeconds, double InTaxRate)
		: NightlyAmount(InNightlyAmount)
		, RegularAmount(InRegularAmount)
		, Seconds(InSeconds)
		, TaxRate(InTaxRate)
	{
	}

	void Call(FCall* InCall) { Calls.push_back(InCall); }
	std::vector<FCall*> GetCalls() const { return Calls; }
	FMoney* GetAmount() const { return NightlyAmount; }
	FDuration GetSeconds() const { return Seconds; }

	FMoney* CalculateFee() const
	{
		FMoney* Result = const_cast<FMoney*>(FMoney::ZERO);
		for (FCall* Call : Calls)
		{
			if (Call->GetFrom().GetHour() >= LATE_NIGHT_HOUR)
			{
				Result = Result->Plus(NightlyAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
			}
			else
			{
				Result = Result->Plus(RegularAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
			}
		}

		return Result->Plus(Result->Times(TaxRate));
	}

private:
	constexpr int32_t LATE_NIGHT_HOUR = 22;

	FMoney* NightlyAmount;
	FMoney* RegularAmount;
	FDuration Seconds;
	std::vector<FCall*> Calls;

	double TaxRate;
};

• 많은 코드 더미 속에서 어떤 코드가 중복인지를 파악하기 어려움
  • Phone은 수정했는데 NightlyDiscountPhone을 수정하지 않았다면?
  • 만약 두 클래스 코드를 다르게 수정했다면?
• 중복 코드는 새로운 중복 코드를 부름
  • 중복 코드를 제거하지 않은 상태에서 코드를 수정하려면 중복 코드를 추가할 수 밖에 없음
  • 중복 코드가 늘어날 수록 애플리케이션은 변경에 취약해지고 버그가 발생할 가능성이 높아짐

3. 타입 코드 사용하기

class FPhone
{
	using ThisClass = FPhone;

public:
	enum class EPhoneType
	{
		Regular,
		Nightly
	};

	FPhone(FMoney* InAmount, const FDuration& InSeconds, double InTaxRate)
		: ThisClass(EPhoneType::Regular, InAmount, const_cast<FMoney*>(FMoney::ZERO), const_cast<FMoney*>(FMoney::ZERO), InSeconds)
	{}

	FPhone(FMoney* InNightlyAmount, FMoney* InRegularAmount, const FDuration& InSeconds)
		: ThisClass(EPhoneType::Nightly, const_cast<FMoney*>(FMoney::ZERO), InNightlyAmount, InRegularAmount, InSeconds)

	{}

	FPhone(EPhoneType InType, FMoney* InAmount, FMoney* InNightlyAmount, FMoney* InRegularAmount, const FDuration& InSeconds)
		: Type(InType)
		, Amount(InAmount)
		, NightlyAmount(InNightlyAmount)
		, RegularAmount(InRegularAmount)
		, Seconds(InSeconds)
	{}

	void Call(FCall* InCall) { Calls.push_back(InCall); }
	std::vector<FCall*> GetCalls() const { return Calls; }
	FMoney* GetAmount() const { return Amount; }
	FDuration GetSeconds() const { return Seconds; }

	FMoney* CalculateFee() const
	{
		FMoney* Result = const_cast<FMoney*>(FMoney::ZERO);
		for (FCall* Call : Calls)
		{
			if (Type == EPhoneType::Regular)
			{
				Result = Result->Plus(Amount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
			}
			else
			{
				if (Call->GetFrom().GetHour() >= LATE_NIGHT_HOUR)
				{
					Result = Result->Plus(NightlyAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
				}
				else
				{
					Result = Result->Plus(RegularAmount->Times(Call->GetDuration().ToSeconds() / Seconds.ToSeconds()));
				}
			}
		}

		return Result;
	}

private:
	constexpr int32_t LATE_NIGHT_HOUR = 22;

	EPhoneType Type;
	FMoney* Amount;
	FMoney* NightlyAmount;
	FMoney* RegularAmount;
	FDuration Seconds;
	std::vector<FCall*> Calls;
};

• 중복 코드를 제거하는 한 가지 방법은 클래스를 하나로 합치는 것
  • 요금제를 구분하는 타입 코드를 추가하고 타입 코드의 값에 따라 로직을 분기
  • 타입 코드를 사용하는 클래스는 낮은 응집도와 높은 결합도

2. 상속을 이용해서 중복 코드 제거하기