修复返回状态

handler/http_handler.go

@@ -20,19 +20,24 @@ func SaveGeometry(ctx context.Context, c *app.RequestContext) {
 	err := c.BindAndValidate(&req)
 	if err != nil {
 		c.JSON(consts.StatusBadRequest, utils.H{"message": "geojson required"})
+		return
 	}
 	if req.Opt == 1 || req.Opt == 2 {
 		resp, err := geo.SaveRoute(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10001, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	} else if req.Opt == 3 {
 		resp, err := geo.SaveOptArea(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10002, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	}
 
 }
@@ -42,71 +47,93 @@ func UpdateGeometry(ctx context.Context, c *app.RequestContext) {
 	err := c.BindAndValidate(&req)
 	if err != nil {
 		c.JSON(consts.StatusBadRequest, utils.H{"message": "geojson required"})
+		return
 	}
 	if req.Opt == 1 || req.Opt == 2 {
 		resp, err := geo.UpdateRoute(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10003, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	} else if req.Opt == 3 {
 		resp, err := geo.UpdateOptArea(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10004, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	}
 
 }
 
 func GetGeometry(ctx context.Context, c *app.RequestContext) {
-
-	id, err := strconv.Atoi(c.Params.ByName("id"))
+	check_str := ""
+	id, err := strconv.Atoi(c.Query("id"))
 	if err != nil {
-		c.JSON(consts.StatusBadRequest, utils.H{"message": "id required"})
+		check_str += "param id required;"
 	}
-	opt, err := strconv.Atoi(c.Params.ByName("opt"))
+	opt, err := strconv.Atoi(c.Query("opt"))
 	if err != nil {
-		c.JSON(consts.StatusBadRequest, utils.H{"message": "opt required"})
+		check_str += "param opt required;"
+	}
+	if check_str != "" {
+		c.JSON(consts.StatusBadRequest, utils.H{"message": check_str})
+		return
 	}
 
 	if opt == 1 || opt == 2 {
 		resp, err := geo.GetRoute(id, opt)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10005, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	} else if opt == 3 {
 		resp, err := geo.GetOptArea(id, opt)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10006, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	}
 }
 
 func DeleteGeometry(ctx context.Context, c *app.RequestContext) {
-	id, err := strconv.Atoi(c.Params.ByName("id"))
+	check_str := ""
+	id, err := strconv.Atoi(c.Query("id"))
 	if err != nil {
-		c.JSON(consts.StatusBadRequest, utils.H{"message": "id required"})
+		check_str += "param id required;"
 	}
-	opt, err := strconv.Atoi(c.Params.ByName("opt"))
+	opt, err := strconv.Atoi(c.Query("opt"))
 	if err != nil {
-		c.JSON(consts.StatusBadRequest, utils.H{"message": "opt required"})
+		check_str += "param opt required;"
+	}
+	if check_str != "" {
+		c.JSON(consts.StatusBadRequest, utils.H{"message": check_str})
+		return
 	}
 
 	if opt == 1 || opt == 2 {
 		resp, err := geo.DeleteRoute(id, opt)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10007, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	} else if opt == 3 {
 		resp, err := geo.DeleteOptArea(id, opt)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10008, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	}
 }
 
@@ -115,19 +142,24 @@ func FindRelatedGeometry(ctx context.Context, c *app.RequestContext) {
 	err := c.BindAndValidate(&req)
 	if err != nil {
 		c.JSON(consts.StatusBadRequest, utils.H{"message": "geojson required"})
+		return
 	}
 	if req.Opt == 1 || req.Opt == 2 {
 		resp, err := geo.GetRelatedRoute(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10009, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	} else if req.Opt == 3 {
 		resp, err := geo.GetRelatedOptArea(req)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10010, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, resp)
+		return
 	}
 
 }
@@ -137,22 +169,28 @@ func CheckTwoOptAreaIsCross(ctx context.Context, c *app.RequestContext) {
 	err := c.BindAndValidate(&req)
 	if err != nil {
 		c.JSON(consts.StatusBadRequest, utils.H{"message": "geojson required"})
+		return
 	}
 	if len(req) != 2 {
 		c.JSON(consts.StatusBadRequest, utils.H{"message": "need tow geojson body"})
+		return
 	}
 	if req[0].ID == 0 || req[1].ID == 0 {
 		res, err := geo.CheckAreasCross(req[0], req[1])
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10011, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, res)
+		return
 	} else {
 		res, err := geo.CheckOptAreasCrossById(req[0].ID, req[1].ID)
 		if err != nil {
-			c.JSON(consts.StatusInternalServerError, err.Error())
+			c.JSON(consts.StatusOK, utils.H{"code": 10012, "message": err.Error()})
+			return
 		}
 		c.JSON(consts.StatusOK, res)
+		return
 	}
 
 }