Python, как красивее написать

Тут 2 проверки, потому в этом случае точно не знаю как бы заменить, а вообще глянь вот тут Пример:

d = {}
d.setdefault('one', []).append('two')
In [4]: d
Out[4]: {'one': ['two']

ad_photos = AdPhoto.objects.filter(ad=self.post)
if ad_photos.exists():
    if ad_photos.filter(main=True).exists():
        ad_photos = ad_photos.filter(main=True)
    context['header_photo'] = ad_photos.first()
else:
    context['header_photo'] = None

Как вариант. Правда БД дергается на 1 раз больше.

Во-первых пойми уже наконец то ограничение длинны строки — атавизм рака хипстеров.
Во-вторых в ecxept надо обязательно указывать что эксцептать, а не просто всё подряд.
В-третьих для выборки одного объекта нужно get использовать в случае с обработкой исключения.
В-четвёртых [:1][0] это какой-то бздец, гет и [0] от фильтра вернут то же самое.

Код за тебя писать не хочется, давай сам.

если больше одного объекта возвращает через get то ругается и падает приложение.

Тогда уж

filter = AdPhoto.objects.filter
context['header_photo'] = filter(ad=self.post, main=True).first() or filter(ad=self.post).first()

В-третьих для выборки одного объекта нужно get использовать в случае с обработкой исключения.

Нет, .first(). MultipleObjectsReturned — не исключительная ситуация.

ну тогда фильтрай, но тогда исключений быть не должно вообще

Затенять stdlib'овую функцию локальной переменной? Идея лучше, но назвать переменную надо по-другому.

Окай, пусть будет filter().first()

Да, подбором названия я не парился. Хотя если оно в скопе дефа — проблем не будет

Правда БД дергается на 1 раз больше

Это диагноз говнокода

Спасибо так выглядит чище.

Супер, не знал что так можно делать :) спасибо всем большое.

Во-первых пойми уже наконец то ограничение длинны строки — атавизм рака хипстеров.

тот же pep8/pyflakes будет ругаться на строку длиннее 79 символов

--max-line-length=100

Их можно настроить на 119 символов.

ими можно вообще не проверять :)

но в PEP8 написано английским по белому:

Maximum Line Length

Limit all lines to a maximum of 79 characters.

А в гуглегайде — уже нет: там разрешают, например, URL'ы длиннее 80 символов.

но в PEP8 написано английским по белому:

This document gives coding conventions for the Python code comprising the standard library in the main Python distribution.

The standard library in the main Python distribution БЛДЖАД.
Хватит уже дрочить на него как на священную корову, в питоне его приняли исключительно потому, что толпа кривожопых хипстеров иначе не может между собой договоритсья. Если у тебя не толпа кривожопых хипстеров код пишет — не надо, блджад, пихать этот пеп в свой проект.

И гуглегайд тоже не имеет отношения ни к чему кроме проектов гугля. Купите уже в магазине голову и думайте ей.

мне наплевать, что там написано. я отвечал на вполне конкретную претензию (написанную, правда, на нерусском языке, поэтому я мало что понял, больше догадался) «откуда вы взяли, что нельзя длинных строк?». я и указал, откуда.

Some teams strongly prefer a longer line length. For code maintained exclusively or primarily by a team that can reach agreement on this issue, it is okay to increase the nominal line length from 80 to 100 characters (effectively increasing the maximum length to 99 characters), provided that comments and docstrings are still wrapped at 72 characters.

Я обычно сам с собой договариваюсь на сто.

Во-первых пойми уже наконец то ограничение длинны строки — атавизм рака хипстеров.

Мы гоним тебя, насмехаемся над тобой. https://github.com/torvalds/linux/pull/17#issuecomment-5654674

я на 79, причём в любых языках. Помогает думать лаконичными переменными и комментариями, не превышать допустимую вложенность, разрабатывать в тайлинге, а не фуллскине с вкладками.

https://docs.djangoproject.com/en/1.9/internals/contributing/writing-code/cod...

An exception to PEP 8 is our rules on line lengths. Don’t limit lines of code to 79 characters if it means the code looks significantly uglier or is harder to read. We allow up to 119 characters as this is the width of GitHub code review; anything longer requires horizontal scrolling which makes review more difficult. This check is included when you run flake8.

Молодец.

льстюк %)

Просили красиво, а не производительно.

Говнокод === некрасиво

А больше никак?

А больше 119 не влезает в гитхабовый просмотр диффа без переноса.

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

Если море вложенных циклов, значит ты делаешь что-то не правильно. Можно наверное раздробить на отдельные методы.

я бы с радостью, но там весьма специфическая функция внутри, пока никаких идей в голову не пришло