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

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

ただいまの
回答率

88.92%

Promise.all(promises).then...で更新したいが処理が走らない

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,808

rms398

score 38

前提・実現したいこと

html javascript phpで画像や動画を削除する機能を作っています。
チェックボックスにチェックをつける→削除ボタンを押す→チェックされた画像のソース(相対パス)をajaxでphpに送る→unlink()で削除
全てajaxでphpにソース(相対パス)を送り終えたら
Promise.allで更新をするというのをしたいのですが

    Promise.all(promises).then(function (results) {
        console.log('通りましたよ');
        window.location.reload(true);
    });


この処理が走りません、console.log()で試したら何も表示されませんでした

でもブラウザのF12でちゃんとdelete.phpにpostで適正な値が渡されていました

該当しているソースコード

$(function(){
    //フォームデータを一時保存する配列
    var formDataArray = []
    //クリックした時にデータ送信
    $('#delete').on('click',function(){
        //チェックボックス
        var im = document.forms.fm.Checkbox;
        var cnt = 0;
        // Array オブジェクトを作成する
        var img_id = new Array();
        //チェックボックスの数だけ繰り返す
        for(var i = 0; i < im.length; i++){
            //チェックボックスにチェックがついてるか
            console.log(im[i]);
            console.log(im[i].checked);
            if(im[i].checked){
                //チェックボックスのvalue(imgのidが入っている)をimg_idにプッシュ
                console.log(im[i].value);
                img_id.push(im[i].value);
                //チェックついてる数を数える
                cnt++;
            }
        }
        //チェックついてる数だけ繰り返す
        console.log(cnt);
        for(var i = 0; i < cnt; i++){
            var formData = new FormData();
            console.log(img_id[i]);
            var id = img_id[i];
            console.log(document.getElementById(img_id[i]));
            //チェックのついている画像のソースをformDataにappend
            formData.append('path',document.getElementById(img_id[i]).getAttribute("src"));
            //formDataをArrayにpush
            formDataArray.push(formData);
        }
        //入ってる配列を繰り返す
        var promises = [];
        formDataArray.forEach(function(formData){
            //画像のsrcが入っているか確認
            console.log(formData.get("path"));
            //ajaxで送る
            //deleteFiles(resolve,formData);
            promises.push(deleteFilesPromise(formData));
        });
        Promise.all(promises).then(function (results) {
            console.log('aaaa');
            window.location.reload(true);
        });    
    });
    function deleteFilesPromise(formData) {
    return new Promise(function (resolve, reject) {
    deleteFiles(formData, function (err, result) {
        if (err != null) {
            reject(err);
            return;
        }
        resolve(result);
    });
    });
    }
    var deleteFiles = function(formData,aaa){
        //ajaxで送信
        $.ajax({
            //POSTで送信
            type: 'POST',
            //ajaxが適切なcontentTypeに自動変換するのを防ぐ
            contentType: false,
            //データを文字列に自動変換するのを防ぐ
            processData: false,
            //送信先
            url: './delete.php',
            //送るデータ
            data: formData,
            //通信成功時に呼ばれる
            //data 接続先PHPファイルが出力したデータ
            success: function(data,dataType) {
                console.log("成功" +data);
                //res(formData);
            },
            //通信失敗時に呼ばれる
            //XMLHttpRequest.status HTTPステータス
            //textStatus            timeout、error、parsererror等の文字列
            //errorThrown           例外情報
            error: function(XMLHttpRequest, textStatus, errorThrown){
                console.log(XMLHttpRequest + " : " + textStatus + " : " + errorThrown);
            }
        });  
    }
});
  • 気になる質問をクリップする

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

0

ブラウザのF12をおしてdelete.phpにpostで適正な値が渡されているか確認してください

 テスト

テストサンプルです。
iは連番で渡しているのでdelete.phpで間違っても削除を走らせないで下さい
以下で処理すれば、10個のpostデータをおくってすべてがもどってきたら
thenが走りますよね?

<script src="https://www.promisejs.org/polyfills/promise-6.1.0.min.js"></script>
<script>
var prm=[];
console.log("start");
for(var i=0;i<10;i++){
  (function(i){
    prm[i]= new Promise(function(resolver){
      var fd=new FormData();
      fd.append('del_id',i);
      var xhr = new XMLHttpRequest();
      xhr.open('POST','delete.php');
      xhr.addEventListener("load" ,function() {
        console.log(this.response)
        resolver(this);
      });
      xhr.send(fd);
    });
  })(i);
}
Promise.all( prm ).then(function(){
  console.log("end");
});

</script>

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/05/24 14:46 編集

    旧:適正な値が表示されました


    テストサンプルはちゃんと走りました

    キャンセル

  • 2017/05/24 16:05

    そうなるとpromiseがresolverをきちんと返してないのかもしれませんね
    var deleteFiles = function(formData,aaa){
    の第二引数のaaaは参照されていないようですが
    successの中から利用してresolverを返さないとまずいんじゃないですか?

    キャンセル

  • 2017/05/24 16:25

    aaa にあたる引数をreturn new Promise(function (resolve, reject)のresolveにしたらちゃんと動きました!

    キャンセル

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

  • ただいまの回答率 88.92%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る