我的网站上有广告。对于要公开展示的广告,必须付费。如果广告没有未付款,或者所有付款都已付款,则该广告是公开的或付费的。
我想要的是检索所有公开广告,但这需要使用此方法进行大量查询。我正在寻找更好的方法。
这有更多字段,但让我们关注 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;
}
我正在获取所有广告,如果它们是公开的,我将它们放入一个数组中,稍后将转到 twig View 。
如果所有付款都已支付,则广告是公开的,因此:
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 的工作原理。它创建一个 token ,然后在您使用该 token 为您的对象付款后,它会使用有关付款的详细信息填充该对象的付款字段详细信息。你可以找到类 here .我上面的支付类扩展了它。所以我正在检查这些详细信息是否已填写,或者“付费”字段是否为真。
public function paidPayment(Payment $payment)
{
$details = $payment->getDetails();
if (array_key_exists("paid", $details) && $details["paid"]) return true;
else return false;
}
问题是这会创建大量不必要的数据库查询。有可能缩短这个吗?我不确定我是否可以在没有 Doctrine 的情况下使用普通查询,因为我需要检查 details['paid'] 字段,我只能在收到数据库结果后在 Controller 或服务中执行此操作。如果甚至可以在查询中执行此操作,我既不知道也似乎找不到方法来执行此操作。
最佳答案
您应该使用 QueryBuilder
而不是 findAll
方法。在这种情况下,您可以指定需要匹配的条件(就像在普通 sql
中一样)。
所以:
$paidAds = $this->em->getRepository('MyBundle:Ad')
->createQueryBuilder('a')
->innerJoin('a.Payments', 'p')
->getQuery()
->getResult();
我没有深入分析你的数据库结构,事实上 innerJoin
可能只是你需要的条件,因为你只会得到那些与付款相关的广告
要阅读更多关于 QueryBuilder
的信息,请查看 doc
关于php - 如何根据值查询相关实体,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/34880634/