- refactor.

business/jd/controller/order.go

@@ -116,7 +116,10 @@ func (c *OrderController) OrderStatus(order *jdapi.JDOrderMsg) *jdapi.JDCallback
 						rec.OrderStatus = status
 						rec.OrderStatusTime = order.Timestamp
 						rec.Code = MsgNotHandledCode
-						db.Update(rec, "OrderStatus", "OrderStatusTime", "Code")
+						utils.CallFuncLogError(func() error {
+							_, err := db.Update(rec, "OrderStatus", "OrderStatusTime", "Code")
+							return err
+						}, globals.ErrStrAccessDB)
 						addOrderMsg(order)
 					}
 				} else {
@@ -183,10 +186,10 @@ func newOrder(order *jdapi.JDOrderMsg) error {
 			err = globals.FreshFoodAPI.NewJDOrder(rec)
 			if err == nil {
 				db := orm.NewOrm()
-				_, err = db.Update(rec, "Data", "Code", "Msg", "Success", "CityName", "OrderStatus", "OrderStatusTime")
-				if err != nil {
-					globals.SugarLogger.Errorf("update order error:%v", err)
-				}
+				utils.CallFuncLogError(func() error {
+					_, err := db.Update(rec, "Data", "Code", "Msg", "Success", "CityName", "OrderStatus", "OrderStatusTime")
+					return err
+				}, globals.ErrStrAccessDB)
 			} else {
 				globals.SugarLogger.Errorf("Error when calling NewJDOrder error:%v", err)
 			}
@@ -227,10 +230,9 @@ func normalOrderStatus(order *jdapi.JDOrderMsg) error {
 	}
 
 	rec.Code = "0"
-	_, err = db.Update(rec, "OrderStatus", "OrderStatusTime", "Code")
-	if err != nil {
-		globals.SugarLogger.Warnf("error when accessing db:%v", err)
-	}
-
+	utils.CallFuncLogError(func() error {
+		_, err := db.Update(rec, "OrderStatus", "OrderStatusTime", "Code")
+		return err
+	}, globals.ErrStrAccessDB)
 	return err
 }