检测到服务的循环引用

Question

我有一个简单的类，如下所示：

<?php
namespace App\Algorithm;

use App\Dao\MatchDao;
use App\Service\MatchService;

class Calculator {
    private $users;
    private $matchDao;

    function __construct(MatchService $matchService, MatchDao $matchDao) {
        $this->users = $matchService->users;
        $this->matchDao = $matchDao;
    }

    public function hourlyRate() {
        $query = $this->matchDao->getSingleColumn('Payment', 'hourly_rate', 32);
        var_dump($query);
    }
}

但我收到以下错误消息：

检测到服务“App\Algorithm\Calculator”的循环引用，路径：“App\Algorithm\Calculator”-

MatchService.php

<?php
namespace App\Service;

use App\Algorithm\Calculator;
use App\Algorithm\Collection;

class MatchService {
    public $users;
    private $collection;
    private $calculator;

    function __construct(Collection $collection, Calculator $calculator) {
        $this->collection = $collection;
        $this->calculator = $calculator;
    }

    public function getMatch($data) {
        $this->users = $this->collection->getAllUsers($data);
        $this->calculator->hourlyRate();
        return 1;
    }

}

问题是MatchService，但我到底做错了什么？

Answer 1

这通常发生在类相互注入依赖关系时，因此循环引用。

给出上述示例，您的类MatchService注入集合和计算器。其中一个（假设calculator作为集合可能是一个条令类）依赖注入您的MatchService。

以下是我对你们的课程的看法：

class MatchService 
{
    public $users;
    private $collection;
    private $calculator;

    public function __construct(Collection $collection, Calculator $calculator) {
        $this->collection = $collection;
        $this->calculator = $calculator;
    }
}

class Calculator
{
    private $matchService;

    public function __construct(MatchService $matchService)
    {
        $this->matchService = $matchService;
    }
}

你有几个选择：

• 更多的服务具有更少的依赖性
• 使用静力学

我们很难为您解决这一问题，因为这取决于您如何构建应用程序。

Answer 2

很明显，您正在将服务A注入服务B，并且将服务B注入服务A。这样做似乎不合逻辑，但有时是必要的。就我而言，我有两项服务：

_迷思-

_我的客户-

我使用我的会话来存储这些凭据，因为它将对整个系统可用，但是，要使用我的客户端获取这些凭据，我需要一些信息存储到我的会话......看，两个需要彼此工作的服务...

我开始看到这一切

检测到服务的循环引用

刚刚升级到Symfony 5。sfy5本身也提出了解决方案：

composer require symfony/proxy-manager-bridge

请记住，服务可以设置为

lazy : true

关于Symfony文档的更多信息

Answer 3

正如一些人指出的那样，循环依赖性来自这样一个事实，即您试图将计算器注入MatchService，同时将MatchService注入计算器。在创建另一个之前，无法创建一个。

看起来更深入一点，似乎计算器正在使用匹配服务来获取用户列表。第二个问题是，计算器试图在匹配服务生成用户之前获取用户。

这里有一个可能的重构：

class Calculator
{
    private $matchDao;

    public function __construct(MatchDao $matchDao)
    {
        $this->matchDao = $matchDao;
    }
    public function getHourlyRate($users) // Added argument
    {
        $query = $this->matchDao->getSingleColumn('Payment', 'hourly_rate', 32);
    }
}
class MatchService
{
    private $collection;
    private $calculator;

    public function __construct(Collection $collection, Calculator $calculator)
    {
        $this->calculator = $calculator;
        $this->collection = $collection;
    }
    public function getMatch($data)
    {
        $users = $this->collection->getAllUsers($data);
        $this->calculator->getHourlyRate($users);
    }
}

从计算器的构造函数中删除MatchService可以解决循环依赖性问题。将$users传递给getHourlyRate可以解决在用户可用之前尝试获取用户的问题。

这当然只是一个可能的解决方案。从您发布的代码中不清楚计算器是否真的需要$users。