인수가 여러 메소드를 통해 전달 될 때 인스턴스 변수에 저장해야합니까?

Question

개체의 여러 메서드를 통해 인수를 전달하려는 경우 해당 인수를 인스턴스 변수에 저장하는 것이 좋습니다? 예를 들어

:이 경우

public class Example { 
    private int[] array; 

    public void doThingsToArray(int[] array) { 
     this.array = array; 
     // `array` doesn't need to be passed as an argument. 
     doFirstThingToArray(); 
     doSecondThingToArray(); 
     ... 
    } 

    ... 
} 

는 Example의 동일한 인스턴스가 여러 배열에 doThingsToArray()를 호출하는 데 사용됩니다, 그래서 당신은 단지 Example의 생성자에 인수로 array을 전달할 수 없습니다.

내가 볼 수있는 프로는 인자로 array을 넘겨 줄 필요가 없다는 것입니다. - Clean Code에 따르면, 적어도 메소드가 가진 매개 변수가 적을수록 좋습니다.

나는 단단한 죄수를 생각해 낼 수 없지만,이 상황에 뭔가 잘못 됐음을 느낍니다.

Answer 1

개체의 여러 메서드를 통해 인수를 전달하려는 경우 해당 인수를 인스턴스 변수에 저장하는 것이 좋습니다.

아니요, 그렇게하는 것이 현명하지 않습니다.

디자인 관점에서 인스턴스 변수에 숨기도록 제안한 값은 개체의 속성이 아닙니다. 오히려 그것은 주어진 계산의 속성입니다. 따라서 메서드 인수 및/또는 로컬 변수는 해당 변수가 속한 곳이며 인스턴스 변수에 절대 속하지 않습니다.

실용적인 관점에서 인스턴스 변수에 숨겨진 값에 의존하면 스레드 안전이 손상됩니다. 하나의 객체에서 여러 개의 계산을 동시에 수행 할 수는 없습니다. 왜냐하면 각각의 객체가 동일한 목적을 위해 동일한 인스턴스 변수를 사용할 수 있기를 기대하기 때문입니다.