客户端未从服务器多线程接收数据

时间:2016-03-26 16:32:24

标签: c# multithreading sockets tcpclient

我想聊聊天。服务器在控制台应用程序中生成,客户端在winforms中生成。

在客户端,我写了一个昵称并连接到服务器。服务器从客户端接收名称。我在字典列表中添加了连接到服务器的所有客户端,其中包含(字符串)名称(TcpClient)套接字。之后,我想向每个客户端发送客户端列表。

当我在服务器上调试时,套接字出现 DualMode,EnableBroadcast 错误。在客户端,当我必须收到列表时,它会停止并且不会执行任何操作。

服务器

namespace MyServer
{
    class MyServer
    {
        public Dictionary<string, TcpClient> clientList = new Dictionary<string, TcpClient>();
    TcpListener server = null;
    NetworkStream stream = null;
    StreamReader streamReader = null;
    StreamWriter streamWriter = null;
    TcpClient clientSocket;

    String messageReceived;
    int number_clients = 0;
    public MyServer(TcpClient clientSocket_connect)
    {
        stream = clientSocket_connect.GetStream();
        streamReader = new StreamReader(stream);
        streamWriter = new StreamWriter(stream);

        receiveMessage(clientSocket_connect); // receive messages
        }
        public MyServer()
        {
            Thread thread = new Thread(new ThreadStart(run));
            thread.Start();
        }           
        public void receiveMessage(TcpClient client_Socket)
        {
            messageReceived = streamReader.ReadLine();             

            if (messageReceived.Substring(messageReceived.Length - 4) == "user")
            {
                String name = messageReceived.Substring(0, messageReceived.Length - 4);
                bool found = false;
                foreach (var namefound in clientList.Keys)
                {
                    if (namefound == name)
                    {
                        found = true;
                        streamWriter.WriteLine("The user already exists");
                        streamWriter.Flush();
                    }
                }
                if (!found)
                {
                    //show who's connected
                    Console.WriteLine(name + " is online");
                    number_clients++;
                    clientList.Add(name, client_Socket);

                    //send to client clientlist
                    String send = null;
                    foreach (var key in clientList.Keys)
                    {
                        send += key + ".";
                    }
                    foreach (var value in clientList.Values)
                    {
                        TcpClient trimitereclientSocket = value;
                        if (trimitereclientSocket != null)
                        {
                            NetworkStream networkStream = trimitereclientSocket.GetStream();
                            StreamWriter networkWriter = new StreamWriter(networkStream);
                            networkWriter.WriteLine(send + "connected");
                            networkWriter.Flush();
                        }
                    }
                }
            }

        }
        void run()
        {
            IPAddress ipAddress = IPAddress.Parse("127.0.0.1");
            server = new TcpListener(ipAddress, 8000);
            server.Start();
            Console.WriteLine("Server started!");
            while (true)
            {
                clientSocket = server.AcceptTcpClient();                
                new MyServer(clientSocket);
            }
        }
    }
static void Main(string[] args)
        {
            MyServer server = new MyServer();
        }
}

客户端

 namespace MyClient
    {
        class MyClient
        {
            List<string> clientList = new List<string>();

            TcpClient client = null;
            NetworkStream stream = nul

l;
        StreamReader streamReader = null;
        StreamWriter streamWriter = null;

        bool connected;
        String received_message;
        public MyClient()
        {
            client = new TcpClient("127.0.0.1", 8000);
            stream = client.GetStream();
            streamReader = new StreamReader(stream);
            streamWriter = new StreamWriter(stream);     
        }
        public void sendClientName(String name)
        {
            streamWriter.WriteLine(Convert.ToString(name));
            streamWriter.Flush();
        }
        public List<ClientName> receiveClientList()
        {
            List<ClientName> val = new List<ClientName>();         
                string name = Convert.ToString(streamReader.ReadLine());
                if (name.Substring(0, name.Length - 9) == "connected")
                {
                    ClientName client = new ClientName();
                    client.Nume = name;
                    val.Add(client);
                }          
            return val;
        }

    }
}

客户表格

 public partial class Form1 : Form
{
    MyClient client = new MyClient();
    public Form1()
    {
        InitializeComponent();
        Thread receiveClients = new Thread(new ThreadStart(getMessages));
    }

