web-dev-qa-db-ja.com

golang-構造体内のマップフィールドを初期化する方法は？

私は、マップを含む構造体を初期化する最良の方法について混乱しています。このコードを実行すると、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)
}

他のオプションはありますか？これを行う一般的に受け入れられている方法があるかどうかを確認したかっただけです。

dictionarystructgo
26
Matt

私はおそらくこれを行うためにコンストラクタを使用します：

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

データ構造を正しく初期化すると仮定することは、コード（特に完全に制御下のコード）で非常に一般的です。この場合、通常、構造体リテラルが使用されます

g := &Graph{
    connections: make(map[Vertex][]Vertex),
}
13
JimB

複合リテラルは、コンストラクター内でうまく機能します。最初の質問を使用した例の作成（およびマップの頂点のコピーの単純な保存）：

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

https://play.golang.org/p/Lf4Gomp4tJ

2
durp