2013-02-27 9 views
11

マップのスライスを作成しようとしています。コードは罰金コンパイルが、私は下のランタイムエラーを取得:ランタイムエラー:「nilマップのエントリへの割り当て」

mapassign1: runtime·panicstring("assignment to entry in nil map"); 

私は2つなインデックスを含む各地図、「ID」と「投資家」で、地図の配列を試みます。私が最初にどちらかの問題を解決しなかったが、以下のようなものが働くだろうと思った

for _, row := range rows { 
     var inv_ids []string 
     var inv_names []string 

     //create arrays of data from MySQLs GROUP_CONCAT function 
     inv_ids = strings.Split(row.Str(10), ",") 
     inv_names = strings.Split(row.Str(11), ",") 
     length := len(inv_ids); 

     invs := make([]map[string]string, length) 

     //build map of ids => names 
     for i := 0; i < length; i++ { 
      invs[i] = make(map[string]string) 
      invs[i]["Id"] = inv_ids[i] 
      invs[i]["Investor"] = inv_names[i] 
     }//for 

     //build Message and return 
     msg := InfoMessage{row.Int(0), row.Int(1), row.Str(2), row.Int(3), row.Str(4), row.Float(5), row.Float(6), row.Str(7), row.Str(8), row.Int(9), invs} 
     return(msg) 
    } //for 

:私のコードは次のようになります。何か案は?

invs := make([]make(map[string]string), length) 
+1

あなたのコードは私のために正常に動作します。http://プレイ.golang.org/p/DaY5qdBGpy –

+3

マップがうまく定義されている場合は、代わりに構造体を使用したいと思うかもしれません。 struct {Id、Investor string} –

答えて

10

あなたがマップのスライスを作成しようとしています。次の例を考えてみます。

http://play.golang.org/p/gChfTgtmN-

package main 

import "fmt" 

func main() { 
    a := make([]map[string]int, 100) 
    for i := 0; i < 100; i++ { 
     a[i] = map[string]int{"id": i, "investor": i} 
    } 
    fmt.Println(a) 
} 

あなたはこれらの行を書き換えることができます。

invs[i] = make(map[string]string) 
invs[i]["Id"] = inv_ids[i] 
invs[i]["Investor"] = inv_names[i] 

として:これは複合リテラルと呼ばれる

invs[i] = map[string]string{"Id": inv_ids[i], "Investor": inv_names[i]} 

今、より多くの慣用的なプログラムの中で、あなたはおそらく投資家を代表するstructを使用したいと思います:

http://play.golang.org/p/vppK6y-c8g

package main 

import (
    "fmt" 
    "strconv" 
) 

type Investor struct { 
    Id int 
    Name string 
} 

func main() { 
    a := make([]Investor, 100) 
    for i := 0; i < 100; i++ { 
     a[i] = Investor{Id: i, Name: "John" + strconv.Itoa(i)} 
     fmt.Printf("%#v\n", a[i]) 
    } 
} 
+1

私が自分自身に言った質問を書いてしまう前に、それを配列ではなくスライスと呼んでください。私は質問を編集しましたが、すべてが複合リテラルを使用して動作するようです。ありがとう! – user387049

関連する問題