JsonVariant tests false for both a value that doesn't exist and for a value that is zero about arduinojson HOT 7 CLOSED

Hi @judge2005,

Thank you for this feedback but I don't see any problem since this behavior is consistent with the conversion to bool.
For example:

JsonVariant var;
var.as<bool>();  // false

JsonDocument doc;
var = doc.to<JsonVariant>();
var.as<bool>();  // false

var.set(false);
var.as<bool>();  // false

var.set(true);
var.as<bool>();  // true

var.set(0);
var.as<bool>();  // false

var.set(1);
var.as<bool>();  // true

var.set(nullptr);
var.as<bool>();  // false

var.set("hello world");
var.as<bool>();  // true

var = doc.to<JsonArray>();
var.as<bool>();  // true

var = doc.to<JsonObject>();
var.as<bool>();  // true

Best regards,
Benoit

from arduinojson.

The problem is that using operator[] I can’t tell the difference between an attribute that doesn’t exist and one that does exist but who’s value is zero. However that is the solution recommended by the documentation. In reality I have to use containsKey().

from arduinojson.

You can use value.is<int>().
Not only does it tell you that the value is present, but it also confirms it has the expected type, making your code even more robust.

from arduinojson.

That is good, but I am trying to make the point that the documentation is misleading. If it is followed it would lead to code that does not function correctly. The documentation states that operator[] can be used to determine if an attribute is present or not because it returns false if the attribute is not present. In fact it will also return false if the attribute is present but is either a numeric zero or a boolean false.

from arduinojson.

Which part of the documentation are you referring to?

from arduinojson.

https://arduinojson.org/v7/api/jsonvariant/containskey/

The text under the heading "Avoid this function when you can!"

Because ArduinoJson implements the Null Object Pattern, it is always safe to read the object: if the key doesn’t exist, it returns an empty value.

Which is only true in the sense that it will always return something rather than throwing an exception.

from arduinojson.

I'm sorry I don't see any problem with this paragraph.
How would you rephrase it?

from arduinojson.