PHPの配列をjQueryで受け取る

Question

PHPには配列が入っており、POST飛んできた場合のみ配列を返します。またPOSTの値によって返す変数が異なります。
関数Pics,Tagsを使用すると、見つからないという旨のエラーが出ます。問題があるとすれば関数の書き方かなと思うのですが、どこが間違っていますでしょうか。

ReferenceError: Can't find variable: res

<?php
$hotels = array(
  array('hotel_id' => '1', 'country_code'=>'PHL', 'hotel_name'=>'SARROSAホテル', 'hotel_address'=>'セブ シティ, セブ', 'hotel_tel'=>'12345', 'price'=>'30'),
  array('hotel_id' => '2', 'country_code'=>'PHL', 'hotel_name'=>'SPAホテル', 'hotel_address'=>'セブ シティ, セブ', 'hotel_tel'=>'12345', 'price'=>'40'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'hotel_name'=>'MATTHERホテル', 'hotel_address'=>'セブ シティ, セブ', 'hotel_tel'=>'12345', 'price'=>'50')
);
$pics = array(
  array('hotel_id' => '1', 'country_code'=>'PHL', 'pic'=>'hotel_idが1の画像です'),
  array('hotel_id' => '2', 'country_code'=>'PHL', 'pic'=>'hotel_idが2の画像です'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'pic'=>'hotel_idが3の画像です')
);
$tags = array(
  array('hotel_id' => '1', 'country_code'=>'PHL', 'tag'=>'無料Wi-Fi'),
  array('hotel_id' => '1', 'country_code'=>'PHL', 'tag'=>'後日支払い可'),
  array('hotel_id' => '1', 'country_code'=>'PHL', 'tag'=>'宿泊施設へ直接支払い'),
  array('hotel_id' => '2', 'country_code'=>'PHL', 'tag'=>'キャンセル無料'),
  array('hotel_id' => '2', 'country_code'=>'PHL', 'tag'=>'無料Wi-Fi'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'tag'=>'無料Wi-Fi'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'tag'=>'後日支払い可'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'tag'=>'宿泊施設へ直接支払い'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'tag'=>'朝食付き'),
  array('hotel_id' => '3', 'country_code'=>'PHL', 'tag'=>'キャンセル無料')
);
if ($_SERVER['REQUEST_METHOD'] === 'POST') {
    if (isset($_POST['kind']) && $_POST['kind'] == 'pics') {
        header('Content-Type: application/json');
        echo json_encode($pics);
        exit;
    } elseif (isset($_POST['kind']) && $_POST['kind'] == 'tags') {
        header('Content-Type: application/json');
        echo json_encode($tags);
        exit;
    } elseif (isset($_POST['kind']) && $_POST['kind'] == 'hotels') {
        header('Content-Type: application/json');
        echo json_encode($hotels);
        exit;
    }
}

<script>
  $(function() {

    function Pics() {
      $.post('sampleAjax.php', {
        kind: 'pics'
      }, function(reslut) {
        var res = reslut;
      });
      return res;
    };


    function Tags() {
      $.post('sampleAjax.php', {
        kind: 'tags'
      }, function(reslut) {
        var res = reslut;
      });
      return res;
    };


    $('#push').on('click', function() {
      // ajax処理
      $.post('sampleAjax.php', {
        kind: 'hotels'
      }, function(reslut) {
        //配列が返ってくる(動作確認済
        console.log(reslut);
        console.log(Tags());
      });
      return false;
    });
  });
</script>

追記

<script>
  $(function() {

    var array = [];

    function pics() {
      $.post('sampleAjax.php', {
        kind: 'picks'
      }, function(reslut) {
        array = reslut;
        // array.push(reslut);
      });
      return array;
    };

    function tags() {
      $.post('sampleAjax.php', {
        kind: 'tags'
      }, function(reslut) {
        console.log(reslut);
        array = Object.keys(reslut);;
      });
      return array;
    };


    $('#push').on('click', function() {
      $.post('sampleAjax.php', {
        kind: 'hotels'
      }, function(reslut) {
        //配列が返ってくる(動作確認済
        // console.log(reslut);
        // console.log(tags());
        console.log(pics());
      });
      return false;
    });
  });
</script>

