mirror/php-altorouter
1
0
Fork 0
mirror of https://github.com/dannyvankooten/AltoRouter.git synced 2025-08-08 09:26:33 +02:00
Code Issues Releases Wiki Activity

Merge pull request #130 from dannyvankooten/strpos-request-method

Browse Source 
Use `stripos` to compare request method (instead of `explode` + `in_array`) for improved performance.
This commit is contained in:
Danny van Kooten
2016-06-21 22:37:56 +02:00
committed by GitHub
parent 48524b67cb 6fe6f4b196
commit a8ee8e875f
1 changed files with 2 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
AltoRouter.php
View File

@@ -189,18 +189,9 @@ class AltoRouter {
} }
foreach($this->routes as $handler) { foreach($this->routes as $handler) {
list($method, $_route, $target, $name) = $handler; list($methods, $_route, $target, $name) = $handler;
$methods = explode('|', $method); $method_match = (stripos($methods, $requestMethod) !== false);
$method_match = false;
// Check if request method matches. If not, abandon early. (CHEAP)
foreach($methods as $method) {
if (strcasecmp($requestMethod, $method) === 0) {
$method_match = true;
break;
}
}
// Method did not match, continue to next route. // Method did not match, continue to next route.
if(!$method_match) continue; if(!$method_match) continue;