在 foreach 循环 c# 中使用 savechanges()

Posted 2023-02-16

【中文标题】在 foreach 循环 c# 中使用 savechanges()

【英文标题】：Using savechanges() inside foreach loop c#

【发布时间】：2020-02-06 05:08:20

【问题描述】：

我有下面的这个方法，它将数据写入数据库中的两个表。我需要在 foreach 部分中将一个集合写入数据库。为什么 saveChanges 在循环的每次迭代中都不起作用，有更好的方法吗？ 有两件事......第一：函数不返回任何东西....第二：数据库没有更新....当我从函数中删除 savechange() 时，它可以工作但没有改变值在数据库中。

if (_context.ProductPins.Where(a => a.ProductId == id && a.Status==false).Count() > 0)
        
            var c = 0;
            var ProductPins = _context.ProductPins.Where(a=>a.Status==false && a.ProductId == id);
            foreach(var item in ProductPins)
            
                ApplicationUser.Balance = ApplicationUser.Balance - product.BuyingPrice;
                item.Equals(true);
                _context.Statements.Add(new Statement
                
                    Amount = product.BuyingPrice,
                    RecordDate = DateTime.Now,
                    Destination = false,
                    FromApplicationUserId = _userManager.GetUserId(User),
                    //ToApplicationUserId = nu,
                    BalanceType = BalanceType.currentbalance,
                );

                _context.Update(ApplicationUser);
                _context.Update(item);
                _context.SaveChanges();
                c++;
                if (c > count)
                
                    break;
                





            

【问题讨论】：

你在使用实体框架吗？我知道 Dapper 允许您实际执行基于集合的插入，但看起来实体框架是正确的吗？

OP，请说明“不起作用”的含义。你有错误吗？例外？没有错误但数据没有保存？数据已保存但不正确？或者它是否符合规范，但您只是不喜欢代码结构？

@Greg 是的，我正在使用实体框架

@JohnWu 有两件事……第一：函数没有返回任何东西……第二：数据库不是最新的……当我从中删除 savechange()它具有的功能可以工作，但不会更改数据库中的值

@محمدالعاني - 如果您将 sql server 作为后端运行，您可以使用 sql profiler 检查吗？ SaveChanges() 应该返回显示已保存记录数的 int。 （附带说明，您可以将 _context.ProductPins.Where(a => a.ProductId == id && a.Status==false).Count() > 0 更改为 _context.ProductPins.Any(a => a.ProductId == id && a.Status==false) 并将 _userManager.GetUserId(User) 置于循环之外）。

【参考方案1】：

我只把 savechange() 方法放在了 froreach 循环之外....savechanges() 不应该在循环中

if (_context.ProductPins.Where(a => a.ProductId == id && a.Status==false).Count() > 0)
        
            var c = 0;
            var ProductPins = _context.ProductPins.Where(a=>a.Status==false && a.ProductId == id);
            foreach(var item in ProductPins)
            
                ApplicationUser.Balance = ApplicationUser.Balance - product.BuyingPrice;
                item.Equals(true);
                _context.Statements.Add(new Statement
                
                    Amount = product.BuyingPrice,
                    RecordDate = DateTime.Now,
                    Destination = false,
                    FromApplicationUserId = _userManager.GetUserId(User),
                    //ToApplicationUserId = nu,
                    BalanceType = BalanceType.currentbalance,
                );

                _context.Update(ApplicationUser);
                _context.Update(item);

                c++;
                if (c > count)
                
                    break;
                





            
_context.SaveChanges();

【参考方案2】：

如果您要保存到数据库，您应该将这些更改排队，以便您可以在最后进行一次写入并一次处理所有这些，而不是一次处理一次，这样您就可以利用数据库内部写入处理。