Django. Меня эта ошибка достала я качаю фотки в цикле

Question

Django. Меня эта ошибка достала я качаю фотки в цикле

в формате .jpeg и сохраняю их в базу.
Бывает что фотка скачивается битая её размер в 10 раз меньше, там ошибка.
Вот последняя версия как я пытаюсь избежать исключение, я правильно делаю?

def save_images(listing):
i = 0
for i in range(len(listing.Media)):
try:
f = open(f"{MEDIA_DIR}/download_images/img_{i}.jpeg", 'rb')
except UnidentifiedImageError:
logger.error(f"Exception: for {listing.ListingId}, (broken img)")
f = open(f"{STATIC_ROOT}main/img/other/nocard.webp", 'rb')
finally:
photo = Photo(image=File(f), listing=listing)
photo.save()

#backend #programming #python #russian

0

17.10.2023

14 ответов

10 просмотров

Gregory Автор вопроса

Jack Bolt
Этот open разве PIL'овский? Билтин не выкинет тако...

Исключение из библиотеки PIL. см в трейсбеке предпоследню юстроку

0

6 месяцев назад

Алиса Королёва

так… ты пытаешься сохранить картинку в файл открытый на чтение?

0

6 месяцев назад

Gregory Автор вопроса

Алиса Королёва
так… ты пытаешься сохранить картинку в файл открыт...

так я сохранил в цикле первые 3 файлы img_0 img_1 img_2 img_3 они не чем не отличаются от img_4

0

6 месяцев назад

evle

Файл бы за собой закрывать не забывать. Ну и в целом логика немного вывернутая вышла.

0

6 месяцев назад

evle

Ну и да — затенять стандартный open — к путанице.

0

6 месяцев назад

Jack Bolt

Gregory
Исключение из библиотеки PIL. см в трейсбеке пред...

Так лови исключение из библиотеки, зачем ты его ловишь на билтиновском open?

0

6 месяцев назад

Gregory Автор вопроса

Jack Bolt
Так лови исключение из библиотеки, зачем ты его ло...

Как?

0

6 месяцев назад

Jack Bolt

evle
Ну и да — затенять стандартный open — к путанице.

Pillow'ский open кидает ValueError, если mode != 'r'

0

6 месяцев назад

evle

Jack Bolt
Pillow'ский open кидает ValueError, если mode != '...

При чём тут это?

0

6 месяцев назад

Jack Bolt

evle
При чём тут это?

У него стандартный open там, иначе исключение вылезло раньше

0

6 месяцев назад

evle

Jack Bolt
У него стандартный open там, иначе исключение выле...

Хотя фиг знает, то там вообще просходит. Может и так.

0

6 месяцев назад

Jack Bolt

Gregory
Как?

Ну, раз на photo.save ошибка, поставь эксепт там

0

6 месяцев назад

Gregory Автор вопроса

Jack Bolt
Ну, раз на photo.save ошибка, поставь эксепт там

я уже сделал это def save_images(listing): i = 0 for i in range(len(listing.Media)): try: f = open(f"{MEDIA_DIR}/download_images/img_{i}.jpeg", 'rb') photo = Photo(image=File(f), listing=listing) photo.save() except UnidentifiedImageError(): logger.error(f"Exception: for {listing.ListingId}, (broken img)") f = open(f"{STATIC_ROOT}main/img/other/nocard.webp", 'rb') photo = Photo(image=File(f), listing=listing) photo.save()

0

6 месяцев назад

Jack Bolt · Accepted Answer

Jack Bolt

Этот open разве PIL'овский? Билтин не выкинет такой эксепшн

0

6 месяцев назад

170 похожих чатов

Django. Меня эта ошибка достала я качаю фотки в цикле

14 ответов

Похожие вопросы