Laravel中destroy方法报错：Call to a member function delete() on bool求解决

核心错误原因 #

Call to a member function delete() on bool 这个错误的本质是你尝试对一个布尔值调用模型的delete()方法，根源出在这几点：

• Descuento::with('ventas')->where('id',$id)->first() 当数据库中找不到对应id的折扣记录时，first()会返回false（布尔值），而非模型实例。
• 你错误地对单个模型实例（甚至是布尔值）执行了foreach循环，遍历布尔值时$descuentos会变成布尔值false，此时调用delete()自然触发报错。
• 额外问题：关联检查逻辑有误——如果ventas是hasMany关联，它是一个集合而非单个对象，$descuentos->ventas->descuento_id这种写法会触发另一个错误。

修复后的代码 #

public function destroy($id){
    // 查询折扣记录，找不到直接返回404
    $descuento = Descuento::with('ventas')->where('id',$id)->firstOrFail();

    // 检查该折扣是否已被订单使用
    if($descuento->ventas->isNotEmpty()){
        return response()->json(['message' => 'No es posible eliminar el descuento ya que fue utilizado'], 500);
    }

    // 未被使用则删除
    $descuento->delete();
    
    return response()->json(['message' => 'Descuento eliminado correctamente'], 200);
}

关键修正点 #

• 用firstOrFail()替代first()：找不到记录时直接抛出404异常，避免后续处理布尔值的问题。
• 移除多余的foreach：first()返回单个模型实例，无需循环遍历。
• 正确检查关联：用$descuento->ventas->isNotEmpty()判断是否有关联订单，这是Laravel集合的标准可靠用法。

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