Вольгов Даниил Лаб. 1 Группа 6513

[the80hz]

ФИО: Вольгов Даниил Иванович
Номер группы: 6513
Номер лабораторной: 1
Номер варианта: 37
Краткое описание предметной области: Учебный курс
Краткое описание добавленных фич:

• Сделан генератор контрактов на Bogus.
• Добавлено кэширование через Redis + IDistributedCache.
• Добавлено структурное логирование генерации и кэша.
• Настроена оркестрация через .NET Aspire (AppHost + API + Redis).

[alxmcs]

Что мы имеем в остатке помимо мелких замечаний:

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

Из чего я делаю логичный вывод, что ты заставил меня смотреть на 600+ строк навайбкоженного говна, не удосужившись их хотя бы запустить самостоятельно. Потраченного времени жаль, далее я это смотреть больше не планирую.
Ты отправляешься в нейрослоповое чистилище, в котором составишь компанию господину Панюшкину с его #13.

[the80hz]

Здравствуйте. Не согласен с частью замечаний в ревью, и прошу пересмотреть решение.

аспаер не запускается, потому что решение не билдится в принципе,

По замечанию о том, что проект и в частности Aspire не запускается, у меня проект собирается локально и без ошибок

the80hz@MacBook-Pro-Daniil cloud-development %  make build
Версия MSBuild 17.8.49+7806cbf7b для .NET
  Determining projects to restore...
  All projects are up-to-date for restore.
  CloudDevelopment.ServiceDefaults -> /Users/the80hz/Developer/cloud-development/CloudDevelopment.ServiceDefaults/bin/Debug/net8.0/CloudDevelopment.ServiceDefaults.dll
  Client.Wasm -> /Users/the80hz/Developer/cloud-development/Client.Wasm/bin/Debug/net8.0/Client.Wasm.dll
  Client.Wasm (Blazor output) -> /Users/the80hz/Developer/cloud-development/Client.Wasm/bin/Debug/net8.0/wwwroot
  CourseGenerator.Api -> /Users/the80hz/Developer/cloud-development/CourseGenerator.Api/bin/Debug/net8.0/CourseGenerator.Api.dll
  CloudDevelopment.AppHost -> /Users/the80hz/Developer/cloud-development/CloudDevelopment.AppHost/bin/Debug/net8.0/CloudDevelopment.AppHost.dll

Сборка успешно завершена.
    Предупреждений: 0
    Ошибок: 0

Прошло времени 00:00:02.06

в репозитории присутствует мейкфайл, которого в принципе никогда не должно было бы существовать в подобном решении

В чем проблема добавленного мною Makefile? Я стараюсь во всех своих проектах и лабораторных добавлять его, для своего удобства. Извините, если он мешает ревью и корректной сдаче лабораторной, больше не буду.

апи контроллер сервиса отдает несоответствующий контракт тому, который ожидается на клиенте

Да, согласен, есть несоответствие клиента и API по адресу, это стоит поправить

Прошу пересмотреть итоговую оценку с учетом того, что основная часть лабораторной (генерация, кэширование, оркестрация) работает, а найденное несоответствие клиента и API является дорабатываемым замечанием в рамках PR.

Спасибо.

[alxmcs]

Дорогой Вольгов Даниил Лаб. 1 Группа 6513!

Соглашусь с неточностью своей формулировки - оркестратор действительно запускается, только конфигурация сервисов в нем настолько нерабочая, что в рантайме падает исключение уже на старте. Этой действительно не то же самое, что и "не билдится", тут был не прав.
Помог ли тебе это выяснить твой мейкфайл, если допустить, что ты вообще запускал этот код до того, как сделать пул-риквест? Похоже, что нет.

Я искренне тронут до глубины души, что ты согласился с тем фактом ,что несоответствие контракта, который отдает бекенд, контракту, который ожидает фронтенд, действительно нужно поправить.

Итоговая оценка пересматриваться не будет, потому что возместить время моей подходящей к закату жизни, которое я потратил на ревью этого говна, невозможно вернуть обратно.
К счастью, у нас сквозной проект на все лабораторные работы, так что у тебя еще будет шанс сделать 1ю лабораторную вменяемым образом, пока ты будешь выполнять 2ю.

На этой полной надежд ноте предлагаю закончить этот сеанс бокса по переписке.