Có phải biểu mẫu không đúng để chỉ một loại có nguồn gốc trong một loại cơ sở không?

Question

Tôi có cấu trúc cây thể loại/tệp. Cả hai loại và tệp đều có thể có cha mẹ, vì vậy tôi bắt nguồn từ một lớp cơ sở chung có thuộc tính Gốc. Vì tất cả các bậc cha mẹ rõ ràng sẽ luôn là các loại (các tệp không thể là cha mẹ), có vẻ như có ý nghĩa để làm cho thuộc tính Gốc của nút là loại CategoryNode.

Biểu mẫu xấu cho lớp cơ sở có đề cập đến lớp dẫn xuất không? Nếu vậy, tại sao? Cách tốt hơn để cấu trúc cái này là gì?

class Node { 
    public CategoryNode Parent {get; set;} 
} 

class File : Node { 
    ... 
} 

class CategoryNode : Node { 
    ... 
} 

Answer 1

Nếu thuộc tính Parent là thực sự là thuộc tính chung của tất cả các hậu duệ và luôn luôn là loại CategoryNode, nó không phải là vấn đề. Về mặt ngữ nghĩa nói nó đúng và về mặt kỹ thuật, tôi nghĩ rằng nó chính xác quá sớm khi bạn vẫn còn trong cùng một thư viện (để tránh tham khảo vòng tròn).

Đây có thể là một vấn đề khi bạn viết mã như thế này:

// BAD CODE 
if(myProp is subclassA) 
{ ... 
} 
else if (myProp is syubclassB) 
{ ... 
} 

Mã này là xấu, bởi vì bạn mất lợi thế của việc thừa kế.

Ngay cả trong .Net Framework cũng có cấu trúc như vậy. Ví dụ đầu tiên xuất hiện trong tâm trí của tôi là thuộc tính XObject.Parent.

XElement kế thừa XObject và XObject xuất bản thuộc tính kiểu XElement. Giống như đoạn mã của bạn.

Answer 2

Lớp cơ sở không được biết ai xuất phát từ nó.

Nếu bạn có trường hợp như vậy, có thể bạn không muốn kế thừa. Bạn chỉ nên sử dụng một số hình thức khớp nối.

Tệp và danh mục phải giữ thành viên nút trong trường hợp của bạn.

Answer 3

Các tùy chọn khác sẽ là thay đổi thứ bậc lớp để làm cho CategoryNode thành lớp gốc (a) hoặc thay đổi loại thuộc tính thành Nút (b).

Cả hai khả năng này đều không tốt: (a) Tệp sẽ có tất cả chức năng CategoryNode mà nó không cần. (b) Nó sẽ ẩn loại đối tượng (luôn luôn là CategoryNode). Điều này có thể xảy ra với các lỗi truyền không hợp lệ ở một nơi khác trong mã của bạn. Ví dụ, nếu bạn quên luôn luôn có một thể loại CategoryNode.

Xem xét điều này, tôi tin rằng mã hiện tại là OK.

Answer 4

Bạn có thể làm ...

interface IParent { 
    ... 
} 

class Node { 
    public IParent Parent {get; set;} 
} 

class File : Node { 
    ... 
} 

class CategoryNode : Node, IParent { 
    ... 
} 

Bằng cách này, bạn không cần phải đề cập đến một đối tượng có nguồn gốc trong lớp cơ sở, cộng thêm, bạn linh hoạt hơn trong những gì thực sự có thể trở thành cha mẹ , trong trường hợp bạn nhận được các loại đối tượng bổ sung tại một thời điểm sau đó. Ngoài ra, bất kỳ chức năng nào có liên quan chỉ dành cho cha/mẹ đều có thể được khai báo trong giao diện đó.