【GO】構造体を扱った、JSON生成で困っています

Question

実現したいこと

以下のJSONを出力する

//理想のJSON
{
  "command": "ENT",
  "table": [
    {
      "table_name": "basic",
      "clientid": 0,
      "server": "basic"
    },
    {
      "table_name": "contents",
      "clientid": 0,
      "time_control": "0"
    }
   ]
}

//現在のJSON
{
    "command": "ENT",
    "table": [
        {
            "Table1": {
                "table_name": "basic",
                "clientid": 0,
                "server": "basic"
            },
            "Table2": {
                "table_name": "contents",
                "clientid": 0,
                "time_control": 0
            }
        }
    ]
}

"Table1": {
"Table2": {
の部分が不要なのですが、どうしても消せません。。。

プログラムは以下です。
異なる構造体をまとめて、リテラルスライスで出力しようとしています。

package main

import (
	"bytes"
	"encoding/json"
	"fmt"
)

//Command ENT
type Command struct {
	Command string  `json:"command"`
	Tables  []Table `json:"table"`
}

//Table Data
type Table1 struct {
	TableName string `json:"table_name"`
	Clientid  int    `json:"clientid"`
	Server    string `json:"server"`
}

type Table2 struct {
	TableName   string `json:"table_name"`
	Clientid    int    `json:"clientid"`
	TimeControl int    `json:"time_control"`
}

type Table struct {
	Table1 Table1
	Table2 Table2
}

func main() {
	BasicConfig := Table1{TableName: "basic", Clientid: 0, Server: "basic"}
	ContentsFunc := Table2{TableName: "contents", Clientid: 0, TimeControl: 0}
	jsonStr := Command{
		Command: "ENT",
		Tables: []Table{
			{
				BasicConfig,
				ContentsFunc,
			},
		},
	}

	jsonBytes, err := json.Marshal(jsonStr)
	if err != nil {
		fmt.Println("JSON Marshal error:", err)
		return
	}
	out := new(bytes.Buffer)
	// プリフィックスなし、スペース4つでインデント
	json.Indent(out, jsonBytes, "", "    ")
	fmt.Println(out.String())
}

よろしくおねがいします。

Answer 1

JSONに対応するオブジェクトでよく勘違いされるパターンのひとつなんですが、
似て異なるものを別の型として表現しようとすることです。
実際はフィールドがオミットされているだけで同じ型を使っているという解釈で済む場合が多々あります。

そもそもTableというアレイフィールドに
異なる型のオブジェクトを入れ込むような設計はかなり稀です。

JSONの各要素の型を設計をするときはできるだけ型を決めて
その「あるPath」に対する「値の型」は一定であるという設計がお勧めですし、
そうなっていないJSON設計は良くない設計だと思います。

今回の事例も以下のように一つのTable型で済みます。

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

package main

import (
	"bytes"
	"encoding/json"
	"fmt"
)

type Command struct {
	Command string  `json:"command"`
	Table   []Table `json:"table"`
}

type Table struct {
	TableName   string `json:"table_name"`
	Clientid    int    `json:"clientid"`
	Server      string `json:"server,omitempty"`
	TimeControl string `json:"time_control,omitempty"`
}

func main() {
	BasicConfig := Table{TableName: "basic", Clientid: 0, Server: "basic", TimeControl:""}
	ContentsFunc := Table{TableName: "contents", Clientid: 0, Server:"", TimeControl: "0"}
	jsonStr := Command{
		Command: "ENT",
		Table: []Table{
			BasicConfig,
			ContentsFunc,
		},
	}

	jsonBytes, err := json.Marshal(jsonStr)
	if err != nil {
		fmt.Println("JSON Marshal error:", err)
		return
	}
	out := new(bytes.Buffer)
	// プリフィックスなし、スペース4つでインデント
	json.Indent(out, jsonBytes, "", "    ")
	fmt.Println(out.String())
}

Comments

[nobonobo]

ちなみに２つのオミットフィールドの「両方に有効な値がある」「両方がエンプティである」のはNGであるという場合のチェックの責任は「JSONデコード結果を受け取る側」または「JSONエンコードに値を渡す側」の責任です。

[Gunjirk]

nobonoboさん、丁寧なご回答ありがとうございます。
無事に解決いたしました。

nobonoboさんのおっしゃる勘違いをしておりました。
もう少しGOを勉強していきます。
JSON設計に関しても見直していきます。

今後ともよろしくお願いします。

Answer 2

nobonobo さんの回答のとおり、両方のsrtuctのフィールドをもつstructを用いて処理するのが良いと思います。

あえて、汎用的に何でも受け取るようにしたいのであれば、以下のように []interface{} で受けてしまうのも最終手段の方法としてはあるかな、と思います。

//Command ENT
type Command struct {
	Command string        `json:"command"`
	Tables  []interface{} `json:"table"`
}

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

• 出力結果

{
    "command": "ENT",
    "table": [
        {
            "table_name": "basic",
            "clientid": 0,
            "server": "basic"
        },
        {
            "table_name": "contents",
            "clientid": 0,
            "time_control": 0
        }
    ]
}

Comments

[Gunjirk]

d_tutuzさん、引き続きご回答いただきありがとうございます。

interfaceも使えるのですね！
試してみたいと思います！