Có tệ khi trả lại ICommand mới mỗi lần trong tài sản getter?

Question

Có đáng để viết đoạn mã này:

RelayCommand _saveCommand; 
public ICommand SaveCommand 
{ 
    get 
    { 
     if (_saveCommand == null) 
     { 
      _saveCommand = new RelayCommand(this.Save); 
     } 
     return _saveCommand; 
    } 
} 

thay vì chỉ trở về đối tượng mới mỗi lần:

public ICommand SaveCommand 
{ 
    get { return new RelayCommand(this.Save); } 
} 

Từ những gì tôi biết lệnh thu khí được sử dụng khá hiếm và RelayCommand 's constructor khá nhanh. Tốt hơn là viết mã dài hơn?

Answer 1

Tôi thích null coalescing operator

public ICommand SaveCommand 
{ 
    get { return _saveCommand ?? (_saveCommand = new RelayCommand(this.Save); } 
} 

Nó trả về toán hạng bên trái nếu toán hạng không phải là null, nếu không nó sẽ trả về toán hạng phải.

Answer 2

Thiết kế này có thể gây nhầm lẫn cho người dùng lớp học của bạn. Ví dụ, họ có thể đọc giá trị của thuộc tính trong một vòng lặp với hàng ngàn lần lặp. Điều đó sẽ tạo ra nhiều đối tượng mới và người dùng có thể sẽ không mong đợi điều đó.

Xem tài liệu về cảnh báo StyleCop CA1819: Properties should not return arrays - đây là sự cố rất giống nhau.

Thông thường, người dùng sẽ không hiểu tác động bất lợi của việc gọi điện cho một thuộc tính như vậy. Cụ thể, họ có thể sử dụng thuộc tính làm thuộc tính được lập chỉ mục.

Ngoài ra, SaveCommand == SaveCommand sẽ là sai. Tôi nghĩ điều này phản trực giác.

Tóm lại, đây có thể không phải là thiết kế tốt nhất, tuy nhiên, nếu người dùng mã của bạn biết cách hoạt động và cách sử dụng đúng cách, thì không sao.

Answer 3

Có, thật tệ khi trả lại đối tượng mới mỗi lần. Tại sao làm điều đó? Nếu, vì lý do nào đó, getter đó được gọi nhiều lần, bạn sẽ tạo ra các đối tượng mới trong bộ nhớ mỗi lần. Nếu bạn làm điều đó cho người này, một cá thể bị cô lập, điều đó không quá khủng khiếp. Nhưng nếu bạn tạo thói quen lập trình theo cách này, bạn sẽ tạo ra các vấn đề khó tìm và có cơ sở mã khó duy trì. Tốt hơn là đơn giản, sạch sẽ và thanh lịch mọi lúc, và kết thúc với một cơ sở mã đẹp, sạch sẽ, dễ bảo trì.

Bằng cách này, bạn có thể luôn luôn chỉ cần khởi tạo lĩnh vực này khi bạn khai báo nó:

RelayCommand _saveCommand = new RelayCommand(this.Save); 

Sau đó getter của bạn chỉ cần này trong đó:

return _saveCommand;