Зачем возвращать -1 вместо 0?

Question

Недавно я заметил соглашение о назначении идентификатора для определенной сущности, и мое внимание было обращено на возврат -1, если идентификатор отсутствует. Зачем возвращать -1 вместо 0?

protected long AcqAgreementID 
{    
    get 
    { 
     if(ViewState["AcqAgreementID"] != null) 
     { 
      return Convert.ToInt64(ViewState["AcqAgreementID"]); 
     } 
     else 
     { 
      return -1; 
     } 
    } 
} 

Answer 1

Часто 0 - действительное значение id или возвращаемое значение. Подумайте о контроле, который имеет индексы или поиск в строке. При поиске выделенного индекса для элемента управления, содержащего несколько элементов или при поиске индекса определенного символа в строке, возвращаемое значение 0 является абсолютно нормальным. Индекс 0 означает, что выбран первый элемент или символ найден в первой позиции в строке. В обоих случаях -1 возвращается, когда ничего не было выбрано или не найдено.

Answer 2

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

Казалось бы, разработчик этой функции предназначил абонентам проверить, является ли возвращаемое значение специальным значением дозорного значения -1. Если -1 возвращен, то ожидается, что вызывающий абонент предпримет соответствующие шаги. Например, вызывающий может показывать или записывать сообщение об ошибке.

Answer 3

В большинстве случаев вы не должны возвращать ни того, ни другого. Скорее всего, вы должны выбросить исключение.

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

Answer 4

Это простой случай magic numbers анти шаблон.

Этого не должно быть сделано. Должны использоваться константы или определения (в зависимости от языка).

Магические числа в целом; вызывают WTF и вопросы, подобные вашим.

Answer 5

Вопрос немного открытый и как-то слишком общий; на самом деле легко ответить с чем-то вроде «это зависит». В общем случае вы используете отрицательные значения, если 0 - вполне допустимое значение домена (например, в разрешенном диапазоне для идентификаторов)

Это не очень хороший шаблон, однако: он оставляет пользователей функции/API запутался (как вы). Вы должны понять, что функция возвращает числа в одном домене (положительные целые числа) и что отрицательное значение невозможно и поэтому используется как «invalud» valude.

В C# /. NET, например, вы должны учитывать типы Nullable. Они явно говорят, что «это значение не находится в домене», добавив дополнительное «значение/состояние» (null). Кроме обнуляемых типов, другие действующие соглашения, которым вы должны рассмотреть:

• сгенерирует исключение (как для int.Parse)
• добавить параметр out, чтобы указать срок действия или нет (например, Int.TryParse)

Answer 6

Я использовал этот шаблон в тех местах, где я хотел использовать целое число с нулевым значением для идентификатора. Мне не нравилось использовать 0 как «default»/«unassigned» ID, так как это значение по умолчанию для целочисленных типов. Это позволяет мне смотреть на объекты модели, чтобы определить, следует ли их вставлять или обновлять, а также могу сказать, неизвестно ли оно «0».

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

...

Как примечание, я также видел/используемые модели, где модель имела обнуляемый тип, который был colleased (??) с известным значением, такими как -1 перед вызовом в функцию/модель данных, для которой требуются типы с нулевыми значениями.Это потрясающий мир «зависит от нас», что мы всегда занимаемся проектами развития.

Answer 7

Это стандарт для методов, которые возвращают индекс в структуре .NET.

public int FindIndex( Predicate<T> match) метод возвращает отсчитываемый от нуля индекс первого вхождения элемента, который соответствует условиям, определенным матча, если найден ; в противном случае -1.

(http://msdn.microsoft.com/en-us/library/x1xzf2ca.aspx)

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

Несоблюдение типов может быть использовано вместо, но стандарт для этих методов был разработан до введения типов с нулевым значением. Если у вас есть контроль над этим кодом, вы можете изменить его, но нет ничего плохого в возврате -1. Определенно не возвращать 0, потому что 0 часто является действительным идентификатором/индексом.