我有一个 (PatchFacilityManager) 列表和一个 (Int) facilityManagerId 的列表。我想让下面的代码更加高效。是否有任何方法可以删除这两个 foreach 循环。
foreach (PatchFacilityManager PM in patchFacilityManager)
{
foreach (int FM in facilityManagerId)
{
if (PM.FacilityManagerId == FM)
{
PM.IsSelected = true;
}
}
}
以下是一种方法：
foreach (PatchFacilityManager PM in patchFacilityManager)
{
PM.IsSelected = facilityManagerId.Contains(PM.FacilityManagerId);
}
我认为，与问题中给出的代码相比，这个解决方案在三个方面上更有效率。
首先，它不会测试条件，表达式的结果直接赋值给PM.IsSelected。根据LukeH的评论，强制不将PM.IsSelected设置为false，因此条件是不可避免的。但是，如果要将其设置为false，则适用此改进。 从问题提问者的评论中，他的情况似乎符合这种优化。 因此不需要条件赋值。
第二，它不会迭代整个列表，因为List.Contains(int)会在参数传递的int第一次出现时返回true并退出循环。
第三，当框架为您提供List.Contains(int)功能时，为什么要重新发明轮子呢。因此，从维护的角度来看，这也更有效率。
.Contains调用中使用new HashSet<int>(facilityManagerId)。这样做有问题吗？ - JenspatchFacilityManager
.Where(c => facilityManagerId.Contains(c.FacilityManagerId))
.ForEach(c => c.IsSelected = true);
IEnumerable<T> 没有内置的 ForEach 方法，而你的 Where 调用就是返回了该类型。ForEach 是 List<T> 上的实例方法，所以你需要先调用 ToList，但这并不会提高效率！ - LukeHvar ids = new HashSet<int>(facilityManagerId);
foreach (PatchFacilityManager pfm in patchFacilityManager)
{
if (ids.Contains(pfm.FacilityManagerId))
pfm.IsSelected = true;
}
patchFacilityManager
.Where(m => facilityManagerId.Contains(m.FacilityManagerId))
.ToList()
.ForEach(m => m.IsSelected = true);
或者
patchFacilityManager
.Join(facilityManagerId, m => m.FacilityManagerId, f => f, (m,f) => m)
.ToList()
.ForEach(m => m.IsSelected = true);
使用LINQ语法的另一种变体：
var match = for PM in patchFacilityManager
join FM in facilityManager on PM.FacilityManagerId equals FM
select PM;
foreach(var PM in match)
{
PM.IsSelected = true;
}
只是为了好玩和超越常规——当列表已排序时的一种方法：
static void Main(string[] args)
{
List<int> nums = new List<int>(new int[] { 1, 2, 3, 4, 5, 6, 7, 8 });
List<int> ids = new List<int>(new int[] { 2, 4, 5 });
for (int i = 0, j = 0; i < nums.Count && j < ids.Count; i++)
{
int num = nums[i];
int id = ids[j];
if (num == id)
{
Console.WriteLine("Match = " + id);
j++;
}
}
Console.ReadLine();
}
我假设您对这个想法的性能优劣没有任何了解。当然，如果您不喜欢使用for循环，您可以修改它以使用主foreach来处理数字，并手动使用IDs的枚举器在主foreach内部。
BinarySearch进行查找，而不是使用foreach。