Laravelで関数内のreture viewが実行できない

Question

laravelで、以下のようなコードを実行したのですが、
showPage関数は実行されており$userも受け渡しができています。
しかし、user.showが表示されず白い画面が表示されます。
ご教授お願いします。

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;

class UserController extends Controller
{
    public function show($id)
    {
        function showView($user)
        {
            return view('user.show', compact('user'));
        }

        $user = User::findOrFail($id);
        showView($user);
    }
}

Comments

[退会済みユーザー]

手元のファイルと同じものを提示してください

Answer 1

どうしてもこのスタイルでやりたいなら、

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;

class UserController extends Controller
{
    public function show($id)
    {
        function showView($user)
        {
            return view('user.show', compact('user'));
        }

        $user = User::findOrFail($id);
        return showView($user); // ここで return する
    }
}

普通ならこうする

<?php

namespace App\Http\Controllers;

use App\User;
use Illuminate\Http\Request;

class UserController extends Controller
{
    public function show(User $user)
    {
        return view('user.show', compact('user'));
    }
}

Comments

[wonder_cat]

ありがとうございます。

[退会済みユーザー]

結局どうしたんでしょうか？

[wonder_cat]

やり方が、あっているのかわからないのですが、、、
このように、したかったです。
if (数字か？) {
if (usernameが空か？) {
return redirect(route('user.show', $user->username));
} else {
showView();
}
else {
if (username が存在するか) {
abort('404');
}
showView();
}

[退会済みユーザー]

それは本来バリデーションで行い、そもそも不正なデータが入らないように設計すべきではありませんか？

[wonder_cat]

ご指摘ありがとうございます。
つまり、こうゆうことでしょうか？

// routes/wab.php
Route::get('/u/{user}', 'UserController@show')->where('name', '[0-9]+')->name('user.show');
Route::get('/u/{user}', 'UserController@show')
->where('name', '[A-Za-z]+')
->name('user.show');

[退会済みユーザー]

バリデーションについて、調べてください。

[wonder_cat]

urlからusername もしくは id を取得して、
user情報を表示するだけで、POSTではないので
バリデーションは、必要ないと思ったのですが。。。

[退会済みユーザー]

showメソッドにつけるという話ではありません。
そもそも
show でそのような分岐をしなければいけないという状況がおかしいと言っているのです。
なぜ、そのような不正なデータが入った状況を考慮しなければいけないのですか？