JSのグローバル関数が変更できなない

Question

getMaxpageでAjaxでPHPファイルに飛び配列数を取得し、それを特定お数字で割ったものをグローバル変数に代入します。

maxpage = Math.ceil(res.maxpage / limitation);
        console.log(maxpage);

と代入しているつもりなのですが何が問題でしょうか。スコープの問題なのはわかりますがグローバル変数はどこからでも変更できるのではないのでしょうか。それとも.doneの後の処理は無名関数内であることが問題でしょうか。

参考サイト
https://qiita.com/matsuby/items/1010ae38ee6258fd8020

$(function() {

  var btndisplay = 5;
  var limitation = 5;
  var cpage = 1;
  var maxpage = 100;
  var last = null;
  var first = null;
  var next = null;
  var prev = null;

  function reviewClone() {
    var review = $("<div>", {
      "class": "review"
    });
    var name = $("<p>", {
      "class": "name"
    });
    var country = $("<p>", {
      "class": "country"
    });
    var date = $("<p>", {
      "class": "date"
    });
    var text = $("<span>", {
      "class": "text"
    });
    review.append(name, country, date, text);
    return review;
  };

  function getMaxpage() {
    $.post('reviewsAjax.php', {
        function: 'getMaxpage'
      }).done(function(res) {
        maxpage = Math.ceil(res.maxpage / limitation);
        console.log(maxpage);
      })
      .fail(function(XMLHttpRequest, textStatus, errorThrown) {
        console.log(textStatus);
      });
  };
  getMaxpage();//２６が表示される
  console.log(maxpage);//100が表示される

});

Comments

[m.ts10806]

＞スコープの問題なのはわかりますが
わかってないからいつも「何が問題でしょうか」って聞いてるんでしょ。
$(function() {　の中にある時点で正確な意味でグローバルではない。
それに何度も注意してんじゃん。あなたのは質問じゃなくてデバッグ依頼だって。
非同期実行は何かって100回確認して出直してください。

Answer 1

mts10806さんが指摘されているように「非同期」というところがポイントですね。
仮に、getMaxpage()の中で非同期通信を使わなければどうなるか試してみればわかるんじゃないでしょうか。

Comments

[Chandler_Bing]

非同期なので、終了する前にconsole.log(maxpage);が呼ばれているのだと思いますが、それを防ぐための
doneではないのでしょうか。

[m.ts10806]

参考記事のシリーズに非同期についてしっかり書いてるからそこ読めばいいのにとは思いました。

[cerfweb]

ほんとだ、非同期関数のところよく読めば何が起きているのかわかりますよね。

[退会済みユーザー]

なにも return してないやつでエラーを起こして　１００の処理に行かないようにしたのがBA・・・

Answer 2

  getMaxpage().done(function(res) {
    maxpage = Math.ceil(res.maxpage / limitation);
    console.log(maxpage);
  });

これで解決しました

Comments

[m.ts10806]

新たな問題を起こしてどうする