http.GetからJSON応答を取得する方法


135

WebからJSONデータを読み取ろうとしていますが、そのコードは空の結果を返します。ここで何が間違っているのかわかりません。

package main

import "os"
import "fmt"
import "net/http"
import "io/ioutil"
import "encoding/json"

type Tracks struct {
    Toptracks []Toptracks_info
}

type Toptracks_info struct {
    Track []Track_info
    Attr  []Attr_info
}

type Track_info struct {
    Name       string
    Duration   string
    Listeners  string
    Mbid       string
    Url        string
    Streamable []Streamable_info
    Artist     []Artist_info
    Attr       []Track_attr_info
}

type Attr_info struct {
    Country    string
    Page       string
    PerPage    string
    TotalPages string
    Total      string
}

type Streamable_info struct {
    Text      string
    Fulltrack string
}

type Artist_info struct {
    Name string
    Mbid string
    Url  string
}

type Track_attr_info struct {
    Rank string
}

func get_content() {
    // json data
    url := "http://ws.audioscrobbler.com/2.0/?method=geo.gettoptracks&api_key=c1572082105bd40d247836b5c1819623&format=json&country=Netherlands"

    res, err := http.Get(url)

    if err != nil {
        panic(err.Error())
    }

    body, err := ioutil.ReadAll(res.Body)

    if err != nil {
        panic(err.Error())
    }

    var data Tracks
    json.Unmarshal(body, &data)
    fmt.Printf("Results: %v\n", data)
    os.Exit(0)
}

func main() {
    get_content()
}

回答:


266

理想的な方法はを使用することではなくioutil.ReadAll、リーダーのデコーダーを直接使用することです。これは、URLを取得し、その応答をtarget構造にデコードする素晴らしい関数です。

var myClient = &http.Client{Timeout: 10 * time.Second}

func getJson(url string, target interface{}) error {
    r, err := myClient.Get(url)
    if err != nil {
        return err
    }
    defer r.Body.Close()

    return json.NewDecoder(r.Body).Decode(target)
}

使用例:

type Foo struct {
    Bar string
}

func main() {
    foo1 := new(Foo) // or &Foo{}
    getJson("http://example.com", foo1)
    println(foo1.Bar)

    // alternately:

    foo2 := Foo{}
    getJson("http://example.com", &foo2)
    println(foo2.Bar)
}

*http.Clientこの回答が最初に示したように、本番環境ではデフォルトの構造を使用しないでください。(これはhttp.Get/ etcが呼び出すものです)。その理由は、デフォルトのクライアントにはタイムアウトが設定されていないためです。リモートサーバーが応答しない場合は、悪い日になります。


5
たとえばtype WebKeys struct { Keys []struct { X5t string X5c []string } } 、解析しているJSONの実際のパラメータが小文字の場合でも、構造体の項目の名前に大文字を使用する必要があるようです。JSONの例:{ "keys": [{ "x5t": "foo", "x5c": "baaaar" }] }
Wilson

1
@ローマ、いいえ。エラーが返された場合、応答値はnilです。(エラーは、有効なHTTP応答を読み取ることができなかったことを意味します。閉じるボディがありません!)これは、存在しないURLで.Get()をポイントすることでテストできます。このメソッドは、net / httpドキュメントの2番目のコードブロックで示されています。
Connor Peet

1
@NamGVUは潜在的な割り当てを保存し、httpキープアライブを使用して接続を再利用できるようにします。
Connor Peet、2017年

2
@ConnorPeetありがとうございます!「本番環境ではデフォルトの* http.Client構造を使用するべきではない」とはどういう意味かと思います。&http.Client{Timeout: 10 * time.Second}他のライブラリ/戦略全体を使用する必要があるということですか?
ジョナロドリゲス

6
ただ、他の人への警告は- json.NewDecoder(r.Body).Decode(target)なりません、不正な形式のJSONの特定の種類のエラーを返します!空の応答が返される理由を理解しようとして数時間を無駄にしただけです-ソースJSONに余分なコンマが含まれるべきではなかったことがわかりました。json.Unmarshal代わりに使用することをお勧めします。使用しての他の潜在的な危険性についての良い過去記事もありますjson.Decoder 、ここでは
adamc

25

あなたの問題はあなたのデータのスライス宣言でしたstructs(を除いてTrack、それらはスライスであってはなりません...)。これは、フェッチされたjsonファイルのかなりおかしなフィールド名によって悪化しました。構造体タグで修正できます。godocを参照してください 。

以下のコードは、jsonを正常に解析しました。さらに質問がある場合は、お知らせください。

package main

import "fmt"
import "net/http"
import "io/ioutil"
import "encoding/json"

type Tracks struct {
    Toptracks Toptracks_info
}

type Toptracks_info struct {
    Track []Track_info
    Attr  Attr_info `json: "@attr"`
}

type Track_info struct {
    Name       string
    Duration   string
    Listeners  string
    Mbid       string
    Url        string
    Streamable Streamable_info
    Artist     Artist_info   
    Attr       Track_attr_info `json: "@attr"`
}

type Attr_info struct {
    Country    string
    Page       string
    PerPage    string
    TotalPages string
    Total      string
}

type Streamable_info struct {
    Text      string `json: "#text"`
    Fulltrack string
}

type Artist_info struct {
    Name string
    Mbid string
    Url  string
}

type Track_attr_info struct {
    Rank string
}

func perror(err error) {
    if err != nil {
        panic(err)
    }
}

