2009-12-18 2 views
-2

Предыдущая проблема (я не подключен к своему серверу, и я набираю текст в тексте отправки и нажимаю send, в моей программе будет отображаться предупреждение serverStream.Write(outStream, 0, outStream.Length); NullreferenceException не обрабатывается.) пока я не вставлю этой кодировкуМое сообщение не отображается после того, как я нажимаю отправить

if (serverStream == null) 
     MessageBox.Show("Please connect to a server first!"); 
     return; 

Но после того, как я вставить код, мое сообщение не отображается после того, как я нажимаю отправить

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; 

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(); 
     } 


     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) 
     { 
      readData = "Conected to NYP Chat Server ..."; 
      msg(); 
      clientSocket.Connect("127.0.0.1", 8888); 
      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); 
     } 
    } 
} 
+4

Пожалуйста, изучите, как программировать, а не просить нас делать все для вас. – SLaks

ответ

3

Вы забыли поставить брекеты вокруг if заявления.

Следовательно, if применяется только к вызову MessageBox, а инструкция return выполняется независимо.

Изменить его

if (serverStream == null) { 
    MessageBox.Show("Please connect to a server."); 
    return; 
} 
+1

У меня есть чувство дежавю. –

+0

ОК спасибо, решил –

+0

@Martinho: Я не удивлен. :-) – SLaks

0

Вы должны поставить фигурные скобки {} вокруг, если заявление. Без них в выражении if выполняется только первая строка вашего if. Это означает, что ваше возвращение всегда вызывается.

Смежные вопросы