我有一个Go语言api应用程序。我已经定义了一组路由和处理程序。然而，mux路由器只返回最后一条路由。
当我请求/api/info时，我在日志记录中得到了这个：
9:0:38 app|2018/02/05 09:00:38 GET /api/info用户创建308.132µs
为什么路由到错误的路由？
路由包：
// NewRouter establishes the root application router
func NewRouter(context *config.ApplicationContext, routes Routes, notFoundHandler http.HandlerFunc) *mux.Router {
router := mux.NewRouter()
router.NotFoundHandler = notFoundHandler
for _, route := range routes {
router.
PathPrefix("/api").
Methods(route.Method).
Path(route.Pattern).
Name(route.Name).
// TODO: fix HandlerFunc. Right now, it is overriding previous routes and setting a single handler for all
// this means that the last route is the only router with a handler
HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
logRoute(setJSONHeader(route.HandlerFunc), route.Name)(context, w, r)
})
}
return router
}
func logRoute(inner ContextHandlerFunc, name string) ContextHandlerFunc {
return func(c *config.ApplicationContext, w http.ResponseWriter, r *http.Request) {
start := time.Now()
inner(c, w, r)
log.Printf(
"%s\t%s\t%s\t%s",
r.Method,
r.RequestURI,
name,
time.Since(start),
)
}
}
func setJSONHeader(inner ContextHandlerFunc) ContextHandlerFunc {
return func(c *config.ApplicationContext, w http.ResponseWriter, r *http.Request) {
w.Header().Set("Content-Type", "application/json")
inner(c, w, r)
}
}
主包装：
var context = config.ApplicationContext{
Database: database.NewDatabase().Store,
}
var routes = router.Routes{
router.Route{"Info", "GET", "/info", handlers.InfoShow},
router.Route{"Users Create", "POST", "/users/create", handlers.UsersCreate},
}
func main() {
notFoundHandler := handlers.Errors404
router := router.NewRouter(&context, routes, notFoundHandler)
port := os.Getenv("PORT")
log.Fatal(http.ListenAndServe(":"+port, router))
}
如果我访问/api/info，它将尝试调用POST到/user/create。但是，如果我删除第二个路由，它将正确路由到InfoShow处理程序。
-为什么Mux要覆盖第一条路线？-我很肯定有问题
HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
logRoute(setJSONHeader(route.HandlerFunc), route.Name)(context, w, r)
})
但我不知道为什么这会导致它映射到第一条路线上。
想法？
通读你的代码和gorilla/mux，我想我知道这个问题。你在函数文字中使用了for循环变量路由，特别是它的字段HanderFunc，但是由于函数文字的工作方式，在调用that函数文字之前不会评估该字段的值。在Go中，范围循环中的第二个变量在每次迭代中被重用，而不是重新创建，因此在for循环之后，如果它仍然在任何范围内（例如您的函数文字），它将包含上次循环迭代的值。以下是我的意思的一个例子：
https://play.golang.org/p/Xx62tuwhtgG
package main
import (
"fmt"
)
func main() {
var funcs []func()
ints := []int{1, 2, 3, 4, 5}
// How you're doing it
for i, a := range ints {
fmt.Printf("Loop i: %v, a: %v\n", i, a)
funcs = append(funcs, func() {
fmt.Printf("Lambda i: %v, a: %v\n", i, a)
})
}
for _, f := range funcs {
f()
}
fmt.Println("-------------")
// How you *should* do it
funcs = nil
for i, a := range ints {
i := i
a := a
fmt.Printf("Loop i: %v, a: %v\n", i, a)
funcs = append(funcs, func() {
fmt.Printf("Lambda i: %v, a: %v\n", i, a)
})
}
for _, f := range funcs {
f()
}
}
在第一个示例中，i和a在每次循环迭代中都被重用，并且在lambda（函数字面量）中不会评估它们的值，直到lambda被实际调用（由funs循环）。为了解决这个问题，您可以通过在循环迭代的范围内（但在lambda的范围之外）重新声明a和i来隐藏它们。这为每次迭代制作了一个单独的副本，以避免重用同一变量的问题。
特别是对于您的代码，如果您将代码更改为以下内容，它应该可以工作：
for _, route := range routes {
route := route // make a copy of the route for use in the lambda
// or alternatively, make scoped vars for the name and handler func
router.
PathPrefix("/api").
Methods(route.Method).
Path(route.Pattern).
Name(route.Name).
// TODO: fix HandlerFunc. Right now, it is overriding previous routes and setting a single handler for all
// this means that the last route is the only router with a handler
HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
logRoute(setJSONHeader(route.HandlerFunc), route.Name)(context, w, r)
})
}
