[리팩토링 cheat sheet] 9.1 변수 쪼개기(Split Variable)

30 May 2020

# Before

temp = 2 * (height + width)
print(temp)
temp = height * width
print(temp)

# After

PERIMETER = 2 * (height + width)
print(PERIMETER)
AREA = height * width
print(AREA)

위 Before 는 변수에 값을 여러 번 대입할 수 밖에 없는 경우이다. 이때 변수의 목적은 긴 코드를 저장했다가 나중에 쉽게 참조하려는 목적으로 흔히 사용된다. 이런 변수는 값을 단 한 번만 대입해야 한다. 역할이 둘 이상인 변수가 있다면 쪼개야 한다

# Before

def discount(input_value, quantity):
    if input_value > 50:
        input_value = input_value - 2
    if quantity > 100:
        input_value = input_value - 1
    return input_value

# After

def discount(original_input_value, quantity):
    result = original_input_value
    if input_value > 50:
        result -= 2
    if quantity > 100:
        result -= 1
    return result

첫번째 if 문에서, 입력 값에 기초하여 결과값을 누적해 계산한다는 사실을 더 명확히 드러낸다.