diff --git a/app/api/controller/CartController.php b/app/api/controller/CartController.php
index f103554..49f4083 100644
--- a/app/api/controller/CartController.php
+++ b/app/api/controller/CartController.php
@@ -46,7 +46,7 @@ class CartController extends BaseApiController
         if ($res === true) {
             return $this->success('加入成功');
         }
-        $this->fail($res);
+        return $this->fail($res);
     }
 
 
diff --git a/app/api/controller/GoodsController.php b/app/api/controller/GoodsController.php
index a80951a..08db1ef 100644
--- a/app/api/controller/GoodsController.php
+++ b/app/api/controller/GoodsController.php
@@ -40,11 +40,11 @@ class GoodsController extends BaseApiController
      */
     public function getGoodsDetail(){
         $id = $this->request->get('id');
-        $goods = GoodsLogic::getGoodsDetail($this->user_id,$id);
+        $goods = GoodsLogic::getGoodsDetail($this->userId,$id);
         if($goods){
-            $this->_success('获取成功',$goods);
+            return $this->success('获取成功',$goods);
         }
-        $this->_error('商品不存在',[],0,0);
+        return $this->fail('商品不存在',[],0,0);
     }
 
     /**