Как использовать UdpClient.BeginReceive в цикле - PullRequest
8 голосов
/ 07 ноября 2010

Я хочу сделать это

for (int i = 0; i < 100; i++ )
{
    Byte[] receiveBytes = receivingUdpClient.Receive(ref RemoteIpEndPoint);
}

Но вместо использования UdpClient.Receive я должен использовать UdpClient.BeginReceive.Проблема в том, как мне это сделать?Примеров, использующих BeginReceive, не так много, и пример MSDN совсем не помогает.Должен ли я использовать BeginReceive или просто создать его в отдельном потоке?

Я последовательно получаю ObjectDisposedException исключение.Я получаю только первые данные.Следующие данные будут выдавать исключение.

public class UdpReceiver
{
    private UdpClient _client;
    public System.Net.Sockets.UdpClient Client
    {
        get { return _client; }
        set { _client = value; }
    }
    private IPEndPoint _endPoint;
    public System.Net.IPEndPoint EndPoint
    {
        get { return _endPoint; }
        set { _endPoint = value; }
    }
    private int _packetCount;
    public int PacketCount
    {
        get { return _packetCount; }
        set { _packetCount = value; }
    }
    private string _buffers;
    public string Buffers
    {
        get { return _buffers; }
        set { _buffers = value; }
    }
    private Int32 _counter;
    public System.Int32 Counter
    {
        get { return _counter; }
        set { _counter = value; }
    }
    private Int32 _maxTransmission;
    public System.Int32 MaxTransmission
    {
        get { return _maxTransmission; }
        set { _maxTransmission = value; }
    }

    public UdpReceiver(UdpClient udpClient, IPEndPoint ipEndPoint, string buffers, Int32 counter, Int32 maxTransmission)
    {
        _client = udpClient;
        _endPoint = ipEndPoint;
        _buffers = buffers;
        _counter = counter;
        _maxTransmission = maxTransmission;
    }
    public void StartReceive()
    {
        _packetCount = 0;
        _client.BeginReceive(new AsyncCallback(Callback), null);
    }

    private void Callback(IAsyncResult result)
    {
        try
        {
            byte[] buffer = _client.EndReceive(result, ref _endPoint);
            // Process buffer
            MainWindow.Log(Encoding.ASCII.GetString(buffer));
            _packetCount += 1;
            if (_packetCount < _maxTransmission)
            {
                _client.BeginReceive(new AsyncCallback(Callback), null);
            }
        }
        catch (ObjectDisposedException ex) 
        {
            MainWindow.Log(ex.ToString());
        }
        catch (SocketException ex) 
        { 
            MainWindow.Log(ex.ToString()); 
        }
        catch (System.Exception ex)
        {
            MainWindow.Log(ex.ToString()); 
        }
    }
}

Что дает?

Кстати, общая идея такова:

  1. Создать менеджер tcpclient.
  2. Начать отправку / получение данных с помощью udpclient.
  3. Когда все данные будут отправлены, tcpclient manager сообщит получателю, что все данные отправлены, и соединение udpclient должно быть закрыто.

Ответы [ 5 ]

6 голосов
/ 20 февраля 2014

Казалось бы, UdpClient.BeginReceive() и UdpClient.EndReceive() плохо реализованы / поняты.И, конечно, по сравнению с тем, как реализован TcpListener, использовать его намного сложнее.

Есть несколько вещей, которые вы можете сделать, чтобы UdpClient.Receive() работал лучше для вас.Во-первых, установка тайм-аутов на базовом сокете Client позволит провалу управления (исключение), что позволит продолжить или зациклить поток управления.Во-вторых, создавая прослушиватель UDP в новом потоке (создание которого я не показывал), вы можете избежать эффекта полублокировки функции UdpClient.Receive() и эффективно отменить этот поток позже, если вы сделаете это правильно.

Код ниже состоит из трех частей.Первая и последняя части должны быть в вашем главном цикле в точках входа и выхода соответственно.Вторая часть должна быть в новой созданной вами теме.

Простой пример:

// Define this globally, on your main thread
UdpClient listener = null;
// ...


// ...
// Create a new thread and run this code:

IPEndPoint endPoint = new IPEndPoint(IPAddress.Any, 9999);
byte[] data = new byte[0];
string message = "";

listener.Client.SendTimeout = 5000;
listener.Client.ReceiveTimeout = 5000;

listener = new UdpClient(endPoint);
while(true)
{
    try
    {
        data = listener.Receive(ref endPoint);
        message = Encoding.ASCII.GetString(data);
    }
    catch(System.Net.Socket.SocketException ex)
    {
        if (ex.ErrorCode != 10060)
        {
            // Handle the error. 10060 is a timeout error, which is expected.
        }
    }

    // Do something else here.
    // ...
    //
    // If your process is eating CPU, you may want to sleep briefly
    // System.Threading.Thread.Sleep(10);
}
// ...


// ...
// Back on your main thread, when it's exiting, run this code
// in order to completely kill off the UDP thread you created above:
listener.Close();
thread.Close();
thread.Abort();
thread.Join(5000);
thread = null;

В дополнение ко всему этому вы также можете проверить UdpClient.Available > 0, чтобы определить, является лилюбые запросы UDP ставятся в очередь перед выполнением UdpClient.Receive() - это полностью устраняет аспект блокировки.Я советую вам попробовать это с осторожностью, так как это поведение не отображается в документации Microsoft, но, похоже, работает.

