zabr Feb 21 2009 at 19:16

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

12 min

1.1K

.NET*

+16

Comments 23

UFO just landed and posted this here

pieceofsummer Feb 21 2009 at 19: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 Feb 21 2009 at 22:16

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

pieceofsummer Feb 21 2009 at 22:56

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

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

Marsikus Feb 21 2009 at 20:54

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

-2

zabr Feb 21 2009 at 22:19

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

zabr Feb 22 2009 at 02:08

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

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

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

UFO just landed and posted this here

gra Feb 21 2009 at 22:00

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

zabr Feb 21 2009 at 22:21

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

Zeraman Feb 21 2009 at 22:24

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

zabr Feb 21 2009 at 22:30

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

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

ivaliy Feb 23 2009 at 01:22

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

zabr Feb 23 2009 at 15:15

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

ivaliy Feb 23 2009 at 21:03

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

zabr Feb 23 2009 at 21:05

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

AlexeyTokar Feb 22 2009 at 00:29

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

-2

AlexeyTokar Feb 22 2009 at 00:41

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