객체 인스턴스 변수를 사용하는 경우와 메서드에 인수를 전달하는 경우


93

인수를 메서드에 전달하는 것과 단순히 모든 개체의 메서드에 표시되는 개체 인스턴스 변수로 선언하는 것 사이에서 어떻게 결정합니까?

나는 클래스의 끝에있는 목록에 인스턴스 변수를 유지하는 것을 선호하지만,이 목록은 프로그램이 커질수록 길어집니다. 변수가 충분히 자주 전달되면 필요한 모든 메소드에서 볼 수 있어야한다고 생각하지만 "모든 것이 공개되면 아무것도 전달할 필요가 없습니다!"라고 생각합니다.


1
구체적인 예가있는 경우보다 직접적으로 유용한 답변을 얻을 수 있습니다
brabster

답변:


55

인스턴스 변수를 참조하고 있으므로 객체 지향 언어로 작업하고 있다고 가정합니다. 어느 정도 인스턴스 변수 사용시기, 범위 정의 방법 및 지역 변수 사용시기는 주관적이지만 클래스를 만들 때마다 따를 수있는 몇 가지 경험 규칙이 있습니다.

  • 인스턴스 변수는 일반적으로 클래스의 속성으로 간주됩니다. 이것을 여러분의 클래스에서 생성 될 객체의 형용사로 생각하십시오. 인스턴스 데이터를 사용하여 객체를 설명 할 수 있다면 인스턴스 데이터에 적합한 선택이라고 확신하는 것이 안전 할 것입니다.

  • 지역 변수는 작업을 완료하는 데 도움이되도록 메서드 범위 내에서 사용됩니다. 일반적으로 메서드는 일부 데이터를 가져오고, 일부 데이터를 반환하고, 일부 데이터에 대해 알고리즘을 처리 / 실행하는 목적을 가져야합니다. 때로는 지역 변수를 메서드의 처음부터 끝까지 돕는 방법으로 생각하는 것이 도움이됩니다.

  • 인스턴스 변수 범위는 보안뿐만 아니라 캡슐화에도 적용됩니다. "목표는 모든 변수를 비공개로 유지하는 것"이라고 가정하지 마십시오. 상속의 경우 변수를 보호 된 것으로 만드는 것이 일반적으로 좋은 대안입니다. 모든 인스턴스 데이터를 공개로 표시하는 대신 외부 세계에 액세스해야하는 데이터에 대한 getter / setter를 만듭니다. 모두 사용 가능하게 만들지 말고 필요한 것만 사용하십시오. 이것은 개발 라이프 사이클 전반에 걸쳐 올 것입니다. 처음부터 추측하기는 어렵습니다.

클래스 주변에 데이터를 전달할 때 코드를 보지 않고는하고있는 일이 좋은 습관이라고 말하기는 어렵습니다. 때로는 인스턴스 데이터에서 직접 작동하는 것이 좋습니다. 다른 경우에는 그렇지 않습니다. 제 생각에는 이것은 경험과 함께 제공되는 것입니다. 객체 지향적 사고 능력이 향상됨에 따라 약간의 직관을 개발할 것입니다.


내 대답은 H-Man2 답변 (평생) 에이 답변을 추가하는 것입니다. 객체의 지속적인 상태 인 경우에만 멤버 속성이어야합니다. 즉, 값은 현재 메서드 스택의 범위 밖에서 자체적으로 의미가 있습니다.
David Rodríguez-dribeas

1
내 직감 반응은 David와 H-MAn2에 동의하는 것입니다. 그러나 나는 Robert c Martin의 "깨끗한 코드"를 읽고 있으며 3 장에서 그는 많은 매개 변수를 갖는 것이 나쁘기 때문에 메소드 매개 변수에서 멤버 변수로 무언가를 이동하도록 코드를 리팩토링합니다. 균형 적으로 여러분의 클래스가 단 하나의 책임 만 가지고 있다면 객체의 수명은 그 계산의 수명과 같을 것입니다. 따라서 실제 대답은이 질문을해야한다면 클래스가 너무 크다는 것입니다.
Andy

