메소드 정의 시 고려할 점

아래처럼 코드를 만들었다.

public class SwappableTile : Tile
{    
    private Vector3 mousePosition;

    private void GetMousePosition()
    {
        mousePosition = Camera.main.ScreenToWorldPoint(Input.mousePosition);
        mousePosition.z = 0;
    }
}

고려해 볼 사항이 여럿 있다.

 

1. 순수 함수이거나 재사용되는 경우 별도의 메소드로 분리할 가치가 있다.

- 순수 함수: 같은 인자 값을 받았을 때 언제나 같은 반환 값을 돌려주는 함수

- 반면 순수 함수가 아닌 함수, 예를 들면 분리한 메소드 내의 변수들을 사용하지 않고 클래스의 멤버 변수를 사용하는 경우 굳이 별도의 메소드로 분리하지 않는 것이 낫다.(의미 없는 메소드 분리는 차후 코드 추적만 어렵게 한다.)

- 위의 GetMousePosition은 메소드 바깥의 클래스 멤버 변수에 접근하여 사용하고 있으므로 순수 함수가 아니다.

 

 

하지만 사실 SwappableTile 클래스 안에서 mousePositon을 얻어오는 부분이 두 군데 있다. 그렇다면 차라리 저 2줄 짜리 코드를 두 번 입력하더라도 별도 메소드로 분리하는 것이 나을까?

그것은 아니다. 그렇지만...

 

 

2. "Get~~", "Set~~"은 관례적인 의미가 있으므로 주의해야 한다.

- 통상적으로 Get~~은 어떤 값을 return할 것처럼 보인다.

- 그리고 Set~~은 어떤 입력 파라미터를 받아서 변수에 값을 설정해줄 것처럼 보인다.

- 위의 GetMousePosition은 사실 mousePosition을 설정하고 있으므로 Set에 더 가깝다. 그러나 관례적인 Set 메소드의 규칙을 위해 입력 파라미터를 추가로 받아야 한다면...

public class SwappableTile : Tile
{    
    private Vector3 mousePosition;

    private void Update()
    {
        // 이렇게 사용
        SetMousePosition(Input.mousePosition);
    }

    private void SetMousePosition(Vector3 mouseScreenPosition)
    {
        mousePosition = Camera.main.ScreenToWorldPoint(mouseScreenPosition);
        mousePosition.z = 0;
    }
}

 

 

그래도 사실 명료하지 않다. 그래서...

 

 

3. 메소드의 내용을 이름으로 유추 가능하게 한다.

- 메소드를 살펴보지 않고 이름만으로 무슨 작업이 수행되는지 유추할 수 있도록 만드는 것이 가장 좋다.

public class SwappableTile : Tile
{
    private Vector3 mousePosition;

    private void Update()
    {
        //이렇게 사용
        mousePosition = GetMouseWorldPosition();
    }

    private Vector3 GetMouseWorldPosition()
    {
        Vector3 worldPosition = Camera.main.ScreenToWorldPoint(Input.mousePosition);
        worldPosition.z = 0;

        return worldPosition;
    }
}

 

 

이제 메소드도 분리하고, Get의 관례적인 규칙(return)을 지키면서 무슨 메소드인지 유추 가능하게 되었다.

 

 

 

 

 

참조

https://jeong-pro.tistory.com/23

순수 함수란? (함수형 프로그래밍의 뿌리, 함수의 부수효과를 없앤다)