Skip to content

Спринт 10 - Добавлены контроллеры - #1

Merged
milayadasha merged 2 commits into
mainfrom
controllers-films-users
Dec 24, 2025
Merged

milayadasha merged 2 commits into
mainfrom
controllers-films-users

Conversation

@milayadasha

Copy link
Copy Markdown
Owner
  1. Добавлены классы-контроллеры FilmController и UserController c методами по обработке GET, POST, PUT запросов
  2. Добавлены классы модели данных Film и User
  3. Добавлены классы тестов

@maks-avilow maks-avilow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Дарья привет! хорошее начало - молодец! тем не менее предлагаю ещё внести немного доработок. посмотри пожалуйста комментарии

Comment thread pom.xml
Comment thread src/main/java/ru/yandex/practicum/filmorate/controller/FilmController.java Outdated

@maks-avilow maks-avilow left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

привет, принято

/**
* Добавляет нового пользователя.
* Проверяет поля пользователя на соответствие.
* Если всё хорошо, то создаёт копию переданного пользователя, присваивает уникальный ID и сохраняет в контроллер.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

за javadoc плюсую

@milayadasha
milayadasha merged commit b36c4b3 into main Dec 24, 2025
2 checks passed
@milayadasha
milayadasha deleted the controllers-films-users branch December 24, 2025 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants