c# - 如何正确有效地使用 WorkbookBeforeClose 事件？

Question

每天，一个人需要检查特定的工作簿是否已根据彭博和路透社的市场数据正确更新；即所有数据都已通过并且“数字看起来正确”。过去，人们不检查“数字”，导致上传到其他系统的数据不准确。

想法是需要开发“某些东西”来防止用户关闭/保存工作簿，除非他/她检查过更新是否正确/准确。 numbers look correct Action 纯粹是一个直观的练习，因此不会以任何方式编码。

简单的解决方案是在关闭特定工作簿之前提示用户验证数据是否已检查。

使用 VSTO SE for Excel 2007，创建了一个加载项，它挂接到在加载项 ThisAddIn_Startup 中初始化的 WorkbookBeforeClose 事件

private void wb_BeforeClose(Xl.Workbook wb, ref bool cancel)
{
    //.... snip ...

    if (list.Contains(wb.Name))
    {
        DailogResult result = MessageBox.Show("some message", "sometitle", MessageBoxButtons.YesNo);

        if (result != DialogResult.Yes)
        {
            cancel = true; // i think this prevents the whole application from closing
        } 
    } 
}

我找到了以下 ThisApplication.WorkbookBeforeSave vs ThisWorkbook.Application.WorkbookBeforeSave建议应该使用 ThisApplication.WorkbookBeforeClose 事件，我认为这是我正在做的事情，因为它将跨越所有打开的文件。

我对该方法的问题是，假设我打开了多个文件，其中一些在我的 list 中，该事件会阻止 Excel 按顺序关闭所有文件。它现在要求每个文件单独关闭。 编辑:当从文件菜单中使用退出 Excel 时会发生这种情况。

问题

1. 我是否正确使用了 WorkbookBeforeClose 事件？是否有效且高效地使用了该事件？
2. 我应该使用应用程序级事件还是文档级事件？
3. 上述行为是否正常？
4. 在加载项中使用工作簿事件时，欢迎提出任何其他建议

更新 [2010 年 3 月 30 日]:

修改后，我还尝试了以下尝试将 BeforeClose 事件处理程序绑定(bind)到按照上面链接的建议打开的每个工作簿。

private void ThisAddIn_Startup(...)
{
    // snip
    Globals.ThisAddin.Application.WorkbookOpen += Application_Open; 
}

private void Application_Open(XL.Workbook wb)
{
    wb.BeforeClose += Document_WorkbookBeforeClose; // method does the same as above
}

我发现这种方法的问题是我尝试关闭所有 Excel 文件(使用 Exit Excel 选项)事件处理程序不执行。根据我的观察，当要检查的文档不是事件文档时，就会发生这种情况。

与我最初的方法相比，这种方法似乎不稳定。我不确定或不满意的一件事是每次打开文档时都绑定(bind)事件。

更新 [2010 年 4 月 7 日]:

Glen 的建议答案很有用，但没有解决眼前的问题，因此我进一步澄清了最后一个问题。

我也找到了这个博客How to Get an Excel VSTO Workbook Closed Event这与我的问题有些相关，因为它可以在我的解决方案的替代方法中使用，使用监视器类型的方法来处理工作簿(并且可能还使用新引入的 OnWorkbookClosed 事件)。

更新 [2010 年 4 月 8 日]:

似乎有些困惑，我不关心对工作簿本身的任何验证，而是关心我使用的方法(即使用应用程序级 WorkbookBeforeClose 事件)是否正确。 @Mathias 下面的评论显示了对与问题 3 有关的部分问题的正确理解，但我认为这是默认的 excel 行为。克服这个问题的解决方案是创建一个仅关闭我的特定文件的关闭函数。

3. Is the behaviour described above normal? Yes, but why? Because the add-in hooks into the application-level event, the checks and cancellation of the event blocks the application from closing any further workbooks. The key here is the ref bool cancel argument (cancel=false allows normal closing of the workbook(default), cancel=true prevents the workbook from closing)

---

带有 VSTO SE 的 VS 2005

Answer 1

应用程序级 WorkbookBeforeClose 是要使用的。问题是不稳定的行为是由于调用 WorkbookBeforeClose 时未保存文件造成的。如果不是，则 Save 事件将启动并且您现在实际上已经丢失了 WorkbookBeforeClose 事件，因为它已经发生了。下面是一些有助于处理此问题的 VBA 代码。

Private Sub Workbook_BeforeClose(Cancel As Boolean)
    If Not Me.Saved Then
        NotSavedPrompt = Me.Name & " has not been saved. Would you like to save now?"
        SaveYesNo = MsgBox(NotSavedPrompt, vbQuestion + vbYesNoCancel)
        Select Case SaveYesNo
            Case vbYes
                Me.Save
            Case vbNo
                Me.Saved = True
            Case vbCancel
                Cancel = True
                Exit Sub
          End Select
    End If
    Call MyRoutine() //'this should be your sub that does what you want
End Sub