Laravelで他テーブルのリストボックスを作成するには？

Question

実現したいこと

breezeのregisterに自前で作ったテーブルの項目をリストボックスを追加したいです。

発生している問題・分からないこと

初学者なもので、どう書いて良いかわからないのですが、調べたところ、
1.Modelを作成する
2.ControllerでViewに渡す
3.Viewでforeachで回す
という流れだと思うのですが、Viewを開くとforeachしようとしているコレクション？の変数が宣言されていないというエラーになります。
「Undefined variable $workplaces」となります。

該当のソースコード

```Workplace.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class Workplace extends Model
{
    use HasFactory;

    protected $fillable = [
        'name',
    ];
}

<?php

namespace App\Http\Controllers;

use App\Models\Workplace;
use Illuminate\Http\Request;

class WorkplaceController extends Controller
{
    public function getWorkplaces()
    {
        $workplaces = Workplace::all(); // 他のテーブルからデータを取得
        return view('profile.edit', compact('workplaces'));
    }

    /**
     * Display a listing of the resource.
     */
    public function index()
    {
        //
    }

    /**
     * Show the form for creating a new resource.
     */
    public function create()
    {
        //
    }

    /**
     * Store a newly created resource in storage.
     */
    public function store(Request $request)
    {
        //
    }

    /**
     * Display the specified resource.
     */
    public function show(Workplace $workplace)
    {
        //
    }

    /**
     * Show the form for editing the specified resource.
     */
    public function edit(Workplace $workplace)
    {
        //
    }

    /**
     * Update the specified resource in storage.
     */
    public function update(Request $request, Workplace $workplace)
    {
        //
    }

    /**
     * Remove the specified resource from storage.
     */
    public function destroy(Workplace $workplace)
    {
        //
    }
}

<div class="mt-4">
            <x-input-label for="workplace_id" :value="__('workplace_id')" />
            <select class="form-control" id="workplace_id" name="workplace_id">
                @foreach ($workplaces as $workplace)
                    <option value="{{ $workplace->workplace->id }}">{{ $workplace->name }}</option>
                @endforeach
            </select>
            <x-input-error :messages="$errors->get('workplace_id')" class="mt-2" />
        </div>

<?php

use App\Http\Controllers\ProfileController;
use Illuminate\Support\Facades\Route;
use App\Http\Controllers\WorkplaceController;

/*
|--------------------------------------------------------------------------
| Web Routes
|--------------------------------------------------------------------------
|
| Here is where you can register web routes for your application. These
| routes are loaded by the RouteServiceProvider and all of them will
| be assigned to the "web" middleware group. Make something great!
|
*/

// Route::get('/', function () {
//     return view('welcome');
// });
Route::get('/', function () {
    return view('auth.login');
});

Route::get('/dashboard', function () {
    return view('dashboard');
})->middleware(['auth', 'verified'])->name('dashboard');

Route::middleware('auth')->group(function () {
    Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit');
    Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update');
    Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy');
});

require __DIR__.'/auth.php';

<?php

use Illuminate\Database\Migrations\Migration;
use Illuminate\Database\Schema\Blueprint;
use Illuminate\Support\Facades\Schema;

return new class extends Migration
{
    /**
     * Run the migrations.
     */
    public function up(): void
    {
        Schema::create('workplaces', function (Blueprint $table) {
            $table->id();
            $table->string('name');
            $table->timestamps();
        });
    }

    /**
     * Reverse the migrations.
     */
    public function down(): void
    {
        Schema::dropIfExists('workplaces');
    }
};

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

chatGptやいろいろなサイトで調べたので、ここで調べたと出せません。すいません。

補足

一つ疑問があります。
Controllerに記述する際、Viewに返すと記載があるのですが、画面が登録、修正、削除とあった場合、それぞれにリストを変えすコードの記述が必要なのでしょうか？

AWS Lightsail上のLaravel10です。

Answer 1

ルーティング情報に getWorkplaces メソッドを呼び出す記述がありませんので、このメソッドは呼び出されてないのかと思います。

routes/auth.php の方に記述があると思いますが、 GET /register に対応するコントローラは
RegisteredUserControllerのcreateメソッドであり、この内容を修正する必要があるはずです。

コントローラが修正されずにViewに変数だけ増やしてしまっているので未定義の状態になっています。

Answer 2

適切なコントローラーに
public function create(): View
{
$workplaces = Workplace::select('id', 'name')->get();
return view('auth.register', compact('workplaces'));
}
を追加したところ、うまくいきました。
ただ、文字化けしてしまうので問題解決に至っていません。
また別の質問にしようかと思うので、この質問はクローズいたします。

Answer 3

return view('profile.edit', compact('workplaces'));

となっているのに、viewのファイル名がregister.blade.phpとなっているのが気になります。「Undefined variable $workplaces」の原因はこれな気がします。何が正しいかは第三者には分かりませんが、この辺をご確認いただくのが良いのではないかと思います。

Comments

[goola]

return view('register.blade', compact('workplaces'));
return view('register', compact('workplaces'));
共に試してみましたが、エラーに変化ありませんでした。

[AbeTakashi]

そもそもUndefined variable $workplacesはどのファイルの何行目で発生しているのでしょうか？ その情報を元にたどり着けませんかね？ コントローラーから$workplacesを渡しているようには見えるので、あとはコントローラーとビューのファイルが正しくひも付けされてないだけのように見えます。

追記）
エラーメッセージが変わらないということであれば、そもそもルーティング周りが間違ってる可能性が高いかもですね。URLが間違っている、web.phpの内容が間違っている可能性がありそうです。今一度、そちらも確認お願いします。どういうURLを叩いてるかが分からないので、第三者には分からないです。

[goola]

エラーは、
@foreach ($workplaces as $workplace)
この箇所で出ています。
URLは、
https://（ドメイン名）.net/register
です。
selectタグを止めてtextにすると普通に開きます。

https://www.youtube.com/watch?v=kfvLppwhmgQ
これでできそうだと思ったのですが、5:26あたりの、timezone_identifiers_listの意味が分からず、途方にくれています。
ただ、このcontrollerを、
public function create(): View
{
$workplaces = Workplace::all();
dd($workplaces);
return view('auth.register');
}
としたところ、
Illuminate\Database\Eloquent\Collection {#1285 ▼ // app/Http/Controllers/Auth/RegisteredUserController.php:25
#items: array:1 [▼
0 =>
App\Models
\
Workplace {#1235 ▼
#connection: "mysql"
#table: "workplaces"
#primaryKey: "id"
#keyType: "int"
+incrementing: true
#with: []
#withCount: []
+preventsLazyLoading: false
#perPage: 15
+exists: true
+wasRecentlyCreated: false
#escapeWhenCastingToString: false
#attributes: array:4 [▼
"id" => 1
"name" => "いしかわ"
"created_at" => null
"updated_at" => null
]
#original: array:4 [▶]
#changes: []
#casts: []
#classCastCache: []
#attributeCastCache: []
#dateFormat: null
#appends: []
#dispatchesEvents: []
#observables: []
#relations: []
#touches: []
+timestamps: true
+usesUniqueIds: false
#hidden: []
#visible: []
#fillable: array:1 [▶]
#guarded: array:1 [▶]
}
]
#escapeWhenCastingToString: false
}
となりました。
timezone_identifiers_listを無視して::allで突き進めば視界が開けるのでしょうか・・・？

[goola]

ちなみに、WorkplaceControllerを
public function getWorkplaces()
{
$workplaces = Workplace::all();
dd($workplaces);
return view('auth.register', compact('workplaces'));
}
としてもなにも表示されませんでした。（Undefined variable $workplacesのままでした）
routeに/registerも無いですし・・・。

あと、上記の通りしたとして、例えばドロップダウンを2つにしたい場合、returnはどう書くのかも疑問です。

[AbeTakashi]

見た感じコレクションのデータが正しく$workplacesに入ってるようなので、return view('register', compact('workplaces')); とすればview側のregister.blade.phpで$workplacesが参照できるはずです。それができないということは、別のコントローラーかメソッドを経由してregister.blade.phpが呼び出されている、全く別のviewが呼ばれている、などの可能性などが考えられると思います。動画の部分は今回のエラーとあまり関係なさそうですし無視して良さそうですが。xdebugとか使って変数をモニタリングしながらステップ実行してみるとかしてみるといいかもですね。