Хм. Смотрите, у меня возникает в коде задача уже второй

раз. Мне нужно по кнопке "проверить статус платежа" кидать запрос на платёжку, и, в зависимости от его статуса, делать что-то. Прошлый раз решил ифами, этот раз придумал вот такую штуку:

from_cryptomus_status_to_custom = {
**{
s: "pending"
for s in [
'process',
'wrong_amount_waiting',
'confirm_check',
'check'
]
},
**{
s: "done"
for s in [
'paid',
'paid_over'
]
},
**{
s: "canceled"
for s in [
'wrong_amount',
'fail',
'cancel',
'system_fail',
'locked'
]
}
}

...

payment: dict = ...

match get_payment_status(payment['status']):
case 'pending':
...
case 'done':
...
case 'canceled':
...
case 'error':
...

def get_payment_status(status: str) -> str:
return from_cryptomus_status_to_custom.get(status, "error")


Есть ли смысл делать как я, свои кастомные статусы, или через ифы просто разбить на списки?

типа...

status = payment['status']

if status in ['paid', 'paid_over']:
...
elif status in [
'wrong_amount',
'fail',
'cancel',
'system_fail',
'locked'
]:
...

и т.д.?

Скорость выполнения вообще не учитываю, там разница в малые доли секунды. Чисто с точки зрения читабельности и удобства

Для начала выкинуть магические константы. Например завести Enum