使用三元表达式返回会在第一次迭代时中断 foreach 循环

Question

好的，我测试了以下内容，我会让您知道我发现了什么：

echo ('-1' < 0) ? 'true' : 'false'; // will echo "true"
echo ('1' > 0) ? 'true' : 'false'; // will echo "true"
# Notice that '-1' and '1' are strings

现在让我们获取一个数组，该数组在过滤所有结果后来自数据库，以便仅获取具有 UID = 1 的行。

$this->a = array(
    [0] => array(
        'UID' => '1',
        'PID' => '91',
        'Amount' => '-1'
    ),
    [1] => array(
        'UID' => '1',
        'PID' => '92',
        'Amount' => '1'
    ),
    [2] => array(
        'UID' => '1',
        'PID' => '93',
        'Amount' => '1'
    )
);

现在我想创建一个函数 posAmount($PID) ，如果 'Amount' > 则返回 true如果 'Amount' 0 或 false 0 。 （注意：金额 = 0 是我并不关心的）。另外，我想编写一个名为 negAmount($PID) 的类似函数，它返回与第一个完全相反的函数。现在，我想向您介绍我的孪生函数：

public function posAmount($pid)
{
    foreach ($this->a as $a)
    {
        if (count($this->a) == 0) { return false; }
        return ($a['PID'] == $pid and $a['Amount'] > 0) ? true : false;
    }
}

public function negAmount($pid)
{
    foreach ($this->a as $a)
    {

        if (count($this->a) == 0) { return false; }
        return ($a['PID'] == $pid and $a['Amount'] < 0) ? true : false;
    }
}

很酷的事实是，关于第一个数组（我使用 var_dump() 检查过，在整个脚本中保持了其本质）：

$istance->negAmount(91); // Returns true, as expected
$istance->posAmount(92); // Returns false, as NOT expected.
# Why do God wants me to get mad?

Answer 1

问题是您总是在 foreach 循环的第一次迭代中返回。您应该像这样重写函数：

public function negAmount($pid) {
    if (count($this->a) == 0) { return false; }
    foreach ($this->a as $a) {
        if ($a['PID'] == $pid) {
            if ($a['Amount'] < 0) {
                return true;
            }
        }
    }
    return false;
}

public function posAmount($pid) {
    if (count($this->a) == 0) { return false; }
    foreach ($this->a as $a) {
        if ($a['PID'] == $pid) {
            if ($a['Amount'] > 0) {
                return true;
            }
        }
    }
    return false;
}

Answer 2

可能只是演示代码中的拼写错误，但 posAmount 方法正在循环 $this->a，而另一个方法正在循环 $this- >votes - OP更正

你的代码中有一些奇怪的东西。为什么要在 foreach 循环中检查 $this->a 的计数？在开始循环之前检查计数会更有意义。

另外，您的比较中存在一些逻辑错误。您只是比较循环中的第一次迭代...它将为数组的第一个索引返回 true 或 false，甚至从不查看其他索引。在比较并返回任何内容之前，您需要匹配循环中的 PID。就像这样：

public function posAmount($pid)
{
    if (count($this->a) == 0) { return false; }
    foreach ($this->votes as $a) {
        if ($a['PID'] == $pid)
        return $a['Amount'] > 0 ? true : false;
    }
    return false;
}

public function posAmount($pid)
{
    if (count($this->a) == 0) { return false; }
    foreach ($this->votes as $a) {
        if ($a['PID'] == $pid)
        return $a['Amount'] < 0 ? true : false;
    }
    return false;
}

Answer 3

问题是您试图将字符串与 int 进行比较，而不尝试对其进行转换。将 $a['Amount'] 更改为 (int)$a['Amount'] 并看看会发生什么。

Answer 4

所以在这里你迭代 $this->a:

public function posAmount($pid)
{
    foreach ($this->a as $a)

但这里 $this->votes:

public function posAmount($pid)
{
    foreach ($this->a as $a)

打字错误或者什么......