[Refactoring] 7-3. 기능 이동 (함수 옮기기, 중첩함수 제거, 모듈성) - 문장을 함수로 옮기기

문장을 함수로 옮기기

• 함수를 호출하는 쪽에서 항상 그 함수를 호출하기 전이나 후에 동일한 관련 동작을 할 것으로 예상되는 것들은 함수 안으로 이동시키기
  • 함수를 이용하는 관점에서 매번 사용되는 코드를 함수 안으로 이동시킴으로써 코드 중복을 막는게 목적
  • 주의할점은 함수 안으로 이동할 문장이 함수의 일부라는 확신이 있어야함 (한 함수는 하나의 일만 해야지, 응집도가 높은 코드 기대가 가능)

문장을 함수로 옮기기

중요한 이유

• 특정 함수를 호출하는 코드가 나올 때마다 그 앞이나 뒤에서 똑같은 코드가 추가로 실행되면, 그 똑같은 코드를 함수안에 삽입하여 중복 코드를 방지

문장을 함수로 옮기기 예제

ex) 이름을 변경한 후 변경 전과 변경 후의 코드를 출력하는 코드

class ViewController: UIViewController {
    var name: String?
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let beforeName = name
        let newName = "123"
        update(name: newName)
        print("change from:\(beforeName) to:\(newName)")
    }
    
    func update(name: String) {
        self.name = name
    }
}

• 문제점 - 변경된 이름을 확인하는 코드는 update(name:)을 호출하는 쪽에서 매번 호출될 수 있어서 중복 코드를 유발할 수 있음
  • 관련 코드를 update(name:) 함수 내부로 이동시키기
  • update(name:)안에서 관련 작업을 하여, 나중에 추후 반복되는 부분에서 무언가 수정할 일이 생겼을 때 단 한곳만 수정하면 되어서 유지보수에 유리한 코드
  • 만약 여러 변형들로 나눠야하는 순간이 올때는 반대로 함수에서 밖으로 반대 리펙토링을하여 쉽게 다시 뽑기가 가능

class ViewController: UIViewController {
    var name: String?
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
        let newName = "123"
        update(name: newName)
    }
    
    func update(name: String) {
        let beforeName = self.name
        self.name = name
        print("change from:\(beforeName) to:\(name)")
    }
}

* 참고

- Refactoring (Marting Flowler)