29

我目前正在将 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")
)

我初始化 db 会话并创建获取集合和文档值的变量，因此当我需要查询集合时，我使用该变量来创建它。

像那样 ：

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
    }
}

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

4

3 回答 3

65

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

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

于 2014-10-26T18:45:42.897 回答
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()
}
于 2016-11-02T17:31:10.473 回答
0

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

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))
        })
    }
于 2016-11-28T19:58:47.930 回答