Go - 变量初始化且非零,但其他函数为

时间:2016-04-01 02:26:16

标签: go initialization global-variables

编写与我当前问题相符的标题有点困难。 我有一个main()函数,它使用另一个包(database_sql)中的函数。此函数初始化全局变量sql.DB *。初始化之后,变量不是nil,但对于其他函数,这个变量仍然是nil。让我们看看下面的代码!

main.go

package main

import (
    "net/http"
    db "./database_sql"
    router "./router"
)

func main() {

    db.Init_SQL();

    router.Init_routes()

    http.ListenAndServe(":8080", router.GetRouter())

}

db.go

package database_sql

import (
    "fmt"
    "database/sql"
    _ "github.com/go-sql-driver/mysql"
)

var DB *sql.DB // global variable to share it between main and the HTTP handler                                                                                                                              
//var dbi *databaseinfos                                                                                                                                                                                     

func Init_SQL() {
    dbi := databaseinfos{
            user: "something",
            password: "something",
            name: "something",
            address: "something",
            port: "2323",
            url: ""}
    dbi.url = (dbi.user + ":" + dbi.password + "@tcp(" + dbi.address + ":" + dbi.port + ")/" + dbi.name)

    db_init_var, err := sql.Open("mysql", dbi.url)
    if err != nil {
            fmt.Println("Error on initializing database connection: %s", err.Error())
    }

    err = db_init_var.Ping()
    if err != nil {
            fmt.Println("Error on opening database connection: %s", err.Error())
    }

    // Here, as you can test, my variable is initialized
    if err == nil {
            DB = db_init_var
            if DB == nil {
                    fmt.Println("NIL DB !!")
            }
            if db_init_var == nil {
                    fmt.Println("NIL db_init_var !!")
            }
            fmt.Println(dbi.url)
    }
}

现在,一切都好!我现在将使用用户名/密码测试http://xxxxxxx/login。一切正常,现在,是时候在数据库中用我上面声明的DB变量发出请求了。

request_user.go

package database_sql

import (
    "encoding/json"
    "fmt"
    "net/http"
    m "models"
)

func CanLogin(user m.User) (bool, m.User, m.StatusBack) {

    // Prepare statement for reading data                                                                                                                                                                
    stmtOut, err := DB.Prepare("SELECT username, password, token FROM users WHERE username = ? AND password = ?")
    if err != nil {
            return false, user, m.StatusBack{ToString: err.Error(), IdStatus: http.StatusInternalServerError}
    }
defer stmtOut.Close()

    // Query the user                                                                                                                                                                                    
err = stmtOut.QueryRow(user.Username, user.Password).Scan(&user.Username, &user.Password, &user.UUID) // WHERE user is registered                                                                    
    if err != nil {
            return false, user, m.StatusBack{ToString: "User not found.", IdStatus: http.StatusNotFound}
    } else {

            j, err := json.Marshal(user)
            if err == nil {
                    return true, user, m.StatusBack{ToString: string(j), IdStatus: http.StatusOK}
            } else {
                    return false, user, m.StatusBack{ToString: err.Error(), IdStatus: http.StatusInternalServerError}
    }
    }

}

但是,当我提出我的sql请求时,以下行说明DB是零。 stmtOut,错误:= DB.Prepare(" SELECT用户名,密码,令牌FROM用户WHERE用户名=?AND密码=?")

我做了很多测试,例如尝试用' ='来初始化它。并且没有':='。我试过,如果DB == nil'声明和DB总是零。为什么?我在初始化路由器之前初始化我的数据库var,所以从技术上讲,当我的路由器给一个函数重定向时,这个函数不会使用数据库var(DB)nil,isn'

感谢您的回答!

编辑2 我重新点了你的答案,然后,我编辑了我的代码就像你问了一些补充。在init_SQL之后,DB不是没有!

main.go

package main

import (
    "fmt"
    "net/http"
    db "./database_sql"
    router "./router"
)

func main() {

    if db.Init_SQL() == true {

            if db.DB == nil {
                    fmt.Println("db.DB is nil !")
            }

            if router.Init_routes() == true {

                    http.ListenAndServe(":8080", router.GetRouter())

            }
    }
}

所以,现在我的2个init函数在成功时返回true,在失败时返回false。我认为这可能是一个异步问题,我就是这样做等待每个函数的结束继续我的步骤'

1 个答案:

答案 0 :(得分:1)

首先,导出像这样的全局变量并不是偶像的。复杂类型应该由运行时初始化(您使用main()执行),但是从运行时全局保留以控制处置。

您也错过了db.Close()

我相信几年前第一次使用MySQL时,我遇到了与你的模式相同的问题。创建局部范围指针的方式很奇怪,然后将其分配给全局变量。通常最好将其直接分配给全局变量。但我认为核心问题是您需要将指针指定给指针。

我使用的模式是将可测试数据库/ sql *DB保持在初始化状态的全局状态。为什么在车轮工作时重新发明轮子:

package main

import ( 
  ...
  "database/sql"

  "mypackage-that-inits-db/database"
)

var db *sql.DB

find main() {

  var err error
  db, err = database.Init_SQL(...params)
  if err != nil {
    ...
  }
  defer db.Close()

  InitUsers(db)
  InitStats(db)
  ...

  http.ListenAndServe(...)

  // now inject db global dependency in other code
  //

  globalStats := NewStatsEngine(db)
  globalStats.RecordUpdateToSomething(...)

  users := getUsers(db)

   ... etc

}

这通常是您在其他代码中看到的模式。

请注意对延迟和关闭的控制,它在调用者所在的位置。

无法轻松测试

另请注意,现在,通过创建模拟sql.DB对象并将其注入NewStatsEngine,getUsers等,您可以轻松测试其他包中的代码,而无需再次模拟全局变量,测试,拆除和设置因为Go 1.5,测试可以同时运行,所以你甚至需要在全局数据库变量周围放置一个mutex.Lock,如果你离开你的包那样的话。切换到IoC模式,比如我在上面的代码中演示的依赖注入,使得测试(和调试)代码变得更加容易。