6412 Лаб.3 Ксеневич Максим #225

maeosha · 2025-12-21T11:34:10Z

ФИО: Ксеневич Максим Станиславович
Номер группы: 6412
Номер лабораторной: 3
Номер варианта: 10
Краткое описание предметной области: Поликилиника
Краткое описание добавленных фич: Добелены EfCore репозитории, DbContex, Aspire проект. Данные хранятся в MySql с оркестрацией через Aspire.

…d patients classes into a separate class

…Info. fix: add summary

… and DTOs

…s and DTOs

…and DTOs

…ions and DTOs

…tors, specializations, and appointments

…patients, doctors, specializations, and appointments

…ls in the Clinic API

…e registrations and Swagger integration

…octor data retrieval in the Clinic API

…e handling

…s to use base controller pattern

- Removed IClinicDataBase interface and its implementation. - Deleted DataSeed class for test data. - Updated Program.cs to configure Entity Framework and database context. - Created AnalyticsServices for analytics-related functionalities. - Refactored AppointmentServices, DoctorServices, PatientServices, and SpecializationServices to use specific data interfaces. - Updated appsettings for database connection strings and logging levels.

… environment

alxmcs

Самый главный вопрос - где оркестратор?

Также со времен 2й лабы пропали инмемори репозитории. Если ты их ввел в свою архитектуру, будь добр поддерживать их до конца так же, как это делают все остальные студенты, которые сдавали 2 и 3 лабу отдельно.

Clinic/Clinic.DataBase/EntityFramework/EfAppointmentDataBase.cs

Clinic/Clinic.Api/DTOs/Appointment/CreateAppointmentDto.cs

Clinic/Clinic.Api/DTOs/Appointment/UpdateAppointment.cs

maeosha · 2025-12-21T14:39:46Z

Зачем поддерживать инмемори хранение, если смысл лабораторной работы как раз таки уйти от них?

alxmcs · 2025-12-21T17:01:06Z

Философскими вопросами стоило задаваться в начале семестра, а не в 20х числах декабря.

Уход от использования инмемори репозиториев не означает отказ от их поддержки. Интерфейсы инфраструктурного уровня проекта нужно было спроектировать так, чтобы они оставались совместимы как с хранением в памяти, так и с хранением данных в бд через ef core.

maeosha · 2025-12-21T19:50:44Z

Так про дальнейшую поддержку инмемори репозиториев не было сказано ни слова в условиях лабораторной работы...

…ucture and functionality

alxmcs

Как-то поздно я, конечно, заинтересовался вопросом. Но почему вариант 10?

Когда должен быть 36

Предыдущие лабы, я, к сожалению, уже принял, так что заставлять тебя менять доменную область было бы нечестно. Но данную лабу я принимать не буду, пока не будет сменено СУБД, потому что по варианту должен быть Postgres, а не MySql.

Кстати, об отсутствии в условии лабораторных необходимости поддерживать инмемори репозитории.

Clinic/Clinic.Tests/ClinicTests.cs

Clinic/Clinic.Api/Clinic.Api.csproj

Clinic/Clinic.AppHost/Clinic.AppHost.csproj

Clinic/Clinic.DataBase/Clinic.DataBase.csproj

Clinic/Clinic.Models/Clinic.Models.csproj

Clinic/Clinic.ServiceDefaults/Clinic.ServiceDefaults.csproj

Clinic/Clinic.Tests/Clinic.Tests.csproj

…tructure and add Clinic.Application project

maeosha · 2025-12-22T11:14:15Z

