C＃TimerCallback戻り値エラー

Question

私はF11デバッグを使用すると、NetStatus値はtrueですが、私はF5デバッグを使用する場合、NetStatus値はfalseです。どうして？どうすれば修正できますか？

_connectの値が何時変わるか分かりません。 これはマルチスレッド同期の問題ですか？

namespace conn 
{ 
    using System; 
    using System.Collections.Generic; 
    using System.Text; 
    using System.Net; 
    using System.Net.Sockets; 
    using System.Threading; 

    class Program 
    { 
     static void Main(string[] args) 
     { 
      wait_conn(); 
     } 

     private static void wait_conn() 
     { 
      bool NetStatus = false; 
      string DataS = "127.0.0.1"; 

      CheckIpConn conn = new CheckIpConn(); 
      conn.CheckIp(DataS); 

      //Thread.Sleep(3000); 
      NetStatus = conn.Connected; 

      if (!NetStatus) 
       Console.WriteLine(conn.Connected.ToString()); 

      Console.Read(); 
     } 
    } 

    public class CheckIpConn 
    { 
     private int _port = 135; 
     private volatile bool _connect; 
     private IPEndPoint _iPEndPoint; 
     private TcpClient _client; 
     private TimerCallback _timerCallBack; 
     private System.Threading.Timer _timer; 

     public bool Connected 
     { 
      get { return _connect; } 
     } 

     public void CheckIp(string Ip) 
     { 
      _iPEndPoint = new IPEndPoint(IPAddress.Parse(Ip), _port); 
      _timerCallBack = new TimerCallback(CheckConnect); 
      _timer = new System.Threading.Timer(_timerCallBack, null, 10, 1000); 
      _timer.Dispose(); 
     } 

     public void CheckConnect(object o) 
     { 
      try 
      { 
       _client = new TcpClient(); 
       _client.Connect(_iPEndPoint); 
       _connect = true; 
       _client.Close(); 
      } 
      catch 
      { 
       _connect = false; 
      } 
     } 
    } 
} 

Answer 1

あなたのコードは極めて不明である - あなたが非同期的に物事をやろうとしているが、ひどいので、やっているように見えます。たとえば、Timerを作成してからすぐにを処理するとします。

Disposeコールの前にタイマーが起動する可能性があるため、デバッガで1行ずつステップ実行すると...しかし、コードを実行するだけで、タイマーが終了する前に処分されます火災のチャンス。さらに、CheckIpを呼び出した直後にconn.Connectedの値を使用しています。したがって、というタイマーが起動した場合は、が発砲したと考えられます。

私はを強くにしてください。あなたはちょうど同期してチェックをすることをお勧めします。それを稼働させて、次にを非同期的なアプローチと考えてください。

Answer 2

スレッドプール上で関数を実行するタイマーを作成します。したがって、CheckIPがタイマーコールバックが返されたときに、実質的に多くの保証を持たないようにします。

さらに、タイマーを作成したらすぐに処分するのはなぜですか？ MSDNは、コールバックがキューに入れられているためにdispose後にコールバックが発生する可能性があると述べていますが、これは確かに悪いスタイルです。また、これはあなたが一度だけチェックしていることを意味しています - それで、あなたはそれを同期的に行うかもしれません - 特にwait_connは、接続が確立されるまで待つべきであることを示します。

using (var client = new TcpClient()) 
{ 
    client.Connect(_iPEndPoint); 
    _connect = true; 
    client.Close(); 
} 

あなたはとにかくそれで何もしない場合は、クラスのメンバ変数を持つはポイントがありません。また

：あなたが行う必要がありますのでTcpClientは使い捨てです。