Existem algumas coisas em que você pode trabalhar;
- não passando por referência; portanto, usando memória extra, você pode passar objetos, mas as matrizes não podem ser passadas por referência por padrão. Ou adicione um
&
na declaração de parâmetro de função comofunction hello(array &$world)
- cheques inválidos, se algo não existir, retorne imediatamente. Não tente encontrar algo que não está lá
- a legibilidade pode ser difícil às vezes
- adicione algum documento (para que você possa entender se o vir dentro de alguns dias, meses, anos)
if
declarações mais inteligentes para obter menos recuo
- As funções devem ter apenas um propósito, atualizar o estoque ou atualizar os relacionados, não os dois, portanto, talvez até corte algumas funções em funções ainda menores. Tente criar essa lógica em sua mente e refaça a partir daí.
- Dê uma olhada
->cleanModelCache()->clearInstance()
de Mage_Core_Model_Model_Abstract
para limpar os dados subjacentes para alguns objetos, pode acelerar as coisas.
- de grosseiro todas as outras coisas que já foram ditas.
Adicionada uma versão atualizada do seu código com algumas recomendações embutidas no seu código atual, eu poderia continuar um pouco, mas atualmente não adicionaria mais.
Função 1: Objetivo é caminhar pela coleção
/**
* Walk collection
*
* @param Mage_Core_Model_Resource_Db_Collection_Abstract $collection
* @return void
*/
public function functionOne($collection)
{
// ...
// Walk collection, create references instead of passing array data
foreach ($collection as $item) {
// Update stock for product
if (!$this->_functionTwo($item)) {
// Not updated, continue next
continue;
}
// Update related products
$this->_functionThree($item); // Use same object again, no extra memory is used
}
// ...
}
Função 2: Objetivo é atualizar o estoque se alterado
/**
* Update stock item if changed, returns true if updated
*
* @param Mage_Core_Model_Model_Abstract $item
* @return bool
*/
function _functionTwo($item)
{
$model = Mage::getModel('catalog/product');
/** @var $model Mage_Catalog_Model_Product */
$id = $model->getIdBySku($item->getData('sku'));
if (!$id) {
// no id found, so stop looking nothing up
return false;
}
// Get option value for store 1
$inventoryStatus = $model->getResource()
->getAttributeRawValue($id, 'product_inventory_status', 1);
if (!$inventoryStatus) {
// No need for another lookup in db, because the status isn't set
return false;
}
$invStatus = $model->getResource()
->getAttribute('product_inventory_status')
->setStoreId(0) // Get admin value
->getSource()
->getOptionText($inventoryStatus);
if (!$invStatus) {
// No need for another lookup in db, because the status has no text
return false;
}
if ($invStatus === 'Z') {
// Inventory status to not change something
return false;
}
$stockItem = Mage::getModel('cataloginventory/stock_item');
/** @var $stockItem Mage_CatalogInventory_Model_Stock_Item */
// $stockItem->setData(array()); // unneeded piece of code
$stockItem->loadByProduct($id);
if ($stockItem->getQty() == $item->getData('quantity')) {
// Valid stock
return false;
}
// Update stock
$stockItem->setQty($item->getData('quantity'));
$stockItem->save();
// End function and call function three separately, does something else
return true;
}
Função 3: Objetivo de atualizar itens de estoque relacionados
/**
* Update related stock items, return false if no related items are found
*
* @param Mage_Core_Model_Model_Abstract $item
* @return bool
*/
function functionThree($item)
{
$collectionOfKits = Mage::getModel('kitinventory/kitinventory')
->getCollection()
->addFieldToFilter('related_sku', $item->getData('sku')); // Check if your indexes are set on these columns
if (!$collectionOfKits->getSize()) {
// Nothing found to relate to
return false;
}
$connection = Mage::getSingleton('core/resource')
->getConnection('core_write');
// Walk kits
foreach ($collectionOfKits as $kit) {
// getData is slightly faster then getSku(unless you've implemented it in your model)
// getSku -> __call('getSku') -> get -> lowercase('sku') -> getData('sku') | note, Magento has some internal caching in this
$kitSku = $kit->getData('sku');
$kitCollection = Mage::getModel('kitinventory/kitinventory')
->getCollection()
->addFieldToFilter('kit_sku', $kitSku)
->setOrder('related_sku', 'ASC');
// Use just a fetchAll to create a fast db query
$select = $kitCollection->getSelect();
$select->reset(Zend_Db_Select::COLUMNS)
->distinct()
->columns('related_sku')
->columns('required_quantity');
// Fetch component sku
$componentSkus = $connection->fetchAll($select, 0);
// Fetch required quantity
$componentRequiredQuantity = $connection->fetchCol($select, 1);
// ...
$componentProductCollection = Mage::getModel('catalog/product')
->getCollection()
->joinField('qty',
'cataloginventory/stock_item',
'qty',
'product_id = entity_id',
'{{table}}.stock_id = 1',
'left');
$componentProductCollection->addAttributeToFilter('sku', array('in' => $componentSkus));
// Next line will invoke a load on the product collection
foreach ($componentProductCollection as $component) {
$quantity = $component->getQty();
// ...
}
// You could choose to do a fetchAll here instead to get just the data you need
$connection = $componentProductCollection->getConnection();
foreach ($connection->fetchAll($componentProductCollection->getSelect()) as $row) {
// Will have a array here
$quantity = $row['quantity'];
// ... -- do not not which funky magic happens here
}
$kitId = Mage::getModel('catalog/product')
->getIdBySku($kitSku);
if (!$kitId) {
// No id
continue;
}
// You could also take a look if you can sum the stock and do a single update instead
$kitStockItem = Mage::getModel('cataloginventory/stock_item')
->loadByProduct($kitId);
$this->functionFour($kitStockItem, $kitSku, $amountOfKitsPossible);
// Or something like this, update single field
$connection->update($kitStockItem->getResource()->getMainTable(), array('qty' => $quantity), 'item_id = ' . $kitStockItem->getId());
}
return true;
}
Função 4: teve que fazer algumas suposições de sorte (ou azar), pois agora é uma função inútil, pode ser adicionada como na Função 3.
/**
* Save stock item if changed and something else, rather not say ;-)
*
* @param Mage_Catalog_Inventory_Model_Stock_Item $kitStockItem
* @param string $kitSku
* @param int $amountOfKitsPossible Guessed it
*/
function functionFour($kitStockItem, $kitSku, $amountOfKitsPossible)
{
// ...
// Do not know the rest of the code, so I wouldn't know which I could optimize here
// If it isn't to serious, you could look at a single query and not hitting extra functions
// Check if changed
if ($quantity !=$kitStockItem->getData('qty')) {
$kitStockItem->setQty($quantity);
$kitStockItem->save();
}
// ...
}
}
functionOne($collection)
? Em que ordem seria o tamanho / número de itens? É necessário fazer um loop sobre ele para obter os SKUs?