我正在尝试编写投票服务器和客户端,因此您启动该程序并显示投票表单,您可以对各种项目进行投票。对于服务器部分,我让服务器在一个单独的线程中运行,但它使用了大量的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);
}
答案 0 :(得分:1)
您正在使用阻止类型的连接。由于TcpListener.AcceptConnection(),您创建的循环会导致CPU开销。您的解决方案是接受非阻塞套接字连接,这是通过异步接收来自套接字的数据来完成的。
这是msdn链接,解释了它的工作原理。 http://msdn.microsoft.com/en-us/library/dxkwh6zw.aspx
答案 1 :(得分:0)
我看到你有字符串连接,这基本上会影响性能;尝试使用StringBuilder - 消息变量应该是StringBuilder类型。