ajaxで取得したデータを戻したいです。

Question

ajaxで取得したデータを加工して戻したいです。
下記のコードで実行するとcategoryListがundefinedとなります。
ajaxが非同期のためリクエストの結果の前に処理が先に進むため結果が戻せないのではと思っているのですが。。。

ajaxで取得したデータを戻す方法をご教授いただけないでしょうか。
足りない情報があれば追記いたします。

よろしくお願いします

var getCategoryL = function () {
        $.ajax({
            url: "URL",
            type: "GET",
            dataType: 'json',
        }).done(function (data){
// ここでdataを加工
            return data_kakougo;
        })
        .fail()
}

var categoryList = getCategoryL;
// この後にcategoryList をループでまわしながら処理をしたい

Comments

[kei344]

ループ部分のコードがどうなっているかがわかりません。

Answer 1

同期で良いなら、yambejp さんの方法で。

もし非同期で処理したいなら、コールバックを渡すのが良いと思います。

---

テスト用 clist.json

{
    "list" : [
        "いち",
        "に",
        "さん",
        "よん"
    ]
}

var getCategoryL = function (callback) {
    $.ajax({
        url: "clist.json",
        type: "GET",
        dataType: 'json',
    }).done(function (data){
        var categoryList = data.list;

        // ここでdataを加工（頭に★を付ける）
        var data_kakougo = [];
        for (var i = 0, imax = categoryList.length-1; i <= imax; i++) {
            data_kakougo[data_kakougo.length] = "★" + categoryList[i];
        }

        // 加工後のデータをコールバックに渡す
        callback(data_kakougo);
    })
    .fail()
}

getCategoryL(function(categoryList) {
    // 取得したらここから続き
    for (var i = 0, imax = categoryList.length-1; i <= imax; i++) {
          document.writeln("<p>" + categoryList[i] + "</p>");
    }
});

コールバックって御存知ですかね。
以下みたいな感じなんですけど。

var getCategoryL = function (データ取得後に呼び出すファンクションを受け取る) {
}

 ...

getCategoryL(データ取得後に呼び出すファンクションを渡す);

どうでしょうか。

Answer 2

jQuery.Defferd(Promise)をそのまま使用したいのであれば.done以降をループさせたい場所で書けばいいかと思います。
Promise(ちょっと違うところがあるけど)ですので、Promiseを返えして、呼び出し先でdone()またはthen()を呼び出せばいいかと思います。

var getCategoryL = function () {
    return $.ajax({
        url: "https://ext.nicovideo.jp/api/search/search/MMD?mode=watch&page=1&sort=f&order=d",
        type: "GET",
        dataType: 'json',
    }).then(function (data) {
        var kakougo = data;
        // 加工処理
        return kakougo;
    });
}

var categoryList = getCategoryL();
categoryList.then(function(data) {
    console.log('呼び出し先', data);
}).fail(function() { 
    console.log('err');
});

Answer 3

問題点

/* 関数を定義しているが、最終的に値を返す形になっていない。*/
var getCategoryL = function () {
    $.ajax({
        url: "URL",
        type: "GET",
        dataType: 'json',
    }).done(function (data){
    // ここでdataを加工
        return data_kakougo; // ここで値を返すのは合っている。
    })
    .fail() // セミコロン抜け
} // セミコロン抜け

var categoryList = getCategoryL; // getCategoryLを関数実行してない

現状の関数定義の部分を単純化するとこういう形と同じです。
内部で別の関数を実行こそしていますが、getCategoryLの実行結果として値を返していない形なのが分かるかと思います。

var getCategoryL = function () {
    hoge();
};

// こうするべき
var getCategoryL = function () {
    return hoge();
};

改善例

とりあえずできるという例ですが、下記のような形にすれば加工はできます。

var getCategoryL = function() {
  return $.ajax({
      url: "URL",
      type: "GET",
      dataType: 'json',
    }).done(function(data) {
      // 加工
      return data;
    })
    .fail();
};

getCategoryL().then(function(data){
  var categoryList = data;
  //ここで加工の続きをする
});

Answer 4

ajaxは必ずしも非同期ではありません
asyncで調整してみてください

$(function(){
  var data_kakougo=0;
  $.ajax({
  url: "URL",
  type: "GET",
  async:false,//非同期処理を中止
  dataType: 'json',//デバッグ中は'text'のほうがよい
  success:function (data){
    data_kakougo=1;//仮にデータを代入
  },
  error:function (data,err){
    console.log(err);
  },
  });
  console.log(data_kakougo);
});