Какое исключение выбрали из сеттера?

Question

У меня есть свойство строки, которое требует максимальной длины, потому что данные связаны с базой данных. Какое исключение следует делать, если вызывающий пытается установить строку, превышающую эту длину?

Например, это C# код:

public string MyProperty 
{ 
    get 
    { 
     return _MyBackingField; 
    } 
    set 
    { 
     if (value.Length > 100) 
      throw new FooException("MyProperty has a maximum length of 100."); 

     _MyBackingField = value; 
    } 
} 

Я считал ArgumentException, но он просто не кажется правильным. Технически, это функция - MyProperty_set(string value) - так может быть сделан случай для ArgumentException, но он не называется функцией для глаз потребителя - он находится справа от оператора присваивания.

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

Answer 1

Посмотрите через mscorlib.dll с отражателем, в подобной ситуации, такие как Microsoft System.String.StringBuilder.Capacity использовать ArgumentOutOfRangeException(), аналогичный:

public int PropertyA 
{ 
    get 
    { 
     return //etc... 
    } 
    set 
    { 
     if (condition == true) 
     { 
      throw new ArgumentOutOfRangeException("value", "/* etc... */"); 
     } 
     // ... etc 
    } 
} 

Answer 2

Вы можете использовать InvalidOperationException. Это компромисс. Я бы тоже не стал использовать исключение ArgumentException.

Answer 3

мне ArgumentException (или ребенок) имеет смысл, потому что аргумент (значение), который вы указали, недействителен, и для этого было создано ArgumentException.

Answer 4

Я бы не стал исключать вообще. Скорее я бы допустил строку любой длины и затем имел отдельный метод «Validate» в классе, который вызывается перед сохранением. Существует несколько сценариев, особенно если вы используете привязку данных, когда бросание исключений из настроек свойств может привести вас в беспорядок.

Проблема с отбрасыванием исключений со стороны разработчиков недвижимости заключается в том, что программисты забывают их поймать. Это зависит от того, насколько вы чисты, какие данные вы получаете. В этом случае я ожидал бы, что длинные длины строк являются общими не исключительными, и поэтому использование исключения будет «управлением потоком с исключениями».

процитировать от Microsoft Design Guidelines for Developing Class Libraries:

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

Answer 5

Помните, сколько проблем в информатике разрешено путем добавления дополнительного уровня косвенности?

Один из подходов - создать новый тип FixedLengthString. Это были бы случаи , которые типа, которые проверяют длины строк, которые они инициализируются, с помощью оператора преобразования для преобразования типов из простой строки.Если ваш установщик свойств принимает такой тип, как его аргумент, любое нарушение станет исключением преобразования типа вместо исключения аргумента/свойства.

На практике я бы редко это делал. Это немного пахнет OO слишком далеко, но в некоторых случаях это может быть полезной техникой, поэтому я упомянул об этом здесь для полноты.

Answer 6

Попробуйте использовать существующие исключения, если возможно. В этом случае используйте InvalidOperationException, потому что входящее значение приводит объект в несогласованное состояние. Пользовательские исключения могут быть созданы, когда требуется конкретная обработка с настраиваемым исключением. В этом случае вы генерируете исключение только с некоторым текстом, поэтому используйте InvalidOperationException.

При метании InvalidOperationException отображается значение, переданное этому установщику.

Answer 7

public IPAddress Address 
{ 
    get 
    { 
     return address; 
    } 
    set 
    { 
     if(value == null) 
     { 
      throw new ArgumentNullException("value"); 
     } 
     address = value; 
    } 
} 

с помощью MSDN