2 回答

TA贡献1784条经验 获得超9个赞
您有嵌套循环实现
// O(N) : Loop over IDSIntersections
foreach (var item in IDSIntersections)
{
// O(N) : Again, loop over ae_alignedPartners_olds
var itemOld = ae_alignedPartners_olds.First(p => p.ObjectID == item);
var itemNew = ae_alignedPartners_news.First(p => p.ObjectID == item);
...
在最坏的情况下,您将具有时间复杂性;数十亿个循环: .让我们借助字典摆脱内部循环:O(N) * O(N) = O(N**2)70k * 70k ~ 5e9
// O(N)
var dictOld = ae_alignedPartners_olds
.GroupBy(p => p.ObjectID) // ObjectID should be a int, string or provide good GetHashCode()
.ToDictionary(chunk => chunk.Key,
chunk => chunk.First());
// O(N)
var dictNew = ae_alignedPartners_news
.GroupBy(p => p.ObjectID)
.ToDictionary(chunk => chunk.Key,
chunk => chunk.First());
// O(N)
foreach (var item in IDSIntersections)
{
// O(1) : no loops when finding value by key in dictionary
var itemOld = dictOld[item];
var itemNew = dictNew[item];
...
现在我们将有关于循环:3 * O(N)3 * 70k ~ 2e5

TA贡献1877条经验 获得超6个赞
自定义会很好,但不是因为它提高了性能,它需要做同样的比较。但是因为在那里封装逻辑使其更易于维护,可读和可重用。您可以将其用于许多 LINQ 方法。IEqualityComparer<AE_AlignedPartners>
缓慢的是,您总是在 -循环中搜索旧项和新项。ObjectIdforeach
您不需要选择两者的共同点,如果您已经加入了旧的和新的,只需将整个实例存储在匿名类型中:ObjectID
var intersections = from itemNew in ae_alignedPartners_news
join itemOld in ae_alignedPartners_olds on itemNew.ObjectID equals itemOld.ObjectID
select new { New = itemNew, Old = itemOld };
foreach(var x in intersections)
{
var itemOld = x.Old;
var itemNew = x.New;
// ...
}
- 2 回答
- 0 关注
- 117 浏览
添加回答
举报