Conversation
…я данных соответствовал серверу
github-actions
bot
added
In progress
| namespace Generator.Controller; | ||
|
|
||
| /// <summary> | ||
| /// Контроллер для объектов жилого строительства. |
There was a problem hiding this comment.
Здесь и далее по коду -
Зачем отступы?
There was a problem hiding this comment.
в ide нажал кнопку code cleanup на всём проекте и они появились
| } | ||
|
|
||
| logger.LogInformation("Getting residential building with Id={id}.", id); | ||
| ResidentialBuildingDto result = await residentialBuildingService.GetByIdAsync(id); |
There was a problem hiding this comment.
Когда-нибудь слышал про var?
| ResidentialBuildingDto result = await residentialBuildingService.GetByIdAsync(id); | |
| var result = await residentialBuildingService.GetByIdAsync(id); |
There was a problem hiding this comment.
в ide нажал кнопку code cleanup на всём проекте var поменялись на типы
There was a problem hiding this comment.
Могу только посоветовать сконфигурировать профайл клинапа вменяемым образом 🤷♂️
| { | ||
| logger.LogInformation("Generating Residential Building for Id={id}", id); | ||
|
|
||
| ResidentialBuildingDto? generatedObject = _faker!.Generate(); |
There was a problem hiding this comment.
Аналогично
В остальных местах мне стало уже лень искать
| logger.LogInformation( | ||
| "Residential building generated: Id={Id}, Address='{Address}', PropertyType='{PropertyType}', " + | ||
| "BuildYear={BuildYear}, TotalArea={TotalArea}, LivingArea={LivingArea}, Floor={Floor}, " + | ||
| "TotalFloors={TotalFloors}, CadastralNumber='{CadastralNumber}', CadastralValue={CadastralValue}", | ||
| generatedObject.Id, generatedObject.Address, generatedObject.PropertyType, generatedObject.BuildYear, | ||
| generatedObject.TotalArea, generatedObject.LivingArea, generatedObject.Floor, generatedObject.TotalFloors, | ||
| generatedObject.CadastralNumber, generatedObject.CadastralValue | ||
| ); |
| /// <summary> | ||
| /// Создаёт настройки кэша - задаёт время жизни кэша. | ||
| /// </summary> | ||
| private DistributedCacheEntryOptions CreateCacheOptions() | ||
| { | ||
| return new DistributedCacheEntryOptions | ||
| { | ||
| AbsoluteExpirationRelativeToNow = _cacheExpirationTimeMinutes | ||
| }; | ||
| } |
There was a problem hiding this comment.
Мощно, конечно, ты выиграл на выделении этого метода. Используешь его в коде ровно один раз
| .AllowAnyOrigin() | ||
| .AllowAnyHeader() | ||
| .AllowAnyMethod(); |
There was a problem hiding this comment.
Сконфигурировать корс нормальным образом
| using System.Text.Json; | ||
| using Generator.DTO; | ||
| using Generator.Generator; | ||
| using Microsoft.Extensions.Caching.Distributed; |
There was a problem hiding this comment.
у меня клинап с этим ничего не делает, может в Rider клинап неправильный какой-то
There was a problem hiding this comment.
По идее, это должно по алфавиту отсортироваться
| builder.Host.UseSerilog((hostingContext, services, loggerConfiguration) => | ||
| { | ||
| loggerConfiguration | ||
| .MinimumLevel.Information() | ||
| .ReadFrom.Configuration(hostingContext.Configuration) | ||
| .ReadFrom.Services(services) | ||
| .Enrich.FromLogContext() | ||
| .WriteTo.Console( | ||
| new CompactJsonFormatter() | ||
| ); | ||
| }); |
There was a problem hiding this comment.
Класс, серилог ты добавил, а синк в опентелеметрию - нет
В итоге в консоль (которая бесполезна) у тебя логи падают:
А в структурных логах пусто:
Либо добавляй недостающий синк, либо сноси серилог 🤷♂️
| using System.Text.Json; | ||
| using Generator.DTO; | ||
| using Generator.Generator; | ||
| using Microsoft.Extensions.Caching.Distributed; |
There was a problem hiding this comment.
По идее, это должно по алфавиту отсортироваться
| [HttpGet("{id:int}")] | ||
| public async Task<ActionResult<ResidentialBuildingDto>> GetResidentialBuilding(int id) |
There was a problem hiding this comment.
Атрибутов с возвращаемыми типами не хватает
2 participants
ФИО: Елагин Денис
Номер группы: 6513
Номер лабораторной: 1
Номер варианта: 38
Краткое описание предметной области: Объект жилого строительства
Краткое описание добавленных фич: Добавлен сервис генерации данных на основе Bogus, кеширование с использованием Redis, структурное логирование для сервиса генерации данных, оркестрация Aspire.