Pep8 как правильно переносить строку в таком варианте?

Question

[Refiru]

Доброго дня, как же правильного переносить строки?

# мне кажется так читается лучше
cursor.execute("""CREATE TABLE IF NOT EXISTS Users
(ID INTEGER NOT NULL PRIMARY KEY, Text TEXT, Configuration Text) """)
# в коде часто вижу так
cursor.execute("""CREATE TABLE IF NOT EXISTS Users
                (ID INTEGER NOT NULL PRIMARY KEY,
               Text TEXT, Configuration Text) """)

Оба варианта умещаются в строку 79 символов.
Но первый не выравнивается с другими параметрами пробелами.
А второй читается хуже, ИМХО. Точнее мне проще читать SQL запрос в первом варианте, но кажется люди пытаются выделить "cursor.execute"?
Для pep8 это вообще важно? или это тот случай, где можно писать как удобнее?

Comments

[pcdesign]

Имхо, лучше использовать sqlalchemy и забыть о raw sql запросах, которые вперемежку с кодом питона представляют из себя дикое нечитабельное месиво.
И помимо sqlalchemy сейчас есть еще другие модные orm, можно выбрать на вкус.

Answer 1

[Ivan Yakushenko]

В первую очередь это аргумент и в pep8 все довольно ясно прописано:

The closing brace/bracket/parenthesis on multiline constructs may either line up under the first non-whitespace character of the last line of list, as in:
or it may be lined up under the first character of the line that starts the multiline construct, as in:

В документации Python используется первый вариант.
Но и второй вариант более чем допустим:

cursor.execute("""
    CREATE TABLE IF NOT EXISTS Users
    (ID INTEGER NOT NULL PRIMARY KEY,
    Text TEXT, Configuration Text)
""")

P.S. Лично мне гораздо удобнее избегать в аргументах формирование строк, так что я всегда делаю так:

query = (
    'CREATE TABLE IF NOT EXISTS Users('
    'ID INTEGER NOT NULL PRIMARY KEY,'
    'Text TEXT,'
    'Configuration Text)'
)
cursor.execute(query)

В целом довольно часто встречаю подобный подход.

Comments

[Сергей Карбивничий]

В документациях примеры не всегда написаны по стилю. Я сейчас не про Python.
Был у меня случай, когда кодил на C#. Некоторый код копировал с MSDN. Потом в Visual Studio решил проверить код на правильность стиля. Так Студия мне выдала кучу варнингов. При чем на участки кода с MSDN. Я поспрашивал в интернете, на что мне ответили - 'Забей и отключи некоторые варнинги.' Вот так)
Использовал анализатор кода, разработанный MicroSoft)

[Ivan Yakushenko]

Dr. Bacon, да, так и есть, особенно с каким-то конструкциями вроде:

async with session.get(url, params, headers) as response:
    ...

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

Answer 2

[Сергей Карбивничий]

Как по мне, здесь не только pep8, но и SQL Style.
Почитайте интересную статью - Руководство по стилю SQL