Конструкторы и бросающие исключения

Question

Это лучший способ прервать создание экземпляра объекта, если его параметры не переданы с действительными данными?

protected Command(string commandKey)
{
    if(commandKey == null) throw new ArgumentNullException("commandKey", "Command Key cannot be null as it is required internally by Command");
    if(commandKey == "") throw new ArgumentException("Command Key cannot be an empty string");
    CommandKey = commandKey;
}

Answer 1

Да. Обычной практикой является проверка аргументов в конструкторах и создание исключения, если они недействительны.

Answer 2

Прекрасно, если вы проверите внутри конструктора и выбросите исключение, если что-то пойдет не так.

Answer 3

Это именно то, что делает Microsoft, если вы просматриваете исходный код фреймворка, поэтому я подозреваю, что он совершенно действителен.

Answer 4

В этом случае вы можете использовать статический метод string.IsNullOrEmpty (commandKey):

protected Command(string commandKey) 
{ 
    if(string.IsNullOrEmpty(commandKey))
        throw new ArgumentException("commandKey");
    //something
} 

Answer 5

Это прекрасно. Конструкторы ничего не возвращают, так как еще вы узнали бы, что что-то пошло не так? Вы можете иметь bool для установки его в неинициализированное состояние, но я бы пошел с исключениями.

Также:

if(String.IsNullOrEmpty(commandKey)) //throw exectpion