循环每次迭代清空模型内容,求正确实现歌曲追加的方法
解决嵌套循环中播放列表歌曲追加被重置的问题
看起来你卡在了集合操作的两个常见坑上——每次迭代model.Songs被清空,以及用错了集合添加方法导致报错。我来帮你一步步解决:
首先排查model的生命周期问题
如果你的model是在嵌套循环内部初始化的(比如每次循环都写var model = new YourModel();),那每轮迭代都会生成一个全新的实例,之前添加的歌曲自然会被重置。解决方法很简单:把model的初始化代码移到循环外面,确保整个筛选过程都复用同一个实例。
然后修正集合追加的方法
你用model.Songs.Add(...)来添加筛选后的歌曲列表是错的——Add方法是用来添加单个元素的,而item.Songs.Where(...).ToList()返回的是一个List<Song>集合,直接用Add会因为类型不匹配报错。正确的做法是用AddRange方法,它专门用于把一个集合中的所有元素批量追加到另一个集合里。
正确实现的代码示例
// 关键:在循环外初始化model,保证全程用同一个实例 var model = new YourPlaylistModel(); // 遍历专辑组 foreach (var album in targetAlbums) { // 遍历要添加的目标歌曲 foreach (var targetSong in songsToAddToPlaylist) { // 筛选当前专辑中匹配的歌曲 var matchedSongs = album.Songs.Where(s => s.SongID == targetSong.SongID).ToList(); // 批量追加到model的歌曲列表中(先判断非空,避免空集合操作) if (matchedSongs.Any()) { model.Songs.AddRange(matchedSongs); } } }
调试验证小技巧
你可以在每次AddRange之后加一行调试输出,确认歌曲数量在累积:
Console.WriteLine($"当前播放列表歌曲总数:{model.Songs.Count}");
这样就能直观看到每轮迭代后歌曲数量是否增长,快速确认问题是否解决。
如果你的model是绑定到UI的(比如WPF、Blazor这类框架),可能还需要触发属性变更通知,但如果只是后端数据的累积,上面的代码就足够解决你的问题了。
内容的提问来源于stack exchange,提问作者Jordan Lewallen
相关产品推荐
相关产品推荐

