Laravel项目购物车首个商品无法增删 路由匹配异常

核心问题判断 #

报错本质是加购、移除购物车的POST请求，没有匹配到对应的POST路由，反而命中了只允许GET请求的其他路由，才会返回405方法不允许错误。你把POST路由改成Route::match支持GET后不报错但无法加购，就是因为请求以GET方式走到了控制器，拿不到POST提交的参数，自然无法执行加购逻辑。

修复步骤 #

1. 先清新环境缓存，调整路由顺序 #

Laravel路由是从上到下匹配的，只要命中符合规则的路由就不会继续往下遍历，新电脑拉代码后首先执行以下命令清除所有旧缓存：

php artisan route:clear
php artisan config:clear
php artisan cache:clear
php artisan view:clear

然后检查路由文件，把所有固定路径的路由全部移到同前缀带动态参数的路由前面，尤其是/products/addproduct、/products/removeproduct这两个POST路由，必须放在/products/details/{product}这类带参数的路由前面，如果你用了Route::resource生成资源路由，也要把自定义的静态路由放在资源路由之前。
正确的路由顺序示例：

// 固定路径路由放最前面
Route::post('/products/addproduct', [OrderController::class, 'addToCart'])->name('addtocart')->middleware('checklogin');
Route::post('/products/removeproduct', [OrderController::class, 'removeFromCart'])->name('removefromcart')->middleware('checklogin');
// 带动态参数的路由放最后
Route::get('/products/details/{product}', [ProductDetailController::class, 'getProductPage'])->name('productdetail')->middleware('checklogin');

注意：不要用Route::match给写操作路由开放GET方法，既会导致参数拿不到，还会漏掉CSRF防护、造成参数泄露。

2. 修复模板HTML结构问题 #

只有首个条目报错，大概率是模板DOM解析异常导致的：

• 先检查商品列表、购物车页模板，在循环输出条目之前有没有未闭合的<form>标签，HTML不允许表单嵌套，外层如果有没关的form标签，浏览器会直接忽略内层第一个form，导致首个条目的提交按钮走到外层form的地址，触发方法不允许报错。
• 不要在<form>标签内部套<a>标签包裹表单元素，你现在的写法把商品卡片的大部分内容包在form里的a标签中，很容易导致浏览器解析DOM错乱，把表单和链接的结构搞混。把跳转链接和表单分开写，商品详情链接不要用相对路径products/details/xxx，必须用路由助手生成全路径，避免在购物车等其他页面点击跳转出错。
  调整后的表单结构参考：

<div class="card h-100 mx-2" style="width: 18rem;margin-bottom: -1em">
    {{-- 跳转链接单独放，不嵌套表单 --}}
    <a href="{{ route('productdetail', $product->id) }}" style="text-decoration: none; color: inherit;">
        <div class="card-body d-flex flex-column">
            <h5 class="card-title">{{$product->name}}</h5>
            <p class="card-text">{{$product->description}}</p>
        </div>
    </a>
    {{-- 表单独立，不嵌套a标签 --}}
    <form action="{{ route('addtocart') }}" method="post">
        @csrf
        <input type="hidden" name="productId" value="{{$product->id}}">
        <ul class="list-group list-group-flush">
            <li class="list-group-item">
                @if(count($product->categories) < 1)
                    No relevant categories found.
                @else
                    @foreach ($product->categories as $category)
                        {{$category->name}}
                    @endforeach
                @endif
            </li>
        </ul>
        <div class="card-footer mt-auto">
            <div class="input-group mb-3">
                <input class="form-control" aria-describedby="basic-addon1" type="number" name="quantity" value="1" min="1" max="{{$product->quantity}}">
                <div class="input-group-append">
                    <input class="btn btn-outline-secondary" type="submit" value="Buy">
                </div>
            </div>
        </div>
    </form>
</div>

3. 修复控制器逻辑bug #

你现在的购物车移除方法有逻辑错误，循环遍历商品时最终只会取到最后一个商品的库存数，回滚库存时会出错，改成直接查找对应商品的购买数量即可：

public function removeFromCart(Request $request){
    $orderInfo = $this->getOrderInfo();
    $targetProduct = $orderInfo->products()->find($request->post('delete'));
    $orderQuantity = $targetProduct ? $targetProduct->pivot->order_quantity : 0;
    $orderInfo->products()->detach($request->post('delete'));
    Product::where('id', $request->post('delete'))->increment('quantity', $orderQuantity);
    return redirect(route('order'));
}

加购方法也建议加商品非空判断，避免查不到商品时报错：

public function addToCart(Request $request){
    $orderInfo = $this->getOrderInfo();
    if($orderInfo === null){
        $orderInfo = Order::create([
            'customer_id' => Auth::user()->id,
            'order_status' => 0
        ]);
    }
    $product = Product::find($request->post('productId'));
    // 加商品存在性和库存判断
    if(!$product || $product->quantity < $request->post('quantity')){
        return back()->with('error', '商品库存不足');
    }
    if($orderInfo->products->contains($request->post('productId'))){
        $orderInfo->products()->increment('order_quantity', $request->post('quantity'));
    }else{
        $orderInfo->products()->attach($request->post('productId'), ['order_quantity' => $request->post('quantity')]);
    }
    $product->decrement('quantity', $request->post('quantity'));
    return redirect(route('product'));
}

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