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

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

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

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

Q&A

解決済

3回答

8083閲覧

Uncaught (in promise) ReferenceError: deleteFiles is not defined

rms398

総合スコア50

JavaScript

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

Ajax

Ajaxとは、Webブラウザ内で搭載されているJavaScriptのHTTP通信機能を使って非同期通信を利用し、インターフェイスの構築などを行う技術の総称です。XMLドキュメントを指定したURLから読み込み、画面描画やユーザの操作などと並行してサーバと非同期に通信するWebアプリケーションを実現することができます。

0グッド

0クリップ

投稿2017/05/24 04:17

編集2017/05/24 04:29

###前提・実現したいこと
html javascript phpで画像や動画を削除する機能を作っています
チェックボックスにチェックをつける→削除ボタンを押す→チェックされた画像のソース(相対パス)をajaxでphpに送る→unlink()で削除
このサイトを参考にして
promise.allですべてajaxで送ったら更新するということをしたいのですが、
以下のエラーがでてきて進みません
エラーになっている行はjavascriptの後半に書かれています

###発生している問題・エラーメッセージ

Uncaught (in promise) ReferenceError: deleteFiles is not defined js/fileDelete.js:55 at http://localhost/jk3/kodomonojikan/js/fileDelete.js:55:6 at Promise (<anonymous>) at deleteFilesPromise (http://localhost/jk3/kodomonojikan/js/fileDelete.js:54:6) at http://localhost/jk3/kodomonojikan/js/fileDelete.js:44:31 at Array.forEach (native) at HTMLInputElement.<anonymous> (http://localhost/jk3/kodomonojikan/js/fileDelete.js:39:27) at HTMLInputElement.dispatch (http://localhost/jk3/kodomonojikan/jquery-3.0.0.min.js:3:9856) at HTMLInputElement.q.handle (http://localhost/jk3/kodomonojikan/jquery-3.0.0.min.js:3:7883)

###該当のソースコード

javascript

1$(function(){ 2 //フォームデータを一時保存する配列 3 var formDataArray = [] 4 //クリックした時にデータ送信 5 $('#delete').on('click',function(){ 6 //チェックボックス 7 var im = document.forms.fm.Checkbox; 8 var cnt = 0; 9 // Array オブジェクトを作成する 10 var img_id = new Array(); 11 //チェックボックスの数だけ繰り返す 12 for(var i = 0; i < im.length; i++){ 13 //チェックボックスにチェックがついてるか 14 console.log(im[i]); 15 console.log(im[i].checked); 16 if(im[i].checked){ 17 //チェックボックスのvalue(imgのidが入っている)をimg_idにプッシュ 18 console.log(im[i].value); 19 img_id.push(im[i].value); 20 //チェックついてる数を数える 21 cnt++; 22 } 23 } 24 //チェックついてる数だけ繰り返す 25 console.log(cnt); 26 for(var i = 0; i < cnt; i++){ 27 var formData = new FormData(); 28 console.log(img_id[i]); 29 var id = img_id[i]; 30 console.log(document.getElementById(img_id[i])); 31 //チェックのついている画像のソースをformDataにappend 32 formData.append('path',document.getElementById(img_id[i]).getAttribute("src")); 33 //formDataをArrayにpush 34 formDataArray.push(formData); 35 } 36 //入ってる配列を繰り返す 37 var promises = []; 38 formDataArray.forEach(function(formData){ 39 //画像のsrcが入っているか確認 40 console.log(formData.get("path")); 41 //ajaxで送る 42 //deleteFiles(resolve,formData); 43 promises.push(deleteFilesPromise(formData)); 44 }); 45 Promise.all(promises).then(function (results) { 46 location.reload(true); 47 }); 48 }); 49 function deleteFilesPromise(formData) { 50 new Promise(function (resolve, reject) { 51 deleteFiles(formData, function (err, result) {//エラーが出てるところ 52 //ajaxで送信 53 $.ajax({ 54 //POSTで送信 55 type: 'POST', 56 //ajaxが適切なcontentTypeに自動変換するのを防ぐ 57 contentType: false, 58 //データを文字列に自動変換するのを防ぐ 59 processData: false, 60 //送信先 61 url: './delete.php', 62 //送るデータ 63 data: formData, 64 //通信成功時に呼ばれる 65 //data 接続先PHPファイルが出力したデータ 66 success: function(data,dataType) { 67 console.log("成功" +data); 68 //res(formData); 69 resolve(formData); 70 }, 71 //通信失敗時に呼ばれる 72 //XMLHttpRequest.status HTTPステータス 73 //textStatus timeout、error、parsererror等の文字列 74 //errorThrown 例外情報 75 error: function(XMLHttpRequest, textStatus, errorThrown){ 76 console.log(XMLHttpRequest + " : " + textStatus + " : " + errorThrown); 77 reject(err); 78 } 79 }); 80 //} 81 }); 82 }); 83 } 84});

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

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

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

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

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

guest

回答3

0

ベストアンサー

deleteFilesという関数がどこにも定義されていないからでは。

投稿2017/05/24 04:22

kei344

総合スコア69364

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

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

rms398

2017/05/24 04:27

deleteFiles(formData, function (err, result) { ここにdeleteFilesの処理が書かれている } {}内に処理を書いているんですけど function deleteFiles{} と別のところに書かないといけないということでしょうか
kei344

2017/05/24 04:32

deleteFiles('引数')はdeleteFilesというすでにある関数に引数を渡して実行、ということになります。 なので、根本的に書き方がおかしいです。参考にされている「omotai」は「重たい処理をする関数が別にある」という前提のコードです。
guest

0

promises.push(deleteFilesPromise(formData));
としていますが
deleteFilesPromiseのなかでpromiseが戻されてませんよね

return new Promise・・・
とした方がよいのではないですか?

投稿2017/05/24 04:54

yambejp

総合スコア114572

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

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

0

Uncaught (in promise) ReferenceError: deleteFiles is not defined js/fileDelete.js:55

deleteFilesという関数 が定義されてないということなので、定義すればひとまずはこのエラーは回避できそうです。

コメントアウト部分が多すぎてどのコードが本筋なのか分かりづらいのが難点です。少し整理してもらえると助かります。

投稿2017/05/24 04:24

m.ts10806

総合スコア80765

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問