Альтернатива Thread.Sleep () перед чтением сокета - PullRequest
Новая
       6

Альтернатива Thread.Sleep () перед чтением сокета

3 голосов
/ 27 января 2011

Я использую эту библиотеку FtpClient для подключения к мэйнфрейму из приложения WinForms.Я использую thread.Sleep для потока, чтобы дождаться ответа, прежде чем он начнет читать, в противном случае он зависает.Есть ли альтернатива для этого? 

public void Login() 
{
    if (this.loggedin) this.Close();

    Debug.WriteLine("Opening connection to " + this.server, "FtpClient");

    IPAddress addr = null;
    IPEndPoint ep = null;

    try
    {
        this.clientSocket = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);
        addr = Dns.Resolve(this.server).AddressList[0];
        ep = new IPEndPoint(addr, this.port);
        this.clientSocket.Connect(ep);
    }
    catch (Exception ex)
    {
        // doubtfull
        if (this.clientSocket != null && this.clientSocket.Connected) this.clientSocket.Close();

        throw new FtpException("Couldn't connect to remote server", ex);
    }

    **Thread.Sleep(4000);**
    this.readResponse();
    ...
}

private void readResponse()
{
    this.message = "";
    this.result = this.readLine();

    if (this.result.Length > 3)
        this.resultCode = int.Parse(this.result.Substring(0, 3));
    else
        this.result = null;
}

private string readLine()
{
    while (true)
    {
        this.bytes = clientSocket.Receive(this.buffer, this.buffer.Length, 0);
        this.message += ASCII.GetString(this.buffer, 0, this.bytes);

        if (this.bytes < this.buffer.Length) break;
    }

    string[] msg = this.message.Split('\n');
    if (this.message.Length > 2)
    {
        this.message = msg[msg.Length - 2];
        try { response = msg[msg.Length - 3]; }
        catch { }
    }
    else
    {
        this.message = msg[0];
    }

    if (this.message.Length > 4 && !this.message.Substring(3, 1).Equals(" ")) return this.readLine();

    if (this.verboseDebugging)
    {
        for (int i = 0; i < msg.Length - 1; i++)
        {
            Debug.Write(msg[i], "FtpClient");
        }
    }
    return message;
}

public void sendCommand(String command)
{
    if (this.verboseDebugging) Debug.WriteLine(command, "FtpClient");

    Byte[] cmdBytes = Encoding.ASCII.GetBytes((command + "\r\n").ToCharArray());
    clientSocket.Send(cmdBytes, cmdBytes.Length, 0);
    this.readResponse();
}

Ответы [ 3 ]

4 голосов
/ 27 января 2011

Использовать модель асинхронного программирования: 

socket.BeginConnect(ep, new AsyncCallback(Connected), socket);

void Connected (IAsyncResult result)
{
    var socket = (Socket)result.AsyncState;

    // do the stuff

    socket.EndConnect(result);
}
0 голосов
/ 27 января 2011

Письмо Thread.Sleep() действительно плохая практика.

Если вы действительно используете FTP-клиент, используйте FtpWebRequest BeginGetResponse (asynccallback, object)

И передать readResonse в качестве обратного вызова. Он будет вызван именно тогда, когда ответ будет готов.

0 голосов
/ 27 января 2011

Да, если вы можете работать с .NET 4.0, у вас есть API задач.

Вы можете узнать больше, прочитав эту статью: http://www.codethinked.com/post/2010/01/25/NET-40-and-SystemThreadingTasks.aspx

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

...