Роутинг на PHP?

Question

[marakhd]

Всем привет! Столкнулся с проблемой: постоянно выдается ошибка 404
Вот код:

Routing.php

<?php 

class Router {
    private $pages = array();
    
    function addRoute($url, $path){
        $this->pages[$url] = $path;
    }

    function route($url){
        if(isset($this->pages["/".$url])) {     //выяснил что ошибка возникает на этой проверки
            $path = $this->pages["/".$url];
            $file_dir = "app/".$path;
            if ($path == "") {
                require "errors/404.html";
                die();
            }
            if (file_exists($file_dir)) {
                require $file_dir;
            } else {
                require "errors/404.html";
                die();
            }
        } else {
            require "errors/404.html";
            die();
        }
    }

}
?>

index.php

<?php 
require "system/Routing.php";
$url = isset($_GET['url']) ? $_GET['url'] : '';

$router = new Router();

$router->addRoute("/cameras", "apps/cameras/index.php");

$router->route($url);

?>

.htaccess

<IfModule mod_rewrite.c>
RewriteEngine on
RewriteBase /
RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^ index.php [QSA,L]
</IfModule>

В чем может быть проблема? Заранее спасибо!

P.S.
файл apps/cameras/index.php доступен при запросе в формате :80/apps/cameras/index.php

Comments

[nokimaro]

marakhd а почему решили не использовать что-то из готовых решений для роутинга?
из самого примитивного и простого https://github.com/nikic/FastRoute
ну например чтобы дальше не пошли вопросы а как сделать роуты с подстановкой параметров по типу GET: /cameras/camera/<id> ?
или вопросы в духе как мне сделать разные обработчики при одинаковом имени роута например GET: /route и POST: /route ?
а как сделать группированные роуты?
а как лучше прикрутить кеширование для роутов так как у меня их набралось под сотню?
а если я хочу на роуты навесить middleware?

[nokimaro]

Смысл в том что если нужен нормальный роутинг лучше взять готовое решение, кода писать придётся даже меньше
Если нужен просто роутинг на 2-3 маршрута то это делается ещё проще

<?php
$url = $_GET['url'] ?? '';

$handler_file = match($url) {
    "/"        => "apps/index.php",
    "/cameras" => "apps/cameras/index.php",
    default    => "errors/404.html",
};

if(is_file($handler_file)) {
    require($handler_file);
    exit;
}

Answer 1

[Сергей delphinpro]

class Router {
    private $pages = array();
    
    function addRoute($url, $path){
-       $this->pages[$url] = $path;
+       $this->pages[trim('/', $url)] = $path;
    }

    function route($url){
+       url = trim('/', $url);
-       if(isset($this->pages["/".$url])) {
+       if(isset($this->pages[$url])) {
-           $path = $this->pages["/".$url];
+           $path = $this->pages[$url];
            $file_dir = "app/".$path;
            if ($path == "") {
                require "errors/404.html";
                die();
            }
            if (file_exists($file_dir)) {
                require $file_dir;
            } else {
                require "errors/404.html";
                die();
            }
        } else {
            require "errors/404.html";
            die();
        }
    }
}

require "system/Routing.php";
-$url = isset($_GET['url']) ? $_GET['url'] : '';
+[$url] = explode('?', $_SERVER['REQUEST_URI']);

$router = new Router();

$router->addRoute("/cameras", "apps/cameras/index.php");

$router->route($url);

Comments

[marakhd]

Спасибо! Но теперь проблема возникает на
if (file_exists($file_dir))
хотя sitenp.loc/apps/cameras/index.php

[Сергей delphinpro]

marakhd,

Лучше использовать абсолютный путь.
if (file_exists(__DIR__.$file_dir)) {

C путями разберитесь

$router->addRoute("/cameras", "apps/cameras/index.php");
$file_dir = "app/".$path; // "app/apps/cameras/index.php"

[marakhd]

Сергей delphinpro, спасибо

[nokimaro]

trim() поменял бы на ltrim() так как иначе роуты со слешем на конце не будут работать