Excel VBA - セルで指定された範囲を PDF に印刷し、Outlook で送信する

Excel VBA - セルで指定された範囲を PDF に印刷し、Outlook で送信する

私は 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間違ったワークシートで作業を開始すると、奇妙なことが起こります。間違ったセルを読み取ったためにファイルを保存できない、プログラムが間違ったシートを読み取ったために正しく印刷されないなどです。

さて、これが問題の原因であると言っているのではありませんが、コードは十分に記述されているため、問題を排除できますが、これについては排除できません。

関連情報