Whats thiết kế/thực hành tốt hơn: Thuộc tính dễ vỡ hoặc thuộc tính 1 giá trị và 1 bool "có" thuộc tính?

Question

Tôi đang làm việc trên một ứng dụng ASP.NET MVC, thiết kế các mô hình miền, sử dụng (thử nghiệm) tính năng EF Code First mới.

Tôi có một thực thể Hoạt động có thể có hoặc không có Hạn chót, cách tốt nhất để tiếp cận nó là gì?

1 bất động sản:

public DateTime? Deadline {get; set;} 
and check vs null before using 

hoặc

2 thuộc tính:

public DateTime Deadline {get; set;} 
public bool HasDeadline {get; set;} 

Lúc đầu, tôi nghĩ đến các tùy chọn đầu tiên, nhưng sau đó tôi bắt đầu nghĩ rằng có lẽ tùy chọn thứ hai sẽ là tốt hơn về DB ...

Có thực tiễn nào tốt nhất liên quan đến điều này không ?

Answer 1

Tôi muốn sử dụng tùy chọn đầu tiên. Xét cho cùng, chính xác là chính xác dạng đóng gói thứ hai.

Việc đóng gói cho thấy rõ ràng rằng bạn chỉ có một giá trị hợp lý (hoặc thiếu). Ở dạng thứ hai, bạn có thể xem các thuộc tính như thể chúng hoàn toàn độc lập, chúng không hợp lý.

Về cơ sở dữ liệu, tôi cho rằng biểu mẫu đầu tiên cũng dễ dàng như vậy ... presuambly bạn sẽ có trường DATETIME rỗng trong cơ sở dữ liệu, phải không? Nó nên ánh xạ trực tiếp.

Answer 2

Tôi sẽ sử dụng tùy chọn đầu tiên. Về lâu dài, tùy chọn thứ hai có thể sẽ gây ra một số vấn đề về bảo trì vì bạn phải nhớ kiểm tra và sử dụng cả hai thuộc tính.

Cũng có một tùy chọn là sử dụng một thuộc tính nhưng thay vì làm cho nó không có giá trị, bạn có thể trả lại Null object (còn được gọi là Special Case).

Answer 3

Cơ sở dữ liệu được sử dụng để lưu trữ giá trị NULL - lưu trữ giá trị Min trong databsae và sau đó có cờ để cho biết liệu bạn có nên tin tưởng giá trị đó làm cho truy vấn phức tạp hay không.

Tôi thích các loại có thể vô hiệu do phản ánh ý định của tên miền - không có ngày, không phải 'không có ngày, vì vậy giả vờ đầu tiên của tháng 1 năm 1970 nghĩa là không có ngày'.

Ngoài ra còn có chi phí duy trì giá trị HasDealine - bạn cần phải đặt giá trị đó mỗi khi thuộc tính tương ứng được cập nhật. Ngoài ra làm thế nào để bạn xóa nó? Nếu bạn đặt Hạn chót cho một ngày, nó sẽ đặt HasDeadline thành true. Làm cách nào để 'bỏ đặt' nó? Bạn có đặt HasDeadline thành false hay không, nhưng để nguyên trường Hạn chót nguyên vẹn với giá trị trước đó?

Toàn bộ icky.

Answer 4

Bạn nên sử dụng giá trị rỗng, vì nó thực hiện chính xác những gì bạn muốn. Sử dụng hai thuộc tính riêng biệt có nghĩa là bạn mất kết nối giữa chúng và bạn cần giải thích bằng tài liệu có mối quan hệ.

Loại vô hiệu cũng phải phù hợp hơn với kiểu cơ sở dữ liệu, tuy nhiên trước tiên bạn nên thiết kế đối tượng của bạn để nó hoạt động như một đối tượng chứ không phải cách bạn lưu trữ nó trong cơ sở dữ liệu.Nếu việc sử dụng một công cụ tạo cơ sở dữ liệu khiến bạn đưa ra các quyết định tồi khi thiết kế mã, nó sẽ không hiệu quả.

Answer 5

Làm thế nào về sự kết hợp của cả hai chỉ để làm cho mã của bạn dễ đọc hơn?

public DateTime? Dealine{get; set;} 
public bool HasDeadline 
{ 
    get 
    { 
     return (Deadline != null); 
    } 
} 

Dễ đọc và thực hiện chính xác điều tương tự mà người tiêu dùng sẽ phải làm. Bên cạnh đó ...

if(HasDeadline) 
    doStuff(); 

là dễ đọc hơn

if(Dealine != null) 
    doStuff(); 

:)