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

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

新規登録して質問してみよう
ただいま回答率
85.48%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

Q&A

解決済

1回答

302閲覧

javascriptで同じファイルをS3にアップロードしてしまう

lilac001

総合スコア23

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

jQuery

jQueryは、JavaScriptライブラリのひとつです。 簡単な記述で、JavaScriptコードを実行できるように設計されています。 2006年1月に、ジョン・レシグが発表しました。 jQueryは独特の記述法を用いており、機能のほとんどは「$関数」や「jQueryオブジェクト」のメソッドとして定義されています。

0グッド

0クリップ

投稿2018/07/04 15:47

ブラウザからAWSのS3へファイルをアップロードしようとしています。
複数送信する際に成功するように見えますが、実際には最後のファイルの中身になってしまいます。
a.jpg
b.jpg
を送る際に、両方のファイルがS3にアップロード成功するのですが、画像の中身がa.jpgもb.jpgの画像になってしまいます。


html

1<div id="droppable">ファイルをドロップしてください。</div> 2<button id="hugefile" type="submit" class="btn btn-primary">送信</button>

javascripy

1 2<script> 3 let upload_files; 4 let upload_file; 5 $(function () { 6 var droppable = $("#droppable"); 7 // File API が使用できない場合は諦めます. 8 if (!window.FileReader) { 9 alert("File API がサポートされていません。"); 10 return false; 11 } 12 var cancelEvent = function (event) { 13 event.preventDefault(); 14 event.stopPropagation(); 15 return false; 16 } 17 droppable.bind("dragenter", cancelEvent); 18 droppable.bind("dragover", cancelEvent); 19 ////////////////////////////////////////////////////// 20 // ドロップ時のイベントハンドラを設定します. 21 ////////////////////////////////////////////////////// 22 var handleDroppedFile = function (event) { 23 var files = event.originalEvent.dataTransfer.files; 24 $("#droppable").text(""); 25 for (var i = 0; i < files.length; i++) { 26// console.log("size" + files[i].size); 27 $("#droppable").append(files[i].name + "<br>"); 28 } 29 upload_files = files; 30 console.log("upload_files="+files); 31 cancelEvent(event); 32 return false; 33 } 34 droppable.bind("drop", handleDroppedFile); 35 }); 36 37 ////////////////////////////////////////////////////// 38 //// 送信ボタンが押された //// 39 ////////////////////////////////////////////////////// 40 $("#hugefile").on('click', function () { 41 42 if (upload_files == null) { 43 alert('ファイルがありません') 44 return false 45 } 46 //// ファイルの数繰り返す //// 47 for (var i = 0; i < upload_files.length; i++) { 48 console.log("i="+i); 49 upload_file = upload_files[i]; 50 console.log(i+":"+upload_files[i].name); 51 //// S3の著名付きURLを取得する //// 52 $.ajax({ 53 url: "/userdata/onetime", 54 data: { 55 "name": upload_file.name, 56 'id':<?PHP echo $project[0]['id']; ?> 57 }, 58 type: 'GET', 59 dataType: 'json' 60 //// S3の著名付きURLにファイルを送信する //// 61 }).done(data => sendFileCore(data, upload_file) 62 ) 63 } 64 return false 65 }) 66 /////////////////////////////////////////////////////// 67 // // Ajaxで取得した署名付きURLを使用してファイルをアップロードする処理 68 ////////////////////////////////////////////////////// 69 function sendFileCore(data, file) { 70 $("#droppable").append(file.name + " アップロード中<br />"); 71 $.ajax({ 72 url: data.uri, 73 type: 'PUT', 74 data: file, 75 contentType: file.type, 76 processData: false 77 }).done(function (d) { 78 $("#droppable").append(file.name + " アップロード完了<br />"); 79 }) 80 } 81</script>

上記を実行すると、#droppableの中身は

A7301203.jpg
A7301195.jpg
A7301195.jpg アップロード中
A7301195.jpg アップロード中
A7301195.jpg アップロード完了
A7301195.jpg アップロード完了

となります。S3の中を見ると、
A7301203.jpg
A7301195.jpg
が存在しますが、
A7301203.jpg
の画像の中身が
A7301195.jpg
になってしまっています。

data => sendFileCore(data, upload_file
が失敗しているようですが、理由がわかりません。
console.logは
i=0
0:A7301203.jpg
i=1
1:A7301195.jpg
となっています。

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

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

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

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

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

guest

回答1

0

ベストアンサー

原因はajaxが非同期通信のために、S3との通信が完了して
.done()の実行が始まる段階ではメインの処理であるforの処理は回り終わってて、引数のupload_fileの中身が配列の最後の値になってしまっているからですね。

promiseパターンなどを使うのが一般的です。
参考: https://qiita.com/hththt/items/9f193fc10b79cdeea903

投稿2018/07/05 02:42

kuwako

総合スコア387

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

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

lilac001

2018/07/05 08:09

たしかにそういうことですね。 promise調べてみました。 ただ、ループで回す、とはなじまないようですね。 情報ありがとうございました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問