Có OK cho phương thức factory để trả về null không?

Question

Tôi đang tự hỏi về thực hành tốt nhất ở đây. Có thực hành tốt cho phương pháp nhà máy để trả về null nếu nó không thể tạo ra bất kỳ thứ gì? Dưới đây là ví dụ:

ICommand command = CommandFactory.CreateCommand(args); 
if (command != null) 
    command.Execute(); 
else 
    // do something else if there is no command 

Một cách khác là trả lại NullCommand hoặc một điều gì đó, tôi đoán, nhưng cách tốt nhất là gì?

Answer 1

Tôi nghĩ rằng có khả năng hợp lý để phương thức nhà máy trả về giá trị rỗng trong một số trường hợp, nhưng không phải nếu đó là phương thức được gọi là CreateCommand. Nếu đó là GetCommand hoặc FetchCommand, điều đó có thể không sao ... nhưng phương pháp Create sẽ đưa ra một ngoại lệ về lỗi, tôi sẽ đề xuất.

Dù bạn thực sự muốn trả lại null trong trường hợp này tùy thuộc vào bức tranh lớn hơn, tất nhiên. (Có thực hiện đối tượng null hợp lý mà bạn có thể trả lại không, ví dụ?)

Answer 2

Tôi đồng ý với Jon Skeet. CreateCommand ngụ ý xây dựng rõ ràng.

Nếu bạn không ném Exception, thì trong trường hợp đó, tôi sẽ đích thân thực hiện triển khai NullCommand, để tránh báo cáo có điều kiện ở tất cả người tiêu dùng và có thể xảy ra lỗi NullReferenceException.

Answer 3

Trả lại null trong trường hợp này sẽ làm cho phương pháp của bạn khó sử dụng hơn; khách hàng phải nhận thức được tình trạng lỗi ngầm. Thay vào đó, hãy ném ngoại lệ và bạn cũng có thể cung cấp phương pháp riêng để khách hàng thử nghiệm cho điều kiện này:

if (CommandFactory.CanCreate(args)) { 
    ICommand command = CommandFactory.Create(args); 
    command.Execute(); 
} 

Hoặc làm cho nhà máy có thể khởi chạy; đó sẽ là tốt hơn nếu bạn cần phải args quá trình trước:

CommandFactory factory = new CommandFactory(args); 
if (factory.IsValid()) { 
    ICommand command = factory.Create(); 
    command.Execute(); 
} 

Giao diện của nhà máy hiện nay làm cho nó rõ ràng và rõ ràng rằng sự sáng tạo có thể thất bại, nhưng nó vẫn đòi hỏi khách hàng để sử dụng phương pháp kiểm tra. Một lựa chọn khác là thế này:

ICommand command; 
if (CommandFactory.TryCreate(args, out command)) { 
    // creation succeeded ... 
} 

Answer 4

Nó chỉ có ý nghĩa để trở Null nếu có một lý do tại sao bạn sẽ muốn người dùng phải kiểm tra cho null mỗi lần ông gọi là Tạo. Thông thường bạn sẽ xem xét sau một mô hình sử dụng hoàn toàn hợp lệ:

var obj = MyFactory.CreateThing(); 
obj.DoSomething(); 

Nhưng những gì bạn đang đề xuất là để buộc các mô hình sử dụng sau đây:

var obj = MyFactory.CreateThing(); 
if (obj == Null) { 
    // Handle null condition 
} else { 
    obj.DoSomething(); 
} 

Thông thường Null kịch bản sẽ có nghĩa là một số loại thất bại, trong trường hợp đó một ngoại lệ có lẽ sẽ có ý nghĩa nhất. Nhưng cuối cùng bạn là nhà sản xuất âm nhạc ở đây và phải quyết định điều gì là hợp lý trong thế giới bạn đang xây dựng.