fbki.Levitan/PersonalAccount
1
0
Fork 0
Code Issues Pull requests 1 Releases Packages 1 Activity Actions

ДЗ №5: Реализация доменной логики #6

Open
1ffy wants to merge 1 commit from task/5_domain-logic into master
pull from: task/5_domain-logic
merge into: fbki.Levitan:master
Conversation 1 Commits 1 Files changed 20 +1094 -21
1ffy commented 2026-03-10 19:39:37 +08:00
Owner
Copy link
No description provided.
Task 5: Domain Logic
All checks were successful
CI / build-and-test (push) Successful in 1m30s
Details
4cf0b1933f
VolovikovAlexander requested changes 2026-03-10 21:13:58 +08:00
VolovikovAlexander left a comment
First-time contributor
Copy link

В целом, хорошо. Код оформлен - очень хорошо. +1 балл. Есть заинчания. Жду.

В целом, хорошо. Код оформлен - очень хорошо. +1 балл. Есть заинчания. Жду.
src/PersonalAccount.Domain/Repositories/RevenueReportRepository.cs
@ -0,0 +21,4 @@
.Select(g => new RevenueDto
{
Period = g.Key,
CashAmount = g.Where(t => t.PaymentType == PaymentType.Cash).Sum(t => t.Amount),
VolovikovAlexander commented 2026-03-10 21:09:52 +08:00
First-time contributor
Copy link

1 Нет. Суммы не верно рассчитываются. Вы не учитываете сдачу и сумму скидки.
Ваш выбили чек на 500 руб. Сделали скидку - 100 руб. Вы заплатили - 1000 руб.
Ваш вариант - получил 1000 руб.

2 Выходные - это еше и праздники - Новый Год, например.

1 Нет. Суммы не верно рассчитываются. Вы не учитываете сдачу и сумму скидки. Ваш выбили чек на 500 руб. Сделали скидку - 100 руб. Вы заплатили - 1000 руб. Ваш вариант - получил 1000 руб. 2 Выходные - это еше и праздники - Новый Год, например.
src/PersonalAccount.Domain/Repositories/SalesReportRepository.cs
@ -0,0 +17,4 @@
{
return transactions
.Where(t => t.OperationType == OperationType.Sale)
.GroupBy(t => t.Nomenclature.Id)
VolovikovAlexander commented 2026-03-10 21:10:20 +08:00
First-time contributor
Copy link

Нет. Тут двойная группировка. По категории и номенклатуре.

Нет. Тут двойная группировка. По категории и номенклатуре.
tst/PersonalAccount.UnitTests/WorkScheduleReportTests.cs
@ -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 commented 2026-03-10 21:13:01 +08:00
First-time contributor
Copy link

Не нашел в коде где у Вас идет расчет рабочей смены. По идее, нужно брать транзакции. Далее делать отбор "Старт", "Окончание" работы. Потом сопоставлять, чтобы определить рабочую смену.

Укажите пжл где реализация?

Не нашел в коде где у Вас идет расчет рабочей смены. По идее, нужно брать транзакции. Далее делать отбор "Старт", "Окончание" работы. Потом сопоставлять, чтобы определить рабочую смену. Укажите пжл где реализация?
All checks were successful
CI / build-and-test (push) Successful in 1m30s
Details
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin task/5_domain-logic:task/5_domain-logic
git switch task/5_domain-logic

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch master
git merge --no-ff task/5_domain-logic
git switch task/5_domain-logic
git rebase master
git switch master
git merge --ff-only task/5_domain-logic
git switch task/5_domain-logic
git rebase master
git switch master
git merge --no-ff task/5_domain-logic
git switch master
git merge --squash task/5_domain-logic
git switch master
git merge --ff-only task/5_domain-logic
git switch master
git merge task/5_domain-logic
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
VolovikovAlexander
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
fbki.Levitan/PersonalAccount!6
No description provided.