Question

我的网站上有广告。要公开展示广告，必须付费。如果广告没有未支付的付款或支付了所有付款，则该广告是公开的或付费的。

我想要的是检索所有公开广告，但这需要使用此方法进行大量查询。我正在寻找更好的方法。

这有更多字段，但让我们关注payments字段。这是OneToMany。

class Ad
{
    /**
     * @var int
     *
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="AUTO")
     */
    private $id;


    /**
     * @var string
     *
     * @ORM\Column(name="title", type="string", length=255,nullable=true)
     */
    private $adTitle;


    /**
     * @var string
     *
     * @ORM\Column(name="description", type="string", length=255,nullable=true)
     */
    private $description;



    /**
     * @ORM\ManyToOne(targetEntity="Application\FOS\UserBundle\Entity\User")
     * @ORM\JoinColumn(name="advertiser")
     */
    private $advertiser;


    /**
     * @ORM\OnetoMany(targetEntity="MyBundle\Entity\Payment",mappedBy="ad",cascade={"persist"})
     * @ORM\JoinColumn(name="payments", referencedColumnName="id",nullable=true)
     */
    private $payments;

}

对于付款，我使用的是PayumBundle，因此实体看起来像这样：

use Payum\Core\Model\Payment as BasePayment;
class Payment extends BasePayment
{
    /**
     * @ORM\Column(name="id", type="integer")
     * @ORM\Id
     * @ORM\GeneratedValue(strategy="IDENTITY")
     *
     * @var integer $id
     */
    protected $id;

    /**
     * @ORM\ManyToOne(targetEntity="MyBundle\Entity\Ad",inversedBy="payments",cascade={"persist"})
     * @ORM\JoinColumn(name="ad", referencedColumnName="id",nullable=true)
     */
    protected $ad;


}

目前我正在使用这样的服务：

  public function getAllPublicAds()
    {
        $allAds = $this->em->getRepository('MyBundle:Ad')->findAll();

        $publicAds=array();

        foreach($allAds as $ad){
            if($this->isPublicAd($ad)) array_push($publicAds,$ad);
        }

        return $publicAds;

    }

我收到所有广告，如果他们是公开的，我会将它们放入一个数组中，稍后会转到树枝视图。

如果广告的所有付款都已付款，则广告是公开的，因此：

public function isPublicAd(Ad $ad)
{

    $payments = $ad->getPayments();

    if (!count($payments)) {
        return false;
    } else {
        foreach ($payments as $payment) {
            if (!$this->paidPayment($payment)) return false;
        }

        return true;
    }
 }

最后，如果表格中的“付费”字段为真，则支付付款。这就是PayumBundle的工作原理。它会创建一个令牌，然后在您使用该令牌支付您的对象后，它会将该对象的付款字段详细信息填入有关付款的详细信息。您可以找到课程here。我上面的付款类扩展了它。所以我正在检查这些细节是否已填写，或者“付费”字段是否为真。

public function paidPayment(Payment $payment)
{
    $details = $payment->getDetails();

    if (array_key_exists("paid", $details) && $details["paid"]) return true;

    else return false;

}

问题在于，这会对数据库产生非常大且不必要的查询次数。有可能缩短这个吗？我不确定我可以在没有Doctrine的情况下使用普通查询，因为在收到数据库结果后，我需要检查详细信息['paid']字段，我只能在控制器或服务中执行。如果在查询中甚至可以做到这一点，我既不知道也不想找到办法。

Answer 1

您应该使用QueryBuilder代替findAll方法。在这种情况下，您可以指定需要匹配的条件（就像在普通sql中一样）。

所以：

$paidAds =  $this->em->getRepository('MyBundle:Ad')
                ->createQueryBuilder('a')
                ->innerJoin('a.Payments', 'p')
                ->getQuery()
                ->getResult();

我没有深入分析您的数据库结构实际上innerJoin可能只是您需要的条件，因为您只会获得与付款有关的Ads

详细了解QueryBuilder支票doc

如何根据相关实体的值查询相关实体

1 个答案: