質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
ファイル

ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

マクロ

定義された処理手続きに応じて、どのような一連の処理を行うのかを特定させるルールをマクロと呼びます。

Q&A

解決済

1回答

5324閲覧

ExcelのVBAでURLDownloadToFile関数を使ってファイルを保存したいのですが ファイルが壊れてしまい上手くいきません

nobu0377

総合スコア10

ファイル

ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

マクロ

定義された処理手続きに応じて、どのような一連の処理を行うのかを特定させるルールをマクロと呼びます。

0グッド

0クリップ

投稿2020/07/15 03:01

ExcelのVBAでURLDownloadToFile関数を使ってファイルを保存したいのですが
ファイルが壊れてしまい上手くいきません。。

ExcelのA列にtitleを、B列にダウンロード先のリンク、C列に判定を返したいと思っています。
リンク先はdropbox等です。

titleは「数字.pdf」など格納されているファイルに応じて拡張子をつけています。
(dropboxに格納されているフォルダはJPGやPDF等、拡張子がバラバラに入っているので、もしも良い方法があればご教示いただけると嬉しいです。またはフォルダごと落とせたら最高なのですが。。)
何卒よろしくお願いいたします。

Private Declare PtrSafe Function URLDownloadToFile Lib "urlmon" _
Alias "URLDownloadToFileA" (ByVal pCaller As Long, _
ByVal szURL As String, ByVal szFileName As String, _
ByVal dwReserved As Long, ByVal lpfnCB As Long) As Long

Dim Ret As Long

'保存フォルダー
Const FolderName As String = "c:\test"

Sub DownloadFileFromURL()
Dim ws As Worksheet
Dim LastRow As Long, i As Long
Dim strPath As String

'シート名 Set ws = Sheets("Sheet1") '最終行 LastRow = ws.Range("A" & Rows.Count).End(xlUp).Row For i = 2 To LastRow '2行目から strPath = FolderName & ws.Range("A" & i).Value Ret = URLDownloadToFile(0, ws.Range("B" & i).Value, strPath, 0, 0) If Ret = 0 Then ws.Range("C" & i).Value = "成功" Else ws.Range("C" & i).Value = "失敗" End If Next i

End Sub

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

dropbox以外のブログなどで画像などでダウンロードできることを確認しました。
dropboxでは、直接URLを指定しダウンロードする前に、ログイン認証に相当する手順が
必要なのではないでしょうか?
(壊れていると思われるファイルに、実際にはエラーページの内容が保存されているとか)

URL末尾からファイル名を取るようにしたコードについて、参考までに記載します。

Excel

1Private Declare PtrSafe Function URLDownloadToFile Lib "urlmon" _ 2Alias "URLDownloadToFileA" (ByVal pCaller As Long, _ 3ByVal szURL As String, ByVal szFileName As String, _ 4ByVal dwReserved As Long, ByVal lpfnCB As Long) As Long 5 6 7Dim Ret As Long 8 9'保存フォルダー 10Const FolderName As String = "c:\test\" 11 12Sub DownloadFileFromURL() 13 Dim ws As Worksheet 14 Dim LastRow As Long, i As Long 15 Dim strPath As String 16 17 'シート名 18 Set ws = Sheets("Sheet1") 19 20 '最終行 21 'A列ではなくB列を見るようにする 22 LastRow = ws.Range("B" & Rows.Count).End(xlUp).Row 23 24 Dim fileName1 As String 25 Dim urlToDL As String 26 27 For i = 2 To LastRow '2行目から 28 urlToDL = ws.Range("B" & i).Value 29 fileName1 = getBaseName(urlToDL) 30 If fileName1 = "" Then 31 ws.Range("C" & i).Value = "ファイル名の取得に失敗" 32 Exit Sub 33 End If 34 strPath = FolderName & fileName1 35 Ret = URLDownloadToFile(0, urlToDL, strPath, 0, 0) 36 37 If Ret = 0 Then 38 ws.Range("C" & i).Value = "成功" 39 Else 40 ws.Range("C" & i).Value = "失敗" 41 End If 42 Next i 43 44 '変数を解放しておく 45 Set ws = Nothing 46 47End Sub 48 49'/を含む Urlからファイル名を返す 50Public Static Function getBaseName(s As String) As String 51 If s = "" Then 52 getBaseName = "" 53 Exit Function 54 ElseIf InStr(s, "/") < 1 Then 55 getBaseName = "" 56 Exit Function 57 End If 58 59 Dim delimited As Variant 60 delimited = Split(s, "/") 61 getBaseName = delimited(UBound(delimited)) 62 63End Function 64 65 66

投稿2020/07/17 02:51

t-takayama

総合スコア167

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nobu0377

2020/07/20 01:31

ありがとうございます。確かにログイン認証の手順が必要そうですね。また、sampleもありがとうございます。活用させていただきます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問