Почему, при выносе пула соединений на уровень файла, паникует?

Question

[Vadim Rublev]

Вот так запрос к БД работает нормально:

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

func DBRout() {
    // Пул подключений.
    db, err_DB := sql.Open("mysql", ConnectDB)
	    if err_DB != nil {
		    fmt.Println("ERROR creating database connection pool:", err_DB)
		}
		defer db.Close()
	
        db_ss.SetMaxOpenConns(25)
        db_ss.SetMaxIdleConns(2)
        db_ss.SetConnMaxLifetime(time.Minute)
		
	// Запрос.
	var err_DBQuery = db.QueryRow("SELECT COUNT(*) FROM `TabTab` WHERE `Col_User` = ? AND ColumnID = ?, 3, UName).Scan(&row_search)
	    if err_DBQuery != nil {
		fmt.Println("Query error:", err_DBQuery)
	        return
	    }
}

func main() {
    var mux = http.NewServeMux()
	mux.HandleFunc("/routDB/", DBRout)
	var serv = &http.Server {
	    ...параметры...
	}
	log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey))
}

А когда выношу пул соединений на уровень файла - паникует.

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

var db *sql.DB

func DBRout() {		
	// Запрос.
	var err_DBQuery = db.QueryRow("SELECT COUNT(*) FROM `TabTab` WHERE `Col_User` = ? AND ColumnID = ?, 3, UName).Scan(&row_search)
	    if err_DBQuery != nil {
			fmt.Println("Query error:", err_DBQuery)
		    return
		}
}

func main() {
    // Пул подключений.
    db, err_DB := sql.Open("mysql", ConnectDB)
	if err_DB != nil {
	    fmt.Println("ERROR creating database connection pool:", err_DB)
	}
	defer db.Close()
	
        db_ss.SetMaxOpenConns(25)
        db_ss.SetMaxIdleConns(2)
        db_ss.SetConnMaxLifetime(time.Minute)

    var mux = http.NewServeMux()
	mux.HandleFunc("/routDB/", DBRout)
	var serv = &http.Server {
	    ...параметры...
	}
	log.Fatal(serv.ListenAndServeTLS(TLScert, TLSkey))
}

Выводит в консоль:
2021/12/27 04:02:13 http2: panic serving 127.0.0.1:59951: runtime error: invalid memory address or nil pointer dereference
goroutine 136 [running]: ...

Ну не фигня ли?? )

Answer 1

[Roman K]

Ну не фигня ли?? )

Нет, ведь у тебя две разных переменные с одним именем — db

Очень удобное правило

Когда что-то работает не так, как ты ожидаешь, вини в первую очередь себя, компилятор и рантайм на 99.99999% прав, крайне маловероятно, что с опытом меньше пары лет в языке, ты сможешь найти в нём баг.

Да и вообще, никогда не создавай глобальных переменных. В продуктовом коде они не нужны в 100% случаев

Comments

[Vadim Rublev]

Я никого кроме себя и не виню.
И конечно я не могу найти тут баг - иначе бы не задал вопрос.

[Roman K]

Vadim Rublev, выражением db, err := ты создал переменную db, а та db, что объявлена глобально, просто так и осталась быть nil

Лучшее решение - в функцию передавать db аргументом, а от глобальной переменной избавиться вообще

[Vadim Rublev]

Понял, поправил. Спасибо!

[Vadim Rublev]

...Поправил, да недоправил.. - как ненужные аргументы заглушить, или как правильно передать db в аргументе?
mux.HandleFunc("/routDB/", DBRout(db))

Просто у меня функция маленько вот такая (уже с принимаемой переменной-пулом_db):

func DBRout(w http.ResponseWriter, r *http.Request, db *sql.DB) { ...

Он хочет при запуске все три аргумента, или вообще без них.
Недогугливаю... Научно потыкать не вышло. (

[Roman K]

Vadim Rublev, например так, очень типичная практика — создавать функциями функции

package main

import (
	"database/sql"
	"fmt"
	"log"
	"net/http"
)

func makeRoute(db *sql.DB) http.HandlerFunc {
	return func(w http.ResponseWriter, r *http.Request) {
		row := db.QueryRow("SELECT 1")
		fmt.Println("do something with", row)
		_, err := w.Write([]byte("ok"))
		if err != nil {
			log.Println(err)
		}
	}
}

func main() {
	db, err := sql.Open("sqlite", "db.sqlite3")
	if err != nil {
		panic(err)
	}
	defer db.Close()
	mux := http.NewServeMux()
	mux.HandleFunc("/my-very-own-route/", makeRoute(db))
	log.Fatal(http.ListenAndServe(":3000", mux))
}

[Vadim Rublev]

Мудрёно! Буду знать. Спасибо!

[Roman K]

Vadim Rublev, можно сделать структурой, так может быть проще писать и понимать код. Хотя в целом это мало что меняет

package main

import (
	"database/sql"
	"fmt"
	"log"
	"net/http"
)

type Handlers struct {
	db *sql.DB
}

func NewHandlers(db *sql.DB) *Handlers {
	return &Handlers{db: db}
}

func (h *Handlers) MyRoute(w http.ResponseWriter, r *http.Request) {
	row := h.db.QueryRow("SELECT 1")
	fmt.Println("do something with", row)
	_, err := w.Write([]byte("ok"))
	if err != nil {
		log.Println(err)
	}
}

func main() {
	db, err := sql.Open("sqlite", "db.sqlite3")
	if err != nil {
		panic(err)
	}
	defer db.Close()
	handlers := NewHandlers(db)
	mux := http.NewServeMux()
	mux.HandleFunc("/my-very-own-route/", handlers.MyRoute)
	log.Fatal(http.ListenAndServe(":3000", mux))
}

[Vadim Rublev]

Понятно.

[Vadim Rublev]

Roman Kitaev Подскажите пожалуйста насчет return.
Если у нас уже есть return:

func makeRoute(db *sql.DB) http.HandlerFunc {
    return func(w http.ResponseWriter, r *http.Request) {
        ...

то внутри анонимной функции уже не надо писать return (например, чтобы завершить работу после ошибки, или возвращая значение в конце функции)? Или внутри функции как обычно ставим return где надо?
Ведь получается, что если внутри анонимной функции завершаем return'ом, то что первый-"главный" return вернет? Ошибку? Или как?