Question

我有一个golang api应用程序。我已经定义了一组路由和处理程序。但是，mux路由器只返回最后一条路由。

当我请求/api/info时，我在记录中得到了这个：

9:0:38 app | 2018/02/05 09:00:38 GET /api/info Users Create 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，它会尝试向/users/create拨打电话。但是，如果我删除第二个路由，它将正确路由到InfoShow处理程序。

为什么mux会覆盖第一条路线？我很确定

有问题

HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
    logRoute(setJSONHeader(route.HandlerFunc), route.Name)(context, w, r)
})

但我不确定为什么会导致它映射到第一条路线。

想法？

Answer 1

阅读你的代码和gorilla / mux，我想我知道这个问题。你在函数文字中使用for循环变量route，特别是它的字段HanderFunc，但由于函数文字的工作方式，该字段的值直到调用函数文字。在Go中，范围循环中的第二个变量在每次迭代时都是重用，而不是重新创建，因此在for循环之后，如果它仍然在任何范围内（比如你的函数文字），那么将包含 last 循环迭代的值。这是我的意思的一个例子：

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实际上是调用（由funcs循环）。要解决此问题，您可以通过重新声明在循环迭代的范围内（但在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)
        })
}

Golang Api仅匹配最后路线

1 个答案: