【发布时间】:2019-04-30 21:30:24
【问题描述】:
我正在尝试遍历文件夹中的所有文件,打开它们并删除文档信息。我在处理无法打开的文件时遇到问题,或者在打开时弹出我们关于禁用宏的提示。我尝试使用 on error resume next 和 on error goto 0 来解决此问题。但随后出现运行时失败,因为当我尝试关闭已打开的文件时,我的工作簿对象 (wb) 尚未设置。
我已阅读有关“On Error Resume Next”和“On error goto 0”的文档,但我不相信我在这里正确使用它们。 非常感谢任何帮助,谢谢。
Option Explicit
Sub test_Scrubber_New()
Dim directory As String, fileName As String, i As Variant, wb As Workbook
Application.DisplayAlerts = False
Application.ScreenUpdating = False
'directory = "C:\Users\bayli\Desktop\Files for Testing\"
directory = "C:\Users\bayli\Desktop\excel files\"
fileName = Dir(directory & "*.xl??")
i = 0
Do While fileName <> ""
On Error Resume Next
Set wb = Workbooks.Open(directory & fileName)
On Error GoTo 0
'remove info
ActiveWorkbook.RemoveDocumentInformation (xlRDIAll)
wb.Close True
i = i + 1
fileName = Dir()
Application.StatusBar = "Files Completed: " & i
Loop
Application.StatusBar = False
Application.ScreenUpdating = True
Application.DisplayAlerts = True
MsgBox "Complete"
End Sub
我更新了我的代码以包括: If Not wb Is Nothing 然后删除@PatricK 建议的信息,它正在工作,但是它不断停止并弹出关于更新链接的弹出窗口。如果我单击“不更新”,我的代码会根据需要继续工作,但有没有办法处理这个问题。我正在循环浏览超过 5k 个文件,因此您可以想象它需要一段时间。花费的时间不是问题,但目前我坐在这里不得不多次点击“不更新”。我认为 Application.DisplayAlerts = False 会阻止这些弹出窗口,但事实并非如此。
【问题讨论】:
-
您尝试打开一个工作簿,如果失败则忽略错误,然后继续对该工作簿进行文字处理。那当然会失败,因为它无法打开!您需要捕获错误,然后执行其他操作。
If Err.Number <> 0 Then 'do something else -
请注意,如果您在打开文件时出现错误,
ActiveWorkbook.RemoveDocumentInformation (xlRDIAll)行将不管运行,并将在该 Activeworkbook 上。您可能希望将其切换为wb.RemoveDocumentInformation(...)。 -
@MichalRosa“你的代码是正确的”——我不敢苟同。
RemoveDocumentInformation应该针对wb,而不是ActiveWorkbook。而On Error Resume Next不是正确的错误处理。 “正确”代码不会愉快地在未知错误状态下运行,并在一切都着火时愉快地报告“完成”。硬编码的路径也是一个问题,i是Variant是非常值得怀疑的,xlRDIAll是强制通过ByVal无缘无故,Dir()不需要多余的括号,程序名称可能更有意义,PascalCase和明确的Public。 -
@MichalRosa - 文件的存在并不意味着 Excel 可以打开它。此外,命中文件系统的 anything 应该会失败,因为正在运行的代码不是唯一的 IO 使用者。其他进程删除
Dir调用和Workbooks.Open调用之间的文件是完全可行的。 -
判断 Excel 是否无法打开文件的简单方法是在尝试打开文件后检查
wb Is Nothing。
标签: excel vba error-handling