1

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

您正在使用 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 Acme\APIController, '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;
    }

谁能解释我做错了什么？

4

1 回答 1

1

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

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

注意new关键字

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

你应该使用

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

或者

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

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

看来您的方法需要$server, $tokenRepository, $logger，将它们定义为文档在Controller class

于 2020-01-02T13:39:42.097 回答