我只是用 PHP 和 PDO 与 MySQL 数据库一起做一些实验，我有点困惑为什么在得到结果后，将它们正确地存储在一个多维数组中并循环遍历它们输出一个数组数据的两倍。

本质上，这是获取数据的查询:

SELECT b.Price, b.ImgURL, m.Name, f.ID, f.Family, f.URL FROM Products AS b INNER JOIN Manufacturers AS m ON m.ID = b.Manufacturer INNER JOIN FamilyLookUp AS l ON l.Product = b.ID INNER JOIN Families AS f ON f.ID = l.Family GROUP BY f.ID ORDER BY b.Price ASC

我希望用它为每个家庭返回 1 行，它在 PHPMyAdmin 查询和 print_r() 结果中都能正常工作。

然后我存储在:

$families[] = array('ID' => $f['ID'], 'Manufacturer' => $f['Name'], 'Family' => $f['Family'], 'URL' => $f['URL'], 'IMG' => $f['ImgURL'], 'Price' => $f['Price'], 'ScentCount' => 0);

它在执行 print_r() 时也能正常工作，并且在使用 foreach 循环回显每个条目的 ID 时它会返回 1234567(所有 7 个系列 ID)

然后我运行另一个查询:

try{
$sqlCmd = "SELECT COUNT(*) FROM FamilyLookUp WHERE Family=:fID";
$s = $pdo->prepare($sqlCmd);
foreach($families as &$fam){
$s->bindValue(':fID', $fam['ID']);
$s->execute();
$fam['ScentCount'] = $s->fetchColumn();
}
}

这也会得到正确的计数，并将它们正确地存储在数组中，以表示每个家庭中的项目数。所以到目前为止一切都很好。

问题发生在我:

foreach($families as $fam):
        ?>

        <div class="product-listing">
        <?php echo $fam['ID']; ?>
            <div class="product-listing-image">
                <a href="<?php echo $fam['URL']; ?>"><img alt="" src="<?php echo $fam['IMG']; ?>"></a>
            </div>
            <div class="product-listing-details">

                <a href="<?php echo $fam['URL']; ?>"><h3><?php echo strtoupper($fam['Manufacturer']); if($fam['Family'] != ""){ echo strtoupper(' - ' . $fam['Family']);} ?></h3></a>
                <?php if($fam['ScentCount'] == 1): ?>
                <span class="product-scent-count"><?php echo $fam['ScentCount']; ?> Scent</span>
                <span class="product-price-value">£<?php echo $fam['Price']/100; ?></span>
                <?php elseif($fam['ScentCount']>1): ?>
                <span class="product-scent-count"><?php echo $fam['ScentCount']; ?> Scents</span>
                <span class="product-price-value">From £<?php echo $fam['Price']/100; ?></span>
                <?php endif;?>
            </div>
        </div>

        <?php
            endforeach;
        ?>

执行此操作后，它会正确输出前 6 个数据族，但由于某种原因，它会输出第 6 个的副本，而不是实际的第 7 个。在 foreach 循环开始之前对行中的所有数据执行 print_r 时，它会返回所有正确的数据，但在 foreach 循环中，第 7 个原始正确数组的位置变成了 1 个重复数组。

任何建议都会很棒。

编辑 Kohloth 的答案(print_r 后跟 foreach vardump):

Array
(
    [0] => Array
        (
            [ID] => 1
        )

    [1] => Array
        (
            [ID] => 7
        )

    [2] => Array
        (
            [ID] => 2
        )

    [3] => Array
        (
            [ID] => 3
        )

    [4] => Array
        (
            [ID] => 4
        )

    [5] => Array
        (
            [ID] => 6
        )

    [6] => Array
        (
            [ID] => 5
        )

)
            array(7) {
  ["ID"]=>
  string(1) "1"
}

    array(7) {
  ["ID"]=>
  string(1) "7"
}

    array(7) {
  ["ID"]=>
  string(1) "2"
}

    array(7) {
  ["ID"]=>
  string(1) "3"
}

    array(7) {
  ["ID"]=>
  string(1) "4"
}

    array(7) {
  ["ID"]=>
  string(1) "6"
}

    array(7) {
  ["ID"]=>
  string(1) "6"
}

最佳答案

这是正在发生的事情:

在这个循环的最后一次迭代:

foreach($families as &$fam){
    $s->bindValue(':fID', $fam['ID']);
    $s->execute();
    $fam['ScentCount'] = $s->fetchColumn();
}

... $fam 指的是 $families 数组的最后一个元素。

然后当你的下一个循环开始时:

foreach($families as $fam){

...$fam 指向的内存位置不会改变，它仍然锁定到 $families 数组的最后一个元素。因此在第一次迭代中，第一个元素的内容被复制到 $fam 中，即在最后一个条目中，然后在第二次迭代中，第二个值在那里被覆盖，依此类推。当最后一次迭代开始时，最后一个元素包含一个但最后一个值，并且它被 ... 本身覆盖，这又是一个但最后一个值。

这bug report提出了同样的问题，给出的答案是这是预期的行为。在回答one of the duplicate bug reports , 说的很中肯:

The current implementation is consistent. Granted, not very useful, but it would be inconsistent to arbitrarily break the reference here.
PHP has no block scope, and breaking the reference would introduce a special-case block-scope here.

这blog用漂亮的插图解释相同的行为。

解决方案是在第二个循环中使用另一个新变量，如下所示:

foreach($families as $fam2){

或者，对于您可能使用 $fam 的任何其他代码，更安全的方法是在第二个循环之前unset($fam)，如下所示:

unset($fam);
foreach($families as $fam){

这是有效的，因为在 foreach 循环开始时，变量从头开始重新创建，因此指向它自己的新内存位置。

documentation on foreach有关于此行为的警告并建议 unset:

Warning

Reference of a $value and the last array element remain even after the foreach loop. It is recommended to destroy it by unset().

所有这一切都很尴尬，当您阅读对相关“错误”报告的回复时，很明显您不是唯一遇到这种意想不到的副作用的人。因此我想强调这一点:

避免使用危险的&

几乎从来没有真正需要使用这个 & 前缀。通过避免它，这些类型的奇怪副作用将属于过去。

您的代码可以在没有 & 的情况下重写如下:

foreach($families as $i => $fam){
    $s->bindValue(':fID', $fam);
    $s->execute();
    // use the index to put the value in place in the array:
    $families[$i]['ScentCount'] = $s->fetchColumn();
}

请注意，它也不会损害代码的可读性。

关于PHP foreach 循环使用一个条目两次，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/36666753/