From 66ab13b45df9eedc31ea879e7a48a87eca871bcf Mon Sep 17 00:00:00 2001 From: StNicolay Date: Sat, 5 Nov 2022 00:25:23 +0300 Subject: [PATCH] Changed handlers.get_accounts to use a guard clause, added comment to clarify, why we are putting account in backticks --- src/bot/handlers.py | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/src/bot/handlers.py b/src/bot/handlers.py index 697924f..83ab037 100644 --- a/src/bot/handlers.py +++ b/src/bot/handlers.py @@ -26,17 +26,19 @@ def get_accounts( ) -> None: base_handler(bot, mes) accounts = database.get.get_accounts(engine, mes.from_user.id) - if accounts: - accounts = [f"`{account}`" for account in accounts] - return send_tmp_message( - bot, - mes.chat.id, - "Ваши аккаунты:\n" - + "\n".join(accounts) - + "\nНажмите на название, чтобы скопировать", - 30, - ) - send_tmp_message(bot, mes.chat.id, "У вас нет аккаунтов") + if not accounts: + return send_tmp_message(bot, mes.chat.id, "У вас нет аккаунтов") + + # Make accounts copyable and escape special chars + accounts = [f"`{account}`" for account in accounts] + send_tmp_message( + bot, + mes.chat.id, + "Ваши аккаунты:\n" + + "\n".join(accounts) + + "\nНажмите на название, чтобы скопировать", + 30, + ) def delete_all(