From 8c1431165c4b08c7c1e97852ab285b3a1e042b9f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=90=D0=BD=D1=82=D0=BE=D0=BD?= Date: Mon, 15 Sep 2025 08:12:34 +0300 Subject: [PATCH] =?UTF-8?q?DAO=20-=20=D0=9F=D0=BE=D0=BB=D0=BD=D0=BE=D0=B5?= =?UTF-8?q?=20=D0=B7=D0=B0=D0=B2=D0=B5=D1=80=D1=88=D0=B5=D0=BD=D0=B8=D0=B5?= =?UTF-8?q?=20=D0=BA=D0=BB=D0=B0=D1=81=D1=81=D0=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- README.md | 12 ++++++++++-- src/DAO/MemoryAuthDAO.cpp | 2 +- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/README.md b/README.md index 739e0e1..b7400b4 100644 --- a/README.md +++ b/README.md @@ -6,6 +6,14 @@ - Посмотреть, что дает MySQL, какие там есть возможность - Посмотреть и подумать, что лучше - корутины или многопоточность? - Покрыть тестами класс User и AuthRegistrationExecutor +- Добавить clang-format(через CLion) +- Перевести GetByUUID GetByLogin на const ref/string_view в IUserDAO - также не vector, а span(погуглить) +- Привести к единоу виду функции IUserDAO +- Пройтись по коду и максимально наставить const +- Указать возможные исключения в интерфейсах DAO +- Вынести User в структуру. Hashed Password структура должна изначально состоять в другой структуре +- SharedPtr - передавать по константной ссылке. +- Сделать интеграционный тест по ручкам # UseCase'ы приложения: @@ -111,8 +119,8 @@ ``` ##### Errors -* `401 BAD_CREDENTIALS` — неверный логин/пароль (А1) -* `422 VALIDATION_FAILED` — пустые поля (А2) +* `401 BAD_CREDENTIALS` — неверный логин/пароль (B1) +* `422 VALIDATION_FAILED` — пустые поля (B2) #### Logout-Request ``` diff --git a/src/DAO/MemoryAuthDAO.cpp b/src/DAO/MemoryAuthDAO.cpp index 434fadb..3573c29 100644 --- a/src/DAO/MemoryAuthDAO.cpp +++ b/src/DAO/MemoryAuthDAO.cpp @@ -31,7 +31,7 @@ bool MemoryAuthDAO::Logout(const std::string& auth_token) { string user_uuid = auth_tokens_to_users_uuids_[auth_token]; - if (!HasAuthorizedUser()) return false; + if (!HasAuthorized(auth_token)) return false; users_uuids_to_auth_tokens_.erase(user_uuid); auth_tokens_to_users_uuids_.erase(auth_token);