staticで宣言した配列を返却するとNullになってしまう

Question

###前提・実現したいこと
CakePHPでWebシステムを作っていますが、基本的なPHPの文法に関する部分で質問があります。

コントローラー内に宣言したローカル関数で、
引数に指定したidをキーに、DBよりデータを取得します。
取得されたデータをstaticで定義した配列にストックしていくという処理を行いたいです。

###発生している問題・エラーメッセージ
最終的にストックしたIDを呼び出し元の変数に入れ確認するとNullになっています。
ストックされたIDを受け取りたいのですが、なぜNullになるのかがわかりません。

###該当のソースコード

<?PHP
class TestController extends AppController
{
  ...

  public function test() {

    $ids = array(1);

    // ここがNullになってしまう
    $getlist = $this->getList( $ids );

  }


function getList($ids) {
        
        static $stocker_ids = array();
        
        $links = $this->Link->find('all',
            array(
                'conditions' => array('bookmark_id' => $ids),
                'fields' => Array('url')
            ));

        $stocker_ids = array_unique(array_merge($stocker_ids, $ids));
        
        $this->log('★★ID収集★★', LOG_DEBUG);
        $this->log($stocker_ids, LOG_DEBUG);
        
        $getlist = array();
        foreach($links as $link) {
                $contentid = $link['Link']['url'];                
                if (!in_array($contentid, $stocker_ids, true) &&
                    !in_array($contentid, $getlist, true)) {
                    $getlist[] = $contentid;
                }
            }
        }
        
        if (count($getlist) > 0) {
            $this->log('★★再帰呼出し前のリスト★★', LOG_DEBUG);
            $this->log($getlist, LOG_DEBUG);
            $this->getContentsList($getlist);
        } else {
            $this->log('★★リターンするリスト★★', LOG_DEBUG);
            $this->log($stocker_ids, LOG_DEBUG);
            // ここで確認するとIDの配列はストックできている
            return $stocker_ids;
        }
    }

###試したこと
getList関数をpublicで宣言してみました。結果はかわりませんでした。

###補足情報(言語/FW/ツール等のバージョンなど)
PHP 5.3.3

Answer 1

getList関数の最下部が以下のようになっていますが、count($getlist) > 0の場合にreturnがないので、返り値がNULLとなってしまっています。

        if (count($getlist) > 0) {
            $this->log('★★再帰呼出し前のリスト★★', LOG_DEBUG);
            $this->log($getlist, LOG_DEBUG);
            $this->getContentsList($getlist);
        } else {
            $this->log('★★リターンするリスト★★', LOG_DEBUG);
            $this->log($stocker_ids, LOG_DEBUG);
            // ここで確認するとIDの配列はストックできている
            return $stocker_ids;
        }
    }

Comments

[AIUeno]

回答がありがとうございます。
コピペしたソースが間違っておりまして、iif (count($getlist) > 0) {したときに、呼び出すのは$this->getList($getlist);となっています。取得したいIDがなくなった時点で、returnするのような流れにしたいです。

Answer 2

※コメントを受けて修正しました。BA後ですみません。

getList関数をpublicで宣言してみました。結果はかわりませんでした。

アクセス権なしとpublic宣言は同じ意味なので変化はありません。

以下、誤回答です・・・
ーーーーーーーーーーー
function内部で変数の宣言をすると、その変数のスコープは宣言したfunction内部に限定されてしまうので、参照できないのだと思います。

ということで、

static $stocker_ids = array();

をfunctionの外に出してはどうでしょうか。

class TestController extends AppController
{
 static $stocker_ids = array();

※なるべくまとめるために最上部に書きましたが、functionの外に書けば意味合いはどこでも同じです。

Comments

[AIUeno]

回答ありがとうございます！掲示したソースが間違っていて、すいませんでした。
stocker_ids をfunction の外で宣言し、getList()の中で
global $stocker_ids; としてから、取得した IDを格納しながら再帰呼び出ししたいのですが、
if (count($getlist) > 0) {で再帰呼出ししてglobal $stocker_ids;のせいなのか、配列が初期化されてしまうようなのです。で、無限再帰呼出しにハマってしまいました。stocker_idsの宣言の仕方でしょうか？

[m.ts10806]

functionの外（＝class内）で宣言した変数を利用するのにglobal 宣言は不要です。
同class内部からだとどこからでも$this->stocker_ids で参照可能ですし、をおさめることも可能です。「クラスのプロパティ」と呼ばれています。
下記参照にしてください。
http://php.net/manual/ja/language.oop5.properties.php

getlist()の中で再度array()と宣言しているため再帰で呼び出されるたびに初期化されていることになります。（というか新しい変数を宣言しているに過ぎませんね）

[m.ts10806]

再帰呼び出しの件は質問に追記してくださいね。回答内容ががらりと変わります。
もちろんmaisumakunさんの回答にあるとおり、count($getlist) > 0　の場合のreturnは必要です。

$this->getContentsList($getlist);
↓
return $this->getContentsList($getlist);

[AIUeno]

ありがとうございました！
ご指示いただいたとおり、関数の外部でstatic宣言することでarray()にストックできました。
しかし、array_mergeしたあとに配列が初期化されるのか、データがなくなってしまうので、
こちらは別の現象＆質問になるかと思います。本件は、解決済とさせていただきます。

[m.ts10806]

この場合はstaticじゃなくpublicの方が良いです。当該CLASSの外から参照できるようにする必要はないと思うので。
配列が初期化されてしまう件はarray_uniqueの方が影響ありそうですが、色々検証してみてください。