@anonymous 1. Не знаю, как в алхимии, но вангую, что даже по пустой выборке можно итерироваться, так что `if photos:` лишний.
2. Наверняка это можно было на уровне ORM отфильтровать, но раз уж питоном, то хотя бы как-то так: `for photo in filter(bool, photos):` вместо второй и третьей строки.
Итого: эта параша из трёх строк с генерацией ненужного списка (читай массива) (манька, ты про genexpr не слышал, что ли? Пиздец же) заменяется одной, да и та не нужна скорей всего, т.к. разрулить надо было при выборке данных.
@anonymous Хз, мне похуй если честно. Я тогда соцсеть за две недели написал - до сих пор работает, а ты дальше думай как там нужно было. //inb4 житхаб анонимусу не кину, там полный деанон
Ебать вас бомбит в этом треде от какой-то хуйни, никто даже не заметил, какую йобу здесь пришлось провернуть с алхимией, все уткнулись в какой-то "лишний" список, это что, все, на что вы способны? Да этому коду более двух лет и писался он за две недели, наверное, это было какое-то упрощение, которое нужно было потом переписать, но меня забрали в армию и стало похуй. Но вы такие хуегрызы, что даже лучше-то не предложили ничего, один предлагает итерироваться по None, второй pep8 почитать, еще что мне сделать?
@anonymous Плохо зделоно. Используя методы, возвращающие коллекцию, я ожидаю всегда получить коллекцию, даже если выборка пустая. Почему бы не пустая коллекция? В логическом контексте она будет False, `if photos` будет работать так же. Хотите отличать её по типу? Окей, сделайте пустую коллекцию поддтипом основного типа (ненужно). А в случае с None я вынужден явно обрабатывать два случая, даже тогда, когда мог бы без этого обойтись (как здесь, например).
@je Да. В нормальных компаниях (где платят не один жалкий килобакс) макака, не осилившая даже код стайл, не нужна. Так и будешь говнокодить в своём болоте, жрать лосось и дрочить на стодолларовые бумажки, вместо того, чтобы просто жить как человек.
@je так понятно же что все у кого хоть капля мозгов есть уже давно сьебали в столицы
можешь ещё в деревню сьездить и всем рассказывать что ты дохуя самогонки выжираешь
@je Маня, ты долбоёб? Долбоёб? Читай хоть, что пишут. В YWU я сделал *предположение* (явно указав на это) и получил ниже комментарий, что на самом деле не так, как я подумал, Я (тот же самый человек, блядь, понимаешь, не кто-то ещё) прокомментировал ещё ниже, почему я считаю такую реализацию плохой. Ты как вывод вообще сделал, что кто-то что-то объясняет, очевидно же по нити, где автор. Ну ты и еблан, конечно.
@anonymous Дяденька, я вообще всего год работаю программиздом, когда мне съебывать в столицы? Да и не шибко мне они симпатичны, что москва что питер хуйня какая-то, как там на велике кататься?
@anonymous Бля, ну во-первых 58-60 (с проектными премиями до 80), во-вторых больше без анальных требований здесь я не видел. Есть каких-то два тухлых объявления на 90К типа за джангу, но в эту хуйню я не верю, на джанге у нас более 30к редко получают.
@je > какую йобу здесь пришлось провернуть с алхимией
Ээээ, КАКУЮ? Простейшие запросы уровня ехал JOIN через WHERE. В чём хоть минимальная сложность, лол?
> этому коду более двух лет
В том и дело, ты и два года назад был таким же заносчивым переоценивающим себя нарциссом, **нихуя не изменилось**
> писался он за две недели
Ну пошли оправдания
> нужно было потом переписать
Ну я кароч технический долг взял, ну кароч верну через неделю, мамой клянусь)))
@je > всего год
А пафоса и гонора, как будто 30 лет в профессии
К тому же, не
> всего год
а
> уже год
Это немаленький срок, у тебя же был бекграунд уже. Ты же не с нуля пришёл джунить, лол. Считай текущий год у тебя просто проёбан впустую, ты нихуя не развиваешься в этом болоте. Как был долбоёбом, так и остался, как не умел программировать, так и не умеешь. Ну а как тут развиться, если сложность низкая (хотя ты настолько неквалифицированный, что тебе даже это за УУУ УАУ ЕБАТЬ ВОТ ЭТО ЗАДАЧА).
@je В этом и дело.
Вместо того, чтобы написать простой, понятный, эффектинвый и рабочий код, ты занимался онанизмом.
Собственно, любой питонщик такой. Ему программировать некогда, ему КОМПУТОР СЦАЕНС надо.
@je итерироваться по None не надо так же, как и проверять null pointer, оп тебе предложил норм решение
пеп8 даже читать не надо, ставь линтер/форматтер, чтобы пацаны в офисе не унижали, если конечно там не обрыганы ещё хуже тебя
@anonymous Тебе смешно, а я как в столицу приезжаю — у меня здоровье портится.
И начинаются, блядь, мысли о том, что деньги — это не главное, и прочая хуйня.