UnityでJsonが上手く使えない

Question

UnityでDBから取得したJsonデータを配列に変換してアクセスするところまでは行ったのですが、その配列内の要素を上書きすることが中々できなくて躓いています。

下記のJsonをaaaというJsonData型の配列に入れた場合

{"0":{"name":"sample.txt","open":"memo","image":"txt","scan":"0"},"1":{"name":"sample.txt","open":"memo","image":"txt","scan":"0"}}

Debug.Log(JsonMapper.ToObject(mailContent["mailFileContents"].ToString())["0"]["name"]);

でJson内の0番目のsample.txtをよみこめたのですが、逆にこのsample.txtという要素を上書きしたいのですが、上のC#のコードにそのまま

JsonMapper.ToObject(mailContent["mailFileContents"].ToString())["0"]["name"] = "aaaaaaa";

のように代入してもJson内の値が変わりませんでしたので、やり方を教えて欲しいです。

Answer 1

こんにちは。

まず、一行で書くのをやめて、一行ずつ変数に入れてみてください。
質問のコードを直してみると、こうです。

// contentからjson文字列を取得
var filecontents = mailContent["mailFileContents"].ToString();

// json文字列からjsonオブジェクトを作成
var jsonobject = JsonMapper.ToObject(filecontents);

// jsonオブジェクトのnameを書き換え
jsonobject["0"]["name"] = "aaaaaaa";

jsonobjectの値を書き換えてるので、おそらくjsonobject変数の中身は意図したとおり書き換わっているはずです。
しかし、変更の影響が及ぶのは変数jsonobjectまでで、filecontentsには何の手も加えていないことが分かります。
mailContentの中身に反映させたいならこうですかね。mailContentが書き換え可能なオブジェクトであるという前提です。

jsonobject["0"]["name"] = "aaaaaaa";

// 書き換えたjsonオブジェクトをjson文字列化して上書き
mailContent["mailFileContents"] = JsonMapper.ToJson(jsonobject);

ちなみに、普通だったら毎回mailContentから取り出したりせずに、一度取り出したjsonobject変数を使い回すように書きます。

Comments

[m_s]

いつも回答有り難うございます、tamotoさんには本当に感謝してます！
この通りにやってみたら出来たのですが、いくつか質問します
このコードの感じだとjsonobjectにはmailContent["mailFileContents"]のポインタがあるので、jsonobject["0"]["name"]にすると値までアクセスできる。更に、その値を上書した後、その値を含むjsonobjectのポインタを同等なmailContent["mailFileContents"]に代入すると新たに更新されてもとのmailContentの中身も変わるという認識であってますか(；・∀・)
あと、「普通だったら毎回mailContentから取り出したりせずに、一度取り出したjsonobject変数を使い回すように書きます。」というのは、mailContentがオブジェクト型なのでここから直接[name]までアクセスして上書できるということですか？もしそういうことだったら何ですけど、直接[name]まではアクセス出来ませんでした・・・

[tamoto]

えっと、jsonobjectに「ポインタはない」です。jsonobjectの中身はmailContent["mailFileContents"]の中身を取り出して「jsonに変換したもの」となっており、その時点でmailContentとの関係は完全に切れています。なので、jsonobjectに対していろいろ操作するのは、mailContentとは何の関係もないのです。変更をmailContentまで反映させたいときは、書き換えたjsonobjectを「再度json文字列に変換」して、mailContentの中に「新しいJsonでまるごと上書き」しているだけです。

「取り出したりしない」という話は、jsonという「文字列で表されるデータ形式」をC#で取り扱うためにToObjectを使って取り込んでいるので、毎回わざわざ外からデータを読み出してjsonに変換したりせずに、最初に一度だけ取り込んだjsonオブジェクトをずっと使えばいい話にはなりませんか？そうすれば、値を変更したとき、その同じオブジェクトを参照する他の処理でも値は変更されています。変更を明示的に「Save」するときだけ、「外(mailContent)」に書き込むような書き方をします。ToObjectはいわゆるパース処理になるので、最適化を頑張ってもそこそこ重い処理になります。なるべく実行しないで済むならその方がいいのです。

Answer 2

いったんToObject()したものを変数に入れてから、変数で値を変更し、その変数をToJson()しないといけないのでは？

var obj = JsonMapper.ToObject(mailContent["mailFileContents"].ToString());
obj["0"]["name"] = "hoge";
var json = JsonMapper.ToJson(obj);

てか、見た感じ全体がJSONなので全体をいったんToObject()して変数に入れてごにょごにょしたほうが楽かと。

Comments

[m_s]

回答ありがとうございます
その通りにやってみたところできました！