А это уже читатель сможет сделать потом. Я же не прошу все время использовать такие примитивные конструкции, как в этом примере. Это для того, чтобы «пощупать» .xml и явно вытащить из него все, что нужно.

lair 19 мар 2013 в 17:33

А зачем это может быть нужно и кому?

Чтобы «пощупать» xml, достаточно на него один раз посмотреть.

Milording 19 мар 2013 в 17:45

Наверное тем людям, которые не знают LINQ, но хотят любым понятным способом начать работать с API ВКонтакте. Я просто пытался уменьшить порог вхождения.

-1

lair 19 мар 2013 в 17:47

Людям, которые не умеют работать ни с LINQ, ни с XPath, надо сначала научиться работать с этими технологиями, а только потом взаимодействовать с API сторонних сервисов.

(на всякий случай напомню, что речь идет в контексте C#)

Milording 19 мар 2013 в 17:52

Полностью согласен, но незнание этих технологий не мешает понимать такие простые вещи. А потом, если надо будет, можно использовать специальные для этого инструменты.

lair 19 мар 2013 в 17:54

А что тут понимать?

Milording 19 мар 2013 в 17:57

получить все сообщения с пользователем, обработать узлы xml

Это тоже самое, что изучать Лого. Вроде все понятно, но для первого знакомства (конечно, я говорю про детей) подойдет.

lair 19 мар 2013 в 17:59

получить все сообщения с пользователем, обработать узлы xml

Это надо делать средствами, заложенными в язык/платформу. А не так, как это делаете вы.

Milording 19 мар 2013 в 18:02

foreach заложен в платформу. И дело даже не в этом.

lair 19 мар 2013 в 18:03

Дело как раз в этом.

Ваш код — это пример, как писать не надо.

Milording 19 мар 2013 в 18:05

Когда вы объясняете ребенку как складывать числа, вы показываете ему на пальцах или яблоках. Но это же не означает, что вы подразумеваете, что он будет так вычислять всю свою жизнь.

lair 19 мар 2013 в 18:07

Программисты — не дети, у них нормально с абстрактным мышлением и памятью.

darked 20 мар 2013 в 12:56

Мне интересная тема поста, спасибо большое. Но, кроме понимания, мне было бы интересно увидеть как это писать правильно. Могли бы вы написать вариант с использованием LINQ to XML, так сказать, для сравнения?

Milording 20 мар 2013 в 16:10

Вам спасибо. Ниже уже есть комментарий с использованием LINQ. Укажу его в статье.

Rondo 19 мар 2013 в 16:17

Я почему-то текст на вашей картинке прочитал как XML+C≠♥

Atreides07 19 мар 2013 в 17:55

Спасибо за статью!

> Пожалуйста, указывайте на любые недочеты в статье. Буду учиться.

Я позволил себе последний код чуть чуть оптимизировать:

while(true)
{
    var doc = XDocument.Load("https://api.vk.com/method/messages.getHistory.xml?uid=12345&offset=" + m + "&count="+n+"&access_token=" + token);// Загружаем наш xml с сервера и начнем обработку

    foreach (var el in doc.Root.Elements("message"))// Раскрываем корневой узел
    {
        foreach (var el_msg in el.Elements("attachment"))// Теперь в узле message
        {
            foreach (var el_attch in el_msg.Elements("photo"))
            {
                foreach (var el_photo in el_attch.Elements("src_xbig"))
                {
                    // Скачиваем фотографию в img/
                    src_client.DownloadFile(el_photo.Value, "img/img" + k + ".jpg");
                    k++;
                }
            }
        }
    }
    // Ну и конечно увеличение параметра count
    m += n;
}

А для не совсем маленьких, так же сделал linq вариант:

while (true)
{
    var doc = XDocument.Load("https://api.vk.com/method/messages.getHistory.xml?uid=12345&offset=" + m + "&count=" + n + "&access_token=" + token);// Загружаем наш xml с сервера и начнем обработку
    var photoElements=
    doc.Root.Elements("message")
        .SelectMany(el => el.Elements("attachment")
        .SelectMany(el_msg => el_msg.Elements("photo")
        .SelectMany(elAttch => elAttch.Elements("src_xbig"))));

    foreach (var el_photo in photoElements)
    {
        // Скачиваем фотографию в img/
        src_client.DownloadFile(el_photo.Value, "img/img" + k + ".jpg");
        k++;
    }
    // Ну и конечно увеличение параметра count
    m += n;
}

Можно так же еще сделать десериализацию данных без ручного разбора — но в рамках этой задачи наверное смысла нет.

Milording 19 мар 2013 в 18:06

Мне нравится ваш код. Спасибо за отзыв.

Weageoo 19 мар 2013 в 18:05

Мне кажется, или тут достаточно одного XPath-запроса? Дайте посмотреть на приходящий XML.

lair 19 мар 2013 в 18:08

Вам не кажется.

Milording 20 мар 2013 в 18:26

Спасибо. Специально для любителей XPath, написал вариант с ним.

Weageoo 20 мар 2013 в 19:26

Я б писал как-то так:

using System;
using System.Linq;
using System.Net;
using System.Xml.Linq;
using System.Xml.XPath;

namespace Test
{
    class Program
    {
        private readonly WebClient _webClient = new WebClient();

        public string Token
        {
            get { return "token"; }
        }
        public string UserID
        {
            get { return "uid"; }
        }
        public string XQueryString
        {
            get { return "//message//attachment//photo//src_big"; }
        }

        static void Main()
        {
            new Program().Run();
        }

        private void Run()
        {
            const int offset = 0, count = 10;

            var url = MakeUrl(UserID, offset, count, Token);
            var doc = XDocument.Load(url);
            var els = doc.XPathSelectElements(XQueryString);

            foreach (var t in els.Select((val, index) => new { val.Value, Index = index }))
            {
                _webClient.DownloadFileAsync(new Uri(t.Value), string.Format("img/img{0}.jpg", t.Index));
            }
        }

        private string MakeUrl(string uid, int offset, int count, string token)
        {
            const string queryFormatString =
                "https://api.vk.com/method/messages.getHistory.xml?uid={0}&offset={1}&count={2}&access_token={3}";

            return string.Format(queryFormatString, uid, offset, count, token);
        }
    }
}