Как стать автором
Обновить

Комментарии 11

Попробуем использовать для проведения пользовательского конкурса с фотографиями, трафик сохранить. Пикаса не ставит ограничений по частоте запросов?

PS — пользуюясь случаем, ищем django программистов — junior и seniour, пишите на andrew.novoselsky@gmail.com
В Питере
Ждал минусов.
Если бы хх, суперджоб, itmozg, django-russian, softwaremaniacs, python.su и… давали бы результат при поиске питонистов, не стал бы без нужды тревожить.
дефицит реально наблюдается…
Небольшая просьба: старайтесь не использовать locals(), даже в примерах — кому-нибудь может это понравится.

# album.GetPhotosUri() + '&imgmax=%s&thumbsize=%s' % (PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)
'%s&imgmax=%s&thumbsize=%s' % (album.GetPhotosUri(), PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)

Тут можно обойтись без конкатенции, коли все равно формируете строку.

PhotosService().GetFeed(album.GetPhotosUri() +
                       '&imgmax=%s&thumbsize=%s' % (PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)).entry

Если разбить на строки будет лучше выглядеть, да и дебажить станет удобнее.

if(album.gphoto_id.text == album_id):

Тут скобочки не обязательны.

return render_to_response

Начиная с версии 1.3 можно использовать шорткат render — код станет чище и понятнее.

# style = 'width: %s; height: %s;'
# thumbnail = get_thumbnail(obj)
# return style % ('%spx' % thumbnail.width, '%spx' % thumbnail.height)
thumb = get_thumbnail(obj)
return 'width: %spx; height: %spx;' % (thumb.width, thumb.height)

Тут немного лишнего.

Еще немного смущает как вы смело обращаетесь к артибутам, элементам по индексам, получаете экземпляры классов и тут же в бой. Возможно это излишняя осторожность, но я бы понапихал проверок и «затычек», еще ни разу об этом не пожалел.
Надеюсь ничего не напутал :)
Спасибо за замечания. Поправил
спасибо за render ))
Для таких вещей
> album.GetPhotosUri() + '&imgmax=%s&thumbsize=%s' % (PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)
создан str.format()
Если честно, то я не вижу смысла делать это через django. Можно же на стороне клиента например так: oss.oetiker.ch/jquery/jquery.EmbedPicasaGallery.html
Я понимаю еще можно применить django чтобы еще и через админку django загружать или редактировать фотографии на picasa. А так только в качестве примера использования gdata.
Странно все оформлено в архиве — лучше просто requirements.txt создать с зависимостями а не класть туда папку gdata(иначе увидев эту папку возникают мысли что gdata у вас измененная и другие версии совсем не совместимы).
поставьте кнопочку Мне нравится возле «Для тех, кто дальше читать не хочет, сразу готовый результат.»
Очень круто получилось. Спасибо.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации