2

我正在尝试编写一个投票服务器和客户端,因此您启动程序并显示投票表格,您可以对各种项目进行投票。对于服务器部分,我让服务器在单独的线程中运行,但它使用了大量 CPU,如何减少它使用的 CPU 量?

这是我的服务器:

        Form1 main = new Form1();

        try
        {
            IPAddress ipAd = IPAddress.Parse(main.ipAddress); //use local m/c IP address, and use the same in the client

            /* Initializes the Listener */
            TcpListener myList = new TcpListener(ipAd, 55548);

            /* Start Listeneting at the specified port */
            myList.Start();

            while (true)
            {
                string message = "";
                Socket s = myList.AcceptSocket();



                if (main.users.Contains(s.RemoteEndPoint.ToString()) == false)
                    main.users.Add(s.RemoteEndPoint.ToString());



                byte[] b = new byte[500];
                int k = s.Receive(b);



                for (int i = 0; i < k; i++)
                {
                    message += (Convert.ToString(b[i]));
                }

                string[] messageArray = message.Split('/');

                MessageBox.Show("help");

                if (messageArray[0].CompareTo("vote") == 0)
                {

                    if (main.votes.ContainsKey(messageArray[1]) != true) main.votes.Add(messageArray[1], 1);
                    else main.votes[messageArray[1]]++;

                    string[] temp = main.textBox1.Text.Split(' ');
                    int numVotes = Convert.ToInt32(temp[1]);
                    numVotes++;
                    main.textBox1.Text = temp[0] + " " + Convert.ToString(numVotes);

                }
                if (messageArray[0].CompareTo("start") == 0)
                {
                    main.updateEverything();
                }
                if(messageArray[0].CompareTo("withdraw") == 0)
                {
                    main.votes[messageArray[1]]--;

                    string[] temp = main.textBox1.Text.Split(' ');
                    int numVotes = Convert.ToInt32(temp[1]);
                    numVotes--;
                    main.textBox1.Text = temp[0] + " " + Convert.ToString(numVotes);
                }

                /* clean up */
                s.Close();
                myList.Stop();
            }
        }
        catch (Exception e)
        {
            Console.WriteLine("Error..... " + e.StackTrace);
        }
4

2 回答 2

1

您正在使用阻塞类型的连接。由于 TcpListener.AcceptConnection(),您创建的循环会导致 CPU 开销。您的解决方案是接受非阻塞套接字连接,这是通过从套接字异步接收数据来完成的。

这是解释其工作原理的 msdn 链接。 http://msdn.microsoft.com/en-us/library/dxkwh6zw.aspx

于 2012-12-18T00:19:05.510 回答
0

我看到您的字符串连接基本上会影响性能;尝试使用 StringBuilder - message 变量应该是 StringBuilder 类型。

于 2012-12-18T00:14:21.497 回答