Request.get python?

Question

[adron90]

качаю фотки с контакта
вместо 641 качает 300 с чем то
хотя в консоль выводит и номер и ссылку

почему так?

import os
import vk
import requests
import datetime
import time
user = '1'
vkapi = vk.API(app_id, login, pass, access_token = ********)
user_info = vkapi.users.get(user_ids = user)
name = str(user_info[0]['first_name'])+' '+str(user_info[0]['last_name'])
def timestamptodate(timestamp):
    return datetime.datetime.fromtimestamp(timestamp).strftime('%Y.%m.%d-%H.%M')
def save_photo(img, date):
    if (os.path.exists('C:\\Users\\home\\Desktop\\vk_photo\\'+name) == False):
        os.makedirs('C:\\Users\\home\\Desktop\\vk_photo\\'+name)
    f = requests.get(img)
    foto =  open('C:\\Users\\home\\Desktop\\vk_photo\\'+name+'\\'+date+'.jpg', 'wb')
    foto.write(f.content)
    foto.close()
def get_photo(n):
    return vkapi.photos.getAll(owner_id=user, photo_sizes=0, rev=0, count=200, offset=n)
def parce_photo():
    n = 0
    y = 0
    while (n<=800):
        photo = get_photo(n)
        for i in photo['items']:
            key = i.keys()
            max = 0
            for x in key:
                if x.startswith('photo_')and(int(x[6:])>max):
                    max = int(x[6:])
            y = y+1
            img = i['photo_'+str(max)]
            date = timestamptodate(i['date'])
            print(y, i['photo_'+str(max)])
            time.sleep(1)
            save_photo(img, date)
        n += 200
parce_photo()

Answer 1

[Oscar Django]

Быть может для некоторых фоток date совпадает и более поздние перезаписывают предыдущие.

Comments

[adron90]

Может быть. Попробую секунды добавить в название

[adron90]

Спасибо. Ты был прав.
Секунды тоже не помогли. Пришлось еще счетчик к названию прикрутить

Answer 2

[Владимир Татарников]

Что сверяет уловие максимума if x.startswith('photo_')and(int(x[6:])>max) по сути? Запусти отладку pdb и все увидишь своими глазами. Проверь, что возвращает твое условие.

Comments

[adron90]

Как сравнить ключи в словаре?

[Владимир Татарников]

Ну здесь логика такова, что у тебя методом пузыря отлавливаются все значения, больше max. Но не факт, что у тебя все значения являются больше этого. Может ли быть так, что сначала "photo_55555", а за ним "photo_55554"? Если такое возможно, то скрипт это не обрабатывает.

[Владимир Татарников]

Т.е. у тебя может быть фоток с большим числом разрешения не 641, а как раз 300? Еще раз советую пройтись отладчиком pdb. Тогда не ошибешься

[adron90]

Владимир Татарников: vk.com/dev/photo
photo_x это ссылки на фото с максимальным разрешением. их всего 6
if x.startswith('photo_')and(int(x[6:])>max) как раз сравнивает которое из 6 копий фоток самое большое. но может быть что photo_x в ответе от контакта будет меньше 6. и все ровно будет выбрано максимальное разрешение.

проблема решена. когда я сохранял фото, то в названии писал

foto =  open('C:\\Users\\home\\Desktop\\vk_photo\\'+name+'\\'+date+'.jpg', 'wb')

дата была в таком формате '%Y.%m.%d-%H.%M'
из за того что фотки в вконтакте были загружены в одну и туже минуту, а некоторые даже в одну и туже секунду, то фотки просто перезаписывались.
в итоге добавил в начале названия счетчик. а дату записывал с секундами(хотя это уже не обязательно было)

foto =  open('C:\\Users\\home\\Desktop\\vk_photo\\'+name+'\\'+str(y)+' '+date+'.jpg', 'wb')

Про отладчик pdb не слышал. четвертый день только учусь. Но обязательно погуглю