Как минимизировать цикл с, если заявление

Question

Я пытаюсь свести к минимуму этот кусок кода

public static void UnfavSong(Song song) 
{ 
    List<string> favorites = FileManagement.GetFileContent_List(FAVS_FILENAME); 

    foreach (string s in favorites) 
    { 
     Song deser = SongSerializer.Deserialize(s); 
     if (deser.ID == song.ID) 
     { 
      favorites.Remove(s); 
      break; 
     } 
    } 

    FileManagement.SaveFile(FAVS_FILENAME, favorites); 
} 

Но я чувствую, что весь foreach часть может быть значительно короче. Есть ли способ в C#, чтобы сократить это до ядра?

Answer 1

вы можете использовать LINQ Where() фильтровать их:

List<string> result = favorites.Where(x=>SongSerializer.Deserialize(x).ID != song.ID).ToList(); 

Это даст вам все элементы, за исключением с согласования ID с song.ID

Answer 2

Использование LINQ

favorites.RemoveAll(s => SongSerializer.Deserialize(s).ID == song.ID) 

Btw. ваш код не должен работать вообще, поскольку вы не можете изменять список во время его итерации.