EF Core, как правильно тащить нужные поля?

Question

[OwDafuq]

Доброго времени суток.
Есть примерно такая база:

Модели

public class BaseIdEntity
{
    public int Id { get; set; }
    public bool IsDeleted { get; set; }
}
 
//Основной класс, с которым и производится вся работа.
public class MAOrder : BaseIdEntity
{
    public DateTime DateStart { get; set; }
    public DateTime? DateEnd { get; set; }
    public DateTime? DateComplete { get; set; }
    public DateTime? PaymentDate { get; set; }
    public bool IsPaid { get; set; }
    public decimal Amount { get; set; }
    
    public MACounterparty? Counterparty { get; set; }
    public int? CounterpartyId { get; set; }
    
    public MAOrderStatus? Status { get; set; }
    public int? StatusId { get; set; }
 
    public MAPaymentType? PaymentType { get; set; }
    public int? PaymentTypeId { get; set; }
 
    public ApplicationUser? Creator { get; set; }
    public string? CreatorId { get; set; }
 
    public ApplicationUser? Worker { get; set; }
    public string? WorkerId { get; set; }
 
    public ICollection<MAOrderPosition> Positions { get; set; } //позиции заказа
}
 
public class MACounterparty : BaseIdEntity
{
    public CounterpartyCategory Category { get; set; }
 
    [MARequired]
    public string Name { get; set; }
 
    [MARequired]
    public string Telephone { get; set; }
 
    [EmailAddress]
    public string? EMail { get; set; }
    public string? Comment { get; set; }
 
    public ICollection<MAOrder> Orders { get; set; }
}
 
public class MAOrderPosition : BaseIdEntity
{
    public DateTime DateStart { get; set; }
    public DateTime? DateEnd { get; set; }
    public DateTime? DateComplete { get; set; }
    [MARequired]
    [MAMaxLength(150)]
    public string Description { get; set; }
 
    [MAMaxLength(150)]
    public string? Comment { get; set; }
 
    public decimal Cost { get; set; }
    public decimal CostPrice { get; set; }
    public int Count { get; set; }
    public bool IsReOrder { get; set; }
 
    public MAOrder? Order { get; set; }
    public int? OrderId { get; set; }
    
    public MAOrderStatus? Status { get; set; }
    public int? StatusId { get; set; }
    
    public MAOrderCategory? Category { get; set; }
    public int? CategoryId { get; set; }
}
 
public class NameColor : BaseIdEntity
{
    [MARequired]
    public string Name { get; set; }
    [MARequired]
    public string Color { get; set; }
}
 
public class MAOrderCategory : NameColor
{
    public ICollection<MAOrderPosition> Positions { get; set; }
}
 
public class MAOrderStatus : NameColor
{
    public ICollection<MAOrder> Orders { get; set; }
    public ICollection<MAOrderPosition> Positions { get; set; }
}
 
public class MAPaymentType : BaseIdEntity
{
    public string Name { get; set; }
    public ICollection<MAOrder> Orders { get; set; }
}

Есть страницы с отображением заказов (MAOrder), но туда нужны данные: все НЕ навигационные св-ва, контрагент (Counterparty - Только имя и телефон), позиции заказа (Positions, все поля кроме 2-х), Статус (Status).

Как правильно тащить такие данные из базы? Через Include попадает очень много мусора (особенно в контрагентов). Через Select - очень много кода. Так же в определенных моментах нужно дополнять Where, чтобы обирать нужные заказы.
Какие Best-practies в этом моменте?

Answer 1

[Василий Банников]

Select/Automapper/Mapster. Больше вариантов нет.

На счёт количества кода я бы поспорил - в случае с Select никто не запрещает переиспользовать уже написанный код, если он используется в многих местах.

Comments

[OwDafuq]

Не совсем понимаю как использовать тут AutoMapper, раньше использовал его только как зависимость через IMapper. Как переиспользовать Select? В любом случае я не смогу использовать Select на Positions, ибо он будет ругаться на то, что там получается тип IEnumerable, а не ICollection

.Select(x => new MAOrder
{
	Positions = x.Positions.Select(p => new MAOrderPosition
	{
		Status = p.Status == null ? null : new MAOrderStatus
		{
			Name = p.Status.Name,
			Color = p.Status.Color
		}
	})
})

[OwDafuq]

Единственное, что смог придумать:
(x.BLABLA = x.BLABLA == null ? null : ... сделано для того, чтобы если там null, то null и остался, может я пропустил какой-то новый синтаксический сахар, чтобы такие конструкции не городить?)

private IQueryable<MAOrder> GetOrders()
{
	var query = context.Orders.AsSplitQuery()
								.Include(x => x.Counterparty)
								.Include(x => x.Positions).ThenInclude(x => x.Category)
								.Include(x => x.Positions).ThenInclude(x => x.Status)
								.Include(x => x.Status)
								.Include(x => x.PaymentType)
								.Include(x => x.Creator)
								.Include(x => x.Worker);

	return query;
}

return GetOrders().Where(x => !x.IsCompleted && x.IsInArchive == isInArchive && !x.Presets.Any()).Select(x => new MAOrder
{
	Id = x.Id,
	DateStart = x.DateStart,
	Amount = x.Amount,
	IsCompleted = x.IsCompleted,
	IsPaid = x.IsPaid,
	IsReady = x.IsReady,
	Positions = x.Positions.Select(p => new MAOrderPosition
	{
		Comment = p.Comment,
		Cost = p.Cost,
		Count = p.Count,
		Description = p.Description,
		IsCompleted = p.IsCompleted,
		IsReady = p.IsReady,
		IsReOrder = x.IsReOrder,
		Status = p.Status == null ? null : new MAOrderStatus
		{
			Id = p.Id,
			Name = p.Status.Name,
			Color = p.Status.Color,
		},
		Category = p.Category == null ? null : new MAOrderCategory
		{
			Id = p.Category.Id,
			Name = p.Category.Name,
			Color = p.Category.Color,
		}
	}).ToList(),
	Counterparty = x.Counterparty == null ? null : new MACounterparty
	{
		Id = x.Counterparty.Id,
		Name = x.Counterparty.Name,
		Telephone = x.Counterparty.Telephone
	},
	Status = x.Status == null ? null : new MAOrderStatus
	{
		Id = x.Status.Id,
		Name = x.Status.Name,
		Color = x.Status.Color
	}
}).AsNoTracking().ToListAsync();

[Василий Банников]

LiptonOlolo,

Как переиспользовать Select?

Просто в отдельный метод вынеси.