Примечание:

MSDN exmaple code Вы, возможно, обнаружили, что при исследовании этой проблемы требуется дополнительный определенный пользователем класс - UdpState.Это не класс библиотеки .NET.Это, кажется, сбивает с толку многих людей, когда они исследуют эту проблему.

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

Команда listener.Close () важна, потому что она заставляет UdpClient генерировать исключение и выходить из цикла, позволяя Thread.Abort() чтобы быть обработанным.Без этого вы не сможете корректно завершить работу потока слушателя, пока не истечет время ожидания или не будет получен пакет UDP, в результате чего код будет продолжаться после блока UdpClient.Receive ().


Просто добавьтена этот бесценный ответ вот рабочий и проверенный фрагмент кода.(Здесь, в контексте Unity3D, но, конечно, для любого c #.)

// minmal flawless UDP listener per PretorianNZ

using System.Collections;
using System;
using System.Net.Sockets;
using System.Net;
using System.Threading;

void Start()
   {
   listenThread = new Thread (new ThreadStart (SimplestReceiver));
   listenThread.Start();
   }

private Thread listenThread;
private UdpClient listenClient;
private void SimplestReceiver()
   {
   Debug.Log(",,,,,,,,,,,, Overall listener thread started.");

   IPEndPoint listenEndPoint = new IPEndPoint(IPAddress.Any, 1260);
   listenClient = new UdpClient(listenEndPoint);
   Debug.Log(",,,,,,,,,,,, listen client started.");

   while(true)
      {
      Debug.Log(",,,,, listen client listening");

      try
         {
         Byte[] data = listenClient.Receive(ref listenEndPoint);
         string message = Encoding.ASCII.GetString(data);
         Debug.Log("Listener heard: " +message);
         }
      catch( SocketException ex)
         {
         if (ex.ErrorCode != 10060)
            Debug.Log("a more serious error " +ex.ErrorCode);
         else
            Debug.Log("expected timeout error");
         }

      Thread.Sleep(10); // tune for your situation, can usually be omitted
      }
   }

void OnDestroy() { CleanUp(); }
void OnDisable() { CleanUp(); }
// be certain to catch ALL possibilities of exit in your environment,
// or else the thread will typically live on beyond the app quitting.

void CleanUp()
   {
   Debug.Log ("Cleanup for listener...");

   // note, consider carefully that it may not be running
   listenClient.Close();
   Debug.Log(",,,,, listen client correctly stopped");

   listenThread.Abort();
   listenThread.Join(5000);
   listenThread = null;
   Debug.Log(",,,,, listener thread correctly stopped");
   }
3 голосов
/ 07 ноября 2010

сначала посмотрите на MSDN.Они дают хороший пример.http://msdn.microsoft.com/en-us/library/system.net.sockets.udpclient.beginreceive.aspx

2 голосов
/ 07 ноября 2010

Я бы осуществлял сетевое взаимодействие в фоновом потоке, чтобы он не блокировал ничего в вашем приложении.

Проблема с BeginReceive заключается в том, что вы должны вызывать EndReceive в какой-то момент (в противном случае у вас есть дескрипторы ожидания, просто сидящие без дела) - и вызов EndReceive будет блокироваться до тех пор, пока не будет завершен прием. Вот почему проще поместить сообщение в другой поток.

2 голосов
/ 07 ноября 2010

Я думаю, вам не следует использовать его в цикле, но вместо этого всякий раз, когда вызывается обратный вызов BeginReceive, вы снова вызываете BeginReceive и сохраняете открытую переменную для count, если хотите ограничить число до 100.

0 голосов
/ 20 октября 2017

Вы должны выполнять сетевые операции, манипуляции с файлами и такие вещи, которые зависят от других вещей, а не от вашей собственной программы в другом потоке (или задача ), поскольку они могут заморозить вашу программу.Причина в том, что ваш код выполняется последовательно.Вы использовали это в цикле, который не в порядке.Каждый раз, когда вызывается BeginRecieve обратный вызов, вы должны вызывать его снова.Взгляните на следующий код :

public static bool messageReceived = false;

public static void ReceiveCallback(IAsyncResult ar)
{
  UdpClient u = (UdpClient)((UdpState)(ar.AsyncState)).u;
  IPEndPoint e = (IPEndPoint)((UdpState)(ar.AsyncState)).e;

  Byte[] receiveBytes = u.EndReceive(ar, ref e);
  string receiveString = Encoding.ASCII.GetString(receiveBytes);

  Console.WriteLine("Received: {0}", receiveString);
  messageReceived = true;
}

public static void ReceiveMessages()
{
  // Receive a message and write it to the console.
  IPEndPoint e = new IPEndPoint(IPAddress.Any, listenPort);
  UdpClient u = new UdpClient(e);

  UdpState s = new UdpState();
  s.e = e;
  s.u = u;

  Console.WriteLine("listening for messages");
  u.BeginReceive(new AsyncCallback(ReceiveCallback), s);

  // Do some work while we wait for a message. For this example,
  // we'll just sleep
  while (!messageReceived)
  {
    Thread.Sleep(100);
  }
}
...