私は、マップを含む構造体を初期化する最良の方法について混乱しています。このコードを実行すると、panic: runtime error: assignment to entry in nil map
が生成されます。
package main
type Vertex struct {
label string
}
type Graph struct {
connections map[Vertex][]Vertex
}
func main() {
v1 := Vertex{"v1"}
v2 := Vertex{"v2"}
g := new(Graph)
g.connections[v1] = append(g.coonections[v1], v2)
g.connections[v2] = append(g.connections[v2], v1)
}
this answer のように、コンストラクターを作成することも考えられます。
別のアイデアは、マップが空の場合にマップを初期化できるadd_connection
メソッドを使用することです。
func (g *Graph) add_connection(v1, v2 Vertex) {
if g.connections == nil {
g.connections = make(map[Vertex][]Vertex)
}
g.connections[v1] = append(g.connections[v1], v2)
g.connections[v2] = append(g.connections[v2], v1)
}
他のオプションはありますか?これを行う一般的に受け入れられている方法があるかどうかを確認したかっただけです。
私はおそらくこれを行うためにコンストラクタを使用します:
func NewGraph() *Graph {
var g Graph
g.connections = make(map[Vertex][]Vertex)
return &g
}
私は標準でこの例を見つけました image/jpeg
パッケージ(ただし、マップではなく、スライス付き):
type Alpha struct {
Pix []uint8
Stride int
Rect Rectangle
}
func NewAlpha(r Rectangle) *Alpha {
w, h := r.Dx(), r.Dy()
pix := make([]uint8, 1*w*h)
return &Alpha{pix, 1 * w, r}
}
データ構造を正しく初期化すると仮定することは、コード(特に完全に制御下のコード)で非常に一般的です。この場合、通常、構造体リテラルが使用されます
g := &Graph{
connections: make(map[Vertex][]Vertex),
}
複合リテラルは、コンストラクター内でうまく機能します。最初の質問を使用した例の作成(およびマップの頂点のコピーの単純な保存):
func NewGraph(v1 Vertex, v2 Vertex) *Graph {
return &Graph{ map[Vertex][]Vertex{ v1: []Vertex{v2}, v2: []Vertex{v1} }}
}
func main() {
v1 := Vertex{"v1"}
v2 := Vertex{"v2"}
g := NewGraph(v1, v2)
fmt.Println(g)
}