
У меня совсем нет опыта работы с Excel VBA, но я пытаюсь создать макрос, который печатает в PDF-файл диапазон указанных ячеек и отправляет его через Outlook.
Я не смогу работать над следующим:
- Не сохраняется в указанную папку в ячейке N9 моего документа.
- Не сохраняется PDF с именем, указанным в ячейке N10.
- Самое главное, что PDF-файл не печатается в указанном мной диапазоне в ячейке N4.
- Также, есть ли способ обойти часть, чтобы указать папку, где сохранить PDF. Просто перейдите прямо в email
Вотпример файла
Вот код, который у меня есть на данный момент, я не опытен в VBA
Sub Saveaspdfandsend()
Dim xSht As Worksheet
Dim xFileDlg As FileDialog
Dim xFolder As String
Dim xYesorNo As Integer
Dim xOutlookObj As Object
Dim xEmailObj As Object
Dim xUsedRng As Range
Set xSht = ActiveSheet
Set xFileDlg = Application.FileDialog(msoFileDialogFolderPicker)
If xFileDlg.Show = True Then
xFolder = xFileDlg.SelectedItems(1)
Else
MsgBox "You must specify a folder to save the PDF into." & vbCrLf & vbCrLf & "Press OK to exit this macro.", vbCritical, "Must Specify Destination Folder"
Exit Sub
End If
xFolder = ActiveSheet.Range("n9")
xFolder = xFolder + "\" + xSht.Name + ".pdf"
'Check if file already exist
If Len(Dir(xFolder)) > 0 Then
xYesorNo = MsgBox(xFolder & " already exists." & vbCrLf & vbCrLf & "Do you want to overwrite it?", _
vbYesNo + vbQuestion, "File Exists")
On Error Resume Next
If xYesorNo = vbYes Then
Kill xFolder
Else
MsgBox "if you don't overwrite the existing PDF, I can't continue." _
& vbCrLf & vbCrLf & "Press OK to exit this macro.", vbCritical, "Exiting Macro"
Exit Sub
End If
If Err.Number <> 0 Then
MsgBox "Unable to delete existing file. Please make sure the file is not open or write protected." _
& vbCrLf & vbCrLf & "Press OK to exit this macro.", vbCritical, "Unable to Delete File"
Exit Sub
End If
End If
Set xUsedRng = xSht.UsedRange
If Application.WorksheetFunction.CountA(xUsedRng.Cells) <> 0 Then
'Save as PDF file
xSht.ExportAsFixedFormat Type:=xlTypePDF, FileName:=xFolder, Quality:=xlQualityStandard
'Create Outlook email
Set xOutlookObj = CreateObject("Outlook.Application")
Set xEmailObj = xOutlookObj.CreateItem(0)
With xEmailObj
.Display
.To = ActiveSheet.Range("n5")
.CC = ActiveSheet.Range("n6")
.Subject = ActiveSheet.Range("n7")
.HTMLBody = "<font face=" & Chr(34) & "Calibri" & Chr(34) & " size=" & Chr(34) & 4 & Chr(34) & ">" & "Good day dear Master," & "<br> <br>" & ActiveSheet.Range("n8") & "<br> <br>" & Signature & "</font>"
.Attachments.Add xFolder
If DisplayEmail = False Then
'.Send
End If
End With
Else
MsgBox "The active worksheet cannot be blank"
Exit Sub
End If
End Sub
решение1
"Я совсем неопытен с Excel VBA"
Кто-то скромничает.
Я не вижу ничего, на что можно было бы указать и сказать: «Вот в чем проблема»; но ваш код постоянно ссылается на , ActiveSheet
и это может быть очень опасно.
Проблема с ActiveSheet
таким долблением в том, что вы не можете быть уверены, что это ActiveSheet
то, что вы ожидаете. Я насчитал 6 вызовов для этого, это 6 возможностей для любого количества вещей изменить активный лист и сорвать вашу программу.
Странные вещи случаются, когда вы начинаете работать с неправильным листом. Например, невозможно сохранить файл, потому что он считывает неправильную ячейку. Или он не печатается должным образом, потому что программа считывает UsedRange
не тот лист.
Я не говорю, что именно это является причиной проблемы, но ваш код написан достаточно хорошо, чтобы я мог исключить наличие проблемы, а вот эту я исключить не могу.