Skip to content

Замотохина Мария. Лаб. 1. Группа 6511#54

Open
ZamotohinaMaria wants to merge 9 commits intoitsecd:mainfrom
ZamotohinaMaria:main
Open

Замотохина Мария. Лаб. 1. Группа 6511#54
ZamotohinaMaria wants to merge 9 commits intoitsecd:mainfrom
ZamotohinaMaria:main

Conversation

@ZamotohinaMaria
Copy link

ФИО: Замотохина мария
Номер группы: 6511
Номер лабораторной: 1
Номер варианта: 6
Краткое описание предметной области: Медицинский пациент
Краткое описание добавленных фич:
Реализовано:

  • Добавлен сервис генерации сотрудников с кешированием
  • Выполнена оркестрация Aspire
  • Логирование основных действий

@github-actions github-actions bot added In progress Код в процессе проверки Lab 1 Лабораторная №1. Кэширование labels Mar 14, 2026
@github-actions github-actions bot requested a review from danlla March 14, 2026 17:09
Copy link

@danlla danlla left a comment

Choose a reason for hiding this comment

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

  • Папка wwwroot не нужна в проекте с генератором
  • При запуске aspire приложение запускается лишний браузер с клиентом, чтобы этого не происходило можно в профиле запуска клиента в launchsettings.json проставить launchBrowser в false
  • Нет ни структурированных логов, ни обычных логов в консоли, если использовать serilog, то его нужно настроить на вывод логов в консоль и на отправку логов в OpenTelemetry, откуда они потом попадают в структурированные логи в дашбордах aspire

/// <summary>
/// Вес - округляется до двух знаков после запятой
/// </summary>
public required double Width { get; set; }
Copy link

Choose a reason for hiding this comment

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

Как будто бы это не вес

/// <param name="faker">Генератор случайных данных</param>
/// <param name="height">Рост</param>
/// <returns>Вес пациента, округленный до 2х знаков после запятой</returns>
private static double GenerateWidth(Faker faker, double height)
Copy link

Choose a reason for hiding this comment

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

Тут тоже наименование нужно поправить

private static double GenerateWidth(Faker faker, double height)
{
var bmi = faker.Random.Double(15, 40);
return (int)(bmi * height * height);
Copy link

Choose a reason for hiding this comment

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

Здесь не нужно приводить к int, но нужно округлить до 2 знаков


<ItemGroup>
<PackageReference Include="Bogus" Version="35.6.5" />
<PackageReference Include="Microsoft.Extensions.Caching.StackExchangeRedis" Version="10.0.5" />
Copy link

Choose a reason for hiding this comment

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

Лучше использовать пакет Aspire.StackExchange.Redis.DistributedCaching

Comment on lines +25 to +33
builder.Services.AddCors(options =>
{
options.AddDefaultPolicy(policy =>
{
policy.AllowAnyOrigin()
.AllowAnyMethod()
.AllowAnyHeader();
});
});
Copy link

Choose a reason for hiding this comment

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

cors лучше настроить так, чтобы только клиент мог делать запросы
например передавать нужные origins в appsettings.json

Copy link
Author

Choose a reason for hiding this comment

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

Не совем понимаю, как это сделать, с cors не очень дружу...

ILogger<Program> logger,
CancellationToken cancellationToken) =>
{
logger.LogInformation("������� ������ �� ���������� �������� � ID: {Id}", id);
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 +3 to +16
<Sdk Name="Aspire.AppHost.Sdk" Version="9.0.0" />
<PropertyGroup>
<OutputType>Exe</OutputType>
<TargetFramework>net8.0</TargetFramework>
<ImplicitUsings>enable</ImplicitUsings>
<Nullable>enable</Nullable>
<IsAspireHost>true</IsAspireHost>
<UserSecretsId>fe743bf2-08eb-41c7-817b-f144b739dce3</UserSecretsId>
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Aspire.Hosting.AppHost" Version="13.1.2" />
<PackageReference Include="Aspire.Hosting.Redis" Version="13.1.2" />
</ItemGroup>
Copy link

Choose a reason for hiding this comment

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

Версия aspire 9.0.0, а версии пакетов 13.1.2, нужно прийти к одной версии здесь
Лучше всего наверно это версия 9.5.2

Comment on lines +6 to +8
var generator = builder.AddProject<Projects.MedicalPatient_Generator>("medicalpatient-generator")
.WithReference(redis)
.WithExternalHttpEndpoints();
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 +10 to +11
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.

А клиенту стоит подождать генератор

using Bogus.DataSets;
using MedicalPatient.Generator.Models;
using Microsoft.AspNetCore.Mvc.RazorPages;
using Microsoft.Extensions.Logging;
Copy link

Choose a reason for hiding this comment

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

cleanup

@ZamotohinaMaria
Copy link
Author

ZamotohinaMaria commented Mar 19, 2026

Папка wwwroot не нужна в проекте с генератором
image
Думаю, что все-таки нужна....

При запуске aspire приложение запускается лишний браузер с клиентом, чтобы этого не происходило можно в профиле запуска клиента в launchsettings.json проставить launchBrowser в false
Этот мемент увидела, где омжно сделать, но оставлю, мне так удобнее все сразу запускать и сразу закрывать, чтобы по вкладкам не бегать

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