Truy cập vào trường riêng của đối tượng khác trong các nhà xây dựng sao chép - Thực sự là một vấn đề?

Question

Trong ứng dụng Java của tôi, tôi có một số bản sao-nhà xây dựng như thế này

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

Bây giờ Netbeans 6.9 cảnh báo về vấn đề này và tôi tự hỏi những gì là sai với mã này?

mối quan tâm của tôi:

• Sử dụng thu khí có thể giới thiệu không mong muốn tác dụng phụ. Đối tượng mới có thể không còn được coi là bản sao của bản gốc.
• Nếu nó được khuyến nghị sử dụng getters, nó sẽ không nhất quán hơn nếu người ta sẽ sử dụng setters cho dụ mới không?

EDIT: Cảnh báo thực tế là "Truy cập của lĩnh vực tư nhân của một đối tượng" và hành động chỉ có sẵn Netbeans cung cấp sẽ bổ sung một @SuppressWarnings("AccessingNonPublicFieldOfAnotherObject")

Mã của tôi thực sự là như tầm thường như ví dụ đưa ra.

Answer 1

Tôi thấy không có vấn đề với mã. Trong thực tế, tôi chia sẻ mối quan tâm của bạn và không muốn sử dụng getters.

Cảnh báo bạn nhận được chính xác là gì? Có lẽ nó liên quan đến một cái gì đó bạn không cho chúng ta thấy?

Cập nhật: Dường như Netbeans thực sự phàn nàn về điều đó. Đó là một cảnh báo khá gây tranh cãi để gửi cùng với một IDE, tôi nghĩ vậy.

Answer 2

Có lẽ lớp học có thể cung cấp phương thức sao chép nông để biết dữ liệu nào cần trả về trong đối tượng được sao chép. Nếu cần, bạn cũng có thể cung cấp bản in sâu.

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

Answer 3

Tôi không biết tại sao NetBeans cảnh báo, nhưng bạn đang tạo một bản sao nông hiệu quả. Bản sao của bạn chia sẻ field1, field2 và field3 với src và như vậy một sửa đổi của field3, nói, một danh sách, sẽ phản ánh trong bản gốc.

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

Nó nhắc nhở tôi về các cuộc thảo luận ADT vs Object.

ADT có thể truy cập trạng thái nội bộ của phiên bản ADT khác. Triết lý đối tượng sẽ ngăn chặn điều đó và thực thi truy cập thông qua getter/setter để đảm bảo độc lập đại diện.

Đó là một sự lựa chọn cân nhắc rằng trường hợp biến trong Java là đẳng cấp riêng, không đối tượng tin (Trong trường hợp sau chỉ this.privateInstVar được cho phép, không obj.privateInstVar).

Điều này có cả điểm mạnh lẫn điểm yếu. Tính năng này đặc biệt hữu ích khi nói đến nhân bản và bình đẳng. Mặt khác, nó có thể bị lạm dụng và đóng gói phá vỡ.

Nó gần như là một cuộc tranh luận triết học. Nhưng IMHO, những gì bạn đang làm là tốt.