zabr21 фев 2009 в 16:16

Скачиваем альбомы из Picasa

12 мин

1.2K

.NET *

+16

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

НЛО прилетело и опубликовало эту надпись здесь

pieceofsummer 21 фев 2009 в 16:51

Честно говоря, по коду рефакторинг плачет.

Первая половина LoadThumbnailsForListView() сводится к вызову LoadAlbumList(), итератор бы там применить было элегантнее, чем три раза обращаться по индексу (а еще элегантнее — linq), какая-то дикая смесь из continue и вложенных if-ов, не говоря уже о том, что

if (atomLinkCollection != null)
{
   if (atomLinkCollection[1] != null)
   {

вполне может кинуть IndexOutOfRangeException на ровном месте…

Конструкция

if (File.Exists(FullName)) File.Delete(FullName);
if (File.Exists(FullName)) return;

честно говоря, ввела меня в ступор.

Функция CheckIntArrayOfDownloads() — это, надо полагать, велосипед к Array.TrueForAll()? :)

Ну да ладно. С задачей продемонстрировать работу с Пикасой код справляется — и хорошо.

zabr 21 фев 2009 в 19:16

плачет согласен, проект писался второпях ибо были убиты фото на локальном харде у жены.
может кинуть.
if (File.Exists(FullName)) return; — лишнее, просто наверно подчищал try/catch а это запямятовал.
О не знал что такое есть :))) Спасибо. Да велосипедец :)))

pieceofsummer 21 фев 2009 в 19:56

> if (File.Exists(FullName)) return; — лишнее, просто наверно подчищал try/catch а это запямятовал.

На самом деле первая проверка тоже не обязательна.
File.Delete() не эксептит, если файл не существует.

Marsikus 21 фев 2009 в 17:54

Полезная штучка, может выложите как проект на sourceforge или подобном месте?

zabr 21 фев 2009 в 19:19

имхо стоит если бы проект был стоящий, а одному мусолить не вижу смысла,
да и не проект это, а как пример по сути.

zabr 21 фев 2009 в 23:08

Минусуя, аргументируйте…

Вы видите смысл выкладывания такой программы на sourceforge, у которой будет пользовалей 5 контингент и которая умрет сразу же как только появится более удобная интеграция Picasa с веб частью?

При этом вряд ли кто то захочет присоединиться в доделке продукта для введения новых фич.
А у меня нехватка времени… вот праздники — на хабр забежал :)

НЛО прилетело и опубликовало эту надпись здесь

gra 21 фев 2009 в 19:00

извините, я не разбирась в C#, просто интересуюсь, а так и надо вызывать
PicasaService pService = new PicasaService(«PicasaService»);
почти в каждом методе класса MainLogic?
я думал такое обычно выносят в конструктор

zabr 21 фев 2009 в 19:21

да вполне можно вынести в поле класса, а в конструктор не вижу смысла пихать.

Zeraman 21 фев 2009 в 19:24

а что это у вас там за девайс на фотке?

zabr 21 фев 2009 в 19:30

это не у меня :)

просто чужую связанную галерею проверял — Lenovo X61
а сначала тренировался на своих, когда у жены хард полетел.

ivaliy 22 фев 2009 в 22:22

херасе чужую, мою!!! в общем, я против такого выкладывания на публичные русурсы.

zabr 23 фев 2009 в 12:15

Ну во-первых она не скрытая, ибо она по умолчанию паблик, сейчас уберу.
А как вы думаете идет поиск по картинкам в галереях picasa? :))

ivaliy 23 фев 2009 в 18:03

То, что галлерея паблик, не говорит еще о том, что ее можно использовать как угодно и где угодно. Это раз, да думаю и самому бы не понравилось, если бы я где-нибудь использовал фотографии из вашего альбома. Но, в прочем, сам виноват, закрыл уже.

zabr 23 фев 2009 в 18:05

используйте ради бога :)

AlexeyTokar 21 фев 2009 в 21:29

Сегодня уже ложусь спать, но завтра обязательно напишу статью, как скачивать фотоальбомы picasaweb используя ~~PHP~~ picassa… Ага…

AlexeyTokar 21 фев 2009 в 21:41

Да. Замечу что это шутка юмора такая если кто не понял, а то автор уже обиделся и минусов натыкал :)
Просто не каждый велосипед заслуживает внимания. Например этот не заслуживает, так как это решение конкретной задачи для которой есть конкретное богатое решение.