Пожалуйста сжальтесь, дайте возможность не возвращать мне инмемори репозитории, мне под них инфрастурктуру менять придется:(

alxmcs · 2025-12-23T14:03:36Z

Лол, нет

… and specializations

alxmcs

Проект обрастает функционалом, а лучше от этого никому не становится. В том числе и мне. К 4 лабораторной у меня, наверное, окончательно сосуды в мозгу лопнут.

Я уже несколько ревью подряд пишу о необходимости выделить dto в отдельную класслибу. Ты как планируешь реализовывать совершенно автономный и никак не связанный с апи сервером сервер генерации контрактов без отдельной либы с контрактами? Апи сервер в нем зареференсишь? А если бы нужно было писать фронтенд, ты бы в клиентском приложении сервер зареференсил?

Может быть, вместо бокса по переписке под конец семестра стоило начать писать код пораньше (и получше)?

alxmcs · 2025-12-24T10:52:11Z

Clinic/Clinic.Api/Controllers/SpecializationControllers.cs

+/// </summary>
+[ApiController]
+[Route("api/specializations")]
+public class SpecializationControllers(ISpecializationServices specializationServices) : BaseControllers<GetSpecializationDto, CreateSpecializationDto, UpdateSpecializationDto>(specializationServices);


Документации ни к методам контроллера

alxmcs · 2025-12-24T10:52:27Z

Clinic/Clinic.Api/DTOs/Appointment/CreateAppointmentDto.cs

Ни к контрактам нет

alxmcs · 2025-12-24T10:52:47Z

Clinic/Clinic.Api/Clinic.Api.csproj

+    <TargetFramework>net8.0</TargetFramework>
+    <Nullable>enable</Nullable>
+    <ImplicitUsings>enable</ImplicitUsings>


Потому что ее никто не билдит

alxmcs · 2025-12-24T10:54:45Z

Clinic/Clinic.Api/Program.cs

+builder.Services.AddAutoMapper(cfg => cfg.AddProfile<MappingProfile>());
+
+builder.Services.AddEndpointsApiExplorer();
+builder.Services.AddSwaggerGen();


И не добавляет в сваггерген

alxmcs · 2025-12-24T10:56:20Z

Clinic/Clinic.Models/Entities/Specialization.cs

+public class Specialization
+{
+    /// <summary>
+    /// Gets or sets the id specialization.
+    /// </summary>
+    required public int Id { get; set; }


Ни при помощи атрибутов

alxmcs · 2025-12-24T10:59:03Z

Clinic/Clinic.Api/Controllers/AnalyticsControllers.cs

+/// </summary>
+[ApiController]
+[Route("api/analytics")]
+public class AnalyticsController(AnalyticsServices testServices) : ControllerBase


Контроллеры в проекте почему-то все проименованы во множественном числе

alxmcs · 2025-12-24T11:02:02Z

Clinic/Clinic.Api/DTOs/Appointment/CreateAppointmentDto.cs

+    /// <summary>
+    /// The full name of the patient for the appointment.
+    /// </summary>
+    public required string PatientFullName { get; set; }
+
+    /// <summary>
+    /// The full name of the doctor for the appointment.
+    /// </summary>
+    public required string DoctorFullName { get; set; }


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

alxmcs · 2025-12-24T11:03:04Z

Clinic/Clinic.Api/DTOs/Appointment/UpdateAppointment.cs

+    public int? PatientId { get; set; } = null;
+
+    /// <summary>
+    /// Optional: The ID of the doctor to update for the appointment.
+    /// </summary>
+    public int? DoctorId { get; set; } = null;


Почему-то здесь здравый смысл все-таки одержал верх, а в CreateAppointmentDto - нет

alxmcs · 2025-12-24T11:07:20Z

Clinic/Clinic.Api/Interfaces/Services/IAppointmentService.cs

@@ -0,0 +1,25 @@
+using Clinic.Api.DTOs.Appointment;
+
+namespace Clinic.Api.Interfaces.Services;


Весь семестр мы говорили о грамотной декомпозиции кода проекта, о разделении слоев, о логической связности, о domain-centered grouping

Не, зачем, тут будет папка "интерфейсы" и в ней будут валяться все интерфейсы 👍 👍 👍 👍

alxmcs · 2025-12-24T11:08:56Z

Clinic/Clinic.Application/Ports/IAppointmentRepository.cs

@@ -0,0 +1,52 @@
+using Clinic.Models.Entities;
+
+namespace Clinic.Application.Ports;


Архитектурная концепция портов-адаптеров никогда не подразумевала под собой помойку из интерфейсов в отдельной директории с названием "порты"

maeosha added 30 commits October 2, 2025 19:31

feat: add enums: BloodGroup, Gender and RhesusFactor

3d9fdf1

feat: add contract Appointment

b6a1c23

feat: add reference book Specialisation

44a437e

feat: add entities: Doctor and Patient

628947b

feat: add main class Clinic

e96ed1c

feat: add dataseed for unit-tests

7cd6a44

feat: add unit-tests

95cf9fe

feat: add build configuration for C# project

594985e

feat: add solution

f2e2575

add summary and chage blood goup names

ab3db8c

feat: extracted identical variables and functions from the doctors an…

faba51e

…d patients classes into a separate class

feat: Now these classes inherit the basic characteristics from Person…

3e1eda4

…Info. fix: add summary

fix: add summary

06d6b89

fix: Rewrite the tests and datasid, bring them to an adequate form.

2fa6ec7

fix: delete old Contracts/Appointment

b7a94a7

fix: delete SpecialiSation.cs

e2c2f53

feat: implement Patient management functionality with CRUD operations…

5027d2e

… and DTOs

feat: add Specialization management functionality with CRUD operation…

9fa5d71

…s and DTOs

feat: implement Doctor management functionality with CRUD operations …

2c79b53

…and DTOs

feat: implement Appointment management functionality with CRUD operat…

8223f2f

…ions and DTOs

feat: implement ClinicDataBase with CRUD operations for patients, doc…

3f5f1aa

…tors, specializations, and appointments

feat: add DataSeed class to populate the database with test data for …

4f9e9a7

…patients, doctors, specializations, and appointments

feat: add DateConverter for JSON serialization of DateOnly type

ad3ca39

feat: add AutoMapper profile for mapping between DTOs and entity mode…

62c75e7

…ls in the Clinic API

feat: initialize Clinic API with main program setup, including servic…

beb4bcb

…e registrations and Swagger integration

feat: add TestServices and TestControllers for enhanced patient and d…

272ee8c

…octor data retrieval in the Clinic API

feat: update DataSeed

a602e50

fix: change DateOnly to DateTime in GetAppointmentDto for correct dat…

ca81d9a

…e handling

fix: The Clinic.APi folder has been moved

6957ee2

fix: Now Specialization is also an entity

ee57321

maeosha added 6 commits December 21, 2025 14:15

feat: implement analytics controller and refactor existing controller…

7d33040

…s to use base controller pattern

feat: add unit tests and project configuration for Clinic.Tests

f48a48b

feat: add CI workflow for build and test automation

557ee16

feat: configure Swagger UI and enable automatic launch in development…

1202cae

… environment

chore: remove outdated issue and discussion templates, and PR template

257dd06

maeosha changed the title ~~Lab3~~ 6412 Лаб.3 Ксеневич Максим Dec 21, 2025

maeosha closed this Dec 21, 2025

maeosha reopened this Dec 21, 2025

github-actions bot added In progress Код в процессе проверки Lab 3 Лабораторная №3. ORM labels Dec 21, 2025

github-actions bot requested a review from alxmcs December 21, 2025 11:37

alxmcs requested changes Dec 21, 2025

View reviewed changes

Clinic/Clinic.DataBase/EntityFramework/EfAppointmentDataBase.cs Outdated Show resolved Hide resolved

Clinic/Clinic.Api/DTOs/Appointment/CreateAppointmentDto.cs Show resolved Hide resolved

Clinic/Clinic.Api/DTOs/Appointment/UpdateAppointment.cs Show resolved Hide resolved

alxmcs assigned maeosha Dec 21, 2025

feat: update DTO namespaces and add service defaults for improved str…

670e4fd

…ucture and functionality

maeosha requested a review from alxmcs December 21, 2025 20:40

alxmcs requested changes Dec 22, 2025

View reviewed changes

maeosha added 2 commits December 22, 2025 13:02

feat: refactor repository interfaces and implementations for better s…

33b472c

…tructure and add Clinic.Application project

fix: changed the database from MySQL to PostgreSQL

28438c0

maeosha requested a review from alxmcs December 22, 2025 11:13

maeosha and others added 2 commits December 23, 2025 22:51

feat: add in-memory repositories for appointments, doctors, patients,…

c81f41c

… and specializations

Fix project SDK declaration in csproj file

0dc0c2a

alxmcs approved these changes Dec 24, 2025

View reviewed changes

alxmcs added Approved Лабораторная зачтена and removed In progress Код в процессе проверки labels Dec 24, 2025

		@@ -0,0 +1,25 @@
		using Clinic.Api.DTOs.Appointment;

		namespace Clinic.Api.Interfaces.Services;

		@@ -0,0 +1,52 @@
		using Clinic.Models.Entities;

		namespace Clinic.Application.Ports;

6412 Лаб.3 Ксеневич Максим #225

Are you sure you want to change the base?

6412 Лаб.3 Ксеневич Максим #225

Uh oh!

Conversation

maeosha commented Dec 21, 2025

Uh oh!

alxmcs left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maeosha commented Dec 21, 2025

Uh oh!

alxmcs commented Dec 21, 2025

Uh oh!

maeosha commented Dec 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alxmcs left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

maeosha commented Dec 22, 2025

Uh oh!

alxmcs commented Dec 23, 2025

Uh oh!

alxmcs left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

alxmcs left a comment •

edited

Loading

maeosha commented Dec 21, 2025 •

edited

Loading