关于 route.thephpleague.com 的正确DI方法

Question

>我收到了关于我的代码的反馈，说我错误地使用了依赖注入：

您正在使用 DI，但除了Request和Response.以下 10 行将始终 实例化对象，即使您从未使用过它们。

在这些方面，我有这样的东西

$router->map('GET', '/tokens/{id}', [new APIController($server, $tokenRepository, $logger), 'get']);
$router->map('GET', '/tokens', [new APIController($server, $tokenRepository, $logger), 'list']);
....
$response = $router->dispatch($container->get('request'));

根据文档，这似乎是正确的方法。引导.php：

$container  = new Container;
$logger = new Logger('book');
$tokenRepository = new RedisTokenRepository($predis, $logger);
$container->add(Logger::class);
$container->add(Server::class);
$container->add(TokenController::class)->addArguments([Server::class, $logger]);
$container->add(APIController::class)->addArguments([Server::class, $tokenRepository, $logger]);
$strategy = (new ApplicationStrategy)->setContainer($container);
$router   = (new Router)->setStrategy($strategy);
$router->map('GET', '/', [new AcmeAPIController, 'someMethod']);

控制器

class APIController
{
private $server;
private $tokenRepository;
private $logger;
public function __construct(
Server $server,
TokenRepositoryInterface $tokenRepository,
LoggerInterface $logger
)
{
$this->server = $server;
$this->tokenRepository = $tokenRepository;
$this->logger = $logger;
}

谁能解释我做错了什么？

Answer 1

反馈很可能是指您实际实例化了路由定义的类;并且您正在手动将依赖项传递给它们。

$router->map('GET', '/tokens/{id}', [new APIController($server, $tokenRepository, $logger), 'get']);

注意new关键字

使用这种方法，在解析文件时，系统将创建APIController的实例 - 是的，几个。

你应该使用

$router->map('GET', '/', [APIController::class, 'get']);

或

$router->map('GET', '/', 'APIController::get');

这样，路由器只会在需要时实例化它们

看来你的方法需要$server, $tokenRepository, $logger， 按照文档在Controller class上描述它们来定义它们