一个一对多的关系，带孩子的父母

Question

public void getForm(string scode) {    
Form result = DBContext.Forms.Where(f => f.Code == fCode && f.SCode == sCode).FirstOrDefault();

    result.Products = result.Products.Where(p => p.Deleted== false).ToList(); // commenting this line fix the problem
return result;
}

我们如何将两行以上合并在一起以避免出现以下错误。

操作失败：由于一个或多个外键属性不可为空，因此无法更改该关系。 对关系进行更改时，相关的外键属性将设置为空值。 如果外键不支持空值，则必须定义新的关系，必须为外键属性分配另一个非空值，或者必须删除不相关的对象。

更新：

当在另一个函数中调用时，它抛出错误

public void savetrans(string fcode)
{
                    Form form = GetForm(fCode);
                    var transDb = new DbContext.Data.Transaction()
                    {
                        FId = form.FId,
                        field1= "test",
                        field2= "test",
                        field3= "test",
                    };

                    DbContext.Transactions.Add(transactionDb);

                    DbContext.SaveChanges();
}

Answer 1

如果要删除标记为已删除的产品：

var deletedProducts = result.Products.Where(x => x.Deleted).ToList();
foreach(var deletedProduct in deletedProducts)
  result.Products.Remove(deletedProduct);

如果Form.Products是List<Product>则可以使用.RemoveAll(x => x.Deleted)

如果您只想在与实体合作时排除已删除的产品，则建议为此目的使用未映射的属性：

public class Form
{
   public ICollection<Product> Products { get; set; } = new List<Product>();

   [NotMapped]
   public IReadOnlyCollection<Product> ActiveProducts => Products.Where(x => !x.Deleted).ToList().AsReadOnly();
   // or
   public IReadOnlyCollection<Product> ActiveProducts
   {
     get { return Products.Where(x => !x.Deleted).ToList().AsReadOnly();
   }
}

然后，当您只想使用活动产品时，请使用.ActiveProducts 。 这种方法的警告是您不能在EF Linq表达式中使用此属性，而只能以只读方式使用。 例如，请勿尝试以下操作：

var products = context.Forms.Where(x => x.FormId == formId).SelectMany(x => x.ActiveProducts);

这将出错，因为EF不会映射ActiveProducts。 您必须将.Products与“已删除”的相应过滤器一起使用。

我通常会在填充视图模型时而不是在实体级别上放置诸如处理活动/非活动之类的逻辑。 实体应反映数据模型，而适用于您如何查看/与该逻辑交互的业务逻辑则由视图模型表示。 使用视图模型的好处是，填充视图模型后，只需检查一次“活动/已删除”状态，它们就不会使用在某些用途中无效的属性来毒害实体。

Answer 2

显然，您的数据库具有Forms和Products 。 Forms和Products之间存在一对多的关系：每个Form都有零个或多个Products ，每个Product使用外键（可能是Product.FormId ）完全属于一个Form

您的第一条语句将提取您的一个满足某些要求的Forms ，如果没有这样的Form ，则为null。 在不检查null返回值的情况下，您尝试更改此Form的Products 。

问题是，可能有多个Products对此Form具有不可为空的外键。 它们是表单Products的ICollection中的项目。 如果您分配一个新的ICollection来Form.Products ，实体框架想要设置的外键Products即是在Form.Products为零，表明该产品不属于任何形式了。 但是在模型描述中，您定义了每个Product应完全属于一个Form 。 因此，错误。

因此，您应该做什么取决于您对程序的要求。 您是否只想获取包含其未删除产品的表单 ，然后应该执行查询。 如果要从数据库中删除此表单中所有已删除的产品 ，则应执行更新

查询某个表格及其未删除的产品

public ICollection<Form> GetFormWithNonDeletedProducts(string scode)
{
    using (var dbContext = new MyDbContext(...))
    {
        return dBContext.Forms                      // from the collection of all Forms
            .Where(form => form.Code == fCode      
                    && form.SCode == sCode)         // keep only the ones that I want
            .Select(form => new Form()              // and create a new Form object
            {                                       // with the properties I plan to use
                 Id = form.Id,
                 Name = form.Name,
                 ...
                 Products = form.Products           // Fetch only the non-deleted products
                     .Where(product => !product.Deleted)
                     .ToList(),
            }
            .FirstOrDefault();
        }
    }
}

我需要创建一个新的Product对象的唯一原因是因为我想将其放入返回值中。 如果不需要返回值，则可以将获取的属性放入匿名对象。 这通常会更有效，因为您不会从数据库中获取不会使用的数据。

例如，上面的示例将分配Product.FormId 。 你不需要它，becasue你知道，这种形式的所有千的产品将具有相同的值FormId：即价值Form.Id 。

相同的查询但未获取您不使用的属性（匿名类型）

using (var dbContext = new MyDbContext(...))
{
    return dBContext.Forms                  // from the collection of all Forms
        .Where(form => form.Code == fCode      
            && form.SCode == sCode)         // keep only the ones that I want
        .Select(form => new                 // and create a new Form object
        {                                   // with the properties I plan to use
                 Id = form.Id,
                 Name = form.Name,
                 ...
                 Products = form.Products   // Fetch only the non-deleted products
                     .Where(product => !product.Deleted)
                     .Select(product => new
                      {                     // Select only properties you plan to use
                          Name = product.Name,
                          Price = product.Price,

                          // not meaningful: you already know the value:
                          // FormId = product.FormId,
                      })
                     .ToList(),
        }
        .FirstOrDefault();
}

更新数据库：删除表单中已删除的产品

尽管您的函数称为GetForms但是您似乎想使用它来删除已删除的产品。

最简单的方法是使用DbSet<Products>.RemoveRange ：

using (var dbContext = new MyDbContext(...))
{
    // remove all Deleted products of the form with Code equal to fCode
    // and SCode equal to sCode
    var productsToDelete = dbContext.Products
        .Where(product => product.Deleted
            && product.Form.Code == fCode
            && product.Form.Scode == sCode);
    // note: query is not executed yet!

    dbContext.Products.RemoveRange(productsToDelete);
    dbContext.SaveChanges();
}

可能是您必须要做RemoveRange(productsToDelete.ToList()) ，您必须检查一下。