Если возвращаемый тип — коллекция, что лучше: возвращать null
или пустую коллекцию? Есть ли общепринятая практика?
Аналогичный вопрос на en.SO: Is it better to return null or empty collection?
Stack Overflow на русском — это сайт вопросов и ответов для программистов. Присоединяйтесь! Регистрация займёт не больше минуты.
Присоединиться к сообществуЕсли возвращаемый тип — коллекция, что лучше: возвращать null
или пустую коллекцию? Есть ли общепринятая практика?
Аналогичный вопрос на en.SO: Is it better to return null or empty collection?
Вопрос здесь в семантике.
На запрос «дайте мне список всех пользователей» пустая коллекция имеет очевидный смысл: «окей, вот вам список всех пользователей, их ровно 0 штук».
С другой стороны, возвращённый null
может означать что угодно: «я не знаю, сколько пользователей», «количество пользователей ещё не подсчитано», «текущая база данных вообще не имеет понятия _пользователь_», что угодно.
Смысл пустой коллекции всегда ясен, смысл null
не определён.
Статья по теме: Eric Lippert, Null Is Not Empty.
Лучше возвращать пустые коллекции - так вы избавите пользователя от необходимости писать утомительный код проверок на null и, соответственно, избавите его от энного количества NullReferenceException
, если он таковой код написать забыл. Хорошим примером могут быть случаи, когда коллекции задействованы в цепочках методов вида
var res = FirstMethod().SecondMethod().ThirdMethod();
и в случае, если коллекция равна null
, вся цепочка может быть обрушена исключением, тогда как проверки на null
превратят этот лаконичный и довольно элегантный синтаксис в кашу из вложенных if
-ов. Для коллекций это можно считать общепринятой практикой.
Кстати, обратите внимание, что в LINQ именно так и делается. Например, следующий код вполне нормально работает без выбрасывания исключений, хотя очевидно, что ни один из элементов коллекции не удовлетворяет условию лямбды в Where
:
var res2 = new[] { 1, 2, 3 }.Where(x => x > 10).Select(x => x);
Пустую коллекцию легче использовать и код получается менее подвержен ошибкам.
Но я не согласен с мнением, что нужно всегда возвращать пустую коллекцию.
Создание любого объекта означает потребление ресурсов: памяти и процессора. Рано или поздно этот объект (коллекция) будет удалён - это работа сборщика мусора, то есть опять потребление ресурсов.
В случае, если нужна производительность, придётся отказаться от создания ненужных объектов (пустых коллекций). В частности, разрабочики Resharper в своих докладах много рассказывали о memory traffic и как они с ним боролись. В частности, там используются хитрые коллекции, в которых несколько значений хранятся в полях класса. И лишь когда их становится много - выделяется память под полноценную коллекцию в куче.
Enumerable.Empty
позволяет не расходовать память на создание "пустых коллекций".
Enumerable.Empty
- хороший выход. Однако, там всё равно создаётся экземпляр массива нулевого размера (заглянул сейчас в Reference Source) и кэшируется.
2 дек 2015 в 10:21
new List<T>()
Всегда когда есть некостыльная альтернатива возвращению null
, нужно пользоваться этой альтернативой. Меньше проблем потом будет.
Так что конечно лучше возвращать пустую коллекцию.
Если не хотите ловить исключение типа, связанное с тем, что значение отсутсвует - то лучше пустую коллекцию. А вообще нужно исходить из поставленной задачи. Если метод должен возвращать список каких-то элементов, то в случае отсутствия элементов - должен быть возвращен пустой список. Значение null должно возвращаться в особых случаях, когда это заранее оговорено и поставлена задача.
Лучше возвращать пустую коллекцию.
Совершенно не хочется писать код вроде этого:
if (container.ItemsProperty != null)
{
foreach (var item in container.ItemsProperty)
// Ужас!
}
Лучше возвратите пустую коллекцию коллекцию/IEnumerable, это общепринятая практика.
Это поможет избежать вышеупомянутого безумия, и так ваши коллеги и пользователи вашего класса не проколят шины вашего автомобиля.
Если говорить о свойствах, присвойте и забудьте:
public IEnumerable<Item> Items { get; private set; }
public Container() { Items = Enumerable.Empty<Item>(); }
Если это метод - то вернуть пустую коллекцию легко:
public IEnumerable<Song> GetListOfMyFavoriteEmoSongs()
{
return Enumerable.Empty<Song>();
}
Enumerable.Empty<T>()
немного эффективнее чем, например, создавать новый пустой массив, или делать yeild break
.
Это перевод ответа с en.SO: https://stackoverflow.com/a/1970001/2790048