Concat Список <type> где списки могут быть нулевыми

Question

У меня есть три списка

List<Project> intProjects = ProjectRepo.GetAllInternalProjects(); 
List<Project> extProjects = ProjectRepo.GetAllExternalProjects(); 
List<Project> mgmProjects = ProjectRepo.GetAllManagementProjects(); 
List<Project> projects = intProjects.Concat(extProjects).Concat(mgmProjects).ToList(); 

Если у меня есть пункты во всех списках он работает нормально, но я получать null value исключения, когда один из списков null.

Да, я мог бы сделать

if (extProjects != null && mgmpProjects != null && intProjects != null) 
    ... 
else if (extProjects == null && mgmpProjects != null && intProjects != null 
    ... 

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

Итак, мой вопрос: как я могу составить списки, где списки могут быть пустыми, не получая ошибку?

Answer 1

Вы могли бы сделать что-то вроде этого с помощью оператора ??:

List<Project> empty = new List<Project>(); 
List<Project> intProjects = ProjectRepo.GetAllInternalProjects() ?? empty; 
List<Project> extProjects = ProjectRepo.GetAllExternalProjects() ?? empty; 
List<Project> mgmProjects = ProjectRepo.GetAllManagementProjects() ?? empty; 
List<Project> projects = intProjects.Concat(extProjects).Concat(mgmProjects).ToList(); 

?? Operator (C# Reference)

Answer 2

Внутри вашего ProjectRepo убедитесь, что функции GetAllInternalProjects(), GetAllExternalProjects(), GetAllManagementProjects() все возвращают пустой список, если не найдено. Для функций, возвращающих списки, я инициализирую возвращаемое значение в начале функции, а затем при необходимости делаю что-то в этом списке и всегда возвращаю это значение. Таким образом, функция всегда будет возвращать объект списка, пустой или непустой. Это должно очистить ваш код.

Пример ниже:

List<object> GetAllInternalProjects(){ 
    List<object> results = new List<object>(); 

    /do something here 

    return results; 

} 

Answer 3

Не обязательно эффективно, но не требует много печатать:

List<Project> projects = new[] { intProjects, extProjects, mgmProjects } 
         .Where(list => list != null) 
         .SelectMany(_ => _) 
         .ToList(); 

Answer 4

Вы можете использовать метод расширения как этот

public static class EnumerableExtension 
{ 
    public static IEnumerable<TSource> ConcatOrSkipNull<TSource>(this IEnumerable<TSource> first, IEnumerable<TSource> second) 
    { 
     if (first == null) 
      first = new List<TSource>(); 
     if (second == null) 
      return first; 
     return first.Concat(second); 
    } 
} 

и перепишите код, а затем:

var projects = intProjects 
        .ConcatOrSkipNull(extProjects) 
        .ConcatOrSkipNull(mgmProjects) 
        .ToList(); 

Answer 5

Другой возможностью является метод расширения как

public static partial class EnumerableExtensions { 
    public static IEnumerable<T> EmptyIfNull<T>(this IEnumerable<T> source) { 
     return source ?? Enumerable.Empty<T>(); 
    } 
    } 

Таким образом, вы можете поместить

List<Project> projects = intProjects 
    .EmptyIfNull() 
    .Concat(extProjects.EmptyIfNull()) 
    .Concat(mgmProjects.EmptyIfNull()) 
    .ToList(); 

Лучше подход необходимости модифицировать GetAllInternalProjects(), GetAllExternalProjects() и ProjectRepo.GetAllManagementProjects() методы: эти методы должны всегда возвращать неnull сбор (который может быть пустой, однако)