Как сохранить несколько фотографий, которые получены по api?

Question

[Jordan_Belforts]

Всем привет.
Не пойму как можно сохранить все полученные фотографии из телеграм.
Следующий код сохраняет только первую пришедшую фотку.
Предполагаю, что нужно использовать async, но не силен в нем и не знаю как можно это решить.
Подскажите, пожалуйста, в какую сторону мне смотреть, заранее спасибо за внимание.

@bot.message_handler(content_types=['photo'])
def getting_photos(message):
    if connect.select_status(chat_id=message.chat.id) == "waiting photo":

        response_getfile = requests.post(URL + "getfile", {"file_id": "%s" % message.photo[0].file_id}).json()
        file_path = response_getfile["result"]["file_path"]
        getting_photo = requests.get(URL_FILE + file_path)
        name_file_photo = file_path.split("/")
        directory = add_folder(connect.get_way_folder(chat_id=message.chat.id))
        with open("{}/{}".format(directory, name_file_photo[1]), "wb") as f:
            f.write(getting_photo.content)
        connect.update_status(status="waiting command", chat_id=message.chat.id)

        connect.default_way(message.chat.id)
        bot.send_message(chat_id=message.chat.id, text="Сохранено")

Comments

[sotanodroid]

Никакая асинхронщина тут не нужна. Нужно понять что приходит в запросе (внутри message). Если там много фоток то скорее всего это будет массив с ними.
А сохраняет одну фотку потому что возможно в этом месте явно выбирается только она одна

message.photo[0].file_id

(не уверен что находится в этом объекте)

Возможно стоит проверять массив или обходить его в цикле. Если пришла одна -- обработается одна, если много -- в цикле обработаются все.

Например

for photo in message.photo:
    file_id = photo.file_id

и дальше делать запросы

[Jordan_Belforts]

На каждую фотку своя ссылка приходит. Мысль понял и сейчас попробую. Спасибо!

И такой кратный вопрос по адекватности написанного кода ?!

[shurshur]

Jordan_Belforts, вместо "{}/{}" использовать os.path.join (оно ещё и OS-переносимее станет), вместо file_path.split("/")[1] - os.path.basename.

Вместо "%s" % message.photo[0].file_id можно написать просто str(message.photo[0].file_id) - будет выглядеть чуть более человечно.

Это на тему небольших красивостей.

[Jordan_Belforts]

shurshur, Спасибо =) поправлю