복사 생성자에서 다른 객체의 비공개 필드에 액세스 - 정말 문제가 있습니까? 내 자바 응용 프로그램에서

Question

나는 6.9 이것에 대해 경고이

public MyClass(MyClass src) { 
    this.field1 = src.field1; 
    this.field2 = src.field2; 
    this.field3 = src.field3; 
... 
} 

이제 넷빈즈 같은 일부 복사 생성자를 가지고 있고이 코드에 어떤 문제가 있는지 궁금해?

내 문제 :

• 원치 않는 부작용을 소개 할 수 게터를 사용. 새 객체는 더 이상 원본 사본으로 간주 될 수 없습니다.
• getter를 사용하는 것이 좋습니다 경우 새로운 인스턴스에도 setter를 사용하는 것이 더 일관성이 없습니까?

편집 : 실제 경고 및 넷빈즈는 @SuppressWarnings("AccessingNonPublicFieldOfAnotherObject")

내 코드를 추가 제공하는 유일한 가능한 액션 "다른 개체의 private 필드의 액세스"입니다 실제로 주어진 예처럼 간단하다.

Answer 1

코드에 아무런 문제가 없습니다. 사실, 나는 당신의 우려를 공유하고 getter를 사용하지 않는 것을 선호합니다.

경고는 정확히 무엇입니까? 어쩌면 당신이 우리에게 보여주지 않는 것과 관련이 있을까요?

업데이트 : 넷빈즈가 정말로 불만을 나타 냈습니다. 그것은 IDE와 함께 제공되는 오히려 논란의 경고입니다, 나는 생각합니다.

Answer 2

아마도 클래스는 복사 된 개체에서 반환 할 데이터를 알고있는 얕은 복사 메서드를 제공 할 수 있습니다. 필요한 경우 깊은 사본을 제공 할 수도 있습니다.

public MyClass shallowCopy() { 
    MyClass aCopy = new MyClass(); 
    aCopy.field1 = this.field1; 
    aCopy.field2 = this.field2; 
    aCopy.field3 = this.field3; 
} 

Answer 3

NetBeans가 경고하는 이유를 모르지만 실제로는 을 얕은 복사본으로 만듭니다. 사본은 field1, field2 및 field3을 src와 공유하며 field3의 수정 사항 (예 : 목록)은 원본에 반영됩니다.

private List field1; 

public MyClass(MyClass src) { 
    this.field1 = src.field1; 
    this.field2 = src.field2; 
    this.field3 = src.field3; 

    field1.add(new Object()); // field1 of src also modified 
... 
} 

Answer 4

그것은 토론 ADT 대 객체 생각 나게한다.

ADT는 ADT의 다른 인스턴스의 내부 상태에 액세스 할 수 있습니다. 객체 철학은이를 방지하고 getter/setter를 통해 액세스를 시행하여 표현 독립성을 보장합니다.

그것은 자바에서 변수 인스턴스가있는 심의를 선택이었다 클래스 (개인, 후자의 경우에는 단지 this.privateInstVar이 허용되지 obj.privateInstVar) 객체 개인하지.

약점과 약점이 있습니다. 및 등호에 관해서는 대단히 편리합니다. 반면에 그것은 오용 될 수 있으며 은 캡슐화를 해제.

거의 철학적 논쟁 거리입니다. 그러나 IMHO, 당신이하고있는 일은 괜찮습니다.