style(Bot): Полировка кода

Author: intervisionlord

handlers/bot_routes/route_edit_items.py

@@ -1,17 +1,16 @@
 """Логика редактирования элементов карточки"""
 
-from aiogram import Router, types, F
+from aiogram import F, Router, types
 from aiogram.utils.keyboard import InlineKeyboardBuilder
 
-from handlers.handler_nc_deck import update_card_description
-from handlers.bot_routes.states import CardCallback
+from handlers.bot_routes.route_list_cards import list_handler
 from handlers.bot_routes.route_view_card import (
-    get_cached_card,
     _show_card_view,
+    get_cached_card,
 )
-from handlers.bot_routes.route_list_cards import list_handler
-
-# from handlers.handler_logging import logger
+from handlers.bot_routes.states import CardCallback
+from handlers.handler_logging import logger
+from handlers.handler_nc_deck import update_card_description
 
 edit_router = Router()
 
@@ -48,13 +47,15 @@ async def toggle_item_handler(
         success = await update_card_description(target_card.id, new_description)
 
         if success:
+            logger.success(f"Статус карочки {target_card.id} обновлен")
             await callback.answer("✅ Статус обновлен")
-            # Обновляем кэш и показываем обновленную карточку
             target_card.description = new_description
             await _show_card_view(callback.message, target_card)
         else:
+            logger.error(f"Статус карточки {target_card.id} не был обновлен")
             await callback.answer("❌ Ошибка обновления")
     else:
+        logger.error(f"Элемент карточки {target_card.id} не был найден")
         await callback.answer("❌ Элемент не найден")
 
 

handlers/bot_routes/route_view_card.py

@@ -69,13 +69,13 @@ async def _show_card_view(message: types.Message, card) -> None:
     for index, item in enumerate(items):
         description = card.description or ""
         lines = description.split("\n")
-        EMOJI_CHECKED = "✅"
-        EMOJI_UNCHECKED = "🔳"
+        emoji_checked = "✅"
+        emoji_unchecked = "🔘"
 
         emoji = (
-            EMOJI_CHECKED
+            emoji_checked
             if index < len(lines) and "[x]" in lines[index]
-            else EMOJI_UNCHECKED
+            else emoji_unchecked
         )
 
         display_item = item
@@ -184,16 +184,16 @@ async def _parse_new_items(text: str) -> list:
     :return: Список очищенных элементов
     :rtype: list
     """
-    ITEM_SEPARATOR = ","
+    item_separator = ","
 
     new_items_text = text.strip()
     if not new_items_text:
         return []
 
-    if ITEM_SEPARATOR in new_items_text:
+    if item_separator in new_items_text:
         return [
             item.strip()
-            for item in new_items_text.split(ITEM_SEPARATOR)
+            for item in new_items_text.split(item_separator)
             if item.strip()
         ]
     return [new_items_text]
@@ -213,20 +213,12 @@ async def _add_items_to_card(message: types.Message, card) -> None:
         return
 
     logger.info(f"Парсинг элементов: {new_items}")
-
-    # Получаем текущие элементы с их состояниями
     current_items_with_states = card.get_list_items_with_states()
     current_item_texts = [item["text"] for item in current_items_with_states]
-
-    # Добавляем только новые элементы (исключаем дубликаты)
     for new_item in new_items:
         if new_item not in current_item_texts:
             current_items_with_states.append({"text": new_item, "checked": False})
-
-    # Создаем обновленный список текстов элементов для обновления
     updated_item_texts = [item["text"] for item in current_items_with_states]
-
-    # Обновляем описание карточки
     new_description = card.update_list_items(updated_item_texts)
 
     logger.info(f"Обновление карточки {card.id}: {len(updated_item_texts)} элементов")
@@ -235,7 +227,6 @@ async def _add_items_to_card(message: types.Message, card) -> None:
 
     if success:
         await message.answer(f"✅ Добавлено {len(new_items)} элементов в '{card.title}'")
-        # Обновляем кэш
         card.description = new_description
         _card_cache[card.id] = card
         await list_handler(message)

handlers/handler_server.py

@@ -46,7 +46,6 @@ async def lifespan(_: FastAPI):
     logger.info("Сервер остановлен")
 
 
-# Создаем приложение с lifespan
 app = create_application()
 app.router.lifespan_context = lifespan
 

models/model_card.py

@@ -26,11 +26,11 @@ class ModelCard(BaseModel):
     @property
     def short_title(self) -> str:
         """Сокращенный заголовок для кнопок"""
-        MAX_TITLE_LENGTH = 30
-        ELLIPSIS_LENGTH = 3
+        max_title_length = 30
+        ellipsis_length = 3
 
-        if len(self.title) > MAX_TITLE_LENGTH:
-            return self.title[: MAX_TITLE_LENGTH - ELLIPSIS_LENGTH] + "..."
+        if len(self.title) > max_title_length:
+            return self.title[: max_title_length - ellipsis_length] + "..."
         return self.title
 
     def get_list_items(self) -> List[str]:
@@ -95,7 +95,6 @@ def update_list_items(self, items: List[str]) -> str:
                 if not line:
                     continue
 
-                # Определяем состояние элемента с помощью match case
                 match line:
                     case line if line.startswith("- [x] "):
                         item_text = line[6:].strip()
@@ -141,7 +140,6 @@ def get_list_items_with_states(self) -> List[dict]:
             if not line:
                 continue
 
-            # Определяем состояние элемента с помощью match case
             match line:
                 case line if line.startswith("- [x] "):
                     item_text = line[6:].strip()