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

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

ただいまの
回答率

90.34%

  • JavaScript

    22124questions

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

  • Ajax

    1409questions

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

画像・動画を削除する処理が完了したら更新したい

解決済

回答 2

投稿 編集

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

rms398

score 36

前提・実現したいこと

html javascript phpで画像や動画を削除する機能を作っています
promiseで画像・動画を削除する処理(ajaxで画像のソース(相対パス)を送ってphpのunlink()で削除)が終わったら更新をしたいのですが、
promise.then(() => reload(true));が動きません
promise.then(() => console.log('動いた'));でログがでるか確認したところ何も出ませんでした
javascriptの後半にあります

該当のソースコード

$(function(){
    //フォームデータを一時保存する配列
    var formDataArray = []
    //クリックした時にデータ送信
    $('#delete').on('click',function(elem){
        //チェックボックス
        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);
        }
        //入ってる配列を繰り返す
        const promise = new Promise((resolve, reject) => {
            formDataArray.forEach(function(formData){
                //画像のsrcが入っているか確認
                console.log(formData.get("path"));
                //ajaxで送る
                deleteFiles(formData);           
            });
        });
        promise.then(() => console.log('動いた'));
        promise.then(() => reload(true));
    });
    var deleteFiles = function(formData){
        //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);
            },
            //通信失敗時に呼ばれる
            //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ページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

check解決した方法

0

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) {
    window.location.reload(true);
});
function deleteFilesPromise(formData) {
    return new Promise(function (resolve, reject) {
        deleteFiles(formData, resolve, function (err, result) {
            if (err != null) {
                reject(err);
                return;
            }
            resolve(result);
            console.log('aaaa');
        });
    });
}
var deleteFiles = function(formData,resolver){
    //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);
            resolver(this);
        },
        //通信失敗時に呼ばれる
        //XMLHttpRequest.status HTTPステータス
        //textStatus            timeout、error、parsererror等の文字列
        //errorThrown           例外情報
       error: function(XMLHttpRequest, textStatus, errorThrown){
            console.log(XMLHttpRequest + " : " + textStatus + " : " + errorThrown);
        }
    });  
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

new Promise((resolve, reject) => { の resolve メソッドが呼ばれなければ then のコールバックは呼び出されません

deleteFiles に resolve を渡して success コールバックの中で resolve を呼び出すようにすれば then のコールバックが実行されます

ただ、formDataArray を forEach で複数回 deleteFiles を呼び出しているので Promise.all を使って全部の deleteFiles の success が呼び出されたら resolve を実行するようにする必要があると思います

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/05/24 11:22 編集

    Promise.allの使い方がよくわかりません、どういった処理を書けばよいのでしょうか
    ※ここに書いたコードは消しました

    キャンセル

  • 2017/05/24 11:24

    コメント欄ではコード記法は利用できませんよ。どこまで分かっていてどこまで分からないのか、どこまで出来ていてどこが出来ていないのかは自身で切り分けないと、いつまでたっても解決しないし身につきません。

    キャンセル

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

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

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

  • JavaScript

    22124questions

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

  • Ajax

    1409questions

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