Как избавиться от большого количества конструктий if-else?

Question

[MrxAlexo]

Пишу бота, который использует запросы к api сайта. Приходится обрабатывать коды ошибок. Приведу пример:

response = auth() # тут ответ от api сайта.
if response.get('code') == 0:
    get_products = get_products_by_title('Test product')
    if get_products.get('code') == 0:
        buy = buy_product(get_products.get('data').get('product_id'))
        if buy .get('code') == 0:
            print('Товар куплен!')
        else:
            print('Ошибка покупки товара')
    else:
        print('Ошибка получения товаров')
else:
    print('Ошибка авторизации')

И только здесь использовано 3 конструкции, а таких у меня в коде множество. Помимо этого есть множество других кодов ошибок, которые тоже необходимо обрабатывать. Подскажите, как это красивее и эффективнее написать без овнокода, чтобы это имело читабельный вид?

Answer 1

[rPman]

Это нормально.
И да, много это не 2-3 уровня а сотни и тысячи.

Да, у программистов обычно зуд на такие вещи, видишь повторяющийся код или логические конструкции и пытаешься их заменить на какое то их описание и код который по этому описанию делает всю логику.

Всегда нужно найти момент и силы остановиться и не доводить до маразма, а то так дойдет до написания языка программирования с помощью которого описываешь свою логику, сложность кода растет, вероятность ошибок тоже.

Comments

[Марат Нагаев]

Код с сотнями и тысячами уровней вложенности это говнокод.
Если не укладываешься в 4, то скорее всего что-то делаешь не так.
Это тоже самое, что функции на 5к строк.
У хорошего программиста будет условно 100 функций на 50 строк, а эти 100 функций раскиданы по 10 модулям.

[rPman]

я это и сказал, 2-3 if-а это не проблема, проблема когда действительно много

p.s. есть метапрограммирование, когда ты генерируешь код по своему описанию (или какой то препроцессор) или банально пишешь программу которая генерирует код, вот в этом сгенерируемом коде может быть милион if, гиганские функции, и прочий треш, и это будет нормально, так как этот промежуточный сгенерированный код читать никто не будет (кроме исправления ошибок генерации)

Answer 2

[Dmitrii]

Есть также паттерны, например, цепочка обязанностей, которые позволяют немного упростить такие проверки, но этот паттерн полезен для сложных проверок/команд

Answer 3

[thatmaniscool]

Для решения вашей задачи используйте тернарный оператор.

Comments

[Сергей Горностаев]

Чтобы код стал ещё менее читаемым?