循环数组问题

Question

我有一个简化的ajay脚本，其中我删除了所有不相关的代码。我遇到的问题首先是我的列数组和随后的foreach循环。我想仔细检查每个元素，如果为true，则将相应的元素更改为YES，如果为false，则将其更改为NO，我不明白它为什么不起作用。

如果有任何问题，比如语法错误或大括号之类的问题，那都是我简化代码时遇到的问题，在我机器上的版本中没有出现。

<?php
$con = mysqli_connect("localhost", "", "", "");
if (!$con) {
    echo "Can't connect to MySQL Server. Errorcode: %s\n". mysqli_connect_error();
    exit;
}
$con->set_charset("utf8");
    $query1 = 'SELECT EGGS, SALAD, TREES, REVISED FROM AUCTIONS WHERE ARTICLE_NO = ?';
    if ($getRecords = $con->prepare($query1)) {
        $getRecords->bind_param("s", $pk);
        $getRecords->execute();
        $getRecords->bind_result($EGGS, $SALAD, $TREES, $REVISED);
        while ($getRecords->fetch()) {
        $columns = array('EGGS', 'SALAD', 'TREES', 'REVISED');
            foreach($columns as $column) {
                $$column = $columns[$column] ? 'YES' : 'NO';
            }
            imageSize = imageResize($PIC_URL, 250, 300);
            echo "<h1>".$EGGS."</h1>";
        }
    }
function imageResize($imageURL, $maxWidth, $maxHeight) {
    $imageSize["width"] = 0;
    $imageSize["height"] = 0;
    $size = getimagesize($imageURL);
    if ($size) {
        $imageWidth  = $size[0];
        $imageHeight = $size[1];
        $wRatio = $imageWidth / $maxWidth;
        $hRatio = $imageHeight / $maxHeight;
        $maxRatio = max($wRatio, $hRatio);
        if ($maxRatio > 1) {
            $imageSize["width"] = $imageWidth / $maxRatio;
            $imageSize["height"] = $imageHeight / $maxRatio;
            return $imageSize;
        } else {
            $imageSize["width"] = $imageWidth;
            $imageSize["height"] = $imageHeight;
            return $imageSize;
        }
    } else {
        die(print_r(error_get_last()));
    }
}

Answer 1

您的循环错误-$columns‘’EGGS‘不存在：

$columns = array('EGGS', 'SALAD', 'TREES', 'REVISED');
foreach($columns as $column) {
    $$column = $columns[$column] ? 'YES' : 'NO';
}

它应该是：

$columns = array('EGGS', 'SALAD', 'TREES', 'REVISED');
foreach($columns as $column) {
    $$column = $$column ? 'YES' : 'NO';
}

或者更好的是：

$tmp = array();
$columns = array('EGGS', 'SALAD', 'TREES', 'REVISED');
foreach($columns as $column) {
    $tmp[$column] = $$column ? 'YES' : 'NO';
}

这样你就不会重写你的绑定变量。

还要注意，出于性能原因，应该将常量数组声明移到while()循环之外。

Answer 2

但我只想补充一点，虽然使用像$$COLUMN这样的变量似乎是一个很好的功能，但它以后会变得非常混乱，并产生许多额外的变量。

为什么不创建一个临时数组，保存所有的YES/NO对？

此外，PHP不允许变量包含数字。我不知道在创建这样的变量时会发生什么：

$name = "123variable";
$$name = "foo";

我得去查一查。

编辑:我刚刚看到您只将查询结果绑定到一些变量。尽管如此，我认为这不是一个好的编码风格。

Answer 3

$columns[$column]并不存在--我无法想象您想要做什么，但这是一个错误。