開いているTCPポートをスキャンするためのスレッド処理

Question

以下のコードでWaitAllに問題があります。このメソッドはIPアドレスとポート（WMIポート135をスキャンしています）のリストを受け取り、ポートが開いているIPアドレスのリストを返します。私はこのコードを遅くすると、実行して正しいシステムを返すようにWaitAllが正しく実装されていないと信じています。ありがとう。

private static List<IPAddress> openSystems = new List<IPAddress>(); 

public static List<IPAddress> Scan(List<IPAddress> addresses, ushort port) 
{ 
    int count = addresses.Count; 
    Task[] tasks = new Task[count]; 

    //Loop through ip address 
    for(int x = 0; x <= count -1; x++) 
    { 
     tasks[x] = Task.Factory.StartNew(() => 
     { 
      using (TcpClient tcp = new TcpClient()) 
      { 
       try 
       { 
        Console.WriteLine("Trying to get into {0}", addresses[x]); 
        tcp.Connect(addresses[x], port); 
        openSystems.Add(addresses[x]); 
       } 
       catch 
       { 
        Console.WriteLine("Can't get into {0}", addresses[x]); 
        //ignore exceptions 
       } 
      } 
     }); 
    } 

    Task.WaitAll(tasks); 
    return openSystems; 
} 

Answer 1

コードでは無視されるいくつかのベストプラクティスがあります。

あなたがれるtcpClientのConnectAsyncメソッドを呼び出す必要がありますし、あなたがTask.WaitAllしかしawait Task.WhenAll

メソッドのシグネチャは、その後public static Task> ScanAsync(List addresses, ushort port)になり、あなたはどこにコーディングすることを待つことができますを使用してはならないTask.Factory.StartNew

への呼び出しを取り除きますが呼び出されます。

あなたは別のスレッドに対してopenSystemsのリストにアクセスしています。リストはスレッドセーフではありませんので、並行コレクションを使用するか、そうでなければ実行してください。

public static async Task<List<IPAddress>> ScanAsync(List<IPAddress> addresses, ushort port) 
{ 
    var tasks = addresses.Select(a => CheckIpAsync(a, port); 

    await Task.WhenAll(tasks); 
    return openSystems; 
} 

private async Task CheckIpAsync(IPAddress address, ushort port) 
{ 
    using (TcpClient tcp = new TcpClient()) 
    { 
     try 
     { 
      Console.WriteLine("Trying to get into {0}", address); 
      await tcp.ConnectAsync(address, port); 
      openSystems.Add(address); 
     } 
     catch 
     { 
      Console.WriteLine("Can't get into {0}", address); 
      //ignore exceptions 
     } 
    } 
} 

これは私が見つけることができないので、まだあなたの質問に答えないかもしれません。しかし、より良い実装を示しています。