Skip to content

Карпов Владимир Лаб. 1 Группа 6512#44

Open
Vouchiko wants to merge 2 commits intoitsecd:mainfrom
Vouchiko:main
Open

Карпов Владимир Лаб. 1 Группа 6512#44
Vouchiko wants to merge 2 commits intoitsecd:mainfrom
Vouchiko:main

Conversation

@Vouchiko
Copy link

ФИО: Карпов Владимир
Номер группы: 6512
Номер лабораторной: 1
Номер варианта: 23
Краткое описание предметной области: Товар на складе
Краткое описание добавленных фич: Создан DTO “Товар на складе”, написаны генератор данных и сервис с кэшированием в Redis.
photo_2026-03-11_11-23-50

@github-actions github-actions bot added In progress Код в процессе проверки Lab 1 Лабораторная №1. Кэширование labels Mar 11, 2026
@github-actions github-actions bot requested a review from Gwymlas March 11, 2026 14:43
.WaitFor(cache);

builder.AddProject<Projects.Client_Wasm>("client")
.WithReference(generator)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

.WithReference(generator) не нужен

@@ -0,0 +1,23 @@
<Project Sdk="Microsoft.NET.Sdk">

<Sdk Name="Aspire.AppHost.Sdk" Version="9.5.0" />
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно поднять до версии 9.5.2 здесь и далее

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше привести название файла и класса к единому виду

@@ -0,0 +1,54 @@
namespace WarehouseItem.Generator.DTO;

public sealed class WarehouseItemDto
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добавить summary к классу


namespace WarehouseItem.Generator.Service;

public interface IWarehouseItemService
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добавить summary тут и далее по коду

return generated;
}

private async Task<WarehouseItemDto?> TryReadCacheAsync(string cacheKey, CancellationToken cancellationToken)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Работу с кэшем можно вынести в отдельную службу

Comment on lines +11 to +13
"ConnectionStrings": {
"redis": ""
},
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно убрать

@Vouchiko
Copy link
Author

Все исправил

@Vouchiko Vouchiko requested a review from Gwymlas March 13, 2026 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

In progress Код в процессе проверки Lab 1 Лабораторная №1. Кэширование

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants