Pull to refresh

Comments 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)

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

Еще немного смущает как вы смело обращаетесь к артибутам, элементам по индексам, получаете экземпляры классов и тут же в бой. Возможно это излишняя осторожность, но я бы понапихал проверок и «затычек», еще ни разу об этом не пожалел.
Надеюсь ничего не напутал :)
Спасибо за замечания. Поправил
Для таких вещей
> 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 у вас измененная и другие версии совсем не совместимы).
поставьте кнопочку Мне нравится возле «Для тех, кто дальше читать не хочет, сразу готовый результат.»
Очень круто получилось. Спасибо.
Sign up to leave a comment.

Articles