求助:socket发送偶而抛出异常问题!!!

Question

//申明
private Hashtable tempsocketList = new Hashtable();
private Hashtable socketList = new Hashtable();
private TcpListener listenmo;//启动socket
listenmo = new TcpListener(8000);
listenmo.Start();//分配socket的线程
private void ReceiveDeal()
{
try
{
  lock(tempsocketList)
  {
    byte[] buf = new byte[10];
    byte[] tempbuf;
    string command,msg;
    int count = 0;
    IDictionaryEnumerator myEnumerator = tempsocketList.GetEnumerator();
    while (myEnumerator.MoveNext())
    {
       Socket socket = myEnumerator.Value as Socket;
       try
       {
if (socket.Connected)
{
  while (socket.Available > 0)
 {
             //自定了个简单的传输协议
    socket.Receive(buf, 0, 10, SocketFlags.None);
    command = Encoding.ASCII.GetString(buf, 0, 10).Trim('\0');
    count = Convert.ToInt32(command.Substring(5,5));
    tempbuf = new byte[count];
    socket.Receive(tempbuf, 0, count, SocketFlags.None);
    msg = Encoding.ASCII.GetString(tempbuf, 0, count).Trim('\0');     if (msg == "Test")
    {       lock(socketList)
      {
        if (socketList.ContainsKey(command.Substring(0,5)))
        socketList.Remove(command.Substring(0,5));
        socketList.Add(command.Substring(0,5) , socket);
      }      tempsocketList.Remove(myEnumerator.Key);

     return;
             }
          }
        }
        else
        {
          socket.Close();
          tempsocketList.Remove(myEnumerator.Key);
          return;
         }
      }
      catch (Exception ex)
      {  
      }
    }
  }
}
catch (Exception ex)
{  
}}//发送的函数
public void SendMsg(string ip , string msg)
{
try
{
msg = Convert.ToString(msg.Length).PadLeft(5,'0') + msg; byte[] buf;
buf = new byte[msg.Length];
Buffer.BlockCopy(Encoding.ASCII.GetBytes(msg), 0, buf, 0, msg.Length);

lock(socketList)
{
(socketList[ip] as Socket).Send(buf);
}
}
catch (Exception ex)
{ }
}
//以下是发送给客户机的函数 每一分多钟会运行一次  //但是每运行1000次左右 总会抛出一两次的异常//Collection was modified; enumeration operation may not execute.   
//at System.Collections.HashtableEnumerator.MoveNext()
//at lj.lj.SendAllInit()public void SendAllInit()
{
try
{
string ip = "" , msg = "";
int i = 2000; IDictionaryEnumerator myEnumerator = socketList.GetEnumerator();
while (myEnumerator.MoveNext())
{
ip = myEnumerator.Key as String;

msg = "Init";
                            //此处省略一堆 msg += string
msg += Convert.ToString(i).PadLeft(4,' ');
SendMsg(ip,msg);
}
}
catch (Exception ex)
{
}
}各位大侠们 小弟实在不清楚问题出在哪里  帮帮忙忙啊!!!

Answer 1

  抛出了什么异常，并且在哪个函数类抛出的异常？鉴于你说是运行1000次左右会抛出，让你去debug代码不现实，所以这时就凸显出了log的好处了。你可以在相应代码段加上log，然后再判断是在哪里出了问题，出了什么问题。光看代码是永远看不出来的。

Answer 2

代码里有加log  日志报错内容如下:
Collection was modified; enumeration operation may not execute.   
at System.Collections.HashtableEnumerator.MoveNext()
at lj.lj.SendAllInit()可是还是不清楚问题出在哪里啊!!!!!!!!!!!!!!!

Answer 3

日志内容不是很明确了么，因为你之前操作过socketList,有socketList.ReMove()之类的操作，导致列表已经更改，所以影响到SendAllInit()这个函数了。

Answer 4

4 5 楼这么一说  我感觉好像也是 lock 的问题我不应该在 SendMsg() 这个函数里 lock(socketList)应该在 SendAllInit()里  这样锁
lock(socketList)
{
IDictionaryEnumerator myEnumerator = socketList.GetEnumerator();
while (myEnumerator.MoveNext())
{
...
}
}
先这样优化吧  要测试几天才知道ok了没....  先谢了啊