Goのruntime error: invalid memory address or nil pointer dereference

Question

#runtime error: invalid memory address or nil pointer dereference
自作で簡単なAPIサーバーを作っているのですが,サーバーにリクエストを送るとTerminalに上のようなエラーが出てしまいます。解決できる方法を探しいます、よろしくお願いします。

##試したこと
GetAllBooks関数の

rows, err := db.Query("SELECT * FROM book;")
    if err != nil {
        log.Fatal(err)
    }

以下をコメントアウトしてリクエストを送っても、上のエラーが表示されるのですが上のコードもコメントアウトしたらブラウザーにHello worldが表示されます。なのでもしかしたら上のコードに問題があるかもしれません..

##全てのコード

package main

import (
	"database/sql"
	"encoding/json"
	"fmt"
	"github.com/go-sql-driver/mysql"
	"github.com/gorilla/mux"
	"log"
	"net/http"
	"os"
)

var db *sql.DB

type Book struct {
	ID    string `json:"id"`
	Title string `json:"title"`
	Page  int    `json:"page"`
	Price int    `json:"price"`
}

type Error struct {
	Message string `json:"message"`
}

func GetAllBooks(w http.ResponseWriter, r *http.Request){
	fmt.Fprintf(w, "Hello world")
	var error Error
	rows, err := db.Query("SELECT * FROM book;")
	if err != nil {
		log.Fatal(err)
	}
	var books []Book

	if err != nil {
		error.Message = "Cant not get data from database"
		w.Header().Set("Content-Type", "application/json")
		json.NewEncoder(w).Encode(error.Message)
		return
	}

	for rows.Next() {
		var book Book
		if err := rows.Scan(&book.ID, &book.Title, &book.Page, &book.Price); err != nil {
			error.Message = "Cant not get data from database"
			w.Header().Set("Content-Type", "application/json")
			json.NewEncoder(w).Encode(error.Message)
			return
		}
		books = append(books, book)
	}
	if err := rows.Err(); err != nil {
		error.Message = "Cant not get data from database"
		w.Header().Set("Content-Type", "application/json")
		json.NewEncoder(w).Encode(error.Message)
		return
	}
	w.Header().Set("Content-Type", "application/json")
	json.NewEncoder(w).Encode(books)
	defer rows.Close()
}

func main(){
	cfg := mysql.Config{
		User: os.Getenv("DBUSER"),
		Passwd: os.Getenv("DBPASS"),
		Net: "tcp",
		Addr: "127.0.0.1:3306",
		DBName: "bookAPI",
	}
	var err error
	db, err := sql.Open("mysql", cfg.FormatDSN())
	defer db.Close()
	if err != nil {
		log.Fatal(err)
	}
	pingErr := db.Ping()
	if pingErr != nil {
		log.Fatal(pingErr)
	}
	fmt.Println("Connected to database!")

	router := mux.NewRouter()
	router.HandleFunc("/books", GetAllBooks).Methods("GET")
	log.Fatal(http.ListenAndServe(":8000", router))
}

#エラー内容

Comments

[nobonobo]

エラーはどの行に対して出力されましたか？

[退会済みユーザー]

エラー内容を見ても何行目を見ればいいのかわからないです... エラー内容の画像を追加したのでもしよければ見てください

Answer 1

エラーはなにか？

エラーはログから /usr/.../go-book-api/cmd/main.go の30行目で発生しています。

どんなエラーかというと、nil であるポインタを参照している、というエラーです。invalid memory address or nil pointer dereference から分かります。

なぜ発生しているのか？

グローバル変数として main.go の14行目に

var db *sql.DB

を宣言していますが、これは初期化されていません。以下のように db 変数を初期化しているように見えますが、これは 14 行目とは別の変数として扱われています。Goの用語ではシャドーイングという仕組みによって起こっています。

func main(){
    // ...
    var err error
    db, err := sql.Open("mysql", cfg.FormatDSN())
    defer db.Close()
    if err != nil {
        log.Fatal(err)
    }
    // ...

正しくは以下のようにすれば、想定どおり db 変数を初期化でき、nil ポインタエラーは解消されるでしょう。: の有無の違いです。

func main(){
    // ...
    var err error
    db, err = sql.Open("mysql", cfg.FormatDSN())
    defer db.Close()
    if err != nil {
        log.Fatal(err)
    }
    // ...

Comments

[退会済みユーザー]

実際に変えてみたらうまく動きました！本当にありがとうございます。シャドーイングについてよく調べて同じミスをしないように気をつけます！