3

как перенести эту строку кода? просто по правилам максимум 101 символов, а тут 150

lines = *map("".join, field), *map("".join, zip(*field)), *map("".join, zip(*((row[index], row[-index - 1],) for index, row in enumerate(field))))
4
  • 3
    Используй символ \ для переноса, после чего жмякай enter.
    – Xybyyn
    13 янв 2021 в 17:03
  • максимум 80, но ладно
    – FotonPC
    13 янв 2021 в 17:14
  • 5
    Мне больно от этой строки...
    – andreymal
    13 янв 2021 в 17:15
  • 1
    @FotonPC нет, 101, компилятор так показал, а он требует все по правилам PEP8
    – Андр
    13 янв 2021 в 17:15

2 ответа 2

4
lines = *map("".join, field), \
        *map("".join, zip(*field)), \
        *map("".join, zip(*((row[index], row[-index - 1],)
                            for index, row in enumerate(field))))

Проверял с помощью flake8 - ошибок не показывает.

Почему for так нужно форматировать - потому что здесь он часть конструкции (generator expression):

((row[index], row[-index - 1],) for index, row in enumerate(field))

И если переносить for на следующую строку, то он должен быть на 1 пробел правее самой левой скобки этой конструкции (не самой левой скобки предыдущей строки), таким образом как бы оставаясь внутри этой скобки (это создатели pep8 подразумевают под visual indent):

((row[index], row[-index - 1],)
 for index, row in enumerate(field))

Лайфхак: можно было вставить эту строку в PyCharm, добавить переносы строк, все отступы PyCharm сам бы правильно сделал (не всегда работает на 100% правильно, но с переносом generator expression и более простых конструкций работает).

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

2
  • поставьте еще пробелы после запятых, по правилам PEP8 всегда так стоят пробелы, а в остальном все прошло
    – Андр
    13 янв 2021 в 17:33
  • 1
    @Андр, поправил, еще обратите внимание на то что я добавил про PyCharm, это может сильно упростить вам жизнь.
    – insolor
    13 янв 2021 в 17:45
3
lines = *map("".join, field),\
        *map("".join, zip(*field)),\
        *map("".join, zip(*((row[index], row[-index - 1],)
            for index, row in enumerate(field))))

Вот так ^. Используйте слеш для переноса строк (любых).

0

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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