3: Обработка клиентских данных (ADO.net) #4

Merged
1ffy merged 3 commits from task/3_connect-database into master 2026-03-01 23:34:53 +08:00
Owner
No description provided.
Task 3: Connect Database
All checks were successful
CI / build-and-test (push) Successful in 1m1s
bdd0b73364
Author
Owner
No description provided.
1ffy force-pushed task/3_connect-database from bdd0b73364
All checks were successful
CI / build-and-test (push) Successful in 1m1s
to 28ec85f9c2
All checks were successful
CI / build-and-test (push) Successful in 1m2s
2026-02-23 23:07:52 +08:00
Compare
VolovikovAlexander left a comment

В целом верно и даже все работает, но некорректно. Не так, как требовалось в исходном задании.
Я сделал несколько коментариев. Посмотрите пжл и доработайте код.

В целом верно и даже все работает, но некорректно. Не так, как требовалось в исходном задании. Я сделал несколько коментариев. Посмотрите пжл и доработайте код.
@ -0,0 +1,53 @@
using System.Reflection;
using Dapper;
First-time contributor

Мы договаривались, что Вы сделаете через EF. Этого нет. Тогда я ожидаю решение через ADO.net - этого тоже нет.
Для Dapper Вы используете подход в своем решение как для ADO.net через ColumnAttribute
Этого можно избежать штатно. Достаточно SQL запрос по другому написать. Например так:

 select receiptn as ReceiptNumber, categoryid as CategoryId, dater as TransactionDate .....
Мы договаривались, что Вы сделаете через EF. Этого нет. Тогда я ожидаю решение через ADO.net - этого тоже нет. Для **Dapper** Вы используете подход в своем решение как для ADO.net через `ColumnAttribute` Этого можно избежать **штатно**. Достаточно SQL запрос по другому написать. Например так: ``` select receiptn as ReceiptNumber, categoryid as CategoryId, dater as TransactionDate ..... ```
@ -0,0 +8,4 @@
/// Configures Dapper to use <see cref="ColumnAttribute"/> for property-to-column mapping.
/// Call one of the registration methods once at application startup, before any queries run.
/// </summary>
public static class DapperTypeMapper
First-time contributor

Не вижу как Вы решили вариант с определением кода номенклатуры и кода сотрудника, да и в целом - с кодами.
см
https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/_Infra/restore.sql

-- Пример выборки данных
select
    top 10
    
    -- Продажа
    case when transtype = 110 
        then id 
    else
        null
    end as ProductId,
    -- Начало работы
    case when transtype =  386
        then id
    else
        null
    end as EmploeeId
from journal

Не нашел в Вашем решение..

Не вижу как Вы решили вариант с определением кода номенклатуры и кода сотрудника, да и в целом - с кодами. см https://gitverse.ru/VolovikovAlexander/PersonalAccount2026/content/master/_Infra/restore.sql ``` -- Пример выборки данных select top 10 -- Продажа case when transtype = 110 then id else null end as ProductId, -- Начало работы case when transtype = 386 then id else null end as EmploeeId from journal ``` Не нашел в Вашем решение..
@ -0,0 +17,4 @@
{
await using var db = new SqlConnection(connectionString);
return await db.QueryAsync<LogEntryDto>(new CommandDefinition(
"SELECT * FROM [dbo].[journal] WHERE CAST(dater AS DATE) = @Date",
First-time contributor

Не верный SQL запрос. У Вас за один день одна запись получилось. Так быть не может.

Не верный SQL запрос. У Вас за один день одна запись получилось. Так быть не может.
Task 3: Apply requested changes
All checks were successful
CI / build-and-test (push) Successful in 3m38s
71fb6f0a08
Merge branch 'master' into task/3_connect-database
Some checks failed
CI / build-and-test (push) Has been cancelled
36c39d7db4
1ffy merged commit 6605b11072 into master 2026-03-01 23:34:53 +08:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
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!4
No description provided.