维护 mgo 会话的最佳实践

Question

我目前正在将 mongodb 与 mgo lib 用于 Web 应用程序，但我不确定我使用它的方式是否是好的 ..

package db

import (
    "gopkg.in/mgo.v2"
)

const (
    MongoServerAddr = "192.168.0.104"
    RedisServerAddr = "192.168.0.104"
)

var (
    MongoSession, err = mgo.Dial(MongoServerAddr)

    MDB  = MongoSession.DB("message")
    MCol = MDB.C("new")
    MSav = MDB.C("save")

    UDB  = MongoSession.DB("account")
    UCol = UDB.C("user")
)

我初始化数据库会话并创建获取集合和文档值的变量， 所以当我需要查询一个集合时，我使用变量来制作它。

像这样：

func UserExist(username string) bool {
    user := Users{}
    err := db.UCol.Find(bson.M{"username": username}).One(&user)
    if err != nil {
        return false
    } else {
        return true
    }
}

那么有没有最佳实践或者这个很好..？ 谢谢

Answer 1

我建议不要使用这样的全局会话。相反，您可以创建一个负责所有数据库交互的类型。例如：

type DataStore struct {
    session *mgo.Session
}

func (ds *DataStore) ucol() *mgo.Collection { ... }

func (ds *DataStore) UserExist(user string) bool { ... }

这种设计有很多好处。一个重要的一点是它允许您同时运行多个会话，因此，例如，如果您有一个 http 处理程序，您可以创建一个由独立会话支持的本地会话，仅针对该一个请求：

func (s *WebSite) dataStore() *DataStore {
    return &DataStore{s.session.Copy()}
}    

func (s *WebSite) HandleRequest(...) {
    ds := s.dataStore()
    defer ds.Close()
    ...
}

mgo 驱动程序在这种情况下表现良好，因为会话在内部缓存并重用/维护。每个会话在使用时也将由独立的套接字支持，并且可以配置独立的设置，并且还将具有独立的错误处理。如果您使用单个全局会话，这些是您最终必须处理的问题。

Answer 2

虽然没有直接回答您的问题，但关于 mgo 会话检查，您必须使用 defer/recover，因为 mgo 调用（甚至是 mgo.session.Ping）恐慌。据我所知，没有其他方法可以检查 mgo 会话状态 (mgo godocs)。您可以使用Gustavo Niemeyer 的建议并在您的DataStore 类型上添加一个方法。

func (d *DataStore) EnsureConnected() {
    defer func() {
        if r := recover(); r != nil {
            //Your reconnect logic here.
        }
    }()

    //Ping panics if session is closed. (see mgo.Session.Panic())  
    d.Ping()
}

Answer 3

使用 go 1.7，在 web 服务器上处理 mongo 会话最惯用的方法是使用新的标准库包 context 编写一个中间件，该中间件可以在调用请求上下文 Done() 时附加 defer session.Close() .所以你不需要记住关闭

AttachDeviceCollection = func(next http.Handler) http.Handler {
        return http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
            db, err := infra.Cloner()
            if err != nil {
                http.Error(w, err.Error(), http.StatusInternalServerError)
                return
            }
            collection, err := NewDeviceCollection(db)

            if err != nil {
                db.Session.Close()
                http.Error(w, err.Error(), http.StatusInternalServerError)
                return
            }
            ctx := context.WithValue(r.Context(), DeviceRepoKey, collection)
            go func() {
                select {
                case <-ctx.Done():
                    collection.Session.Close()
                }
            }()

            next.ServeHTTP(w, r.WithContext(ctx))
        })
    }