laravel5の２階層の一括ルーティング

Question

###前提・実現したいこと
全てアクセスに関して１つ１つルーティングしていくのはすごく面倒なので
https://ドメイン/ディレクトリ/コントローラファイル名/アクション名/id/
っていうふうにしたいです。
ルーティングの設定をどのようにすればいいのでしょうか？

###発生している問題・エラーメッセージ

他のサイトをいくらぐぐっても無理でした。

エラーメッセージ

###該当のソースコード
php

Route::any('{one?}/{two?}/{three?}/{four?}/{five?}/{six?}/{seven?}/{eight?}/{nine?}/{ten?}/'
, function($one='top',$two=null,$three=null,$four=null) {
$app = app();

$two		= ucfirst($two);
$app_path	= app_path();
if (file_exists($app_path.'/Http/Controllers/'.$one.'/'.$two.'Controller.php')) {
  $controller = "App\Http\Controllers\".$one."\".$two."Controller";
  $foo = new $controller;
  $foo->$three($four);
} else {
	header("HTTP/1.0 404 Not Found");
	die(View::make('common/404'));
}

});

###試したこと
と書くとViewが動かなくなりました。
laravel4なら動いてたのに、5になって動かなくなりました。


###補足情報(言語/FW/ツール等のバージョンなど)
laravel5

Answer 1

https://ドメイン/ディレクトリ/コントローラファイル名/アクション名/id/

このように決めているのであれば、以下のようにしてはどうでしょう。
（Laravelのバージョンは5.4）

ルーティング（汎用的なものに修正）

// その他ルーティング
Route::any('a/b/c', 'AbcsController@abc');

// [ 規定 ]
// https://ドメイン/ディレクトリ/コントローラファイル名/アクション名/id/ 
$paths = explode('/', Request::path());
if (!empty($paths) && count($paths) >= 3) {
    // 規定にマッチするが、別にルーティングをしたい場合
    // 規定と同じURLのものがある場合は、先に書いた方が優先して表示される
    Route::any('aa/bb/cc', 'FugasController@aaa');

    // 規定通りにルーティングをする場合
    $paths[0] = \Doctrine\Common\Inflector\Inflector::classify($paths[0]);
    $paths[1] = \Doctrine\Common\Inflector\Inflector::classify($paths[1]);
    Route::group(['namespace' => $paths[0]], function () use ($paths) {
        Route::any('{directory}/{controller}/{action}/{id?}/', $paths[1].'Controller@'.$paths[2]);
    });
}

コントローラーの例

namespace App\Http\Controllers\Hoge;

class HogesController extends \App\Http\Controllers\Controller
{
    public function show($directory, $controller, $action, $id=null)
    {
        dd([$directory, $controller, $action, $id]);
    }
}

例：
https://example.com/hoge/hoges/show/123/
（ドメイン：example.comを利用）
（ディレクトリー：App\Http\Controllers\Hogeにコントローラーがある）
（コントローラー：HogesController）
（アクション：showアクションにアクセス）
（ID：idは123）

上記ルーティングのコードの\Doctrine\Common\Inflector\Inflector::classify()は以下を使っています。

/**
 * Converts a word into the format for a Doctrine class name. Converts 'table_name' to 'TableName'.
 *
 * @param string $word The word to classify.
 *
 * @return string The classified word.
 */
public static function classify($word)
{
    return str_replace(" ", "", ucwords(strtr($word, "_-", "  ")));
}

Comments

[SeijiroKomatsu]

回答ありがとうございます。
ディレクトリはView内ではなくコントローラ内にディレクトリがほしいです。

[coba-coba]

修正しました。
Route::group()の'namespace'でコントローラー内のディレクトリを指定します。
ディレクトリが変わるので、コントローラーではnamespaceをApp\Http\Controllers\Hogeとして、Controllerについてもextends \App\Http\Controllers\Controllerのように指定します。

[SeijiroKomatsu]

ありがとうございました。解決しました。namespaceを考慮する必要があったんですね。

Answer 2

もうちょっと修正を加えました。　参考までに

$paths = explode('/', Request::path());
$paths[0] = isset($paths[0]) ? ucfirst($paths[0]) : '';
$paths[1] = isset($paths[1]) ? ucfirst($paths[1]) : '';
$paths[2] = isset($paths[2]) ? $paths[2] : 'default';
if (file_exists(app_path().'/Http/Controllers/'.$paths[0].'/'.$paths[1].'Controller.php')) {
$controller = 'App\Http\Controllers'.$paths[0].''.$paths[1].'Controller';
$instance = new $controller;
if (method_exists($instance,$paths[2])) {
Route::group(['namespace' => $paths[0]], function () use ($paths) {
Route::any('{directory}/{controller}/{action?}/{id?}/{five?}/{six?}/{seven?}/', $paths[1] . 'Controller@' . $paths[2]);
});
}
}

Answer 3

if文をついかすれば安全ですね。ありがとうございました。

Answer 4

$paths = explode('/', Request::path());
$paths[0] = isset($paths[0]) ? ucfirst($paths[0]) : '';
$paths[1] = isset($paths[1]) ? ucfirst($paths[1]) : '';
$paths[2] = isset($paths[2]) ? $paths[2] : '';
if (file_exists(app_path().'/Http/Controllers/'.$paths[0].'/'.$paths[1].'Controller.php')) {
  Route::group(['namespace' => $paths[0]], function () use ($paths) {
    Route::any('{directory}/{controller}/{action}/{id?}/', $paths[1].'Controller@'.$paths[2]);
  });
}

ありがとうございました。解決しました。

Comments

[coba-coba]

解決してよかったです！
でも「解決した方法」は細かいところを除いてまんま私のコードですよね…

下記の回答では回答用にポイントを絞って書いており変なところがあったので、汎用的なものに書き換えました。

ルーティングではファイルの有無をチェックする必要は基本的にはありません。
ルーティングで拾えないものは404となるだけです。
ただ、自分の規定通りのアクセスがきた場合のみ、500エラーにならないように気を付ける必要はあります。

あと、以下は好みの話ですが……

コントローラーの名前などは自分で好きにつければいいと思いますが、私はNamesControllerやTableNamesControllerとしたいです。
また、\Doctrine\Common\Inflector\Inflector::classify()を使ってURLを変換したいです。
なぜなら、URLを/table_names/…のように指定した場合、簡単に"TableNames"に変換することができるためです。
ucfirst()を使う場合は"TableNames"に一発で変換するためには、URLを/tableNames/…のように一部大文字を使ってあらわす必要があるためです。
URLに大文字は不格好なイメージがあります。
すみません好みの問題ですが。

[SeijiroKomatsu]

＞ルーティングで拾えないものは404となるだけです。
これは試してみたいと思います。
\Doctrine\Common\Inflector\Inflector::classify()でそういう使い方ができるんですね。
回答ありがとうございました。助かりました。