PHP If/Else逻辑异常：倒计时函数始终返回output2问题排查

你的代码存在几处关键逻辑错误，导致输出结果不符合预期：

1. 条件判断逻辑完全颠倒且判断对象错误
   原代码中用$zero <= 0作为返回剩余天数的条件，但$zero是strtotime($ed)生成的时间戳——只要传入的日期格式有效，这个值必然是正数（哪怕是过去的日期，时间戳也是正数，只是小于当前时间戳）。这就导致该条件几乎永远不成立，所以始终返回output2。

2. 变量名拼写错误
   你将$difference拼写成了$diffference（多了一个f），虽不影响运行，但降低了代码可读性。

3. 未处理无效日期参数
   如果传入的ed参数格式不正确，strtotime()会返回false，后续计算会出现异常，需要添加容错处理。

4. 未处理负数天数显示问题
   当结束日期早于当前时间时，计算出的$days会是负数，直接显示会造成误解。

修正后的代码 #

function calculate_time($atts){
    // 设置短代码参数默认值，避免参数缺失报错
    $atts = shortcode_atts( array(
        'ed' => ''
    ), $atts );

    $ed = $atts['ed'];
    $end_time = strtotime($ed);
    $current_time = strtotime('now');

    // 处理无效日期的情况
    if ($end_time === false) {
        return 'Invalid date format.';
    }

    $difference = $end_time - $current_time;
    $output1 = floor($difference / (60*60*24)) . ' days left';
    $output2 = "sorry - all gone. But keep checking back. Every so often, something does end up here!";

    // 正确逻辑：结束时间晚于当前时间则返回剩余天数，否则返回已结束提示
    if ($difference > 0) {
        return $output1;
    } else {
        return $output2;
    }
}
add_shortcode( 'countdown', 'calculate_time' );

关键修改说明 #

• 用shortcode_atts()为参数设置默认值，防止未传入ed参数时触发报错。
• 将判断条件改为$difference > 0，直接比较结束时间与当前时间的差值，逻辑更直观准确。
• 添加了strtotime()返回false的处理分支，避免无效日期导致的异常。
• 修正变量名拼写错误，提升代码可读性。

内容的提问来源于stack exchange，提问作者Richard Pallaziol WOC