Каков наилучший способ итерации объектов с помощью linq

Question

           var AddQuery = from newdist in newSpec.Distributions
               where !( from oldDist in oSpec.Distributions
               select oldDist.Id).Contains( newdist.Id )
               select newdist;

          foreach (var dist in AddQuery)
          {
            dist.operation="Add";
          }
          var updateQuery = from oldDistributionForUpdate in ospec.Distributions
                            where (from newDistributionForUpdate in newspec.Distributions
                            select newDistributionForUpdate.DistributionId).Contains(oldDistributionForUpdate.DistributionId)
                            &&
                            (from newDistributionForUpdate in newpaymentSpecification.Distributions
                            select newDistributionForUpdate.Amount).Equals(oldDistributionForUpdate.Amount)
                            select oldDistributionForUpdate;

          foreach (var dist in updateDistQuery)
          {
            dist.operation = "Update";
          }

Я планирую собрать коллекцию объектов из результатов запроса и обработать их. Есть ли более простой способ добиться того, что я делаю?

Answer 1

В исходном фрагменте вы соответствуете Id для AddQuery и DistributionId для UpdateQuery.Если это опечатка и она должна быть Id или DistributionId для обоих случаев, то вы можете обрабатывать добавление и обновление намного эффективнее в одном цикле.

var AddUpdateQuery = from newdist in newSpec.Distributions
                         join oldDist in oSpec.Distributions
                         on newdist.Id equals oldDist.Id into matchingDistributions
                         select new { newdist, matchingDistributions };

    foreach (var dist in AddUpdateQuery)
    {
        if (!dist.matchingDistributions.Any())
        {
            dist.newdist.operation = "Add";
            //additional processing for "Add" operation.
        }
        else if (dist.newdist.Amount == dist.matchingDistributions.First().Amount)
        {
            dist.newdist.operation = "Update";
            //additional processing for "Update" operation.
        }
    }