0

Я часто слышу, что вложенные условия не грамоты, что есть более лучший способ реализации. Разве это плохо?;

Почему вложенные if считают неграмотный кодом?;

Как правильно реализовывать условия?;

2
  • проблема "плохого" кода всегда одна - читаемость. лучше - ооп, в частности полиморфизм 14 дек 2022 в 19:44
  • Количество ветвлений (условных операторов) определяет сложность программы, чем больше таких операторов, тем сложнее понять программу. Для уменьшения сложности применяют различные способы декомпозиции / преобразования логических выражений 14 дек 2022 в 23:44

1 ответ 1

4

Зачастую вложенные условия понижают читабельность кода. Для примера возьмем функцию, создающую пользователя. Перед созданием нам нужно проверить имя пользователя и его пароль:

USERNAMES = {'peter', '123', 'progamer'}


def create_new_user(username, password):
    global USERNAMES
    USERNAMES.add(username)


def register_user():
    username = input()
    password = input()

    if len(username) <= 20:
        if len(username) > 3:
            if username in USERNAMES:
                print("User with this username already exists")
            else:
                if len(password) > 3:
                    create_new_user(username, password)
                    print("New user is created")
                else:
                    print("The password is too short")
        else:
            print("Username is too short")
    else:
        print("Username is too long")


if __name__ == '__main__':
    register_user()

Функция register_user() выполняет свою задачу, но его тяжело отнести к категории читаемых - несколько вложенных условий, каждрое из которых отвечает за какую-то проверку. Если потребуется добавить новые проверки, то придется каждый раз вчитываться во вложенные условия и стараться не задеть уже существующие условия и их функционал.

У этой проблемы есть два (не взаимоисключающих) решения.

Разворачивание

Если внутри условий есть места, где функция "досрочно" завершает выполнение, то можно попробовать "развернуть" эти условия. Такое условие:

if len(password) > 3:
    create_new_user(username, password)
    print("New user is created")
else:
    print("The password is too short")

Можно развернуть так:

if len(password) <+ 3:
    print("The password is too short")
    return
else:
    create_new_user(username, password)
    print("New user is created")

И так как при выполнении условия мы покидаем фунцию, мы можем убрать else (тем самым уменьшая вложенность):

if len(password) <+ 3:
    print("The password is too short")
    return
create_new_user(username, password)
print("New user is created")

Применив эту технику несколько раз, в конце получим изначальную функцию register_user() в таком виде:

def register_user():
    username = input()
    password = input()

    if len(username) > 20:
        print("Username is too long")
        return

    if len(username) <= 3:
        print("Username is too short")
        return

    if username in USERNAMES:
        print("User with this username already exists")
        return

    if len(password) <= 3:
        print("The password is too short")
        return

    create_new_user(username, password)
    print("New user is created")

В таком виде код довольно линеен, добавление новых проверок (и действий в случае их нарушения) стало проще.

Вынесение

Еще один способ "облегчения" условий - вынесение их в отдельные функции. Пример:

def is_username_valid(username):
    if len(username) > 20:
        print("Username is too long")
        return False

    if len(username) <= 3:
        print("Username is too short")
        return False

    if username in USERNAMES:
        print("User with this username already exists")
        return False

    return True


def is_password_valid(password):
    if len(password) <= 3:
        print("The password is too short")
        return False
    return True


def register_user():
    username = input()
    password = input()

    if not is_username_valid(username):
        return

    if not is_password_valid(password):
        return

    create_new_user(username, password)
    print("New user is created")

Здесь проверки вынесены в отдельные функции, что повысило читаемость - теперь по одним лишь названиям вызываемых функций точно видно, что register_user() проверяет имя пользователя и пароль, и если они валидны, создает пользователя.

P.S. Для практики и лучшего понимания можно вернуться к своему старому коду и попробовать повысить его читаемость этими способами.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.