Initial serializing about arduinojson HOT 7 CLOSED

Hi @hitecSmartHome,

It looks like you can simplify your code like so:

  boolean ComponentHandler::updateZones(JsonObject newZoneData){
      JsonDocument zonesObj;
      DeserializationError error = deserializeJson(zonesObj, (const char*)zones);
      if(error){ zonesObj.clear(); } // I have tried to just ignore the error
  
-     std::string id = newZoneData["id"];
-     if( zonesObj[id].isNull() ){
-         JsonObject zone = zonesObj[id].to<JsonObject>();
-         zone.set( newZoneData );
-     }else{
-         zonesObj[id].set( newZoneData );
-     }
+     zonesObj[newZoneData["id"]] = newZoneData;
      return serializeJson(zonesObj, zones, ZONES_SIZE) > 0;
  }

But what is your question exactly?

Best regards,
Benoit

from arduinojson.

Thanks for the suggestion.

The question is that if I serialize it why do i get string "2" instead of a string object.

from arduinojson.

It doesn't work like that

boolean ComponentHandler::updateZones(JsonObject newZoneData){
    JsonDocument zonesObj;
    deserializeJson(zonesObj, (const char*)zones); // Doesn't matter if the zones char array is empty or not.
    zonesObj[newZoneData["id"]] = newZoneData; // Does not save it. I have to use set() method
    return serializeJson(zonesObj, zones, ZONES_SIZE) > 0;
}

from arduinojson.

Indeed, I just realized that doc[var] doesn't work when var is a string; it only works with integers.
This is clearly a bug.
Until I fix it, please use zonesObj[newZoneData["id"].as<JsonString>()] = newZoneData; instead.

About your original issue, where serializeJson() produces 2 as the output JSON document, this could be due to the initial data in the zones buffer.
Please add a terminator at the beginning of the buffer to ensure that deserializeJson() sees an empty input.

  if( !hshSystem.alloc(zones,ZONES_SIZE) ){
      printf("FAILED TO ALLOCATE ZONES ARRAY\n");
+     return;
  }
+ zones[0] = 0;

from arduinojson.

I see. Will check it, thank you very much! And I will try with JsonString also.

from arduinojson.

Confirmed it is working with JsonString.

boolean ComponentHandler::updateZones(JsonObject newZoneData){
    JsonDocument zonesObj;
    deserializeJson(zonesObj, (const char*)zones);
    zonesObj[newZoneData["id"].as<JsonString>()] = newZoneData;
    return serializeJson(zonesObj, zones, ZONES_SIZE) > 0;
}

Thank you very much. Will test this

if( !hshSystem.alloc(zones,ZONES_SIZE) ){
      printf("FAILED TO ALLOCATE ZONES ARRAY\n");
+     return;
  }
+ zones[0] = 0;

from arduinojson.

It works fine. Thank you very much for the help. I appreciate it.

from arduinojson.