2010-01-22 10 views
9

I wykazały błąd: Błąd czasowego, to niejednoznaczny odniesienia między System.Windows.Forms.Timer „” i „” System.Threading.Timer

 
'Timer' is an ambiguous reference between 'System.Windows.Forms.Timer' and 'System.Threading.Timer' 
przy dodawaniu kodu zegara

using System; 
using System.Collections.Generic; 
using System.ComponentModel; 
using System.Data; 
using System.Drawing; 
using System.Linq; 
using System.Text; 
using System.Windows.Forms; 
using System.Net.Sockets; 
using System.Threading; 
using System.Security.Cryptography; 


namespace SocketClient 
{ 

    public partial class SocketClient : Form 
    { 
     System.Net.Sockets.TcpClient clientSocket = new System.Net.Sockets.TcpClient(); 
     NetworkStream serverStream = default(NetworkStream); 
     string readData = null; 


     public SocketClient() 
     { 
      InitializeComponent(); 
      Timer timer = new Timer(); 
      timer.Tick += new EventHandler(TimerOnTick); 
      timer.Interval = 1000; 
      timer.Start(); 
     } 



     private void TimerOnTick(object sender, EventArgs ea) 
     { 
      Invalidate(); 
     } 

     protected override void OnPaint(PaintEventArgs pea) 
     { 
      StringFormat strfmt = new StringFormat(); 
      strfmt.Alignment = StringAlignment.Far; 
      strfmt.LineAlignment = StringAlignment.Far; 

      pea.Graphics.DrawString(DateTime.Now.ToString("F"), 
      Font, new SolidBrush(ForeColor), 
      ClientRectangle, strfmt); 

     } 

     private void getMessage() 
     { 
      while (true) 
      { 
       serverStream = clientSocket.GetStream(); 
       int buffSize = 0; 
       byte[] inStream = new byte[10025]; 
       buffSize = clientSocket.ReceiveBufferSize; 
       serverStream.Read(inStream, 0, buffSize); 
       string returndata = System.Text.Encoding.ASCII.GetString(inStream); 
       readData = "" + returndata; 
       msg(); 
      } 
     } 


     private void msg() 
     { 
      if (this.InvokeRequired) 
       this.Invoke(new MethodInvoker(msg)); 
      else 
       textDisplay.Text = textDisplay.Text + Environment.NewLine + " >> " + readData; 
     } 


     private void buttonConnect_Click(object sender, EventArgs e) 
     { 


      // show the message if no input is enter. 
      if (string.IsNullOrEmpty(textName.Text) || string.IsNullOrEmpty(textPort.Text) || string.IsNullOrEmpty(textIP.Text)) 
      { 
       MessageBox.Show("Please enter Name, IP Address & Port #"); 
      } 
      else 
      { 
       //connect to the server if all 3 input is enter 
       readData = "Conected to NYP Server ..."; 
       msg(); 

       clientSocket.Connect(textIP.Text, Convert.ToInt32(textPort.Text)); 
       serverStream = clientSocket.GetStream(); 

       byte[] outStream = System.Text.Encoding.ASCII.GetBytes(textName.Text + "$"); 
       serverStream.Write(outStream, 0, outStream.Length); 
       serverStream.Flush(); 

       Thread ctThread = new Thread(getMessage); 
       ctThread.Start(); 


      } 
     } 

     private void buttonSend_Click(object sender, EventArgs e) 
     { 
      // Show msg box if no server is connected 
      if (serverStream == null) 
      { 
       MessageBox.Show("Please connect to a server first!"); 
       return; 
      } 

      // Send text 
      byte[] outStream = System.Text.Encoding.ASCII.GetBytes(textSend.Text + "$"); 
      serverStream.Write(outStream, 0, outStream.Length); 
      serverStream.Flush(); 

      // Clear text 
      textSend.Text = ""; 

     } 

     private void textDisplay_TextChanged(object sender, EventArgs e) 
     { 
      textDisplay.SelectionStart = textDisplay.Text.Length; 
      textDisplay.ScrollToCaret(); 
      textDisplay.Refresh(); 
     } 

     private void textSend_TextChanged(object sender, EventArgs e) 
     { 
      buttonSend.Enabled = !string.IsNullOrEmpty(textSend.Text); 
     } 
    } 
} 
+1

Należy zauważyć, że .NET Framework naruszył jeden z Wytycznych projektowania systemu .NET Framework: klasy o ogólnych nazwach powinny być klasami ogólnego przeznaczenia. Rozważmy 'String' vs.' XmlTextReader'. 'Timer' to nazwa ogólna i jedna lub obie klasy są wyspecjalizowane. –

Odpowiedz

5

problemem jest to, że jesteś

using System.Windows.Forms; 
using System.Threading; 

Obie te przestrzenie nazw posiadają klasę Timer i kompilator nie może powiedzieć, który z nich korzystać. Kiedy deklarujesz swoją Timer użyć pełnej nazwy, albo:

System.Windows.Forms.Timer 

lub

System.Threading.Timer 

WinForms Timer Class

Threading Timer Class

Na podstawie wykorzystania klasy, myślę, że chcesz System.Windows.Forms.Timer, podobnie jak poniżej:

public SocketClient() 
    { 
     InitializeComponent(); 
     var timer = new System.Windows.Forms.Timer(); 
     timer.Tick += new EventHandler(TimerOnTick); 
     timer.Interval = 1000; 
     timer.Start(); 
    } 
29

Istnieje tutaj są różne opcje:

  • Użyj aliasu:

    using UITimer = System.Windows.Forms.Timer; 
    
    ... 
    UITimer timer = new UITimer(); 
    
  • używać nazwy pełne kwalifikacje:

    System.Windows.Forms.Timer timer = new System.Windows.Forms.Timer(); 
    
  • Użyj alias namespace:

    using WinForms = System.Windows.Forms; 
    ... 
    WinForms::Timer timer = new WinForms::Timer(); 
    

Chciałbym jednak osobiście proponuję podział się ten kod interfejsu użytkownika od kodu sieciowego - w tym momencie jest to mało prawdopodobne, aby być problemem.

Chciałbym również zauważyć, że obecnie czytasz ze strumienia, nie biorąc pod uwagę wartości zwracanej - to zły pomysł, ponieważ nie wiesz, ile bufora rzeczywiście zawiera nowe dane.

+0

Całkowicie zapomniałem o aliasingu, głupku mnie! – Sapph

+0

dlaczego ta odpowiedź nie jest akceptowaną odpowiedzią? – Liban

Powiązane problemy