Magento/Zend 框架中的内存泄漏

Question

运行这个简单的脚本时，我得到了下面发布的输出。 这让我觉得我的代码或 Zend Framework/Magento 堆栈中存在内存泄漏。迭代任何类型的 Magento 集合时会出现此问题。 我有什么遗漏或做错了吗？

脚本：

$customersCollection = Mage::getModel('customer/customer')->getCollection();

foreach($customersCollection as $customer) {
  $customer->load();
  $customer = null;
  echo memory_get_usage(). "\n";
}

输出：

102389104
102392920
...
110542528
110544744

Answer 1

您的问题是每次迭代都会发出相当昂贵的查询，而您可以通过集合查询加载必要的数据：

$collection = Mage::getResourceModel('customer/customer_collection')->addAttributeToSelect('*');

会做同样的事情，但都在一个查询中。这种方法需要注意的是，如果有任何 customer_load_before 或 customer_load_after 事件的自定义事件观察器（这些事件没有核心观察器），则需要为每个数据模型手动运行观察器。

编辑：感谢 osonodoar 发现错误的类引用（客户/客户与客户/客户集合）

Answer 2

对象（或其他值）的内存只有在 PHP 进程中的任何地方都没有引用时才能被释放。在您的情况下，$customer = null 行只会将对该对象的引用数量减少一，但不会使其达到零。

如果你考虑一个更简单的循环，这可能会变得更清楚：

$test = array('a' => 'hello');
foreach ( $test as $key => $value )
{
    // $value points at the same memory location as $test['a']
    // internally, that "zval" has a "refcount" of 2

    $value = null;
    // $value now points to a new memory location, but $test['a'] is unnaffected
    // the refcount drops to 1, but no memory is freed
}

因为您使用的是对象，所以有一个额外的转折 - 您可以在循环内修改对象而无需创建它的新副本：

$test = array('a' => new __stdClass);
// $test['a'] is an empty object

foreach ( $test as $key => $value )
{
    // $value points at the same object as $test['a']
    // internally, that object has a "refcount" of 2

    $value->foo = "Some data that wasn't there before";
    // $value is still the same object as $test['a'], but that object now has extra data
    // This requires additional memory to store that object

    $value = null;
    // $value now points to a new memory location, but $test['a'] is unnaffected
    // the refcount drops to 1, but no memory is freed
}

// $test['a']->foo now contains the string assigned in the loop, consuming extra memory

在您的情况下，->load() 方法可能会依次扩展$customersCollection 的每个成员中的数据量，每个成员都需要更多内存。在循环前后检查$customersCollection 可能会证实这一点。

Answer 3

首先，取消设置变量时使用 unset($variable) 而不是 $variable=null。它基本上做同样的事情，但你的意图要清楚得多。

其次，PHP 注定要死掉——内存泄漏不是一个大问题，因为 PHP 请求可能会持续几秒钟，然后进程就会死掉，并且它正在使用的所有内存都被释放以供下一个请求使用。除非您遇到扩展问题，否则无需担心。

编辑：这并不是说不要担心代码的质量，但是对于这样的事情，除非它引起问题，否则很可能不值得努力阻止它发生。

Answer 4

处理内存泄漏的其他方法是在循环中调用 exec 并让该 exec 函数完成导致内存泄漏的工作部分。

因此，一旦它完成了它的部分并终止了该 exec 中的所有内存泄漏。

因此，随着大量迭代，这种不断增加的内存损失将得到处理。

Answer 5

@benmarks 响应在这里是正确的方法，因为在循环中调用 load() 是一个非常昂贵的调用。

调用 $customer->load() 将增量分配内存，该内存将被 $customersCollection 引用，直到循环结束时才会释放内存。

但是，如果由于某种原因需要调用 load()，下面的代码不会泄漏内存，因为 GC 会在每次迭代中释放模型分配的所有内存。

$customersCollection = Mage::getModel('customer/customer')->getCollection();

foreach($customersCollection as $customer) {
    $customerCopy = Mage::getModel('customer/customer')->load($customer->getId());

    //Call to $customerCopy methods

    echo memory_get_usage(). "\n";
}