而不是扫描多个端口，我只扫描一个端口，当端口关闭时，我的应用程序关闭

Question

我正在尝试使用异步扫描一次扫描多个端口。问题是我只能显示第一个工作端口，然后等待我的应用程序关闭 20 秒，却没有告诉我端口已关闭。

这段代码有什么问题？

private void btnStart_Click(object sender, EventArgs e)
{
    for (int port = 80; port < 100; port++)
    {
        ScanPort(port);
    }
}

private void ScanPort(int port)
{
    var client = new TcpClient();

    try 
    {
        client.BeginConnect(IPAddress.Parse("74.125.226.84"), port, new AsyncCallback(CallBack), client);
    } 
    catch (SocketException) 
    {
        client.Close();
    }
}

private void CallBack(IAsyncResult result)
{
    var client = (TcpClient)result.AsyncState;

    client.EndConnect(result);

    if (client.Connected)
    {
        this.Invoke((MethodInvoker)delegate
        {
            txtDisplay.Text += "open2" + Environment.NewLine;
        });
    }
    else
    {
        this.Invoke((MethodInvoker)delegate
        {
            txtDisplay.Text += "closed2" + Environment.NewLine;
        });
    }
}

Answer 1

在您的回调方法中，我会确保关闭连接并处理 TcpClient。 TcpClient.EndConnect(IAsyncResult) 也可以抛出异常。我也看不到在哪里捕获端口号以显示给用户。我会写这样的回调。

编辑：我实际上并没有编译或执行我的代码（抱歉）。我还发现了另一篇文章，它展示了如何在 C# 中创建端口扫描器，http://www.dijksterhuis.org/building-a-simple-portscanner-in-c/ 在这篇文章中有一条评论指出，

这里有个问题：TCPClient.Close() 函数的 .NET 实现实际上并没有正确关闭连接。所以我们需要在调用 TCPClient.Close 之前执行额外的步骤来获取表示连接的流并关闭它。

private void CallBack(IAsyncResult result) 
{ 
    var client = (TcpClient)result.AsyncState; 
    bool connected = false;

    try
    {
        client.EndConnect(result);
        connected = client.Connected;
    }
    catch (SocketException)
    {
    }
    catch (ObjectDisposedException)
    {
    }
    finally
    {
        if (client.Connected)
        {
            client.Close();
        }

        client.Dispose();
    }

    if (connected) 
    { 
        this.Invoke((MethodInvoker)delegate 
        { 
            txtDisplay.Text += "open2" + Environment.NewLine; 
        }); 
    } 
    else 
    { 
        this.Invoke((MethodInvoker)delegate 
        { 
            txtDisplay.Text += "closed2" + Environment.NewLine; 
        }); 
    } 
}