Undefined variable 未定義のエラーを解消したい

Question

DBの情報をviewで表示させたいのですがエラーが解消できずにいるのでアドバイスをいただけたらと思います。foreachの記述が悪いのか、他が原因なのかがわからずにいますのでご教示をお願いします。

各ファイル

■user.blade.php

<?php

Route::get('/', function () {
    return view('index');
});

Route::get('/','UsersController@index');
//Route::get('/home','HomeController@index')->name('home');

// ユーザ登録
Route::get('signup', 'Auth\RegisterController@showRegistrationForm')->name('signup');
Route::post('signup', 'Auth\RegisterController@register')->name('signup.post');

Route::get('login', 'Auth\LoginController@showLoginForm')->name('login');
Route::post('login', 'Auth\LoginController@login')->name('login.post');
Route::get('logout', 'Auth\LoginController@logout')->name('logout');

Route::resource('users','UsersController',['only'=>['show']]);

■Users.Controller.php

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Employee;

class UsersController extends Controller
{
    public function index(){
        $users = Employee::select('employee_id','employee_name')->get();//orderBy('id','desc')->get();

        return view('users',['employees'=>$users,]);
    }
}

■Users.blade.php

@extends('layouts.app')

@section('content')

  <div class="text-right">

  @if(Auth::check())
      {{ Auth::user()->name }}
  @endif

  </div>

  @foreach($users as $user)
    <div class='kkk'>
      <table class="table">
        <thead class="thead-dark">
          <tr>
            <th scope="col">社員番号</th>
            <th scope="col">氏名</th>
          </tr>
        </thead>
        <tbody>
          <tr>
            <th scope="row">{{ $user->employee_id }}</th>
            <th scope="row">{{ $user->employee_name }}</th>
          </tr>
        </tbody>
      </table>
    </div>

  @endforeach

エラー内容

ErrorException
Undefined variable: users (View: /var/www/html/laravel-app/resources/views/users.blade.php)

※別途画像あり

Comments

[m.ts10806]

普通にPHPをしていたら頻出のNoticeかと思いますが、その辺りどうでしょうか。おおよそ「命名した名前で使ってない」ことが原因です。
「コントローラからの値の受け渡し」でしたらLaravelのドキュメントからわかることもあるでしょうし・・

[plusninesty]

原因は引数の誤りでした。命名について改めて確認したいと思います。
ありがとうございました。

[m.ts10806]

はい。「命名」です。
きちんと役割に見合った名前をつけることで今回のようなミスは減ります。
紛らわしい名前を付けてしまうことで今回のようなミスに繋がります。
employeeとuserを同じ扱いにしてしまっていることに違和感を覚えるところからですね。

Answer 1

コ<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Employee;

class UsersController extends Controller
{
    public function index(){
        $users = Employee::select('employee_id','employee_name')->get();//orderBy('id','desc')->get();

-        return view('users',['employees'=>$users,]);
+        return view('users',['users'=>$users,]);
    }
}

Comments

[plusninesty]

ご指摘いただいた箇所修正したら解決しました。早期ご回答いただきありがとうございました。