ДЗ №5: Реализация доменной логики #6
No reviewers
Labels
No labels
No milestone
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
fbki.Levitan/PersonalAccount!6
Loading…
Reference in a new issue
No description provided.
Delete branch "task/5_domain-logic"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
В целом, хорошо. Код оформлен - очень хорошо. +1 балл. Есть заинчания. Жду.
@ -0,0 +21,4 @@.Select(g => new RevenueDto{Period = g.Key,CashAmount = g.Where(t => t.PaymentType == PaymentType.Cash).Sum(t => t.Amount),1 Нет. Суммы не верно рассчитываются. Вы не учитываете сдачу и сумму скидки.
Ваш выбили чек на 500 руб. Сделали скидку - 100 руб. Вы заплатили - 1000 руб.
Ваш вариант - получил 1000 руб.
2 Выходные - это еше и праздники - Новый Год, например.
@ -0,0 +17,4 @@{return transactions.Where(t => t.OperationType == OperationType.Sale).GroupBy(t => t.Nomenclature.Id)Нет. Тут двойная группировка. По категории и номенклатуре.
@ -0,0 +68,4 @@var day = new DateTimeOffset(2025, 3, 10, 0, 0, 0, TimeSpan.Zero);var shifts = new[]{new WorkShift { Id = 2, Employee = Bob, Company = Company, StartedAt = day.AddHours(14), CompletedAt = day.AddHours(22) },Не нашел в коде где у Вас идет расчет рабочей смены. По идее, нужно брать транзакции. Далее делать отбор "Старт", "Окончание" работы. Потом сопоставлять, чтобы определить рабочую смену.
Укажите пжл где реализация?
Да. Хорошо. Если будете на занятии попрошу Вас рассказать про решение.
@VolovikovAlexander wrote in #6 (comment):
А где?