Comments 11
Попробуем использовать для проведения пользовательского конкурса с фотографиями, трафик сохранить. Пикаса не ставит ограничений по частоте запросов?
PS — пользуюясь случаем, ищем django программистов — junior и seniour, пишите на andrew.novoselsky@gmail.com
PS — пользуюясь случаем, ищем django программистов — junior и seniour, пишите на andrew.novoselsky@gmail.com
0
Небольшая просьба: старайтесь не использовать locals(), даже в примерах — кому-нибудь может это понравится.
Тут можно обойтись без конкатенции, коли все равно формируете строку.
Если разбить на строки будет лучше выглядеть, да и дебажить станет удобнее.
Тут скобочки не обязательны.
Начиная с версии 1.3 можно использовать шорткат render — код станет чище и понятнее.
Тут немного лишнего.
Еще немного смущает как вы смело обращаетесь к артибутам, элементам по индексам, получаете экземпляры классов и тут же в бой. Возможно это излишняя осторожность, но я бы понапихал проверок и «затычек», еще ни разу об этом не пожалел.
Надеюсь ничего не напутал :)
# 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)
Тут немного лишнего.
Еще немного смущает как вы смело обращаетесь к артибутам, элементам по индексам, получаете экземпляры классов и тут же в бой. Возможно это излишняя осторожность, но я бы понапихал проверок и «затычек», еще ни разу об этом не пожалел.
Надеюсь ничего не напутал :)
+4
Для таких вещей
> album.GetPhotosUri() + '&imgmax=%s&thumbsize=%s' % (PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)
создан str.format()
> album.GetPhotosUri() + '&imgmax=%s&thumbsize=%s' % (PHOTO_IMGMAXSIZE, PHOTO_THUMBSIZE)
создан str.format()
+2
Если честно, то я не вижу смысла делать это через django. Можно же на стороне клиента например так: oss.oetiker.ch/jquery/jquery.EmbedPicasaGallery.html
Я понимаю еще можно применить django чтобы еще и через админку django загружать или редактировать фотографии на picasa. А так только в качестве примера использования gdata.
Странно все оформлено в архиве — лучше просто requirements.txt создать с зависимостями а не класть туда папку gdata(иначе увидев эту папку возникают мысли что gdata у вас измененная и другие версии совсем не совместимы).
Я понимаю еще можно применить django чтобы еще и через админку django загружать или редактировать фотографии на picasa. А так только в качестве примера использования gdata.
Странно все оформлено в архиве — лучше просто requirements.txt создать с зависимостями а не класть туда папку gdata(иначе увидев эту папку возникают мысли что gdata у вас измененная и другие версии совсем не совместимы).
+1
поставьте кнопочку Мне нравится возле «Для тех, кто дальше читать не хочет, сразу готовый результат.»
+1
Очень круто получилось. Спасибо.
0
Sign up to leave a comment.
Фотогалерея на Django с использованием Google Picasa в качестве хостинга