Question

我目前正在编写一个与I / O设备交互的程序，需要一种每隔x秒钟轮询设备的方法，以便检查输入/输出连接。为此，我使用了一个按钮，它使用计时器和计时器事件句柄创建一个线程来进行轮询。但是，我注意到在任务管理器中，随着时间的推移，它正在慢慢消耗更多的内存。下面是一些（我认为）与我的问题相关的代码片段。

创建线程的按钮：

private void btnConnect_Click(object sender, EventArgs e)
    {
        new Thread(start).Start();
    }

包含计时器的线程：

        public void start()
    {
        timer = new System.Timers.Timer(1000);
        timer.Elapsed += new ElapsedEventHandler(timerElapsed);
        timer.Enabled = true;
    }

ElapsedEventHandler：

public void timerElapsed(object sender, ElapsedEventArgs e)
    {
        connect();
    }

最后方法是connect（）;：

public void connect()
    {
        StringBuilder sb = new StringBuilder();
        sb.Append(txtIPseg1.Text + "." + txtIPseg2.Text + "." + txtIPseg3.Text + "." + txtIPseg4.Text);
        int Port = int.Parse(txtPort.Text);
        string address = sb.ToString();

        //send data
        byte[] bData = new byte[71];
        bData[0] = 240;
        bData[1] = 240;
        bData[2] = 0;
        bData[3] = 1;
        bData[68] = 240;
        bData[69] = 240;
        bData[70] = this.CalculateCheckSum(bData);

        try
        {
            byte[] result = this.SendCommandResult(address, Port, bData, 72);
            if (result != null)
            {
                this.Invoke((MethodInvoker)delegate
                {
                    txtOutput1.Text = (result[4] == 0x00 ? "HIGH" : "LOW"); // runs on UI thread
                });
            }
        }
        catch (Exception ex)
        {
            MessageBox.Show(ex.ToString());
        }

    }

我很确定泄漏要么来自计时器，要么是方法connect（）中使用的匿名委托;有人有什么想法吗？

Answer 1

每次单击按钮时，您都在创建一个新计时器。此外，您没有保留对它的引用，因此它将被垃圾收集器销毁。无需在新线程上启动计时器，因为计时器将在新线程上引发Elapsed事件。

class Form1 ...
{
    private System.Timers.Timer timer = null;

    public void start()
    {
    if (timer == null)
        {
        timer = new System.Timers.Timer(1000);
        timer.Elapsed += new ElapsedEventHandler(timerElapsed);
        }
    timer.Enabled = true;
    }

    ...
}

就内存泄漏而言，我不会假设存在内存泄漏只是因为您看到在应用程序运行时内存使用量似乎随机波动。当您在.NET之类的复杂框架内运行时，这是正常行为。每次计时器触发时，它都会调用connect方法来创建新对象。这些对象将保留在内存中，直到垃圾收集器到处清理它们为止。因此，看到内存在几分钟后突然上升，然后再次下降，这并不奇怪。我不会怀疑有问题，除非它在更长的时间内不断失控。

使用StringBuilder类的方式也很奇怪。你在做什么：

StringBuilder sb = new StringBuilder();
sb.Append(txtIPseg1.Text + "." + txtIPseg2.Text + "." + txtIPseg3.Text + "." + txtIPseg4.Text);
string address = sb.ToString();

并不是更好（事实上它有点糟糕），而不仅仅是这样做：

string address = txtIPseg1.Text + "." + txtIPseg2.Text + "." + txtIPseg3.Text + "." + txtIPseg4.Text;

如果您正在寻找更有效，更容易阅读的方式，请尝试这样的事情

string address = string.Format("{0}.{1}.{2}.{3}", txtIPseg1.Text, txtIPseg2.Text, txtIPseg3.Text, txtIPseg4.Text);

然而，没有任何东西可以作为任何会导致内存泄漏的东西向我跳出来，所以除非你有充分的理由这样想，否则我不会担心。

c＃内存泄漏系统

1 个答案: