1

这似乎是一个愚蠢的问题,但我只是想确保我做对了。我的主要形式大部分时间都不可见。要打开它,我有一个 NotifyIcon。菜单选项之一是退出应用程序。我还有一些静态全局变量需要在应用程序关闭之前处理掉。所以在program.cs我有这个。

    [STAThread]
    static void Main()
    {
        InitializeApplication();
        InitializeMainForm();
        Application.Run(main);
    }
    private static void InitializeApplication()
    {
        Application.EnableVisualStyles();
        Application.SetCompatibleTextRenderingDefault(false);
        Application.ApplicationExit += Application_ApplicationExit;
    }
    private static void InitializeMainForm()
    {
        main = new AssignButtonForm();
        main.FormClosing += main_FormClosing;
        Globals.StartNotify();
    }

    static void main_FormClosing(object sender, FormClosingEventArgs e)
    {
        var dlg = MessageBox.Show("Turn off Application?", "Exit?", MessageBoxButtons.OKCancel, MessageBoxIcon.Question, MessageBoxDefaultButton.Button1);
        if (dlg == DialogResult.OK)
        {
            Globals.notifyIcon1.Dispose();
            Application.Exit();
        }
        else
        {
            e.Cancel = true;
        }
    }

所以我希望这是正确的称呼方式。

    private void exitToolStripMenuItem_Click(object sender, EventArgs e)
    {
        this.Hide();
        Application.OpenForms[0].Close();
    }

那是对的吗?或者,还有更好的方法?

编辑

好的,所以我的一个类有自己的 Dispose 方法

public class KeymonNotifyIcon : IDisposable
{
    public KeymonNotifyIcon()
    {
        InitializeComponent();
        keymonMenuStrip.SetupKeysSelected += OnSetupKeysSelected;
    }
    ~KeymonNotifyIcon()
    {
        Dispose();
    }
    public void Dispose()
    {
        if (notifyIcon1 != null)
            notifyIcon1.Dispose();

        if (keymonMenuStrip != null)
            keymonMenuStrip.Dispose();
    }
}

全球班

public static class Globals
{
    public static TraceSource trace = new TraceSource("Keymon");
    public static KeymonNotifyIcon notifyIcon1;
    public static void StartNotify()
    {
        notifyIcon1 = new KeymonNotifyIcon();
    }
}
4

2 回答 2

2

实际上,我相当肯定它Application.ExitDispose为你调用你所有的方法(只要你已经实现了IDisposable

看到这个问题,它引用了这个问题

于 2013-02-12T17:48:51.673 回答
1

如果您的程序刚刚关闭,则尚未被垃圾回收的对象将运行其终结器。推荐的模式是对象实现IDisposable具有终结器以确保其IDisposable运行。据我所知,.NET BCL 类始终遵循这种模式。但是,您自己的或第三方/开源组件可能不遵循该模式。

注意:MSDN 上的链接模式不调用 GC.SuppressFinalize。查看如何使用它来减少 GC 开销。

单独实施IDisposable并不足以确保正确处置对象。

如果您有实现 的静态引用对象,则从应用程序关闭事件IDisposable显式调用它们是更可靠的解决方案。IDisposable.Dispose()

编辑

您的 Dispose 实现将导致在拥有的对象上调用 Dispose() 两次,因为:

if (notifyIcon1 != null)
        notifyIcon1.Dispose();

不设置notifyIcon1null,并且终结器无条件地Dispose()再次调用。

此外,您将始终导致终结器运行(这会使该类的 GC 更昂贵,因为您没有在Dispose().

于 2013-02-12T18:03:01.917 回答