Laravelのforeachがうまく回らない

Question

Laravelのcontroller内でforeachを回して、bladeにhtmlタグを出し分けたいのですが、
1番目のデータしか処理がされず、下記の例だと全て「<span>b</span>」になってしまいます。

色々調べたのですが、何が原因なのか一人では解決できず、
初歩的な質問で恐縮ですが、こちらに投稿させていただきました。
どなたかご教授いただけますでしょうか？

----追記----

やりたいこととしては、
user配列に入っているidが100以上なら"<span>a</span>"を、100未満なら"<span>b</span>"をbladeに埋め込みたいです。

public function index(Request $request)
  {
    $user = $this->user_service->searchUser();
    $id = array_column($user, 'id');
　　　//array:3 [▼
        0 => "99"
        1 => "100"
        2 => "101"
      ]
    $label = [];
    foreach($id as $key=>$val) {
      if($val > 100)
        { $label = "<span>a</span>"; }
      else if($val <= 100)
        { $label = "<span>b</span>"; }
    }
    
    return view('user.index')->with([
      'user' => $user,
      'label' => $label,
    ]);
  }

@foreach($users as $user)
 <table>
  <tbody>
   <tr>
    <td>{!! $label !!}</td>
   </tr>
  </tbody>
 </table>
@endforeach

Answer 1

下記で解決しました。

$user = $this->user_service->searchUser();
$id = array_column($user, 'id');
foreach($id as $key=>$val) {
 if($val > 100)
  { $user[$key]["label"] = "<span>a</span>"; }
 else if($val <= 100)
  { $user[$key]["label"] = "<span>b</span>"; }
}

<td>{!! $user["label"] !!}</td>

Comments

[m.ts10806]

そもそもコントローラから$usersで渡さないとundefinedですよね。

[i-nr]

controller

public function index(Request $request)
{
$user = $this->user_service->searchUsers();

$ids = array_column($user, 'id');
foreach($ids as $key=>$val) {
if($val > 100)
{ $user[$key]["label"] = "<span>a</span>"; }
else if($val <= 100)
{ $user[$key]["label"] = "<span>b</span>"; }
}

return view('user.index')->with([
'users' => $user,
]);
}
--------------------
blade

@foreach($users as $user)
<table>
<tbody>
<tr>
<td>{!! $user['label'] !!}</td>
</tbody>
</table>
@endforeach

[m.ts10806]

命名は気をつけられたほうが良いです。紛らわしい名前だったりマジックナンバーつけてしまうとそれだけでバグの温床となります。

[i-nr]

ありがとうございます。
まだ書き慣れておらず、あまり考えずに命名していました。。
今後は命名もきちんと考えてつけようと思います！

Answer 2

@foreach($user as $user)
えっ、何から何に？

単数形の表記で、複数レコードのデータを扱うのが、すでに違和感。

Answer 3

PHP自体の書き方の問題です。
追記でなく代入してるからかと。

初期化関係なく上書きしてるので、本来は

 $label[] = 

とループ内で受けた上で、

implode(PHP_EOL,$label)

としたかったんでしょうかね。

$userと連携とるなら

 $label[$val] = 

とループ内で受けた上で、viewで$user->id
をキーに$labelを参照とか？

どのように表示したいか次第。

Comments

[m.ts10806]

userのほうはどうしたいのか不明ですね。
変数別にしてる時点でuserとlabelとの関連性ないですし。

[i-nr]

追記させていただいたのですが、
やりたいことは、user配列内のidが100以上なら"<span>a</span>"を、100未満なら"<span>b</span>"をbladeに出し分けたいです。

user配列は下記のようになっています。
array:3 [▼
0 => array:20 [▼
"id" => 99
]
1 => array:20 [ [▼
"id" => 100
]
2 => array:20 [ [▼
"id" => 101
]
]

[m.ts10806]

回答内容では何もヒントになりませんか？特に問題ないはずですが。