Богачев Матвей Лаб. 1 Группа 6511

[Matosik]

ФИО: Богачев Матвей
Номер группы: 6511
Номер лабораторной: 1
Номер варианта: 3
Краткое описание предметной области: Транспортное средство
Краткое описание добавленных фич:

• Добавлен сервис генерации данных через Bogus. Для повышения реалистичности генерации была добавлена кастомная логика формирования полей производитель и модель. Стандартные механизмы Bogus генерируют значения независимо, из-за чего возможны некорректные комбинации, например: производитель BMW и модель Rio, хотя модель Rio относится к Kia.
• интеграционные тесты. Тестирование валидатора с помощью генератора и ручных данных
• Настроен Redis для кэширования. Что позволило в среднем меньшить время запроса с 230ms до 160ms. В данных условиях это может показаться незначительным так как тестирование проводилось на локалке, и операции генерации не ресурсоемкая, но всё же прирост по скорости заметен.
• Настройка оркестрация проектов при помощи .NET Aspire
• логирование сервиса генерации

[danlla]

• В репозитории 2 проекта с aspire, нужен 1

[Matosik]

Логика генерации завязана на Id, номер Id = seed, для генерации данных

[Matosik]

Спасибо за ревью. Повысилось желание скачать себе на компьютер, какой нибуль AI-агент и создавать ИИ-слопы, и отправлять код на ревью без запуска проекта

[danlla]

Логика генерации завязана на Id, номер Id = seed, для генерации данных

Я понимаю это, я к тому, что в первую очередь это id объекта, а то что вы его используете для сидирования генерации это уже второстепенное

[danlla]

Спасибо за ревью. Повысилось желание скачать себе на компьютер, какой нибуль AI-агент и создавать ИИ-слопы, и отправлять код на ревью без запуска проекта

Так самостоятельно написанный плохой код не становится хорошим только из-за того, что он написан самостоятельно

В ревью я просто указал очевидные проблемы и возможные улучшения вашего кода, чтобы он стал лучше, а у вас появилась обратная связь по проделанной вами работе, не понимаю к чему столько пассивной агрессии по типу: страшная табуляция превратилась в шедевр искусства

Если что, то как выглядит код очень важно для дальнейшей разработки и поддержки этого кода

Если с какими-то замечаниями не согласны, всегда можно написать об этом и спросить почему лучше так, а не по-другому

Помимо того, что курс направлен на ознакомление вас с разными технологии, он предполагает, что вы будете писать приложения, которые будут смотреть другие люди, а для этого нужно придерживаться некоторых стандартов и соблюдать чистоту кода

[Matosik]

Основные замечания были учтены. Если какие-либо пункты остались без внимания, прошу ещё раз тезисно их обозначить, всё исправлю