LINUX.ORG.RU

А почему autopep8 такой ужас ?

 


0

2

Вот так autopep8 исковеркал переносами текст:

           res = db.session.query(Test).filter(
                and_(
                    test.test_id == test_id,
                    Test.begin > begin -
                    timedelta(
                        hours=24),
                    Test.begin < begin +
                    timedelta(
                        hours=24)))
Это норм ?



Последнее исправление: Jopich1 (всего исправлений: 2)

Чем больше текст похож на стихи Маяковского, тем больше pep8.

SuoiCat
()

Это норм ?

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

Хотя нафига оно делает перенос тут, вообще не понятно:

                    timedelta(
                        hours=24),

Psych218 ★★★★★
()
Последнее исправление: Psych218 (всего исправлений: 1)

слишком сложный код для питона
разбивай на функции и переменные

Bad_ptr ★★★★★
()

Никогда не понимал зачем люди полагаются на PEP8 за пределами его применимости.

anonymous
()
Ответ на: комментарий от Psych218

Предполагаю, там длинна строки уже больше 81, оно в таком случае переносит посимвольно вот так вот. У нас на работе за такое могут и коворкинг устроить)

ykroop
()

С какими ключами запускаешь? Если с каким-нить -a/--aggressive то всё понятно.

true_admin ★★★★★
()
Ответ на: комментарий от ykroop

У нас на работе за такое могут и коворкинг устроить)

Это что у вас, наказание такое?)

dave ★★★★★
()
Ответ на: комментарий от ykroop

дык если это hours=24 не переносить, строка всё равно останется короче, чем предыдущая и следующая. Сам посмотри:

           res = db.session.query(Test).filter(
                and_(
                    test.test_id == test_id,
                    Test.beginefir > begin -
                    timedelta(hours=24),
                    Test.begin < begin +
                    timedelta(hours=24)))
Psych218 ★★★★★
()
Последнее исправление: Psych218 (всего исправлений: 1)

and_(

Спасибо, блеванул. А так каноничный пытон, гыгы.

bread
()

autopep8

Это у вас религия такая? Мда, тяжела и неказиста жизнь простого пытониста.

bread
()

1. Дополнительный таб под аргументы для каждого вложенного вызова функции. Все ок, но я бы так вложенно не делал.

2. Люди отписавшиеся выше не очень умные.

anonymous
()
Ответ на: комментарий от Psych218

Хотя нафига оно делает перенос тут, вообще не понятно:

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

anonymous
()

Консистенция.

Отсутствует.

EXL ★★★★★
()

исковеркал

нече было автоутилитам код доверять. Он тебе еще втюхает 80 символов на строку, потому что когда-то давно это было норм. Проще использовать линтеры в качестве плагинов редактора и опираться на советы.

Dred ★★★★★
()
Ответ на: комментарий от Psych218

нафига оно делает перенос

может он в 80 символов не уложился ? Хотя лишникие перенсы строк ИМХО, убивают всю читабельность

Dred ★★★★★
()
Ответ на: комментарий от Dred

может он в 80 символов не уложился

Да говорю же, строка выше не перенесена при том, что она всё равно ещё длиннее, чем эта без переноса. Вот как оно без этого переноса выглядит:

           res = db.session.query(Test).filter(
                and_(
                    test.test_id == test_id,
                    Test.beginefir > begin -
                    timedelta(hours=24),
                    Test.begin < begin +
                    timedelta(hours=24)))
Psych218 ★★★★★
()

А почему autopep8 такой ужас ?

Это гвидобейсик, детка!

anonymous
()

Нет конечно, нахрена два раза дельту считать?

           delta = timedelta(hours=24)
           res = db.session.query(Test).filter(
                and_(
                    test.test_id == test_id,
                    Test.begin > begin - delta,
                    Test.begin < begin + delta))
Deleted
()

А вообще у нас есть специалист по pep8, Goury он щас расскажет

Deleted
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.