PHP Laravel route path指定について

前提・実現したいこと

以下のような二つのパターンのrouteを通るようにしたい。

①
user/repo/branch
user/repo/branch/path1
user/repo/branch/path1/path2 (pathの部分は何個あっても可能）

下の二つだけなら
Route::get('/{user}/{repo}/{branch}/{path}','SearchController@show')->where('path','.*');
このようなroutingで通りますが一番上の場合(pathの部分がない場合)エラーになってしまいます。

②
user/repo/branch/path1/aaaa.html
user/repo/branch/path1/path2/aaaa.html
user/repo/branch/path1/path2/path3/aaaa.html

末尾のpathに@@@.@@@があった場合にrouteを通したい。
こちらもpathの部分は何個あっても大丈夫なようにしたいです。

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

上記のrouteの通し方がわからない。

試したこと

①
Route::get('/{user}/{repo}/{branch}/{path?}','SearchController@show')->where('path?','.*');
このように指定してみたがsyntaxエラー。

①だけを通すためなら
Route::get('/{user}/{repo}/{branch}/{path}','SearchController@show')->where('path','.*');
Route::get('/{user}/{repo}/{branch}/{path?}','SearchController@show');
とすればbranch以下のpathに何が来ても(user/repo/branchのような場合でも)通るようになりますが、
@@@@.@@@のpathが来た時もshowアクションが動くようになってしまいます。
@@@@.@@@の場合は違うアクションを動かしたいです。

②
Route::get('/{user}/{repo}/{branch}/{path}','SearchController@show')->where('path','.');
Route::get('/{user}/{repo}/{branch}/{path}.{ext}','SearchController@show')->where('path','.');

いずれも試しましたがどちらもうまくいきませんでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

行動規範の内容に同意します

回答2件

前回 https://teratail.com/questions/142664 で回答したものです。

path がない場合もあるようでしたらルーティングを

Route::get('/{user}/{repo}/{args}','SearchController@show')->where('args','^(?!.*.html$).*$');
Route::get('/{user}/{repo}/{args}','SearchController@html')->where('args','.*');

などとし、コントローラー側で

public function show($user, $repo, $args)
{
    $paths = explode('/', $args);
    $branch = array_shift($paths);
    // ...
}

public function html($user, $repo, $args)
{
    $paths = explode('/', $args);
    $branch = array_shift($paths);
    // ...
}

としてあげると $branch もうまく取得でき、path の部分が $paths に配列として格納されます。

そして末尾が .html の時は html メソッドを実行してくれるかと思います。

投稿2018/08/31 07:53

enomotodev

総合スコア27

kkkenn

2018/08/31 08:39

先日、今回共にご丁寧にありがとうございます。うまくルーティングは組めていましたがコーディングが汚かったので参考にさせていただきます！

行動規範の内容に同意します

ベストアンサー

ルートのwhere条件は、正規表現でチェックされていることを覚えれば、そこを工夫することでやりたいことが出来るようになりますよ！

①

user/repo/branch
user/repo/branch/path1
user/repo/branch/path1/path2 (pathの部分は何個あっても可能）

ルート指定はこれだけで充分だと思います。

Route::get('/{user}/{repo}/{branch}/{path?}','SearchController@show')->where('path','.*');

ルートパラメータ名の最後に付けた?は、「存在しない場合もあるよ」という目印のようなものなので、where条件のルートパラメータ名には含めなくて大丈夫です。

②

user/repo/branch/path1/aaaa.html
user/repo/branch/path1/path2/aaaa.html
user/repo/branch/path1/path2/path3/aaaa.html
末尾のpathに@@@.@@@があったは違うアクションを動かしたい。

※　@@@.@@@の前には1つ以上のpathがあるものとして話を進めます。

拡張子部分（.ドットより後ろ部分）が固定（あるいはパターンがある）ならば、それを条件にするのが良いと思うのですが、「@@@.@@@」は拡張子部分も変動する、ということでしょうかね・・？

routeの書き方はこんな感じだと思います。
※●●●部分は@@@.@@@で動かしたいアクションを指定してください

Route::get('/{user}/{repo}/{branch}/{path}/{file}.{ext}','SearchController@●●●')->where('path', '.*[^/]{1}$')->where('file', '[^./]+')->where('ext', '[^./]+');
Route::get('/{user}/{repo}/{branch}/{path?}','SearchController@show')->where('path','.*');

●●●のアクション（@@@.@@@のルート）の方は、
pathの条件は、/で終わらない、
fileの条件は、.（ドット）と/を含まない
extの条件は、.（ドット）と/を含まないとなっています。

showのルートは、●●●のアクションのルート設定よりも後ろに書きます。
これで、●●●のアクションのルートにマッチしない「/{user}/{repo}/{branch}/～～～～」なものはshowで拾うことになります。

この指定で、
●●●アクションには、例えば、
user/repo/branch/path1/path2/aaaa.html　でアクセスした場合、

{path} には "path1/path2"
{file} には "aaaa"
{ext} には "html"

が渡されるようになります。

★上記のルート指定での注意点★
先にも書きましたが、変じゃないかなー？と思うようなURLでも、
「/{user}/{repo}/{branch}/～～～～」の形であれば
showのルートに拾われますので、除外したいものがあれば、showの「path」のwhere条件（正規表現）を変える必要が出てきます。

変じゃないかな？と思うURLの例は、こんなの・・・

user/repo/branch/aaa.html ←path部分がないとか
user/repo/branch/path1/aaa.html/path2　←　URLの順番がおかしいとか
user/repo/branch/path1/path2/.html　　←　拡張子部分だけあるとか
user/repo/branch/path1/path2/aaaa..html　←　ドットがたくさんあるとか

※動作検証を最低限しかしていませんので、もしかしたら正規表現に考慮漏れや記述ミスがあるかもしれません。。。
その場合は、正しくあるべきURLを考えてみて、それに合うように正規表現を組み立てて試してください。

投稿2018/08/30 06:33