@ DavidRodríguez-dribeas 메소드 스택이란 무엇을 의미합니까?
committedandroider

@committedandroider : 값이 현재 함수 호출보다 오래 지속되는 경우
David Rodríguez-dribeas

46

주로 이것은 변수에 저장하는 데이터의 수명에 따라 다릅니다. 데이터가 계산 중에 만 사용되는 경우 매개 변수로 전달하십시오. 데이터가 개체의 수명에 바인딩 된 경우 인스턴스 변수를 사용합니다.

변수 목록이 너무 길어지면 클래스의 일부를 새 클래스로 리팩토링하는 것에 대해 생각하는 것이 좋습니다.


21

제 생각에는 인스턴스 변수는 데이터가 호출간에 사용될 때만 필요합니다.

예를 들면 다음과 같습니다.

myCircle = myDrawing.drawCircle(center, radius);

이제 myDrawing 클래스를 이미징 할 때 15 개의 도우미 함수를 사용하여 myCircle 개체를 만들고 이러한 각 함수에는 중심과 반경이 필요합니다. 여전히 myDrawing 클래스의 인스턴스 변수로 설정해서는 안됩니다. 다시는 필요하지 않기 때문입니다.

반면에 myCircle 클래스는 중심과 반경을 모두 인스턴스 변수로 저장해야합니다.

myCircle.move(newCenter);
myCircle.resize(newRadius);

myCircle 객체가 이러한 새 호출이 수행 될 때 반경과 중심이 무엇인지 알기 위해서는이를 필요로하는 함수에 전달하는 것이 아니라 인스턴스 변수로 저장해야합니다.

따라서 기본적으로 인스턴스 변수는 객체의 "상태"를 저장하는 방법입니다. 개체의 상태를 알기 위해 변수가 필요하지 않은 경우 인스턴스 변수가 아니어야합니다.

그리고 모든 것을 공개합니다. 그것은 순간에 당신의 삶을 더 쉽게 만들 수 있습니다. 그러나 그것은 당신을 괴롭히기 위해 돌아올 것입니다. 완두콩하지 마십시오.


매개 변수 (oldCenter, newCenter)를 사용하도록 이동을 정의 할 수 있습니다.
obesechicken13

4

IMHO :

변수가 인스턴스 상태의 일부를 형성하는 경우 인스턴스 변수-classinstance HAS-A instancevariable이어야합니다.

인스턴스의 메서드에 반복적으로 무언가를 전달하거나 인스턴스 변수가 많다는 사실을 발견하면 무언가를 놓쳤거나 어딘가에서 잘못된 추상화를 만들었을 경우를 대비하여 디자인을 살펴볼 것입니다.

도움이되기를 바랍니다.


3

물론 클래스에 하나의 큰 공용 변수 목록을 유지하는 것은 쉽습니다. 그러나 직관적으로도 이것이 갈 길이 아님을 알 수 있습니다.

사용하기 직전에 각 변수를 정의하십시오. 변수가 특정 메서드의 기능을 지원하는 경우 메서드 범위에서만 사용하십시오.

또한 보안에 대해 생각해보십시오. 공용 클래스 변수는 "외부"코드에서 원하지 않는 변경에 취약합니다. 당신의 주요 목표는 모든 변수를 비공개로 유지하는 것이며, 그렇지 않은 변수는 그렇게해야 할 아주 좋은 이유가 있어야합니다.

매개 변수를 스택 위로 전달하는 것은 매우 빠르다. 경험상 메서드 시그니처를 깨끗하고 우아하게 유지하는 것이 좋습니다. 동일한 데이터를 사용하는 메서드가 여러 개있는 경우 클래스 멤버가 될만큼 중요한지 결정하고 그렇지 않은 경우 코드를 리팩터링하여 더 이해하기 쉽게 만듭니다.

그것은 상식으로 귀결됩니다. 각 새 변수를 선언하는 위치와 이유, 기능이 무엇인지 정확히 생각하고 거기에서 어떤 범위에 있어야하는지 결정하십시오.


메서드를 단위 테스트 할 수 있도록 공용 메서드를 원하는 경우가 많습니다.
obesechicken13 dec.
당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.