php 関数の中で作った配列を関数の外で使用したい

Question

前提・実現したいこと

putList（）　の返り値として　$listを返したいのですが、nullが返ってきてしまいます...

$listを返り値として返すには、どのように書いたらよいのでしょうか？

とても基本的な質問で、、申し訳ありません。。

よろしくお願いします。

＊globalは使わない方法
＊SQLは回さない方法

でお願いできればと思います。。

該当のソースコード

public function putList()
{  
        $users = Users::all();
        $users->each(function ($user) {
            $list = [];
            Shop::where('region', $user->region)->each(function ($n){
                $list[] = [$n->id => $n->name];
            });
           return $list
        });
}

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[m.ts10806]

↓ここにもreturnつけるとどうなりますか？
return $users->each(function ($user) {

[megumi8989]

ありがとうございます！！

コレクションで
$users = Users::all();
と同じものが返ってきます。

Answer 1

PHPからの解決方法はクロージャのuseで参照渡しするとかclassならプロパティにするとかだけど

public function putList()
{  
        $list = [];

        $users = Users::all();
        $users->each(function ($user) use(&$list) {
            Shop::where('region', $user->region)->each(function ($n) use(&$list){
                $list[] = [$n->id => $n->name];
            });
        });

        return $list;
}

そもそもの発想が違っていてDBから解決したほうがいい。
（↑で本当に欲しい$listになってるか怪しい）

1回で$listを返すSQLを組み立てれば速度的にも早いはず。
たぶんjoinかリレーション使うけど本当に欲しいデータが分からないので後は自分で考えてもらうしかない。

public function putList()
{
    return User::...
}

Comments

[megumi8989]

、、ありがとうございます。
参照渡しを使うことで関数の外でもつかうことが出来るのですね。
勉強になりました！