Асинхронный пинг множества хостов

[kpoko]

Пытаюсь написать пингер. Есть множество хостов host[] которые нужно периодически пинговать, и есть массив pingTime[], который нужно заполнять результатами пинга. Как передать делегату значение переменной i? Текущий код неправильный и приводит к ошибке.

for (byte i = 0; i < allhosts; i++)
{
        Ping ping = new Ping();
        ping.PingCompleted += delegate(object sender, PingCompletedEventArgs e)
        {
                pingTime[i] = Convert.ToInt16(e.Reply.RoundtripTime);
                (ping as IDisposable).Dispose();
        }
        ping.SendAsync(host[i], timeout, new AutoResetEvent(false));
}

[Guy]

если по рабочекрестьянски то в PingCompletedEventArgs - e.Reply.Address лежит то куда ты слал. А если как ты хочешь то импользовать Action с 2-мя параметрами 1-вый будет i второй куда слать, но судя по твоему коду лучше по рабочекрестьянски
http://msdn.microsoft.com/en-us/library/bb549311.aspx

[kpoko]

Спасибо что откликнулись. Метод рабочекрестьянский мне не подошел, т.к. он не работает когда хост представлен в виде доменного имени, а также когда в списке есть повторяющиеся IP. Заинтересовал второй метод. Я новичок в C#, по ссылке ходил, но не смог применить тот пример к своему случаю. Покажите пожалуйста как это должно выглядеть в виде кода?

[Guy]

using System;
using System.IO;
using System.Collections.Generic;
using System.Net.NetworkInformation;

public class TestLambdaExpression
{
    private static List<string> hosts = new List<string>() { "google.ru", "ya.ru", "mail.ru", "aaaaa213213123123.ru" };

    public static void Main()
    {
        Action<string> asyn = new Action<string>(Pinger);

        hosts.ForEach(e => 
        {
            asyn.Invoke(e);
        });

        Console.WriteLine("End");
        Console.ReadKey();
    }

    private static void Pinger(string hostAdress)
    {
        Ping png = new Ping();
        try
        {
            PingReply pr = png.Send(hostAdress);
            Console.WriteLine(string.Format("Status for {0} = {1}", hostAdress, pr.Status));
        }
        catch (Exception ex)
        {
            Console.WriteLine(string.Format("Status for {0} = {1}", hostAdress, ex.InnerException.Message));
        }
    }
}

[kpoko]

Спасибо большое, ваш код работает. Еще остался такой вопрос: когда разбирался с классом Ping, пытался пример из http://msdn.microsoft.com/ru-ru/libr...tion.ping.aspx переделать под свою задачу, но до конца не понял какие аргументы нужно отправлять. Можно ли такой вариант применить к моему случаю?

for (byte i = 0; i < allhosts; i++)
{
	Ping ping = new Ping();
	ping.PingCompleted += new PingCompletedEventHandler(PingCompletedCallback(i, ping, /*ЧТО СЮДА ПЕРЕДАТЬ?*/));
	ping.SendAsync(host[i], timeout, new AutoResetEvent(false));
}
////////////////////////////////
private void PingCompletedCallback(int i, object sender, PingCompletedEventArgs e)
{
	pingTime[i] = Convert.ToInt16(e.Reply.RoundtripTime);
}

[Eugene_M]

Сообщение от Guy

using System;
using System.IO;
using System.Collections.Generic;
using System.Net.NetworkInformation;

public class TestLambdaExpression
{
    private static List<string> hosts = new List<string>() { "google.ru", "ya.ru", "mail.ru", "aaaaa213213123123.ru" };

    public static void Main()
    {
        Action<string> asyn = new Action<string>(Pinger);

        hosts.ForEach(e => 
        {
            asyn.Invoke(e);
        });

        Console.WriteLine("End");
        Console.ReadKey();
    }

    private static void Pinger(string hostAdress)
    {
        Ping png = new Ping();
        try
        {
            PingReply pr = png.Send(hostAdress);
            Console.WriteLine(string.Format("Status for {0} = {1}", hostAdress, pr.Status));
        }
        catch (Exception ex)
        {
            Console.WriteLine(string.Format("Status for {0} = {1}", hostAdress, ex.InnerException.Message));
        }
    }
}

Замечательный скрипт, он работает на "ура", но, если можно, не могли бы вы его "адаптировать" для работы не со списком, а с массивом... Прошу помощи))

[Reskov]

Кроко так Ваш первоначальный пример почти рабочий, чуть подправил

for (byte i = 0; i < allhosts; i++)
{
        Ping ping = new Ping();
        ping.PingCompleted += delegate(object sender, PingCompletedEventArgs e)
        {
                int val = Convert.ToInt32(e.UserState);
                pingTime[val] = Convert.ToInt16(e.Reply.RoundtripTime);
                (ping as IDisposable).Dispose();
        }
        ping.SendAsync(host[i], timeout, i);
}

Ps тьфу не заметил что год 2011)))))))))))))))

[Aleksey999]

Сообщение от Reskov

Кроко так Ваш первоначальный пример почти рабочий, чуть подправил

for (byte i = 0; i < allhosts; i++)
{
        Ping ping = new Ping();
        ping.PingCompleted += delegate(object sender, PingCompletedEventArgs e)
        {
                int val = Convert.ToInt32(e.UserState);
                pingTime[val] = Convert.ToInt16(e.Reply.RoundtripTime);
                (ping as IDisposable).Dispose();
        }
        ping.SendAsync(host[i], timeout, i);
}

Ps тьфу не заметил что год 2011)))))))))))))))

Даже не смотря на дату: Спасибо!