PHP foreach循环使用一个条目两次


PHP foreach loop using one entry twice

我只是在MySQL数据库中尝试使用PHP和PDO,我有点困惑,为什么在获得结果后,将它们正确存储在多维数组中并循环使用,它会两次输出其中一个数组数据。

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

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

我希望这样可以为每个Family返回1行,这在PHPMyAdmin查询和打印结果时都能正确工作。

然后我存储在:

$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循环时,它会返回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();
}
}

这样还可以获得正确的计数,并将其正确存储在每个族中项目数的数组中。所以到目前为止一切都很好。

问题发生在I:

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个原始正确数组的位置上有一个重复数组。

任何建议都会很棒。

编辑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$familys数组的最后一个元素。

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

foreach($families as $fam){

$fam指向的内存位置不变,它仍然锁定在$家族数组的最后一个元素。因此,在第一次迭代中,第一个元素的内容被复制到$fam中,即在最后一个条目中,然后在第二次迭代时,第二个值被覆盖,依此类推。当最后一次迭代开始时,最后一个元素包含倒数第一个值,这会被。。。本身,这也是最后一个值。

这个错误报告提出了同样的问题,给出的答案是这是故意的行为。在回答一个重复的错误报告时,这是非常中肯的:

目前的实施是一致的。当然,不是很有用,但是在这里任意中断引用是不一致的
PHP没有块作用域,破坏引用将引入此处为特殊情况块范围。

这个博客用精美的插图解释了同样的行为。

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

foreach($families as $fam2){

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

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

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

foreach上的文档对这种行为发出了警告,并向unset:提出了建议

警告

CCD_ 5和最后一个数组元素的引用即使在CCD_。建议用unset()销毁。

所有这些都很尴尬,当你阅读对相关"bug"报告的回复时,很明显,你并不是唯一一个遇到这种意外副作用的人。因此,我想强调这一点:

避免使用危险的&

几乎从来没有真正需要使用这个&前缀。通过避免它,这些奇怪的副作用将成为过去。

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

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();
}

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

不管怎样,第二双眼睛看不出代码的HTML输出部分有什么问题。这并没有改变你正在迭代的数组的结构,所以在我看来,也许你的诊断是不对的?值得对最后一个foreach循环进行注释,并将其替换为:

<?php foreach($families as $fam): ?>
    <?php var_dump($fam);?> 
<?php endforeach; ?>

在输入HTML重foreach块之前,只需仔细检查$familys数组是否正确。

编辑:我已经在视图代码前面直接运行了一个显式数组声明,并且它不会复制最后的结果。当您将其直接放在视图代码之前时会发生什么?

$families = [
    [
        'ID' => 1,
        'Manufacturer' => 'man1',
        'Family' => 'fam1',
        'URL' => 'url1',
        'IMG' => 'img1',
        'Price' => 'price1',
        'ScentCount' => 1
    ],
    [
        'ID' => 2,
        'Manufacturer' => 'man2',
        'Family' => 'fam2',
        'URL' => 'url2',
        'IMG' => 'img2',
        'Price' => 'price2',
        'ScentCount' => 2
    ],
    [
        'ID' => 3,
        'Manufacturer' => 'man3',
        'Family' => 'fam3',
        'URL' => 'url3',
        'IMG' => 'img3',
        'Price' => 'price3',
        'ScentCount' => 3
    ],
];
?