fix JSON

[Reputeless]

• コピーコンストラクタ・ムーブコンストラクタの問題を修正 JSON を swap すると実行時エラーが発生する #1117
• swap を定義
• Invalid な JSON への代入を有効に
• JSON::operator =(const std::initializer_list<std::pair<String, JSON>>& list) および JSON::operator =(const Array<JSON>& array) が古い内容を消去していなかったバグを修正 一部の JSON::operator =(...) が古い内容を消去していなかったバグを修正 #1166

[tomolatoon]

1. 寿命について

以前触れたように、寿命を延長出来るようにして、次のコードが通るようにするのがいいと思います。
https://discord.com/channels/443310697397354506/999983621408567326/1162390851624898581

#include <Siv3D.hpp>

JSON f() {
	JSON json = UR"({
		"foo": 42,
		"bar": "Hello, world!",
		"baz": [1, 2, 3]
	})"_json;

	// operator[] の返り値はプロキシとして働く、pvalue の JSON なので、nlohmann::json の値を所有していない
	return json[U"baz"];
}

void Main() {
	// 現状: 返り値は元のインスタンスが所有している nlohmann::json を参照しているが、その nlohmann::json は既に破棄されているので実行時エラー
	Console << f();
}

2. メンバ関数clearについて

clearについても、not m_isValidのチェックを無くし、invalid な状態からの復帰が出来るようにするべきです。

[Raclamusi]

1. 寿命について

以前触れたように、寿命を延長出来るようにして、次のコードが通るようにするのがいいと思います。 https://discord.com/channels/443310697397354506/999983621408567326/1162390851624898581

寿命の延長を実装するなら、detail::JSONDetail::RefType を std::shared_ptr<detail::JSONDetail::ValueType> にするのがよさそうですね。

https://gist.github.com/Raclamusi/08fbf4e70d0f1ac418793b7cdce30f7e/revisions#diff-018619d34dd37983bbc72f3e0cdfbd1a131c0670de6bec1d6bf04bddd206dece

[Reputeless]

ありがとうございます。JSON クラス、v0.6.15 以降で根本的に作り直したいので、寿命の延長関連は新規のほうで検討しましょう。そちらは別の Issue を立てます。

[Reputeless]

メンバ関数 clear について → JSON の .clear() は少し特殊で、クリア後も JSONValueType を保持する仕様になっています。例えば配列を .clear() したら空の配列になるなど。

そのため、Invalid の .clear() 結果が Invalid であるのも不自然ではないと考えています。
#1175 ではそもそも Invalid を廃止することも検討しているので、ここら辺の仕様変更はスキップしたいと思います。

[Reputeless]

レビューありがとうございました。 @Raclamusi @tomolatoon
気になった点は修正したので、これでマージする予定です。#1175 についても参加いただけると幸いです。