Как сделать мои маршруты более эффективными?

Question

Как сделать мои маршруты более эффективными?

Делая переписать директивы в .Htaccess файл я пишу мои пользовательские директивы.

class Route {

public $request_url;
public $url_args;
public $request_method;

function __construct() {
$this -> request_url = $_SERVER['PATH_INFO'];
$this -> url_args = explode('/', substr($_SERVER['PATH_INFO'], 1));
$this -> request_method = $_SERVER['REQUEST_METHOD'];
}

public function get($url, $callback) {
if($this -> request_method == 'GET') {
if($url == $this -> request_url){
$callback();
} elseif(strpos($url, ':')) {
$new_url_args = explode('/', substr($url, 1));
if(count($new_url_args) == count($this -> url_args)) {
$match = true;
$args = [];

for($i = 0; $i < count($this -> url_args); $i++) {
if($new_url_args[$i][0] == ':') {
$args[substr($new_url_args[$i], 1)] = $this -> url_args[$i];
} else {
if($new_url_args[$i] != $this -> url_args[$i]){
$match = false;
break;
}
}
}

if($match) {
$callback($args);
}

}
}
}
}
}

И тогда я инициировал и добавил несколько маршрутов следующим образом.

$route = new Route();

$route -> get('/', function() {
echo "Welcome to DocsApp";
die;
});

$route -> get('/test/sample', function() {
echo 'tested';
die;
});

$route -> get('/user/:id/:name', function($args) {
echo  $args['id'] . '<br />' . $args['name'];
die;
});

Все работает нормально.

Но каждая функция get вызывается вместо требуемой.
Чтобы предотвратить это, я звоню die в конце успеха обратный вызов согласованного маршрута.

Есть ли лучший способ вызвать определенную функцию маршрута и предотвратить ненужные вызовы?

1

.htaccess custom-routes php routing

Решение

Другие решения

Прочитав ответ Олафа, я нашел какое-то решение.
Добавление match_found собственность на Route поможет предотвратить ненужные звонки.

class Route {

public $request_url;
public $url_args;
public $request_method;
private $match_found = false;

function __construct() {
$this -> request_url = $_SERVER['PATH_INFO'];
$this -> url_args = explode('/', trim($_SERVER['PATH_INFO'], '/'));
$this -> request_method = $_SERVER['REQUEST_METHOD'];
}

public function get($url, $callback) {
if($this -> match_found) {
return;
}
if($this -> request_method == 'GET') {
if($url == $this -> request_url){
$callback();
$this -> match_found = true;
} elseif(strpos($url, ':')) {
$new_url_args = explode('/', trim($url, '/'));
if(count($new_url_args) == count($this -> url_args)) {
$match = true;
$args = [];

for($i = 0; $i < count($this -> url_args); $i++) {
if($new_url_args[$i][0] == ':') {
$args[trim($new_url_args[$i], ':')] = $this -> url_args[$i];
} else {
if($new_url_args[$i] != $this -> url_args[$i]){
$match = false;
break;
}
}
}
if($match) {
$callback($args);
$this -> match_found = true;
}
}
}
}
}

1

Источник

Accepted Answer

Вы уже знаете, соответствует ли URL. Я бы просто вернулся true или же false в зависимости от результата и выберите код возврата, если вам нужно пойти дальше, например,

public function get($url, $callback) {
if ($this->request_method == 'GET') {
if ($url == $this->request_url) {
$callback();
return true;
} elseif (strpos($url, ':')) {
$new_url_args = explode('/', substr($url, 1));
if (count($new_url_args) == count($this->url_args)) {
$match = true;
// ...
if($match) {
$callback($args);
return true;
}
}
}
}

return false;
}

$route = new Route();

$match = $route->get('/', function() {
echo "Welcome to DocsApp";
});

if (!$match) {
$match = $route->get('/test/sample', function() {
echo 'tested';
});
}

if (!$match) {
$match = $route->get('/user/:id/:name', function($args) {
echo  $args['id'] . '<br />' . $args['name'];
});
}

1