
私は Excel VBA についてはあまり経験がありませんが、指定したセルの範囲を PDF に印刷し、それを Outlook で送信するマクロを作成しようとしています。
以下のことは実行できません:
- ドキュメントのセル N9 の指定されたフォルダーに保存されません
- セル N10 で指定した名前で PDF が保存されない
- 最も重要なのは、セルN4の指定された範囲のPDFが印刷されないことです。
- また、PDFを保存するフォルダを指定する部分をバイパスする方法はありますか?メールに直接アクセスしてください
これがサンプルファイル
これまでのコードは次のとおりです。私は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
間違ったワークシートで作業を開始すると、奇妙なことが起こります。間違ったセルを読み取ったためにファイルを保存できない、プログラムが間違ったシートを読み取ったために正しく印刷されないなどです。
さて、これが問題の原因であると言っているのではありませんが、コードは十分に記述されているため、問題を排除できますが、これについては排除できません。