← Сtrl

ZZZ · Июнь 30, 2009 09:05:31

Я бы сделал так:

orm_sort_key, sort_string = parse_sort_argument(
								request.GET.get('sort', default_sort_value))

piranha · Июнь 30, 2009 09:59:29

lorien
Мне давно мучает вопрос. Возьмём такой код.

Сразу после ( переносишь аргументы на другую строку, и места становится сразу много.

Lexander · Июнь 30, 2009 12:16:04

Получается, только добавлять лишнюю строку и переменную, если по PEP8.

Ferroman · Июнь 30, 2009 14:00:59

Может это и к лучшему
# Разрежённое лучше плотного.
# Удобочитаемость важна.

poltergeist · Июнь 30, 2009 23:53:40

1) Я бы так написал

sort_value = request.GET.get('sort', default_sort_value)
orm_sort_key, sort_string = parse_sort_argument(sort_value)

2) Как вариант, если много аргументов надо передать функции:

orm_sort_key, sort_string = parse_sort_argument(
    request.GET.get('sort', default_sort_value),
)

Сниппет из жизни: в коде встречается много подобных ветвлений:

if one:
    if two:
        if three:
            if four:
                success()
            else:
                print error4
        else:
            print error3
    else:
        print error2
else:
    print error1

Вот придумал такой вот ход (может и не я первый придумал):

class Break(Exception): pass

try:
    if not one:
        raise Break(error1)
    if not two:
        raise Break(error2)
    if not three:
        raise Break(error3)
    if not four:
        raise Break(error4)
    success()
except Break, e:
    print e
except AnotherError, e:
    print e

Как вы считаете, нормальное решение для длинных ветвлений? Мне оно кажется самым линейным, понятным и надёжным.
Я вообще стараюсь использовать вместо логики “if” логику “if not”, это хорошо экономит отступы.

Отредактировано (Июль 1, 2009 00:06:12)

Python-сообщество

Уведомления

#1 Июнь 30, 2009 09:05:31

Стиль именования

#2 Июнь 30, 2009 09:59:29

Стиль именования

#3 Июнь 30, 2009 12:16:04

Стиль именования

#4 Июнь 30, 2009 14:00:59

Стиль именования

#5 Июнь 30, 2009 23:53:40

Стиль именования

Board footer