func get_content() {
    url := "http://ws.audioscrobbler.com/2.0/?method=geo.gettoptracks&api_key=c1572082105bd40d247836b5c1819623&format=json&country=Netherlands"

    res, err := http.Get(url)
    perror(err)
    defer res.Body.Close()

    decoder := json.NewDecoder(res.Body)
    var data Tracks
    err = decoder.Decode(&data)
    if err != nil {
        fmt.Printf("%T\n%s\n%#v\n",err, err, err)
        switch v := err.(type){
            case *json.SyntaxError:
                fmt.Println(string(body[v.Offset-40:v.Offset]))
        }
    }
    for i, track := range data.Toptracks.Track{
        fmt.Printf("%d: %s %s\n", i, track.Artist.Name, track.Name)
    }
}

func main() {
    get_content()
}

1
レスポンスボディに何かがあります。
peterSO 2013年

6
私の場合、「struct」フィールドに大文字の最初の文字がありませんでした。
2014年

以下の答えは正解です。応答で直接デコーダーを使用します。ボディは不要な割り当てを回避し、一般的にはより理想的です。指摘してくれてありがとう。
タイク

@abourget omgこのコメントありがとうございます。パーサーで問題を探して1時間過ごし、wiresharkで応答が正しいことを確認してください...ありがとう
agilob

14

jsonパッケージで使用するには、構造体に大文字のプロパティ名が必要です。

大文字のプロパティ名はexported propertiesです。小文字のプロパティ名はエクスポートされません。

また、データオブジェクトを参照(&data)で渡す必要があります。

package main

import "os"
import "fmt"
import "net/http"
import "io/ioutil"
import "encoding/json"

type tracks struct {
    Toptracks []toptracks_info
}

type toptracks_info struct {
    Track []track_info
    Attr  []attr_info
}

type track_info struct {
    Name       string
    Duration   string
    Listeners  string
    Mbid       string
    Url        string
    Streamable []streamable_info
    Artist     []artist_info
    Attr       []track_attr_info
}

type attr_info struct {
    Country    string
    Page       string
    PerPage    string
    TotalPages string
    Total      string
}

type streamable_info struct {
    Text      string
    Fulltrack string
}

type artist_info struct {
    Name string
    Mbid string
    Url  string
}

type track_attr_info struct {
    Rank string
}

func get_content() {
    // json data
    url := "http://ws.audioscrobbler.com/2.0/?method=geo.gettoptracks&api_key=c1572082105bd40d247836b5c1819623&format=json&country=Netherlands"

    res, err := http.Get(url)

    if err != nil {
        panic(err.Error())
    }

    body, err := ioutil.ReadAll(res.Body)

    if err != nil {
        panic(err.Error())
    }

    var data tracks
    json.Unmarshal(body, &data)
    fmt.Printf("Results: %v\n", data)
    os.Exit(0)
}

func main() {
    get_content()
}

まだ動作しませんが、これでうまくいきますか?同じ空の応答
Akshaydeep Giri 2013

3
「jsonパッケージで使用するには、構造体に大文字のプロパティ名が必要です。」
HVNSweeting 2015

8

json.Unmarshal(into var data interface{})からの結果は、Goタイプおよび変数宣言と直接一致しません。例えば、

package main

import (
    "encoding/json"
    "fmt"
    "io/ioutil"
    "net/http"
    "os"
)

type Tracks struct {
    Toptracks []Toptracks_info
}

type Toptracks_info struct {
    Track []Track_info
    Attr  []Attr_info
}

type Track_info struct {
    Name       string
    Duration   string
    Listeners  string
    Mbid       string
    Url        string
    Streamable []Streamable_info
    Artist     []Artist_info
    Attr       []Track_attr_info
}

type Attr_info struct {
    Country    string
    Page       string
    PerPage    string
    TotalPages string
    Total      string
}

type Streamable_info struct {
    Text      string
    Fulltrack string
}

type Artist_info struct {
    Name string
    Mbid string
    Url  string
}

type Track_attr_info struct {
    Rank string
}

func get_content() {
    // json data
    url := "http://ws.audioscrobbler.com/2.0/?method=geo.gettoptracks&api_key=c1572082105bd40d247836b5c1819623&format=json&country=Netherlands"
    url += "&limit=1" // limit data for testing
    res, err := http.Get(url)
    if err != nil {
        panic(err.Error())
    }
    body, err := ioutil.ReadAll(res.Body)
    if err != nil {
        panic(err.Error())
    }
    var data interface{} // TopTracks
    err = json.Unmarshal(body, &data)
    if err != nil {
        panic(err.Error())
    }
    fmt.Printf("Results: %v\n", data)
    os.Exit(0)
}

func main() {
    get_content()
}

出力:

Results: map[toptracks:map[track:map[name:Get Lucky (feat. Pharrell Williams) listeners:1863 url:http://www.last.fm/music/Daft+Punk/_/Get+Lucky+(feat.+Pharrell+Williams) artist:map[name:Daft Punk mbid:056e4f3e-d505-4dad-8ec1-d04f521cbb56 url:http://www.last.fm/music/Daft+Punk] image:[map[#text:http://userserve-ak.last.fm/serve/34s/88137413.png size:small] map[#text:http://userserve-ak.last.fm/serve/64s/88137413.png size:medium] map[#text:http://userserve-ak.last.fm/serve/126/88137413.png size:large] map[#text:http://userserve-ak.last.fm/serve/300x300/88137413.png size:extralarge]] @attr:map[rank:1] duration:369 mbid: streamable:map[#text:1 fulltrack:0]] @attr:map[country:Netherlands page:1 perPage:1 totalPages:500 total:500]]]
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.