foreachで回すと多次元配列になってしまう

Question

ご覧いただきありがとうございます！
PHP初心者のため、「何言ってんだこいつ」と思われるかもしれませんが、ご容赦ください。

//TestController
  public function friends(Request $request)
  {

　　//foreachでfriend_idをidにもつuserを取得してこの配列に入れたい
    $friends = [];

    //hasManyでfriend_idを配列で取得
    /*[
        {
          "friend_id": 1
        },
        {
          "friend_id": 4
        },
        {
          "friend_id": 2
        }
    ]*/
    $friend_ids = $this->user::find($request['id'])->friends;

    foreach ($friend_ids as $key => $value) {
      $friends[] = $this->user::find($value);
    }

    $data = [
      'friends' => $friends,
    ];

    return response()->json($data);
  }

//このレスポンスが返ってきてしまう
"friends": [
        [
            {
                "id": 1,
                "email": "[e-mail]",
                "password": "kishimoto",
                "name": "岸本",
                "tenant_id": 2,
                "friend_code": 543673,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            }
        ],
        [
            {
                "id": 4,
                "email": "[e-mail]",
                "password": "hashimoto",
                "name": "橋下",
                "tenant_id": 2,
                "friend_code": 284763,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            }
        ],
        [
            {
                "id": 2,
                "email": "[e-mail]",
                "password": "tanaka",
                "name": "田中",
                "tenant_id": 4,
                "friend_code": 857034,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            }
        ]
    ]

//本当はこうしたい
{
    "friends": [
            {
                "id": 2,
                "email": "[e-mail]",
                "password": "tanaka",
                "name": "田中",
                "tenant_id": 4,
                "friend_code": 857034,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            },
            {
                "id": 4,
                "email": "[e-mail]",
                "password": "hashimoto",
                "name": "橋下",
                "tenant_id": 2,
                "friend_code": 284763,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            },
            {
                "id": 1,
                "email": "[e-mail]",
                "password": "kishimoto",
                "name": "岸本",
                "tenant_id": 2,
                "friend_code": 543673,
                "role": 0,
                "valid": 0,
                "comment": null,
                "created_at": "-0001-11-30 00:00:00",
                "updated_at": "-0001-11-30 00:00:00"
            }
    ]
}

返したいJSONが[]に入ってしまい、アプリ側で取り出しづらいです。
返したいレスポンスは
$friends[0] = $this->user::find($friend_ids[0]['friend_id']);
$friends[1] = $this->user::find($friend_ids[1]['friend_id']);
で作れるのですが、foreachで回すと[]の中に入ってしまいます。
{}と[]の違いもいまいち理解できてないのですが、
foreachと$friends[0]とやっている処理は同じではないのでしょうか？

//Userクラスになります
class User extends Model
{

    public function friends()
    {

      $friend_ids = $this -> hasMany('App\Friend')->select(['friend_id']);
      return $friend_ids;

    }
}

Comments

[miyabi_pudding]

おそらく、クラスメソッドだとお見受けしますが、できれば、user::findメソッドの内容も記載いただいたほうが、解決につながりやすいかと思います。
もしかしたら、返している値に原因がある可能性もあるかと・・・。

[tomato513]

修正いたしました！
ご検討・ご指摘いただきありがとうございます！

Answer 1

first()が使えそうに思います。

Comments

[tomato513]

ご回答頂きありがとうございます！
first()はスマートに見えると思います！
私のようなケースは結構あるのでしょうか？

[m.ts10806]

正直なところ、「要件通りできていれば良い」ので競技プログラミングとかするのでなければ「自身が実現可能なやり方」で良いと思います。
一応、フレームワークであればフレームワークが提供している機能を確認して使うのが効率的で確実とは思いますけど。

Answer 2

class User extends Model
{
    public function friends()
    {
      return $this->hasMany(App\Friend::class);
    }
}

class TestController entends Controller
{
    public function friends(Request $request)
    {
        return \collent('friends' => User::find($request['id'])->friends)->toJson();
    }
}

これでいいと思います。

Answer 3

よくわかりませんが単純に考えればこう？

  $friends[] = $this->user::find($value)[0];

Comments

[tomato513]

ご回答ありがとうございます！
確かにインデックスで解決はするのですが、スマートじゃないかなと思いまして。。。
結構yambejs様がご回答いただいた記述方法は現場などで使用されるのでしょうか？

Answer 4

デバッグのために、ORMがデータベースに対してどのようなSQLを発行してデータを取得しているかのログを取るために、以下のようなデータベースクエリログのライブラリを使うと問題解決の糸口に繋がります。
mnabialek/laravel-sql-logger

Answer 5

JSONを「こういう形式で欲しい」時のAPIリソース。
https://readouble.com/laravel/5.8/ja/eloquent-resources.html

その前にまずEloquentの基本を身に付けるのが先。Eloquentの使い方から間違ってる。
よく見るけど簡単にできる便利な機能があるのに無視して素のPHPでなんとかしようと無駄なことしてる例が多い。