「追記」
回答のおかげで、上手く行きました。
画像ではconsole.logの返り値がPHPの配列$hotelsの部分だけ [Object, Object, Object] なっています。
違いはなんなのでしょうか。他のPHPの配列と形は同じなのですが。。。

また順番を変えたり、

function pics() {
      return $.post('sampleAjax.php', {
        kind: 'hotels'
      });
    };

のようにしても同じでした。

Comments

[m.ts10806]

起きている問題に対してタイトルが大きすぎる。

Answer 1

変数のスコープ

Comments

[m.ts10806]

これでピンとこなければあなたがやろうとしていることはあなたの持っている技術知識に対しては難しすぎるので基礎の基礎の基礎からやりなおすことを強くすすめる。
エラー調べません、マニュアル・リファレンス読みません、自分のコードは間違ってない　ではteratailで質問する上で話にならない。

[退会済みユーザー]

ちなみに404 not found でも正常にデータがとれると思ってる模様

[m.ts10806]

私のコメント全無視してる時点でお察し案件です。

[退会済みユーザー]

そして
月曜が～木曜の～とかの質問解決済みになってるのに同じての質問がまた投稿されてるし(笑)

[m.ts10806]

もはや質問ではないですからね
デバッグ依頼。作業依頼もコメントでしてるようなもんです

Answer 2

エラーメッセージから察するに、var resを定義している場所によるスコープの違いの問題のように思います。

// resはfunctionの外で定義する
var res = null;
$.post('sampleAjax.php', {
  kind: 'tags'
}, function(reslut) {
  // function内部で定義された変数は関数の外では使えない
  res = reslut;
});
return res;

追記 : 非同期通信の結果を関数からreturnさせるため、全体の構造を修正しました。

  $(function() {
    var array = [];

    // 関数からは$.postの結果をreturnする
    function pics() {
      return $.post('sampleAjax.php', {
        kind: 'picks'
      });
    };

    function tags() {
      return $.post('sampleAjax.php', {
        kind: 'tags'
      });
    };

    $('#push').on('click', function() {
      $.post('sampleAjax.php', {
        kind: 'hotels'
      }, function(result) {
        // 返り値のdone関数を使って、非同期通信の結果を取得できます。
        pics().done(function(result) {
          console.log('pics', result);
        });
        tags().done(function(result) {
          console.log('tags', result);
        });
      });
      return false;
    });
  });

Comments

[Chandler_Bing]

ありがとうございます。追記します。

[m.ts10806]

スコープもそうですけど、returnの場所ですね。非同期ですし。

[Chandler_Bing]

どこに書けば良いのでしょうか

[noka_blank]

ご指摘にあった通り、非同期通信の結果を取得する部分に問題があるようなので、回答に追記しました。
エラーメッセージは確かに変数のスコープのことを指しておりますが、おそらくYutaNkaiさんの欲しい回答はこちらではないでしょうか？

[Chandler_Bing]

ありがとうございます。上手く機能しました。doneを使用していなかった事が問題なのでしょうか。

[noka_blank]

jQuery(だけではないですが)では、非同期通信の結果はすぐにreturnされずに、コールバック関数を使って後から通知されます。
doneはそのコールバック関数にあたるものなので、間違いではないですが、問題はもう少し基礎的な、非同期通信とは何かという部分にあると思います。

[Chandler_Bing]

ありがとうございます。しかし最初の質問でもコールバック関数をfunction(){}を使い記述しているのではないのでしょうか。

[noka_blank]

最初は、以下の書き方でした。
プログラムはコメントの数字の順番に実行されます。

function Pics() {
// 1. 最初に非同期通信を開始
$.post('sampleAjax.php', {
kind: 'pics'
}, function(reslut) {
// 3. 通信の結果が返ってきたら実行
var res = reslut;
});
// 2. 通信の結果を待たずにreturn
return res;
};

2の時点で未定義の変数をreturnしようとしているため、エラーになりました。
エラーは変数を定義することで解消されますが、それは本質的な解決ではない、という指摘がされましたので、回答のように修正しました。

YutaNkaiさんのご質問のコードは、非同期通信の結果を待たずにreturnされたものを、console.logで出力しようとしていた、という事になります。

[Chandler_Bing]

なるほどよくわかりました。ありがとうございます。