Question

【C#】while循环中控件的问题

在使用套接字的时候，客户端我用了一个while（true）循环来接收数据。在这个循环中，没当收到消息，就显示出来。为此，我用了一个MESSAGEBOX来显示，但是每次收到消息，除了MESSAGEBOX会显示外，还会多出来32个空白的MESSAGEBOX,这是为什么？
如果我用richtextbox来显示数据的话，就没有问题。请教了。
private void ReceiveMsg()
{
while (true)
{
byte[] packet = new byte[_maxPacket];
_nws.Read(packet, 0, packet.Length);
string _cmd = DecodingBytes(packet);

/// "11"-服务器要求客户机更新在线列表
/// "12"-服务器要求客户机做闪屏振动
/// default-接受用户消息或者系统消息的正文
switch (_cmd)
{
case "11":
{
byte[] onlineBuff = new byte[_maxPacket];
int byteCnt = _nws.Read(onlineBuff, 0, onlineBuff.Length);
IFormatter format = new BinaryFormatter();
MemoryStream stream = new MemoryStream();
stream.Write(onlineBuff, 0, byteCnt);
stream.Position = 0;
StringCollection onlineList = (StringCollection)format.Deserialize(stream);
//online_cb.Items.Clear();
foreach (string onliner in onlineList)
{
if (!onliner.Equals(_username))
{
//online_cb.Items.Add(onliner);
}
}
break;

}
case "12":
{
Nudge();
break;
}
default:
{
string displaytxt = Encoding.Unicode.GetString(packet);
////////下面的代码为接收消息后执行的操作
chatrcd_rtb.AppendText(displaytxt);
// MessageBox.Show(displaytxt);//此处会出错。多出来很多空白的MESSAGEBOX.
break;
}
}
}
}
第一次弹出来我想要的那个MESSAGEBOX，我点了确定以后，又出来一个空白的MESSAGE，再点确定，又出来一个空白的MESSAGEBOX，再点，再出来一个，……，就这样，我点了30多下确定后才结束。

Answer 1

错误原因：NetworkStream接受过来的Byte[]不一定占满packet数组的，packet数组后面有剩余，因此你用Encoding.Unicode.GetString(packet); 返回的string一定有错误。

解决方案：
NetworkStream.Read返回一个int来指示有效地读入字节数，当然这个字节数是<=packet.Length，用这个是来返回正确的字符串。

int readLength = _nws.Read(packet, 0, packet.Length);

...

string displaytxt = Encoding.Unicode.GetString(packet, 0, readLength);

Answer 2

nws.Read(packet, 0, packet.Length);
nws应该是一各socket的输入流吧？
当你执行Read的时候，虽然指定了packet.Length，但Read函数会返回一个实际读取长度，应该依据这个长度来判断是否读到了想要的数据。简单的测试是增加这样的代码：
if (_nws.Read(packet, 0, packet.Length) != packet.Length)
continue ;
应该可以解决问题。

如果你一定要读出指定的长度才肯罢休，那么，就应该增加一个循环，或者看看有没有类似readFully 的函数。
比如：
int readLength = 0 ;
int totalLength = 0 ;

while (totalLength < packet.length)
{
readLength = _nws.Read (packet, totalLength, packet.length - totalLength) ;
if (readLength < 0) // 这时socket已经断开了。
return ;
totalLength += readLength ;

}

Answer 3

多出来的messagebox是怎么样一个效果啊？是你按掉一个就又弹出一个
还是一次屏幕刷出33个来？
messagebox中断效果，后面的break;没有执行

Answer 4

把下面的break;改成return;试试

Answer 5

学习中