    private void btnConnect_Click(object sender, EventArgs e)
    {
        client.sendClientName(txtNickname.Text + "user");
    }
    public void getMessages()
    {
        while (true)
        {
            lbClientsConnected.Items.Add(client.receiveClientList());
        }
    }
}

1 个答案:

答案 0 :(得分:2)

运行代码时,我无法重现任何错误。我不知道你的意思是什么“套接字出现在DualMode,EnableBroadcast错误”。也就是说,代码存在许多可修复的问题,包括一些直接与您关注的问题“当我必须收到它停止但不做任何事情的列表时。”

代码最大的问题可能是你从不启动客户端的接收线程。您需要在Start()对象创建后调用Thread方法:

public Form1()
{
    InitializeComponent();
    Thread receiveClients = new Thread(new ThreadStart(getMessages));

    // The receiving thread needs to be started
    receiveClients.Start();
}

现在,即使已修复,您还有其他一些问题。下一个重要问题是您正在正确解析收到的文本。在您的代码中,您应该在字符串末尾查找文本"connected",而是提取文本的其他部分(使用客户端名称列表)。

您的receiveClientList()方法应该如下所示:

private const string _kconnected = "connected";

public List<string> receiveClientList()
{
    List<string> val = new List<string>();
    string name = Convert.ToString(streamReader.ReadLine());

    // Need to check the *end* of the string for "connected" text,
    // not the beginning.
    if (name.EndsWith(_kconnected))
    {
        name = name.Substring(0, name.Length - _kconnected.Length);
        val.Add(name);
    }
    return val;
}

(你没有在你的问题中分享ClientName类,实际上这个例子不需要它;一个简单的string值足以满足本练习的目的。另外,我,'我们引入了名为const string的{​​{1}},以确保在每个需要的地方正确使用字符串文字,以及简化使用。)

但即使修复了这两个问题,你仍然在_kconnected代码中有一对实际处理receive方法返回值的代码。首先,您将从receive方法返回的Form对象传递给List<T>方法,这只会导致ListBox.Items.Add()显示对象的类型名称,而不是元件。

其次,因为代码是在拥有ListBox对象的UI线程以外的线程中执行的,所以必须在调用ListBox时包装该调用。否则,您将获得跨线程操作异常。

解决这两个问题,你得到了这个:

Control.Invoke()

通过这些更改,代码将处理客户端发送的消息,并返回客户端列​​表。这应该会让你更进一步。也就是说,你还有许多其他问题,包括一些相当基本的问题:

  1. 最大的问题是,当您接受服务器中的连接时,您将创建一个全新的服务器对象来处理该连接。这有几个原因并不是一个好主意,但主要原因是代码的其余部分似乎在概念上假设单个服务器对象正在跟踪所有客户端,但每个连接将导致其自己的集合客户对象,每个集合只有一个成员(即客户)。

    请注意,一旦解决了这个问题,您将拥有多个访问单个字典数据结构的线程。您需要学习如何使用public void getMessages() { while (true) { // Need to receive the data, and the call Invoke() to add the // data to the ListBox. Also, if adding a List<T>, need to call // AddRange(), not Add(). string[] receivedClientList = client.receiveClientList().ToArray(); Invoke((MethodInvoker)(() => listBox1.Items.AddRange(receivedClientList))); } 语句来确保跨多个线程安全地共享字典。

  2. 另一个重要问题是,您不是使用第一次接受连接时创建的lock,而是创建一个全新的streamWriter对象(在名为{{1的局部变量中引用)写入套接字。在这个非常简单的例子中,它工作正常,但在缓冲和缺乏线程安全之间,这个设计错误的代码可能会出现严重的数据损坏问题。

  3. 问题较少但值得修复的是,您的服务器代码完全无法利用您将客户端存储在字典中的事实,以及.NET具有有用的辅助函数来执行操作喜欢加入一堆字符串。我会写你的服务器的StreamWriter方法更像这样:

  4. networkWriter

    上述内容并非详尽无遗。坦率地说,您可能应该花更多时间查看现有的网络感知代码示例,例如:在MSDN和Stack Overflow上,以及在网站,博客或书籍上的教程。即使您按照每个连接一个线程的方式编写服务器,就像您在这里尝试做的那样,有很多细节您真正需要得到正确的,而且到目前为止还没有。

    但是我希望上述内容足以让你超越目前的障碍,继续解决下一个大问题。 :)