diff --git "a/_posts/2020-05-31-\353\246\254\355\214\251\355\206\240\353\247\201-\355\225\204\353\223\234 \354\235\264\353\246\204 \353\260\224\352\276\270\352\270\260.md" "b/_posts/2020-05-31-\353\246\254\355\214\251\355\206\240\353\247\201-\355\225\204\353\223\234 \354\235\264\353\246\204 \353\260\224\352\276\270\352\270\260.md" new file mode 100644 index 0000000..e55a3d6 --- /dev/null +++ "b/_posts/2020-05-31-\353\246\254\355\214\251\355\206\240\353\247\201-\355\225\204\353\223\234 \354\235\264\353\246\204 \353\260\224\352\276\270\352\270\260.md" @@ -0,0 +1,70 @@ +--- +layout: post +title: "[리팩토링 cheat sheet] 9.2 필드 이름 바꾸기(Rename Field)" +description: "리팩토링 책을 읽고, 파이썬으로 cheat sheet를 작성한다" +date: 2020-05-31 +tags: [리팩토링, python] +comments: true +share: true + +--- + + + +```python +# Before + +class Organization: + @property + def name(){...} +``` + + + +```python +# After + +class Organization: + @property + def title(){...} +``` + +---- + +* 위 Before 는 변수에 값을 여러 번 대입할 수 밖에 없는 경우이다. 이때 변수의 목적은 긴 코드를 저장했다가 나중에 쉽게 참조하려는 목적으로 흔히 사용된다. 이런 변수는 값을 단 한 번만 대입해야 한다. 역할이 둘 이상인 변수가 있다면 쪼개야 한다 + + + + + + + +```python +# 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 +``` + + + +```python +# 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 문에서, 입력 값에 기초하여 결과값을 누적해 계산한다는 사실을 더 명확히 드러낸다. \ No newline at end of file