Bạn có nên bảo vệ các trường không?

Question

Mã dụ:

unit Foo; 

    TFoo = class 
    protected 
    FList: TList; // Lifetime is managed by constructor and destructor 
    public 
    property List: TList read FList; 
    constructor Create; 
    destructor Destroy; override; 
    end; 

unit Bar; 

    TBar = class(TFoo) 
    procedure MyMethod; 
    end; 

procedure TBar.MyMethod; 
begin 
    // Access of FList goes here 
end; 

Lớp TBar có thể trực tiếp thay đổi giá trị của FList, nhưng đó không phải là thực sự cần thiết, vì nó chỉ có để gọi các phương thức của nó/sử dụng thuộc tính của nó.

Tôi có nên đặt FList ở chế độ riêng tư và sử dụng thuộc tính để truy cập từ TBar thay thế không?

Bạn xử lý các trường hợp như thế nào? Có cân nhắc về hiệu suất nào không?

Answer 1

Mặc dù tôi đồng ý rằng bạn có thể bắt đầu với đặc quyền ít nhất, và di chuyển mọi thứ trong tầm nhìn khi cần thiết, nó chỉ vì nó kết thúc việc tạo ra thiết kế hướng đối tượng phù hợp mà không cần phải suy nghĩ quá chức năng kinh doanh thực tế cần được tiếp xúc.

Bạn nên đóng gói và ẩn càng nhiều sự phức tạp càng tốt trong một đối tượng để giao diện bên ngoài càng nhỏ gọn càng tốt. Một cách để thực hiện điều này là chỉ thêm hoặc hiển thị các thuộc tính khi bạn cần chúng.

Nếu bạn không cần truy cập bên ngoài vào một thành viên cụ thể của lớp, nó có thể chỉ đơn giản là tạo tác triển khai và không phù hợp với việc sử dụng thực tế của lớp. Do đó, sự phức tạp nên được ẩn đi.

Trong trường hợp này, vì TBar kế thừa từ TFoo, Được bảo vệ là mức hiển thị hợp lệ, vì nó được dành riêng cho các lớp kế thừa. Ngoài ra, bởi vì TBar được thừa hưởng từ TFoo, có lẽ bạn đang nghĩ rằng nó sẽ có một số đặc quyền bổ sung cho các hoạt động bên trong của TFoo bởi vì nó là, sau khi tất cả, lớp con của nó. Tại sao chúng ta nên xuống hạng TBar để có cùng cấp truy cập thấp như các lớp khác?

Câu trả lời phụ thuộc vào việc FList là một thành viên lớp thực tế của TFoo, khi chúng ta xem xét mô hình TFoo đại diện, hoặc cho dù đó chỉ là chi tiết triển khai. Ngoài ra, mức truy cập yêu cầu là bao nhiêu? Chúng ta chỉ đơn giản là truy cập nó, hay chúng ta đang thay đổi việc thực hiện?

Tôi đoán rằng bạn không cần quyền truy cập vào FList và bạn không thay đổi việc triển khai, trong trường hợp đó, ngay cả khi hai lớp học trong cùng một đơn vị, tôi vẫn đặt FList Private over Protected .

Nếu bạn chỉ truy cập vào thành viên lớp học từ các lớp con cháu trong cùng một đơn vị, tôi sẽ vẫn giữ riêng tư. Tuy nhiên, nếu FList là thứ bạn cần ghi đè trong TBar (có thể không phải vì nó không phải là một phương thức), hoặc được thiết kế như thứ gì đó thừa hưởng các lớp nên hoặc sẽ ghi đè, cho dù nó nằm trong cùng một đơn vị hay không , sau đó bạn sẽ muốn làm cho nó được bảo vệ.

Bạn cũng sẽ cần nâng cao khả năng hiển thị thành Được bảo vệ nếu bạn cần truy cập FList từ các lớp con cháu bên ngoài cùng một đơn vị.