<refactor>[JSON]: redefined JSON output for Sets

Author: Kalashnikovni

sbg/multidim_inter.cpp

@@ -259,16 +259,13 @@ MaybeMDI MultiDimInter::compact(const MultiDimInter& other) const
 rapidjson::Value toJSON(MultiDimInter mdi
   , rapidjson::Document::AllocatorType& alloc)
 {
-  rapidjson::Value result{rapidjson::kArrayType};
-
   rapidjson::Value interval_array{rapidjson::kArrayType};
   for (const Interval& i : mdi) {
     rapidjson::Value jth = toJSON(i, alloc);
     interval_array.PushBack(jth, alloc);
   }
-  rapidjson::Value mdi_obj{rapidjson::kObjectType};
-  mdi_obj.AddMember("bounds", interval_array, alloc);
-  result.PushBack(mdi_obj, alloc);
+  rapidjson::Value result{rapidjson::kObjectType};
+  result.AddMember("bounds", interval_array, alloc);
 
   return result;
 }

sbg/ord_set.cpp

@@ -681,16 +681,13 @@ void OrderedSet::compact()
 
 rapidjson::Value toJSON(OrderedSet s, rapidjson::Document::AllocatorType& alloc)
 {
-  rapidjson::Value result{rapidjson::kArrayType};
-
   rapidjson::Value mdi_array{rapidjson::kArrayType};
   for (const MultiDimInter& mdi : s) {
     rapidjson::Value jth = detail::toJSON(mdi, alloc);
     mdi_array.PushBack(jth, alloc);
   }
-  rapidjson::Value mdi_obj{rapidjson::kObjectType};
-  mdi_obj.AddMember("pieces", mdi_array, alloc);
-  result.PushBack(mdi_obj, alloc);
+  rapidjson::Value result{rapidjson::kObjectType};
+  result.AddMember("pieces", mdi_array, alloc);
 
   return result;
 }

sbg/ord_unidim_dense_set.cpp

@@ -492,16 +492,16 @@ OrdUnidimDenseSet OrdUnidimDenseSet::traverse(
 rapidjson::Value toJSON(OrdUnidimDenseSet s
   , rapidjson::Document::AllocatorType& alloc)
 {
-  rapidjson::Value result{rapidjson::kArrayType};
-
   rapidjson::Value interval_array{rapidjson::kArrayType};
   for (const Interval& i : s) {
-    rapidjson::Value jth = detail::toJSON(i, alloc);
+    rapidjson::Value jth_array{rapidjson::kArrayType};
+    jth_array.PushBack(detail::toJSON(i, alloc), alloc);
+    rapidjson::Value jth{rapidjson::kObjectType};
+    jth.AddMember("bounds", jth_array, alloc);
     interval_array.PushBack(jth, alloc);
   }
-  rapidjson::Value mdi_obj{rapidjson::kObjectType};
-  mdi_obj.AddMember("pieces", interval_array, alloc);
-  result.PushBack(mdi_obj, alloc);
+  rapidjson::Value result{rapidjson::kObjectType};
+  result.AddMember("pieces", interval_array, alloc);
 
   return result;
 }

sbg/unord_set.cpp

@@ -484,16 +484,13 @@ void UnorderedSet::compact()
 rapidjson::Value toJSON(UnorderedSet s
   , rapidjson::Document::AllocatorType& alloc) 
 {
-  rapidjson::Value result{rapidjson::kArrayType};
-
   rapidjson::Value mdi_array{rapidjson::kArrayType};
   for (const MultiDimInter& mdi : s) {
     rapidjson::Value jth = detail::toJSON(mdi, alloc);
     mdi_array.PushBack(jth, alloc);
   }
-  rapidjson::Value mdi_obj{rapidjson::kObjectType};
-  mdi_obj.AddMember("pieces", mdi_array, alloc);
-  result.PushBack(mdi_obj, alloc);
+  rapidjson::Value result{rapidjson::kObjectType};
+  result.AddMember("pieces", mdi_array, alloc);
 
   return result;
 }