如何实现在EF核心中添加相关数据的方法？

问题描述 投票：0回答：1

如何实现在EF core中添加相关数据的方法？需要将哪些对象提交给add方法以及它应该返回什么？我需要在DB中添加相关数据，例如：

 {
        "productId": 0,
        "number": "xxx",
        "amount": 5.65,
        "primeCost": 20.33,
        "productTypeId": 0,
        "parameters": [
            {
                "id": 0,
                "name": "Type",
                "value": null
            },
            {
                "id": 3,
                "name": "steel grade",
                "value": "CK45"
            },
            {
                "id": 4,
                "name": "diameter",
                "value": "40"
            }
        ]
    }

这些是我的模型类：

public class Product //: BaseObject
{
    public int Id { get; set; }

    public string Name { get; set; }
    public string Number { get; set; }
    public double Amount { get; set; }
    public double PrimeCost { get; set; }

    [ForeignKey("ProductTypeId")]
    public int  ProductTypeId {  get; set; }
    public virtual ProductType ProductType { get; set; }

    public ICollection<ProductParameter> ProductParameters { get; set; } = new List<ProductParameter>();
}
public class ProductType //: BaseObject
{   
    public int Id { get; set; }

    public string NameType { get; set; }

    public ICollection<Parameter> Parameters { get; set; } = new List<Parameter>();
    public ICollection<Product> Products { get; set; } = new List<Product>();
}
 public class Parameter //: BaseObject
{
    public int Id { get; set; }
    public string Name { get; set; }

    [ForeignKey("ProductTypeId")]
    public int ProductTypeId { get; set; }
    public ProductType ProductType { get; set; }

    public ICollection<ProductParameter> ProductParameters { get; set; } = new List<ProductParameter>();
}
 public class ProductParameter //: BaseObject
{
    public int Id { get; set; }

    public int ProductId { get; set; }
    public virtual Product Product { get; set; }

    public int ParameterId { get; set; }
    public virtual Parameter Parameter { get; set; }


    public string Value { get; set; }

}

这些是我的DTO课程：

public class ProductDTO
{   

    public int ProductId { get; set; }
    public string Number { get; set; }
    public double Amount { get; set; }
    public double PrimeCostEUR { get; set; }

    public int ProductTypeId { get; set; }
    public string NameType { get; set; }

    public ICollection<ParameterDTO> Parameters { get; set; } = new List<ParameterDTO>();
}
public class ParameterDTO
{
    public int Id { get; set; }
    public string Name { get; set; }
    public string Value { get; set; }

}

我在DB中添加相关数据的方法的实现：

public async Task<IEnumerable<ProductDTO>> AddProducts(ProductDTO ProductDTO, 
        List<ParameterDTO> ParameterDTO)
    {
        var EntryProduct = await _context.Products.FindAsync(ProductDTO.ProductId);

        if (EntryProduct == null)
        {

            _context.Products.Add(new Product
            {
                Id = ProductDTO.ProductId,
                Number = ProductDTO.Number,
                Amount = ProductDTO.Amount,
                PrimeCostEUR = ProductDTO.PrimeCostEUR,
            });
            _context.SaveChanges();

            foreach (var i in ParameterDTO)
            {
                var EntryParameter = await _context.Parameters.FindAsync(i.Id);
                if (EntryParameter != null)
                {
                    _context.ProductParameters.Add(
                      new ProductParameter
                      {
                          ProductId = ProductDTO.ProductId,
                          ParameterId = i.Id,
                          Value = i.Value
                      });
                    _context.SaveChanges();
                }
            }

        }

        return ProductDTO;
    }

我得到以下异常编译器错误：

严重级代码描述项目文件行抑制状态错误CS0266无法将类型“GdmStore.DTO.ProductDTO”隐式转换为“System.Collections.Generic.IEnumerable”。存在显式转换（您是否错过了演员？）

c# entity-framework-core linq-to-entities
1个回答
1
投票

您的方法期望返回IEnumerable，但您只返回传入的单个Product DTO。

签名应该是：

public async Task<ProductDTO> AddProducts(ProductDTO ProductDTO, List<ParameterDTO> ParameterDTO)

鉴于ProductDTO有一个ParameterDTO集合，还需要第二个参数吗？ （看起来它会发送两次参数）

通过实体定义，我发现了一些问题：

[ForeignKey("ProductTypeId")]
public int  ProductTypeId {  get; set; }
public virtual ProductType ProductType { get; set; }

应该

[ForeignKey("ProductType")] // FK to the reference property.
public int  ProductTypeId {  get; set; }
public virtual ProductType ProductType { get; set; }

所有导航属性（如集合和producttype）都应声明为虚拟，否则您将获得不一致的行为。如果需要，声明虚拟的那些将具有访问延迟加载的权限，其他将被保留为#null。

Product和Parameter都不应该引用ProductType，从我可以看到它可能只是在Product上以避免非规范化问题。 （设置了具有不同ProductTypes的参数的产品。）

在处理导航属性时，我建议从实体中删除FK属性并使用映射（EF6）/阴影属性。 （EF核心）

例如：

 public class ProductParameter 
{
    public int Id { get; set; }

    public virtual Product Product { get; set; } // No ProductId/ParameterId
    public virtual Parameter Parameter { get; set; }

    public string Value { get; set; }
}

modelBuilder.Entity<Product>()
    .HasMany(x => x.ProductParameters)
    .WithOne(x => x.Product)
    .HasForeignKey("ProductId"); // Sets up a shadow property.

映射FK的问题在于，参考有2个真实来源。 ProductParameter.ProductId与ProductParameter.Product.Id。通常这些将指向相同的值，但代码可能依赖于一个路径而不是另一个路径，如果一个路径更改而另一个路径没有，则会导致一致性错误。

谨慎使用异步操作。如果您按ID或任何其他相对较快的操作撤回单个记录，请不要使用异步，因为注册延续会产生性能成本。 （更快地执行同步调用）Async适用于预计需要一段时间的操作。 （即超过一秒钟）

最后，代码可能有效，但它没有很好地利用EF，单独设置所有这些实体，并且您通常不希望多次调用SaveChanges以确保数据全部一起提交或者根本不提交，如果存在问题。

   var EntryProduct = _context.Products.Find(ProductDTO.ProductId);

    if (EntryProduct != null)
        return ProductDTO;

    var product = new Product
    {
        Id = ProductDTO.ProductId,
        Number = ProductDTO.Number,
        Amount = ProductDTO.Amount,
        PrimeCostEUR = ProductDTO.PrimeCostEUR,
    };

    var parameterIds = ParameterDTO.Select(x => x.Id).ToList();
    var parametersToAdd = context.Parameters
        .Where(x => parameterIds.Contains(x.ParameterId))
        .Select(x => new ProductParameter
        {
            Product = product,
            Parameter = x
        }).ToList();

    product.ProductParameters.AddRange(parametersToAdd);
    await _context.SaveChangesAsync();

    return ProductDTO;

我不建议对DbContext（_context）使用模块级变量，因为上下文应该是短暂的，以帮助避免一个工作流打算保存的潜在问题，而其他代码可能不会。如果它由IoC容器注入并且作用于与请求匹配的生命周期，那么这不应该导致任何问题。只是要小心保持开放时间超过需要的时间。

最新问题
© www.soinside.com 2019 - 2024. All rights reserved.