Спорное мнение, но на практике это один из самых частых источников багов.
Когда метод возвращает null вместо списка, ты заставляешь каждого, кто его вызывает, писать дополнительную проверку.
Иначе всё ломается на самом банальном месте:
foreach (var transaction in GetTransactions(userId))
{
// boom, если null
}
В итоге получаешь:
• лишние null-проверки повсюду
• более громоздкий код
• ошибки в рантайме, если кто-то забыл проверить
Всегда этого возвращай пустую коллекцию:
Enumerable.Empty<T>()
new List<T>()
[] в C# 12
var transactions = GetTransactions(userId)
?? Enumerable.Empty<TransactionDto>();
Теперь код становится чище и понятнее.
Ты всегда можешь итерироваться и не думаешь о null каждый раз.
Хороший API - это тот, в работе с которым ошибиться почти невозможно.
