2

我正在尝试为特定端口建立与主机/IP 地址的套接字连接。我试图实现的主要目标是：

  • 获取展位成功/失败连接的响应时间（以毫秒为单位）
  • 使用自定义超时，例如 5 秒 (5000)

所以基本上如果我指定一个主机名google.com和端口，80这应该检查连接是否有效，如果我指定173.194.70.101和端口80它应该可以工作。

如果我指定例如主机名google.com和端口，7788这应该不起作用，因为该端口未打开。如果我指定 IP：173.194.70.101和端口“7788”，这应该不起作用。

如果我指定一些随机主机sdfzsdfaklsf.com和端口，7788这应该不起作用，因为主机不存在。

对于上述所有情况，我需要所有成功/失败的响应时间......

我最终得到了这段代码和接缝，可以正常工作，但是我想问一下这是否是正确的做法？

     public string CheckConnection(string ipAddressOrHostName, string portName)
    {
        Stopwatch timer = new Stopwatch();
        timer.Start();
        Socket server = null;
        string elapsed = string.Empty;

        try
        {

            IPHostEntry hostEntry = Dns.GetHostEntry(ipAddressOrHostName);

            IPAddress ipAddress = hostEntry.AddressList[0];

            IPEndPoint ip = new IPEndPoint(ipAddress, Convert.ToInt32(portName));

            server = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp);

            IAsyncResult result = server.BeginConnect(ip, null, null);

            result.AsyncWaitHandle.WaitOne(5000, true);

            timer.Stop();

            elapsed = timer.ElapsedMilliseconds.ToString();


            if (!server.Connected)
            {                    
                server.Close();
                throw new Exception();
            }
            else
            {   
                string status = string.Format("Connected succesfully to: {0} in: {1} milliseconds", server.RemoteEndPoint.ToString(), elapsed);                   
                server.Shutdown(SocketShutdown.Both);
                server.Close();

                return status;
            }
        }
        catch (Exception)
        {
            timer.Stop();

            elapsed = timer.ElapsedMilliseconds.ToString();

            return string.Format("Connection failed to: {0}:{1} in: {2} milliseconds", ipAddressOrHostName, portName, elapsed);                     
        }

    }
4

1 回答 1

2

这是相当不错。几点：

  1. 捕获更具体的异常类型并对其进行测试。您不想捕获不相关的东西或隐藏错误。
  2. 完成 DNS 解析后启动计时器。
  3. throw new Exception不太好，因为异常并不意味着控制流。重组方法以不需要这个hack。
  4. 您可能想要使用任务而不是IAsyncResult. 没有功能变化，只是更现代的风格。
  5. 您在一些地方有冗余（例如重复关闭，重复 ToString）。
  6. 解析方法之外的端口。它不属于那里。事实上，如果端口是不可解析的，你会在此刻触发感知超时，因为会抛出异常。这就是我用第 (1) 点警告你的地方。过多地使用异常会导致一个你可能永远不会知道的错误，因为它被隐藏了。
于 2013-08-27T20:51:37.177 回答