Не удалось неявно преобразовать тип List << анонимный тип: .. >> в IEnumerable <Product>

Question

Не удалось неявно преобразовать тип List << анонимный тип: .. >> в IEnumerable <Product>

2

Я пытаюсь выполнить запрос linq Join но возникла исключительная ситуация.

Это мой метод:

public async Task<IEnumerable<Product>> TestProducts()
{
    var items = await _context.Products.Join(_context.ProductTypes,
         p => p.ProductId,
         c => c.ProductTypeId,
         (p, c) => new
         {
             productId = p.ProductId,
             number = p.Number,
             amount = p.Amount,
             primeCostEUR = p.PrimeCostEUR,
             nameType = c.NameType

         }).ToListAsync();
    return items;
}

Это моя модель:

public class Product //: BaseObject
{
    public long ProductId { get; set; }
    public string Number { get; set; }
    public double Amount { get; set; }
    public double PrimeCostEUR { get; set; }

    public long ProductTypeId {  get; set; }
    public ProductType ProductType { get; set; }

    public long ProductParameterId { get; set; }
    public ICollection<ProductParameter> ProductParameters { get; set; } = 
    new List<ProductParameter>();
}

public class ProductType //: BaseObject
{
    public long ProductTypeId { get; set; }

    public string NameType { get; set; }

    public ICollection<Product> Products { get; set; }
    public ICollection<Parameter> Parameters { get; set; }

}

Это ошибка, с которой я столкнулся:

Ошибка CS0266 Невозможно неявно преобразовать тип 'System.Collections.Generic.List<>' в 'System.Collections.Generic.IEnumerable <.Models.Product>'.

Stas Iliukovich 15 март 2019, в 22:31

Источник

0

Ошибка кажется очевидной. Разве вы не можете просто выбрать new Product { } вместо анонимного типа new { } ? Ой, подождите, вы включаете NameType , который не является собственностью Product . По сути, поскольку вы определили метод как возвращающий IEnumerable<Product> , то это то, что вы должны вернуть. Так что либо измените тип возврата, либо измените то, что вы возвращаете.
Rufus L 15 март 2019, в 21:15

Теги:

c#

linq

asp.net

entity-framework

2 ответа

0

Изменить это:

 var items = await _context.Products.Join(_context.ProductTypes,
             p => p.ProductId,
             c => c.ProductTypeId,
             (p, c) => new
             {
                 productId = p.ProductId,
                 number = p.Number,
                 amount = p.Amount,
                 primeCostEUR = p.PrimeCostEUR,
                 nameType = c.NameType

             }).ToListAsync();

к этому:

 var items = await _context.Products.Join(_context.ProductTypes,
             p => p.ProductId,
             c => c.ProductTypeId,
             (p, c) => new Product  //as Product not anonymous
             {
                 productId = p.ProductId,
                 number = p.Number,
                 amount = p.Amount,
                 primeCostEUR = p.PrimeCostEUR,
                 nameType = c.NameType

             }).ToListAsync();

Ashkan Mobayen Khiabani 15 март 2019, в 19:14

0

Конечно, имена этих свойств нужно немного изменить, и Product не имеет NameType ...
Rufus L 15 март 2019, в 21:18
0

ну я на самом деле мало обращал внимания на класс продукта;)
Ashkan Mobayen Khiabani 15 март 2019, в 21:19
0

Да, у меня был комментарий, который совпадает с вашим ответом, но потом я заметил, что они включают в результат поле из объединенной таблицы, так что это ломает вещи. Им либо не нужно включать поле, либо определять другой тип, который его включает, и возвращать IEnumerable этого типа.
Rufus L 15 март 2019, в 21:23

Показать ещё 1 комментарий

Ещё вопросы

Ошибка кажется очевидной. Разве вы не можете просто выбрать new Product { } вместо анонимного типа new { } ? Ой, подождите, вы включаете NameType , который не является собственностью Product . По сути, поскольку вы определили метод как возвращающий IEnumerable<Product> , то это то, что вы должны вернуть. Так что либо измените тип возврата, либо измените то, что вы возвращаете.
Конечно, имена этих свойств нужно немного изменить, и Product не имеет NameType ...
ну я на самом деле мало обращал внимания на класс продукта;)
Да, у меня был комментарий, который совпадает с вашим ответом, но потом я заметил, что они включают в результат поле из объединенной таблицы, так что это ломает вещи. Им либо не нужно включать поле, либо определять другой тип, который его включает, и возвращать IEnumerable этого типа.

TanvirArjel · Accepted Answer · 2019-03-16T00-30-00.000Z

Кажется, что вы ожидаете, что результат будет комбинацией класса модели Product и ProductType. Поэтому создайте класс DTO (Data Transfer Object) следующим образом, который будет содержать все поля вашего выходного запроса.

public class ProductDto
{
    public long ProductId { get; set; }
    public string Number { get; set; }
    public double Amount { get; set; }
    public double PrimeCostEUR { get; set; }
    public string ProductTypeName { get; set; }
}

Теперь напишите ваш метод контроллера следующим образом, где возвращаемый тип метода IEnumerable<ProductDto>:

public async Task<IEnumerable<ProductDto>> TestProducts()
{
    var items = await _context.Products.Select(p => new ProductDto
         {
             ProductId= p.ProductId,
             Number= p.Number,
             Amount= p.Amount,
             PrimeCostEUR= p.PrimeCostEUR,
             ProductTypeName = p.ProductType.NameType
         }).ToListAsync();
    return items;
}

Более того, вам не нужно явное LINQ-соединение, как вы это сделали, вы можете достичь того же, используя проекцию EF, как я сделал с большей простотой.

Спасибо за ответ. Я новичок в .NET. Должен ли каждый запрос на соединение иметь свой собственный класс DTO?