优化数据库查询MySQL和PHP

Question

我想知道是否还有修改我的PHP代码甚至在数据库上更轻一点？此代码在cronjob中每分钟运行一次。

希望确保这对数据库的影响尽可能小。

<?php
ini_set('memory_limit', '128M');
define('MAGENTO_ROOT', getcwd());
//exit;

$time     = time();
$time2    = $time - 0;
$to       = date('Y-m-d H:i:s', $time2);
$lastTime = $time - 60; //orders from the past X seconds
$from     = date('Y-m-d H:i:s', $lastTime);
$curDate  = date('Y-m-d');

$mageFilename = '/app/Mage.php';
require_once $mageFilename;
Mage::app();

$order_collection = Mage::getResourceModel('sales/order_collection');
$order_collection->addAttributeToSelect('*')->addAttributeToFilter('updated_at', array(
    'from' => $from,
    'to' => $to
))->getSelect();

print count($order_collection);
foreach ($order_collection->getItems() as $order) {
    $cards = Mage::getModel('giftcards/giftcards')->getCollection()->addFieldToFilter('order_id', $order->getId());
    print count($cards);
    foreach ($cards as $card) {
        if ($card->getCardStatus() == 0) {        
            if ((($card->getMailDeliveryDate() == null) || ($curDate == $card->getMailDeliveryDate())) && $card->getCardType() != 'offline') {
                $card->setCardStatus(1);
                $card->save();
                $card->send();
            }
        } elseif (($card->getCardStatus() == 1) && ($curDate < $card->getMailDeliveryDate())) {
            $card->setCardStatus(0);
            $card->save();
        }
    }
}
?>

Answer 1

你需要立即优化的唯一方法是你如何获取giftcards：你得到order_collections并且你正在迭代这些项目并获得相关数据;这被称为N=1 issue。

我不确定Magento是如何工作的（我很确定他们有一些你可以使用的join功能）但是你需要预先获取你需要的所有内容然后循环结果并根据你的需要格式化数据。