无法正确显示 FOREACH 循环的结果

Question

我将我的代码切换到 PDO 并且我几乎到了，除了这部分，查询的所有结果都应该使用 foreach 语句显示。我知道数据被正确提取并正确存储在 $row2 中。

我需要做的是从 $results2 中获取 'position' 变量并将其加 1，然后运行位置等于该新数字的 foreach 循环...

在我切换到 PDO 之前，我的代码运行良好，并且确实正确显示了 2 个预期结果：

$cont = $results2[0]->position;;
$cont++;

foreach ($results2 as $resulting) {
if ( $results2[0]->position = $cont )
{
echo "<hr><br><br>" . $resulting->text . "&nbsp;" . "   <b>Suggested by: $resulting-
>display_name </b><br>|<b> <a href=\"$url\">Approve this contribution</a></b> | <b><a
href=\"$url\">delete this suggestion</a></b> |";
}
}

但是现在，foreach 在转换为 PDO 之后只返回了两次相同的第一个结果，看起来像这样：

//新的 PDO 代码——不显示两个结果：

$results2->execute();
$row2 = $results2->fetchAll(PDO::FETCH_ASSOC);

$cont = $row2[0]['position'];
$cont++;

foreach ($row2 as $resulting) {
if ( $row2[0]['position'] == $cont )
{
echo "<hr><br><br>" . $row2[0][text] . "&nbsp;" . "   <b>Suggested by:" . $row2[0]
[display_name] . "</b><br>|<b> <a href=\"$url\">Approve this contribution</a></b> | <b>
<a href=\"$url/?WID=$resulting->WID\">delete this suggestion</a></b> |";
}
}

Answer 1

看起来您的旧代码正在获取一个对象数组，而您的新代码正在获取一个数组数组（由PDO::FETCH_ASSOC 指示）。

因此，虽然 $results2[0]->position 过去可以工作，但您错误地将其替换为 $row2[0][position]。但是，position 需要在第二种情况下引用，因为它是一个数组索引。

如果您打开了所有错误报告，您会很容易看到这一点。

这同样适用于$row2[0][text] 和$row2[0][display_name]。除了这些索引上没有引号之外，您可能真的想要$resulting 而不是$row[0]，因为$row[0] 每次循环都会为您提供相同的值。所以那些应该改成$resulting['text']和$resulting['display_name']

所以用$row2[0]['position'] 替换$row2[0][position] 的所有实例。另外，请听从 John Ruddell 的建议。

Answer 2

注意：这不是一个答案...这只是我澄清我在 cmets 中所说的内容的一种方式，因为它很难在 cmets 中显示代码。

通用 IF 代码逻辑：

if (true){
    // code will always execute on true
}

if (false){
    // code will never execute on false
}

样本：

$key = "12345";
if($key){
    // will always be executed because this is a true statement. 
}

您的代码：

if ( $row2[0]['position'] = $cont ){
    // will always execute because the assignment is true
}

现在： 除了所有这些.. 如果您在使用 == 运行它时没有发生任何事情，这意味着计数不等于 $row[0][position]，因此它是一个错误的语句，并且 if 中的代码不会被执行。

建议：

尝试在 if 语句之前添加 echo $cont; 和 echo $row2[0]['position']; 以查看发生了什么。