У мене є фрагмент коду, який можна представити як:
public class ItemService {
public void DeleteItems(IEnumerable<Item> items)
{
// Save us from possible NullReferenceException below.
if(items == null)
return;
foreach(var item in items)
{
// For the purpose of this example, lets say I have to iterate over them.
// Go to database and delete them.
}
}
}Тепер мені цікаво, чи це правильний підхід чи варто кинути виняток. Я можу уникати винятку, тому що повернення було б таким самим, як ітерація над порожньою колекцією, тобто жоден важливий код не виконується, але з іншого боку я, можливо, приховую проблеми десь у коді, бо чому б хто хотів зателефонувати DeleteItemsз nullпараметром? Це може вказувати на те, що в коді є проблема десь ще.
Ця проблема, як правило, у мене з методами в сервісах, тому що більшість з них щось робить і не повертає результат, тому якщо хтось передає недійсну інформацію, то сервісу нічого не робити, тому вона повертається.
2
Це лише моя думка, але я завжди вважав, що має найбільш сенс, що метод повинен викидати виняток, коли він робить щось ненавмисне (виняткове). У вашому випадку я б кинув InvalidOperationException, якби хтось спробував видалити null / 0 елементів.
—
Фальгантіль
@gnat Моє запитання стосується конкретно послуг, через те, як вони використовуються та їх призначення. Цей "дублікат" говорить лише про загальні випадки, і головна відповідь навіть суперечить собі в контексті мого точного методу.
—
FCin
Чи можливо, що нумерація буде порожньою, а не нульовою? Чи поводилися б ви з цим по-іншому?
—
JAD
@Falgantil Я можу побачити, що null заслуговує на виняток, але я вважаю, що недоцільно кидати виняток у порожній список.
—
Кевін