лучший способ обновить объект в одном из многих списков

Question

Я пытаюсь найти лучший способ обновить объект, который может быть в одном из нескольких различных списка/коллекции

Пример:

public class MyItem 
{ 
    public Guid Id { get; set; } 
    public string Name { get; private set; } 
    public string Status { get; private set; } 

    public MyItem(string name) 
    { 
     this.Id = new Guid(); 
     this.Name = name; 
    } 

    public void UpdateStatus(string status) 
    { 
     this.Status = status; 
    } 
} 

public class OtherClass 
{ 
    public ObservableCollection<MyItem> ItemList1; 
    public ObservableCollection<MyItem> ItemList2; 
    public ObservableCollection<MyItem> ItemList3; 

    public UpdateStatus(Guid id, string status) 
    { 
     // Figure out which ItemList needs to be updated 
     var item = ItemList1.FirstOrDefault(s => s.Id == id); 
     if (item == null) 
     { 
      item = ItemList2.FirstOrDefault(s => s.Id == id); 
      if (item == null) 
      { 
       item = ItemList3.FirstOrDefault(s => s.Id == id); 
       if (item == null) 
       { 
        Debug.WriteLine("Unable to update Status"); 
        return; 
       } 
      } 
     } 
     item.UpdateStatus(status); 
    } 
} 

Мне не нравится вложенный, если это так, я уверен, что есть лучший способ, но его пятница и мой мозг жарятся :(

Я понимаю, что я мог бы добавить что-то вроде этого, вместо вложенных IFS:

public MyItem UpdatedStatus(ObservableCollection collection, Guid id) 
{ 
    return collection.FirstOrDefault(s => s.Id == id); 
} 

или

public bool UpdatedStatus(ObservableCollection collection, Guid id, string status) 
{ 
    var item = collection.FirstOrDefault(s => s.Id == id); 
    if (item != null) 
    { 
     item.UpdateStatus(status); 
     return true; 
    } 
    return false; 
} 

Но мне кажется, что там должно быть лучше

Любые советы?

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

Answer 1

LINQ для помощи. Он работает для вас, как и ожидалось?

var item = ItemList1.Union(ItemList2) 
        .Union(ItemList3) 
        .FirstOrDefault(s => s.Id == id); 

Важно:

Union() возвращает уникальные элементы, это может быть медленным на больших наборов данных, поэтому рекомендуется использовать Concat() для больших списков вместо»

var item = ItemList1.Concat(ItemList2) 
        .Concat(ItemList3) 
        .FirstOrDefault(s => s.Id == id); 

Также стоит принять взглянуть на Jon Skeet's Concat() реализация от edulinq serie, Concat() действительно проста: (Full Article)

private static IEnumerable<TSource> ConcatImpl<TSource>( 
    IEnumerable<TSource> first, 
    IEnumerable<TSource> second) 
{ 
    foreach (TSource item in first) 
    { 
     yield return item; 
    } 
    foreach (TSource item in second) 
    { 
     yield return item; 
    } 
} 

Answer 2

Вы можете использовать null-coalescing operator (??), чтобы упростить код:

public UpdateStatus(Guid id, string status) 
{ 
    var item = ItemList1.FirstOrDefault(s => s.Id == id) 
     ?? ItemList2.FirstOrDefault(s => s.Id == id) 
     ?? ItemList3.FirstOrDefault(s => s.Id == id) 
    if (item == null) 
    { 
     Debug.WriteLine("Unable to update Status"); 
     return; 
    } 
    item.UpdateStatus(status); 
}