PHP - 简单的 IF ... ELSE IF... 产生意想不到的结果，请教我？

Question

我在一段简单的 php 代码上遇到了麻烦。

我正在使用 2 个产品定价层。这些基于用户是否已登录。

如果用户未登录，且第一个价格为空；那么价格就是price1。如果不是，则为price1。

这工作得很好。

如果用户已登录，并且第一个价格为空；那么价格就是price1。如果不是，则为price2。

这是它应该工作的方式，但实际发生的情况是这样的：

如果用户登录，并且第一个价格为空；那么价格就是0。如果不是，那就是price2。

为什么我的代码会产生这种效果？

if (!userIsLoggedIn())
{
    if (empty($prPrice2))
    {
        $prPrice = $prPrice1;
    }
    else
    {
        $prPrice = $prPrice1;
    }
} else if (userIsLoggedIn())
{
    if (empty($prPrice2))
    {
        $prPrice = $prPrice1;
    }
    else
    {
        $prPrice = $prPrice2;
    }
} else 
{
    $prPrice = $prPrice1;
}

如果有人有任何建议可以帮助我解决这个问题，我将不胜感激。

谢谢你！

@Pekka，这相当复杂。我只是希望这种情况发生：

product 1 -> price 1 = 1.00
product 1 -> price 2 = 0.00
product 2 -> price 1 = 1.00
product 2 -> price 2 = 0.80

如果用户已登录但price2字段为空，则price变量将为price1。如果不是，则为price 2。

另一方面，如果用户未登录但price2 字段为空，则price 变量将为price1。如果没有，则价格为1。

Answer 1

您解释规则的方式有点令人困惑。您应该能够修改以下代码以适应：

if (userIsLoggedIn()) {
    $prPrice = !empty($prPrice2) ? $prPrice2 : $prPrice1;
} else {
    $prPrice = !empty($prPrice1) ? $prPrice1 : $prPrice2;
}

Answer 2

该代码与您描述的算法矛盾。

你告诉：

如果用户未登录且第一个价格为空，则
价格是价格1。如果没有，那就是价格2。如果用户已登录并且
第一个价格为空，则价格为price1。如果没有，则为price2。

因此，事实上，无论用户是否登录，算法都应该执行完全相同的操作。

在你的代码中，这也很奇怪：

你有三个条件：

1. 用户未登录
2. 用户已登录
3. 其他

用户已登录或未登录。我没有看到任何其他可能性。

而且，以下行：

if (empty($prPrice2))
{
    $prPrice = $prPrice1;
}
else
{
    $prPrice = $prPrice1;
}

可以减少为，

$prPrice = $prPrice1;

因为您在两个代码块中执行相同的操作。

Answer 3

您的代码过于复杂并且包含一些奇怪的结构。这是一个等效但简化的版本，但请检查我在第一个 else... 上添加的注释...

if (!userIsLoggedIn())
{
    if (empty($prPrice2))
    {
        $prPrice = $prPrice1;
    }
    else
    {
        $prPrice = $prPrice1;  // this is highly suspicious...
    }
} else {    // user is logged or not, no need to recheck that boolean var
    if (empty($prPrice2))
    {
        $prPrice = $prPrice1;
    }
    else
    {
        $prPrice = $prPrice2;
    }
} 

用三元语句表达相同条件的稍微奇特的方式（示例相当于外部 if 的 else 块：

$prPrice = (empty($prPrice2)) ? $prPrice1 : $prPrice2;

编辑

假设有一种方法可以区分尚未登录的帐户持有人和没有帐户的用户，您需要在外部条件下处理它，如下所示：

if ($UserHasAnAccount) {  
// but i don't understand (yet) how you'd know that at this point
    if (userIsLoggedIn()) {  // the price logic described before
        ....
    } else {
        ....
    }
} else {  // unknown user, price1
    $prPrice = $prPrice1;
}

Answer 4

else 
{
    $prPrice = $prPrice1;
}

您无法达到此条件，因为据我了解 userIsLoggedIn 是布尔值，并且可能只发生两种情况：当 userIsLoggedIn 为 true 时和当 userIsLoggedIn 为 false 时。
你写的

如果用户未登录且第一个价格为空，则价格为price1。如果没有，则为price2。

但在你的代码中会发生这种情况：

如果用户未登录且第二价格为空，则价格为price1。如果不是，则也是price1。

同样在 userIsLoggedIn 条件下您写道

如果用户登录并且第一个价格为空

但在您的代码中，您检查第二个价格是否为空。