我的网站上有广告。要公开展示广告,必须付费。如果广告没有未支付的付款,或者所有付款都已付款,则该广告是公开的或付费的。
我想要的是检索所有公共广告,但这需要使用此方法进行大量查询。我正在寻找更好的方法。
这有更多的字段,但让我们专注于payments
字段。这是一对多。
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的工作方式。它会创建一个令牌,然后在您使用该令牌为您的对象付款后,它会使用该对象的付款字段详细信息填充有关付款的详细信息。你可以在这里找到课程。上面的付款类扩展了它。所以我正在检查这些详细信息是否已填写,或者"付费"字段是否为真。
public function paidPayment(Payment $payment)
{
$details = $payment->getDetails();
if (array_key_exists("paid", $details) && $details["paid"]) return true;
else return false;
}
问题是这会对数据库创建大量不必要的查询。有没有可能缩短这个时间?我不确定我是否可以在没有 Doctrine 的情况下使用普通查询,因为我需要检查详细信息['paid'] 字段,这是我在收到数据库结果后只能在控制器或服务中执行此操作。如果甚至可以在查询中做到这一点,我也不知道,似乎也没有找到一种方法来做到这一点。
你应该使用QueryBuilder
而不是findAll
方法。在这种情况下,您可以指定需要匹配的条件(就像在普通sql
中一样)。
所以:
$paidAds = $this->em->getRepository('MyBundle:Ad')
->createQueryBuilder('a')
->innerJoin('a.Payments', 'p')
->getQuery()
->getResult();
我没有深入分析您的数据库结构innerJoin
事实上可能只是您需要的条件,因为您只会得到那些与付款相关的Ads
要了解有关QueryBuilder
检查文档的更多信息