司南聚合遗漏bug

SiNan.Business/AggregationBusiness.cs

@@ -810,27 +810,25 @@ namespace SiNan.Business
                 var adSkuGoi_AggregationDate_PopularizeLevel = aggregationDate_PopularizeLevelList.FirstOrDefault(x => x.Sku == adSku.Sku && x.BusinessType == adSku.BusinessType);
                 var adSkuGoi_AggregationDate_ProductLevel = aggregationDate_ProductLevelList.FirstOrDefault(x => x.Sku == adSku.Sku && x.BusinessType == adSku.BusinessType);
 
-                if (adSkuGoi_AggregationDate_PopularizeLevel != null || adSkuGoi_AggregationDate_ProductLevel != null)
+                var adSkuDailyAggregation = new AggregationJDPopularizeAdSkuDaily()
                 {
-                    var adSkuDailyAggregation = new AggregationJDPopularizeAdSkuDaily()
+                    Id = idGenerator.NewLong(),
-                    {
+                    CreateTime = DateTime.Now,
-                        Id = idGenerator.NewLong(),
+                    Date = aggregationDate,
-                        CreateTime = DateTime.Now,
+                    ShopId = shopId,
-                        Date = aggregationDate,
+                    Cost = adSkuGoi_AggregationDate_PopularizeLevel?.Cost ?? 0M,
-                        ShopId = shopId,
+                    PopularizeLevelProfit = adSkuGoi_AggregationDate_PopularizeLevel?.Profit ?? 0M,
-                        Cost = adSkuGoi_AggregationDate_PopularizeLevel?.Cost ?? 0M,
+                    PopularizeLevelGOI = adSkuGoi_AggregationDate_PopularizeLevel?.GOI ?? 0M,
-                        PopularizeLevelProfit = adSkuGoi_AggregationDate_PopularizeLevel?.Profit ?? 0M,
+                    ProductLevelProfit = adSkuGoi_AggregationDate_ProductLevel?.Profit ?? 0M,
-                        PopularizeLevelGOI = adSkuGoi_AggregationDate_PopularizeLevel?.GOI ?? 0M,
+                    ProductLevelGOI = adSkuGoi_AggregationDate_ProductLevel?.GOI ?? 0M,
-                        ProductLevelProfit = adSkuGoi_AggregationDate_ProductLevel?.Profit ?? 0M,
+                    AdGroupId = adSku.AdGroupId,
-                        ProductLevelGOI = adSkuGoi_AggregationDate_ProductLevel?.GOI ?? 0M,
+                    CampaignId = adSku.CampaignId,
-                        AdGroupId = adSku.AdGroupId,
+                    BusinessType = adSku.BusinessType,
-                        CampaignId = adSku.CampaignId,
+                    SkuId = adSku.Sku,
-                        BusinessType = adSku.BusinessType,
+                    AdId = adSku.AdId
-                        SkuId = adSku.Sku,
+                };
-                        AdId = adSku.AdId
+                insertAggregationAdSkuDailyList.Add(adSkuDailyAggregation);
-                    };
+
-                    insertAggregationAdSkuDailyList.Add(adSkuDailyAggregation);
-                }
                 #endregion
             }
 

SiNan.Business/GOIBusiness.cs

@@ -228,27 +228,28 @@ namespace SiNan.Business
             foreach (var skuId in skuIdList)
             {
                 {
-                    var cost = costs.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 2)?.Cost ?? 0M;
+                    var kuaicheCost = costs.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 2);
-                    var profit = profits.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 2)?.Profit ?? 0M;
+                    var kuaicheProfit = profits.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 2);
-
+                    if (kuaicheCost != null || kuaicheProfit != null)
-                    if (cost != 0 || profit != 0)
                     {
+                        var cost = kuaicheCost?.Cost ?? 0M;
+                        var profit = kuaicheProfit?.Profit ?? 0M;
                         var adskuGoi = new GOIByAdSku() { Sku = skuId, Cost = cost, Profit = profit, BusinessType = 2 };
                         list.Add(adskuGoi);
                     }
-
                 }
 
                 {
-                    var cost = costs.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 134217728)?.Cost ?? 0M;
+                    var zntfCost = costs.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 134217728);
-                    var profit = profits.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 134217728)?.Profit ?? 0M;
+                    var zntfProfit = profits.FirstOrDefault(x => x.Sku == skuId && x.BusinessType == 134217728);
-                    if (cost != 0 || profit != 0)
+                    if (zntfCost != null || zntfProfit != null)
                     {
+                        var cost = zntfCost?.Cost ?? 0M;
+                        var profit = zntfProfit?.Profit ?? 0M;
                         var adskuGoi = new GOIByAdSku() { Sku = skuId, Cost = cost, Profit = profit, BusinessType = 134217728 };
                         list.Add(adskuGoi);
                     }
                 }
-
             }
             return list;
         }