时间:2019-03-17 标签:c#system.timers内存泄漏

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(); 中使用的匿名委托(delegate)，有人有什么想法吗？

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);

但是，没有什么会引起内存泄漏，所以除非你有充分的理由这么认为，否则我不会担心它。