Created DTO logic #3

jezv · 2026-02-23T21:26:30+08:00

jezv commented

2026-02-23 21:26:30 +08:00

Структура такая:
Внутри DTO модели есть метод Map, который заполняет её по значениям из БД.
Метод Map реализует цепочку обязаностей из 3-ёх звеньев:
OperationHandler - создаёт Enum операции по айди,
IdHandler - заполняет соответсвующий айди от типа операции,
FieldsHandler - заполняет все остальные поля, согласно их названиям из кастомного атрибута

Структура такая: Внутри DTO модели есть метод Map, который заполняет её по значениям из БД. Метод Map реализует цепочку обязаностей из 3-ёх звеньев: OperationHandler - создаёт Enum операции по айди, IdHandler - заполняет соответсвующий айди от типа операции, FieldsHandler - заполняет все остальные поля, согласно их названиям из кастомного атрибута

jezv added 1 commit

2026-02-23 21:26:30 +08:00

Created DTO logic db766b72d1

VolovikovAlexander requested changes

2026-02-24 00:20:14 +08:00

VolovikovAlexander left a comment

First-time contributor

Код нужно доделывать. Значительно.

Src/PersonalAccount.Domain/Core/Asbtract/DTOBase.cs Outdated

					
				@ -0,0 +1,10 @@

				﻿namespace Src.PersonalAccount.Domain.Core;

				public abstract class DTOBase

Оформление кода.
https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/_docs/CONTRIBUTING.md
Нейминг

Оформление кода. https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/_docs/CONTRIBUTING.md Нейминг

Src/PersonalAccount.Domain/Core/FieldsHandler.cs Outdated

					
				@ -0,0 +4,4 @@

				{

				  public override void HandleRequest(object condition)

				  {

				    throw new NotImplementedException();

Зачем нужны классы в которых нет реализации?

Src/PersonalAccount.Domain/Models/DTO/JournalNoteDTO.cs Outdated

					
				@ -0,0 +28,4 @@

				  /// <summary>

				  /// Идентификатор сотрудника

				  /// </summary>

				  public uint? EmployeeId { get; set; }

Нет атрибутов. Как определяется код сотрудника?

jezv added 1 commit

2026-02-24 09:17:01 +08:00

FieldsHandler logic 942edbc4d8

jezv added 1 commit

2026-02-24 12:22:14 +08:00

Added logic to handlers 3839c793a5

VolovikovAlexander requested changes

2026-02-24 12:46:59 +08:00

VolovikovAlexander left a comment

First-time contributor

В целом, решение понятно. Принято.
Однако, нужно доработать

Коментарии в код. Коментарии к тестам
Сделать загрузку данных: День, Месяц, Квартал

В целом, решение понятно. Принято. Однако, нужно доработать - Коментарии в код. Коментарии к тестам - Сделать загрузку данных: `День, Месяц, Квартал`

Src/PersonalAccount.Domain/Core/Asbtract/DTOBase.cs Outdated

					
				@ -0,0 +2,4 @@

				namespace Src.PersonalAccount.Domain.Core;

				public abstract class DTOBase

Коментарии в код добавьте пжл.
Назначение данного класса? Для чего используется каждое свойство?

Коментарии в код добавьте пжл. Назначение данного класса? Для чего используется каждое свойство?

Src/PersonalAccount.Domain/Core/EnumHandler.cs

					
				@ -0,0 +16,4 @@

				        Attribute = p.GetCustomAttribute<DTOFieldName>()

				      })

				      .Where(f => f.Attribute != null && f.Attribute.FieldType.IsEnum)

				      .ToList()

Понятное, но спорное решение. Ведь у Вас в каждом Handler разница только в Where

 .Where(f => f.Attribute != null && f.Attribute.FieldType.IsEnum)
 .Where(f => 
                  f.Attribute != null && (f.Attribute.FieldType.IsPrimitive || f.Attribute.FieldType == typeof(string)))

Полагаю, что можно унифицировать.
Но в целом - годиться.

Понятное, но спорное решение. Ведь у Вас в каждом Handler разница только в Where ``` .Where(f => f.Attribute != null && f.Attribute.FieldType.IsEnum) .Where(f => f.Attribute != null && (f.Attribute.FieldType.IsPrimitive || f.Attribute.FieldType == typeof(string))) ``` Полагаю, что можно унифицировать. Но в целом - годиться.

Tst/PersonalAccount.UnitTests/TestsHandlers.cs

					
				@ -0,0 +158,4 @@

				        var fieldsHandler = new FieldsHandler();

				        var journalHandler = new JournalIdHandler();

				        enumHandler.Successor = fieldsHandler;

Вот тут у Вас в тестах - формируется цепочка ответственности. Ок
Но где же решение для:

В рамках данного модуля создать методы
Загрузкой транзакций задень
Загрузка транзакций за месяц
Загрузка транзакций за квартал
Для каждой загрузки сделать замер по времени работы.

Тогда сделайте тесты. Так же, можно сделать менеджер класс, который для определенного варианта сформирует нужную цепочку ответственности.

Вот тут у Вас в тестах - формируется цепочка ответственности. Ок Но где же решение для: ``` В рамках данного модуля создать методы Загрузкой транзакций задень Загрузка транзакций за месяц Загрузка транзакций за квартал Для каждой загрузки сделать замер по времени работы. ``` Тогда сделайте тесты. Так же, можно сделать менеджер класс, который для определенного варианта сформирует нужную цепочку ответственности.

jezv added 1 commit

2026-02-24 19:16:21 +08:00

Final Performance f6b6b48bc1

jezv added 1 commit

2026-02-24 19:16:54 +08:00

Some fixes b13cd4716f

jezv commented

2026-02-24 19:17:41 +08:00

Обработка за один день сделана за 314 ms
Обработка за один месяц сделана за 814 ms
Обработка за один квартал сделана за 438 ms

Короче вся обработка занимает очень мало времени и всё время уходит на задержку обращения к БД, поэтому показатели нестабильные между 300-800 мс везде.

Обработка за один день сделана за 314 ms Обработка за один месяц сделана за 814 ms Обработка за один квартал сделана за 438 ms Короче вся обработка занимает очень мало времени и всё время уходит на задержку обращения к БД, поэтому показатели нестабильные между 300-800 мс везде.

jezv changed title from ~~WIP: Created DTO logic~~ to Created DTO logic

2026-02-24 19:17:42 +08:00

VolovikovAlexander approved these changes

2026-02-24 20:13:54 +08:00

VolovikovAlexander left a comment

First-time contributor

В целом хорошо. Запросы не верно. Но не страшно. 99% не верно сделали)

Src/PersonalAccount.Performance/Program.cs

					
				@ -0,0 +23,4 @@

				using var connect = new SqlConnection(options.ConnectionString);

				connect.Open();

				var sql = @"SELECT t2.storeid, t1.PreviewGroupType as transtype, t2.receiptn,

Не верно. У Вас согласно запросу не 1, 3, 6 месяцев получится. А три раза ДО указанной даты.

jezv merged commit e4e9eab259 into develop

2026-03-02 20:10:05 +08:00

jezv referenced this pull request from a commit

2026-03-02 20:10:05 +08:00

Merge pull request 'Created DTO logic' (#3) from feature/dto into develop

Sign in to join this conversation.

No reviewers

No labels

No milestone

No project

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

jezv/VolovikovASPNET!3

Rows
Columns