Лаб. 4 6413 Зюзин Егор #267
Conversation
This reverts commit 2a917f8.
…moved to subjects folder)
…nd add some comments
…data (for good results)
… make class DataSeed static class
commit 17: add PolyclinicTests.yml - action for my tests
…nd get it in swagger. lab3
…c. rename to pascalcase namespace.
# Conflicts: # README.md
github-actions
bot
added
In progress
DmitryKrakhmalev
left a comment
DmitryKrakhmalev
left a comment
There was a problem hiding this comment.
Я не буду проверять лабораторную работу пока не сведете с 2-3 лабораторной. Откладываю.
После того как получите апрув - актуализируете код данной лабораторной и направляете повторный запрос на review
| /// <summary> | ||
| /// Background service for appointment Kafka consumer | ||
| /// </summary> | ||
| public class AppointmentKafkaBackgroundService( |
| /// <summary> | ||
| /// Background service for doctor Kafka consumer | ||
| /// </summary> | ||
| public class DoctorKafkaBackgroundService( |
| /// <summary> | ||
| /// Factory for generating contracts | ||
| /// </summary> | ||
| public static class ContractGenerator |
| /// <summary> | ||
| /// Entry point for Kafka contracts generator | ||
| /// </summary> | ||
| public static class Program |
There was a problem hiding this comment.
Зачем вы их удаляли? Они создаются изначально. Такое чувство что текст с нейронки вставляли и не заморачивались.
DmitryKrakhmalev
left a comment
DmitryKrakhmalev
left a comment
There was a problem hiding this comment.
Я не засчитываю данную лабораторную. Я несколько раз просил устранить замечания по 2-3 лабораторке но так ничего и не добился. Проверять как работает четвертая без Aspire - не имеет вообще никакого смысла.
По коду тоже еще много замечаний.
| /// <summary> | ||
| /// Kafka consumer for patient contracts | ||
| /// </summary> | ||
| public class PatientKafkaConsumer( |
There was a problem hiding this comment.
Все консьюмеры написаны без обработок ошибок. Любая ошибка положит BackgroundService на лопатки.
| { | ||
| var config = new ProducerConfig | ||
| { | ||
| BootstrapServers = "localhost:9092", |
There was a problem hiding this comment.
Коннекты не должны хардкодится. Они должны выноситься в настройки. Если нужно - переопределяться Aspire.
| PassportNumber = Guid.NewGuid().ToString(), | ||
| FullName = $"Patient {Random.Shared.Next(1, 1000)}", | ||
| DateOfBirth = DateTime.Now.AddYears(-Random.Shared.Next(18, 80)) |
There was a problem hiding this comment.
Где Bogus? Это одно из условий задания.
DmitryKrakhmalev
added
Approved
2 participants
ФИО: Зюзин Егор
Номер группы: 6413
Номер лабораторной: 4
Номер варианта: 67
Краткое описание предметной области: Поликлиника
Краткое описание добавленных фич: «Инфраструктура» - Реализация сервиса генерации данных и его интеграция с сервером, настроены Aspire и Kafka, интеграция PostgeSQL.