Add interfaces for Route and Handler

This commit is contained in:
PJ Dietz 2013-05-23 19:49:43 -04:00
parent 47ec8f5f96
commit b28b53aa11
5 changed files with 137 additions and 36 deletions

View File

@ -15,8 +15,15 @@ namespace pjdietz\WellRESTed;
*
* @property-read Response response The Response to the request
*/
abstract class Handler
abstract class Handler implements HandlerInterface
{
/**
* Matches array from the preg_match() call used to find this Handler.
*
* @var array
*/
protected $args;
/**
* The HTTP request to respond to.
*
@ -31,32 +38,6 @@ abstract class Handler
*/
protected $response;
/**
* Matches array from the preg_match() call used to find this Handler.
*
* @var array
*/
protected $args;
/**
* Create a new Handler for a specific request.
*
* @param Request $request
* @param array $args
*/
public function __construct($request, $args = null)
{
$this->request = $request;
if (is_null($args)) {
$args = array();
}
$this->args = $args;
$this->response = new Response();
$this->buildResponse();
}
// -------------------------------------------------------------------------
// Accessors
@ -75,6 +56,38 @@ abstract class Handler
return null;
}
/**
* @param array $args
*/
public function setArguments(array $args)
{
$this->args = $args;
}
/**
* @return array
*/
public function getArguments()
{
return $this->args;
}
/**
* @param \pjdietz\WellRESTed\Request $request
*/
public function setRequest($request)
{
$this->request = $request;
}
/**
* @return \pjdietz\WellRESTed\Request
*/
public function getRequest()
{
return $this->request;
}
/**
* Return the instance's Reponse
*
@ -82,6 +95,8 @@ abstract class Handler
*/
public function getResponse()
{
$this->response = new Response();
$this->buildResponse();
return $this->response;
}

View File

@ -0,0 +1,12 @@
<?php
namespace pjdietz\WellRESTed;
interface HandlerInterface
{
public function getRequest();
public function setRequest($request);
public function getArguments();
public function setArguments(array $args);
public function getResponse();
}

View File

@ -15,7 +15,7 @@ use \Exception;
/**
* A Route connects a URI pattern to a Handler.
*/
class Route
class Route implements RouteInterface
{
/**
* Regular expression matching URL friendly characters (i.e., letters,
@ -47,14 +47,14 @@ class Route
*
* @var string
*/
public $pattern;
private $pattern;
/**
* Name of the Handler class to use
*
* @var string
*/
public $handler;
private $handler;
/**
* Create a new Route
@ -68,6 +68,38 @@ class Route
$this->handler = $handler;
}
/**
* @param string $handler
*/
public function setHandler($handler)
{
$this->handler = $handler;
}
/**
* @return string
*/
public function getHandler()
{
return $this->handler;
}
/**
* @param string $pattern
*/
public function setPattern($pattern)
{
$this->pattern = $pattern;
}
/**
* @return string
*/
public function getPattern()
{
return $this->pattern;
}
/**
* Create a new Route using a URI template to generate the pattern.
*

View File

@ -0,0 +1,37 @@
<?php
namespace pjdietz\WellRESTed;
/**
* Interface for a route to relate a pattern for matching a URI to a handler class.
*/
interface RouteInterface
{
/**
* Return the regex pattern used to match the URI
*
* @return string
*/
public function getPattern();
/**
* Provide a regex pattern that matches the URI
*
* @para string $pattern
*/
public function setPattern($pattern);
/**
* Return the name of the class the route will dispatch.
*
* @return string
*/
public function getHandler();
/**
* Provide the classname to instantiate to handle the route.
*
* @param string $className
*/
public function setHandler($className);
}

View File

@ -36,9 +36,9 @@ class Router
/**
* Append a new Route instance to the Router's route table.
*
* @param Route $route
* @param RouteInterface $route
*/
public function addRoute(Route $route)
public function addRoute(RouteInterface $route)
{
$this->routes[] = $route;
}
@ -58,10 +58,15 @@ class Router
$path = $request->path;
foreach ($this->routes as $route) {
if (preg_match($route->pattern, $path, $matches)) {
if (is_subclass_of($route->handler, '\pjdietz\WellRESTed\Handler')) {
$handler = new $route->handler($request, $matches);
return $handler->response;
/** @var RouteInterface $route */
if (preg_match($route->getPattern(), $path, $matches)) {
$handlerClassName = $route->getHandler();
if (is_subclass_of($handlerClassName, '\pjdietz\WellRESTed\HandlerInterface')) {
/** @var HandlerInterface $handler */
$handler = new $handlerClassName();
$handler->setRequest($request);
$handler->setArguments($matches);
return $handler->getResponse();
} else {
return $this->getNoRouteResponse($request);
}