这是一个错误还是我错了？

Question

/**
 * Returns nodes found by xpath expression
 *
 * @param string $xpath
 * @return array
 */
public function getXpath($xpath)
{
    if (empty($this->_xml)) {
        return false;
    }

    if (!$result = @$this->_xml->xpath($xpath)) {
        return false;
    }

    return $result;
}

此代码取自 Magento。您可以在其公共 svn 中查看特定文件：

http: //svn.magentocommerce.com/source/branches/1.5/lib/Varien/Simplexml/Config.php

现在我认为 (!$result = @$this->_xml->xpath ($xpath)) 永远不可能计算为 true，因此 return false 语句永远不会发生。

因为xpath的返回值的赋值，无论对变量$result是true还是false，总是返回true，取反总是返回false。

所以这是一个错误或完全冗余的代码，还是我错了？

仅供参考：我目前正在调试一个问题，其中一些配置元素“丢失”，我认为错误就在那里。

Answer 1

这是建立这样的条件的一种奇怪的方式，但我认为这是有道理的。

$result = 会将 xpath 操作的结果分配给 $result。

如果结果为 false （值 xpath( ) 将在出现错误时返回），条件将匹配并且函数将返回 false。

Answer 2

“因为将xpath的返回值赋值给变量$result总是返回true”这个说法是错误的，我不知道你为什么得出这个结论。赋值运算符的值始终等于分配的值，因此它很容易为 false 或 true (($x = false) == false )

Answer 3

您可以使用以下命令进行测试，

var_dump(!$result = FALSE);    // TRUE
var_dump(!$result = array());  // TRUE
var_dump(!$result = array(1)); // FALSE

因此如果 SimpleXml::xpath 返回空数组或 FALSE，Magento 函数将返回 FALSE。对数组取反将首先将其类型转换为布尔值。 空数组类型转换为 Boolean 将为 FALSE< /a>.这意味着，当您的 XPath 在语法上是正确的，但没有找到任何结果时，Magento 将返回 FALSE。

Answer 4

我发现将内部分配括起来会很有帮助：

if (! ($result = @$this->_xml->xpath($xpath)) ) {

从视觉上看，分配现在是一个子任务，必须在反转之前执行。如果没有额外的括号，它的执行完全相同，这只是有助于您自己的代码阅读。

任何赋值命令都会返回所返回的值。这可以让您执行以下操作：

$a = $b = $c = $d = $e = 'A made-up value';

$e 的新值将返回到 $d，而 $d 将返回到 $c，并且很快...