[LR] Routing do widoków, 2 pytania Opcje

[markonix]

[PHP] pobierz, plaintext 
Route::get('page/{id}', function ($name) {
    return view('pages.static.'. $name);
});
[PHP] pobierz, plaintext 

Mam taki prosty route, który pozwoli na tworzenie takich statycznych podstron bez mojej ingerencji.
Wystarczy stworzyć widok w folderze i wsio.

Mam jednak zagwozdki:
1) Jak wyświetlić 404 gdy widoku nie ma, przy obecnym rozwiązaniu wyrzuca błąd o braku widoku (oczywiste)?
2) Chciałbym pominąć te "page" czyli mieć example.com/faq zamiast example.com/pages/faq? Jak zrobić taki główny route, który zbierze wszystkie requesty ale zostawi te należące do aplikacji?

edit:
W sumie z tym punktem 2 przyszedł mi najoczywistszy pomysł.. Po prostu dać:

[PHP] pobierz, plaintext 
Route::get('{id}', function ($name) {
    return view('pages.static.'. $name);
});
[PHP] pobierz, plaintext 

na samym końcu pliku.

Czy to "ładne" i bezpieczne rozwiązanie?

Ten post edytował markonix 13.01.2016, 00:18:08

[Pyton_000]

Rzucany masz taki wyjątek:

Kod

throw new InvalidArgumentException("View [$name] not found.");

Dodaj obsługę tutaj: app/Exceptions/Handler.php

np:/

[PHP] pobierz, plaintext 
if ($e instanceof InvalidArgumentException) {
            abort(404);
        }
[PHP] pobierz, plaintext 

[markonix]

[PHP] pobierz, plaintext 
        if ($e instanceof InvalidArgumentException)
        {
            abort(404);
        }
 
        $class = get_class($e);
        if ($class == 'InvalidArgumentException')
        {
            abort(404);
        }
[PHP] pobierz, plaintext 

Jakaś podpowiedź czemu działa tylko ta druga, brzydsza metoda? :/

[Pyton_000]

\InvalidArgumentException ?

Zrób dd($e) i będziesz wiedział jakiego dokładnie jest typu.

Ten post edytował Pyton_000 13.01.2016, 12:54:04

[markonix]

Dumpowałem i wskazywało na InvalidArgumentException dlatego zgłupiałem.
Slash oczywiście zadziałał, oczywiście bo nawet PHPSTORM to podpowiadał.

Wracając do tematu właściwego mam coś takiego:

[PHP] pobierz, plaintext 
    public function report(Exception $e)
    {
        if ($e instanceof \InvalidArgumentException && strstr($e->getMessage(), 'pages.static') !== false)
        {
            abort(404);
        }
        return parent::report($e);
    }
[PHP] pobierz, plaintext 

Ten drugi warunek jest po to aby zachować funkcjonalność wyjątków gdy faktycznie brakuje widoku w przypadku normalnych kontrolerów (np. zrobimy literówkę z metodzie view()).
Brzydko ale działa jak powinno.

[KrzychuKa]

Czy nie byłoby gustowniej poprostu sprawdzić czy widok istnieje?

[PHP] pobierz, plaintext 
view()->exists('pages.static.'. $name)
[PHP] pobierz, plaintext 

[markonix]

Czyli coś takiego:

[PHP] pobierz, plaintext 
Route::get('{id}', function ($name) {
    if (view()->exists('pages.static.'. $name)) {
        return view('pages.static.' . $name);
    }
    return abort(404);
});
[PHP] pobierz, plaintext 

Też o czymś takim myślałem, ale nie jestem przyzwyczajony że w LR wszystkie metody i funkcje są tak "magicznie" wszędzie dostępne.
W każdym razie praktyczna nauka Exceptions/handler też się przyda na przyszłość.