Возможно ли, чтобы метод фабрики возвращал значение null?

Question

Мне интересно узнать о лучшей практике здесь. Является ли хорошей практикой метод фабрики возвращать null, если он не может создать что-либо? Вот пример:

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

Альтернативой было бы вернуть NullCommand или что-то, я думаю, но что лучше практика?

Answer 1

Я думаю, что потенциально разумно, чтобы фабричный метод возвращал значение null в некоторых ситуациях, но не, если это метод под названием CreateCommand. Если бы это было GetCommand или FetchCommand, это может быть хорошо ... но метод Create должен бросить исключение при неудаче, я бы предложил.

Действительно ли вы действительно хотите, чтобы он вернулся null в этой ситуации, в зависимости от большей картины, конечно. (Есть ли разумная реализация нулевого объекта, которую вы могли бы вернуть, например?)

Answer 2

Я согласен с Джоном Скитом. CreateCommand ясно подразумевает строительство.

Если вы не выбрали Exception, то в этом случае я лично воспользовался бы реализацией NullCommand, чтобы избежать условных утверждений у всех потребителей и возможных ошибок NullReferenceException.

Answer 3

Возврат null в этом случае сделает ваш метод более сложным в использовании; клиенты должны знать о неявном состоянии отказа. Вместо этого, бросить исключение, и вы также можете предоставить отдельный метод для клиентов, чтобы проверить для этого условия:

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

Или сделать заводскую instantiatable; который был бы лучше, если вам нужно предварительно обработать args:

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

интерфейс завода в настоящее время делает его четким и понятным, что создание может потерпеть неудачу, но он по-прежнему требует, чтобы клиент использовать метод проверки. Другой вариант заключается в следующем:

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

Answer 4

Это имеет смысл только вернуть Null, если есть причина, почему вы хотите пользователю иметь для проверки нуль каждый раз, когда он называет Create. Как правило, вы бы рассмотреть следующие совершенно реально существующей модели использования:

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

Но что вы предлагаете, чтобы заставить следующую схему использования:

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

Обычно Null сценарий будет означать некоторые вид сбоя, и в этом случае исключение, вероятно, будет иметь наибольший смысл. Но в конечном итоге вы здесь создатель музыки и должны решить, что разумно в мире, который вы строите.