Laravel中使用Scope校验限时折扣时间异常问题排查

你的怀疑方向是对的，但还有几个关键问题需要一起修正，才能解决折扣不恢复、计时器负数以及正确控制计时器显示的问题：

1. 模型Scope的使用错误（核心问题） #

你定义的scopeValidDiscount是用于批量查询处于有效折扣期的商品集合，但在控制器中，你已经获取了单个$product模型，再调用->ValidDiscount()->get()会返回一个包含符合条件商品的集合（即使当前商品不在折扣期，也会返回空集合），而视图里@if($validdiscount)会把空集合视为true（因为Laravel集合是对象，布尔判断永远为真），这就导致计时器一直显示，哪怕折扣过期。

修正方案： #

给Product模型添加一个判断单个商品是否处于有效折扣期的方法，替代直接用Scope查询单个模型：

// Product模型中新增方法
public function hasValidDiscount()
{
    // 先判断折扣日期是否为空，满足第二个不显示条件
    if (is_null($this->start_discounted) || is_null($this->end_discounted)) {
        return false;
    }
    // 判断当前时间是否在折扣区间内
    return Carbon::now()->between($this->start_discounted, $this->end_discounted);
}

然后在控制器中，直接用这个方法判断，同时调整日期获取逻辑：

public function product($slug){
    $product = Product::where('slug', $slug)->firstOrFail();
    $hasValidDiscount = $product->hasValidDiscount();
    
    // 只有当折扣有效时才获取日期
    $end = null;
    if ($hasValidDiscount) {
        $end = $product->end_discounted->format('M d, Y');
    }

    return view('your-product-view', compact('product', 'hasValidDiscount', 'end'));
}

2. 视图中计时器的显示判断修正 #

现在视图里直接用$hasValidDiscount这个布尔值判断，就能完美满足你要求的两个不显示条件：

• 折扣日期为空：hasValidDiscount返回false
• 折扣日期过期：hasValidDiscount返回false

修正后的Blade代码：

@if($hasValidDiscount)
<div class="deals_timer row align-items-center">
    <div class="col-md-6 deals_timer_title_container">
        <div class="deals_timer_title">{{__('store.hurryup')}}</div>
        <div class="deals_timer_subtitle">{{__('store.endsin')}}</div>
    </div>
    <div class="col-md-6 deals_timer_content ml-auto">
        <div class="deals_timer_box clearfix" data-target-time="{{$end}}">
            <div class="text-center deals_timer_unit">
                <div id="deals_timer3_hr" class="deals_timer_hr"></div>
                <span>{{__('store.hours')}}</span>
            </div>
            <div class="text-center deals_timer_unit">
                <div id="deals_timer3_min" class="deals_timer_min"></div>
                <span>{{__('store.mins')}}</span>
            </div>
            <div class="text-center deals_timer_unit">
                <div id="deals_timer3_sec" class="deals_timer_sec"></div>
                <span>{{__('store.secs')}}</span>
            </div>
        </div>
    </div>
</div>
@endif

3. 计时器脚本添加负数判断，避免显示异常 #

即使服务器端做了判断，也可能因为客户端时间差或者页面缓存导致计时器出现负数，所以在JS里添加判断逻辑，当剩余时间小于等于0时，自动清除计时器并隐藏模块：

<script>
$(document).ready(function(){
    if($('.deals_timer_box').length) {
        var timers = $('.deals_timer_box');
        timers.each(function() {
            var timer = $(this);
            var targetTime;
            var target_date;
            if(timer.data('target-time') !== "") {
                targetTime = timer.data('target-time');
                target_date = new Date(targetTime).getTime();
            } else {
                var date = new Date();
                date.setDate(date.getDate() + 2);
                target_date = date.getTime();
            }
            var h = timer.find('.deals_timer_hr');
            var m = timer.find('.deals_timer_min');
            var s = timer.find('.deals_timer_sec');
            
            var timerInterval = setInterval(function () {
                var current_date = new Date().getTime();
                var seconds_left = (target_date - current_date) / 1000;
                
                // 新增：剩余时间<=0时，清除计时器并隐藏整个折扣模块
                if (seconds_left <= 0) {
                    clearInterval(timerInterval);
                    timer.closest('.deals_timer').hide();
                    return;
                }

                var days, hours, minutes, seconds;
                days = parseInt(seconds_left / 86400);
                seconds_left = seconds_left % 86400;
                hours = parseInt(seconds_left / 3600);
                hours = hours + days * 24;
                seconds_left = seconds_left % 3600;
                minutes = parseInt(seconds_left / 60);
                seconds = parseInt(seconds_left % 60);
                
                // 补零逻辑保持不变
                if(hours.toString().length < 2) {
                    hours = "0" + hours;
                }
                if(minutes.toString().length < 2) {
                    minutes = "0" + minutes;
                }
                if(seconds.toString().length < 2) {
                    seconds = "0" + seconds;
                }
                
                h.text(hours);
                m.text(minutes);
                s.text(seconds);
            }, 1000);
        });
    }
});
</script>

4. 商品原价恢复的逻辑补充 #

针对折扣到期后商品未恢复原价的问题，需要在视图中根据$hasValidDiscount来切换显示折扣价或原价：

<!-- 示例：商品价格显示逻辑 -->
@if($hasValidDiscount)
    <span class="discount-price text-danger">${{$product->discounted_price}}</span>
    <span class="original-price text-muted ml-2"><del>${{$product->original_price}}</del></span>
@else
    <span class="original-price">${{$product->original_price}}</span>
@endif

这样一套修正下来，就能彻底解决你遇到的所有问题：

• 折扣到期后自动隐藏计时器、恢复显示原价
• 折扣日期为空时不显示计时器
• 避免计时器出现负数的异常情况

内容的提问来源于stack exchange，提问作者mafortis