JSON Objekt Feld erzeugen und auswerten

[Christian J. (Gast)]

Hallo,

leider ist für mich C++ böhmische Dörfer und die Denkweise hinter der 
JSON API noch nicht ganz klar. Ich möchte einfach nur einen Satz von 10 
Wetterberichten, der sich bereits im JSON Buffer befindet als Feld 
auswerten.

Bisher habe ich mir nur das zusammen gesucht, was auch klappt:

Das Parsen... ich hole 10 Datensätze von der OpenweatherMap, d.h. 10*3 
Stunde = 30 Stunde Vorhersagen

const size_t capacity = JSON_ARRAY_SIZE(3) + 2*JSON_OBJECT_SIZE(1) + JSON_OBJECT_SIZE(2) + 3*JSON_OBJECT_SIZE(4) +JSON_OBJECT_SIZE(5) + JSON_OBJECT_SIZE(6) + JSON_OBJECT_SIZE(12) + 480;

DynamicJsonBuffer jsonBuffer(10 * capacity);
JsonObject& root = jsonBuffer.parseObject(client);

Wurzel der Liste festlegen:
JsonArray& list = root["list"];

/* 2 Sätze auslesen: now und later */

JsonObject& now = list[0];
JsonObject& later = list[1];

PressureNow        = now["main"]["pressure"];
TempNow         = now["main"]["temp"];
WSpeedNow        = now["wind"]["speed"];

usw.

Was nicht klappt ist ein Feld an zu legen, wo ich zb 10 Sätze 
hineinschreibe :-(  Mit now und later habe ich zwei aber ich will das in 
einem Array haben. also zb feld[n]

JsonObject& feld[10];    <- Compiler Mecker: declaration of feld as 
array of referces.

for (int i = 0; i < 10; i++) {
Auswertung.....
}

Kann da mal jemand helfen? Möglichst mit Code?

[Rufus Τ. F. (rufus)]

Christian J. schrieb:
> und die Denkweise hinter der JSON API noch nicht ganz klar

"die JSON API" - C++ selbst kennt kein JSON; Du benutzt also irgendeine 
Library/irgendwelchen Code.

Welchen? Woher kommt der? Ist der dokumentiert?

[Christian J. (Gast)]

Ja, die Arduino Lib für esp8266.... und da ist alles dokuemntiert, 
erschlagend sogar. Ich will nur dieses eine Problem lösen.

https://arduinojson.org/v5/doc/

[Rufus Τ. F. (rufus)]

Dann zeig' doch mal das Json, das Du von der Wetterseite erhältst.

[Christian J. (Gast)]

das ist eines mit 2 Datensätzen, ich kann aber auch 10 abrufen. Und 
genau das will ich jetzt tun. Natürlich erzeuge ich nicht 10 Variablen 
now,later,nochlater,viellater etc. sondern will die 10 Sätze einfach nur 
auslesen und in meine eigenen Vars packen, damit ich auf vertrauten 
Boden komme und weiterverarbeiten kann.

JsonArray& list = root["list"];
    
JsonObject& now = list[0];
JsonObject& later = list[1];

  /* Aktuellen Druck und Temperatur holen */
  debug(F("Hole JSON Buffer Daten..."));
  /* Aktuelle Daten */
  PressureNow        = now["main"]["pressure"];
  TempNow         = now["main"]["temp"];      /* Temperatur */
  WSpeedNow        = now["wind"]["speed"];
  WeatherIDNow      = now["weather"][0]["id"];
  CloudsAllNow      = now["clouds"]["all"];
  String Date       = now["dt_txt"];
  String TextNow       = now["weather"][0]["description"];
  /* Vorhersage */
  String TextLater     = later["weather"][0]["description"];
  CloudsAllLater      = later["clouds"]["all"];
  TempLater         = later["main"]["temp"];      /* Temperatur */

{
  "cod": "200",
  "message": 0.0023,
  "cnt": 2,
  "list": [
     {
      "dt": 1547726400,
      "main": {
        "temp": 5.86,
        "temp_min": 3.93,
        "temp_max": 5.86,
        "pressure": 998.26,
        "sea_level": 1012.84,
        "grnd_level": 998.26,
        "humidity": 98,
        "temp_kf": 1.93
      },
      "weather": [
        {
          "id": 500,
          "main": "Rain",
          "description": "light rain",
          "icon": "10d"
        }
      ],
      "clouds": {
        "all": 44
      },
      "wind": {
        "speed": 6.17,
        "deg": 241.002
      },
      "rain": {
        "3h": 0.0825
      },
      "sys": {
        "pod": "d"
      },
      "dt_txt": "2019-01-17 12:00:00"
    },
 
  {
      "dt": 1547737200,
      "main": {
        "temp": 4,
        "temp_min": 2.55,
        "temp_max": 4,
        "pressure": 997.53,
        "sea_level": 1012.32,
        "grnd_level": 997.53,
        "humidity": 100,
        "temp_kf": 1.45
      },
      "weather": [
        {
          "id": 500,
          "main": "Rain",
          "description": "light rain",
          "icon": "10d"
        }
      ],
      "clouds": {
        "all": 92
      },
      "wind": {
        "speed": 4.98,
        "deg": 259.003
      },
      "rain": {
        "3h": 1.8475
      },
      "snow": {
        "3h": 0.006
      },
      "sys": {
        "pod": "d"
      },
      "dt_txt": "2019-01-17 15:00:00"
    }
  ],
  "city": {
    "id": 2952444,
    "name": "Barntrup",
    "coord": {
      "lat": 51.991,
      "lon": 9.1128
    },
    "country": "DE",
    "population": 9513
  }
}

[Christian J. (Gast)]

JsonObject& now = list[0];
JsonObject& later = list[1];

Damit habe ich schon die Infos, die ich brauche für die ersten beiden 
Sätze, die 3h auseinander liegen. 0 für jetzt und 1 für in 3 Stunden. 
Und nun nur noch das ganze erweitern auf 10 Sätze. Aber JsonObject& mit 
dem & hinter dem Typ ist mir aus C nicht bekannt, das scheint eine 
Instanz eines Objektes zu sein. Now wird zum Zeiger auf den Inhalt bzw 
die Methoden dieser Klasse. So mein rudimentäres Verständnis von c++....

Und da soll es alles rein:

/* Wetter Datensatz */
typedef struct {
    int      FC_Hours;        /* Stunden in Zukunft: 0,3,6,9,12,15,.... */
    int     wid;          /* Die ID der aktuellen Wetterlage */  
    int     Pressure;        /* Druck */
    double     Temp;          /* Temperatur */
    double    WSpeed;          /* Windgeschwindigkeit */
    int     CloudsAll;        /* Bewölkung in % */
    String    DateStamp;        /* Zeitstempel als String */ 
    String    WetterText;        /* Wetter als Satz */
} wetter_t;

wetter_t wetter[10] = {0,};

[Markus F. (mfro)]

Christian J. schrieb:
> JsonObject& feld[10];    <- Compiler Mecker

Zu recht.

C++ Standard §8.3.2/4:
    There shall be no references to references, no arrays of references, 
and no pointers to references.

[Christian J. (Gast)]

Das nützt mir aber auch nichts. Warum sagst du mir nicht wie es richtig 
gemacht wird?

[Rufus Τ. F. (rufus)]

Christian J. schrieb:
> sondern will die 10 Sätze einfach nur auslesen und in meine eigenen Vars
> packen,

Ich würde hier anstelle von

   PressureNow =  now["main"]["pressure"];

etwas à la

   PressureNow =  List[0]["main"]["pressure"];

ausprobieren.

Und stattt "PressureNow" ein Array von Drücken:

for (i = 0; i < Anzahl; i++)
{
  Pressure[i] = List[i]["main"]["pressure"];
}

bzw. sehen, ob es einen tauglichen Listeniterator gibt und den dann 
verwenden.

Oder hab' ich das Problem jetzt komplett missverstanden?

[Christian J. (Gast)]

Rufus Τ. F. schrieb:
> Oder hab' ich das Problem jetzt komplett missverstanden?

Nee, ich denke schon richtig. Das problem vieler Anleitungen wie auch 
bei dieser ist, dass sie von leuten geschrieben werden, die so tief in 
der Materie stecken, dass sie die Fragen eines Neulings nicht 
"verstehen". Im Grund ist diese nur, wie die Daten in dem Container 
abgelegt sind (nach dem Parsen) und wie der Zugriff erfolgt. Json an 
sich ist nix Besonders, ein Datenformat wie xml, lesbar von Menschen und 
Maschinen zur strukturierten Ablage von Daten, die eine hierarische 
Ordnung haben können.  Ihc probiers mal heute abend aus, wenn ich Zeit 
habe...

[Christian J. (Gast)]

Weisst du zufällig was diese 0 da bedeutet?
WeatherIDNow      = now["weather"][0]["id"];
Ist auch nur abgeschrieben.....

PS: Deine Lösung kompiliert schonmal durch.... das gibt Hoffung.

PSS: Und sie funktioniert auch :-)))) Klasse, danke Meister!

[M.K. B. (mkbit)]

Christian J. schrieb:
> Weisst du zufällig was diese 0 da bedeutet?
> WeatherIDNow      = now["weather"][0]["id"];

Das ist der Index für ein Array.
Hier (http://json.org/) ist die JSON Syntax ganz gut dargestellt. Wenn 
du die Daten genau anschaust, dann siehst du, dass weather ein Array 
ist.

[Christian J. (Gast)]

Ah ok. Es reicht wennn man das versteht, was man grad braucht denke ich. 
Bin ja kein Entwickler für solche Anwendungen.

Und warum muss ich solche Verrenkungen machen über die foo Variablen? 
Direkte Zuweisung auf den gleichen Typ (global) geht nicht. Da wird 
gemeckert. Es können auch keine rechnungen gemacht werden. Die Ausdrücke 
müssen so da stehen wie sie sind. Also die *3.6 kann nicht geklammert 
werden.

  for (int i = 0; i < (HFcN-1);i++) {
    wetter[i].wid     =  list[i]["weather"][0]["id"];
    wetter[i].Pressure  = list[0]["main"]["pressure"];
    wetter[i].Temp     = list[0]["main"]["temp"];
    wetter[i].WSpeed   =  list[0]["wind"]["speed"];
    wetter[i].WSpeed   *= 3.6;
    wetter[i].CloudsAll = list[i]["clouds"]["all"];
    String foo1         = list[i]["dt_txt"];
    wetter[i].DateStamp = foo1;
    String foo2      = list[0]["weather"][0]["description"];
    wetter[i].WetterText = foo2;
  }

[Rufus Τ. F. (rufus)]

Christian J. schrieb:
> Direkte Zuweisung auf den gleichen Typ (global) geht nicht. Da wird
> gemeckert.

Aha. Der Compiler gibt eine Meldung "Gemecker" aus, oder wie muss man 
sich das vorstellen?

Fehlermeldungen haben in der Regel eine Bedeutung, der man entnehmen 
kann, was das Problem ist.

[Christian J. (Gast)]

Rufus Τ. F. schrieb:
> Aha. Der Compiler gibt eine Meldung "Gemecker" aus, oder wie muss man
> sich das vorstellen?

Es ist ein ganzer Wust, der sich nicht auf die Zeile bezieht, sondern 
aus den Quelltext der Json Lib. U.a. dass keine passende Methode 
gefunden wurde. Nur der Typ String macht diese probleme, die anderen 
nicht.

[Christian J. (Gast)]

Ich habe mal etwas gekürzt :-)

Arduino: 1.8.5 (Windows 7), Board: "NodeMCU 1.0 (ESP-12E Module), 80 
MHz, Flash, 4M (1M SPIFFS), v2 Lower Memory, Disabled, None, Only 
Sketch, 256000"

Build-Optionen wurden verändert, alles wird neu kompiliert
K:\ESP8266\WetterOled\tools.ino: In function 'int GetWeatherData()':

tools:186: error: ambiguous overload for 'operator=' (operand types are 
'String' and 'ArduinoJson::Internals::EnableIf<true, 
ArduinoJson::Internals::JsonObjectSubscript<const char*> >::type {aka 
ArduinoJson::Internals::JsonObjectSubscript<const char*>}')

   wetter[i].DateStamp        =  list[i]["dt_txt"];

exit status 1
ambiguous overload for 'operator=' (operand types are 'String' and 
'ArduinoJson::Internals::EnableIf<true, 
ArduinoJson::Internals::JsonObjectSubscript<const char*> >::type {aka 
ArduinoJson::Internals::JsonObjectSubscript<const char*>}')

Dieser Bericht wäre detaillierter, wenn die Option
"Ausführliche Ausgabe während der Kompilierung"
in Datei -> Voreinstellungen aktiviert wäre.

[Rufus Τ. F. (rufus)]

Probier' doch mal aus, was passiert, wenn Du in die betreffende Zeile 
einen typecast nach String einbaust, d.h.

>     wetter[i].DateStamp = (String) list[i]["dt_txt"];

Was passiert dann?

[Christian J. (Gast)]

Rufus Τ. F. schrieb:

> Was passiert dann?

K:\ESP8266\WetterOled\tools.ino: In function 'int GetWeatherData()':

tools:186: error: call of overloaded 
'String(ArduinoJson::Internals::EnableIf<true, 
ArduinoJson::Internals::JsonObjectSubscript<const char*> >::type)' is 
ambiguous

   wetter[i].DateStamp  =  (String)list[i]["dt_txt"];

Das Problem bezieht sich nur auf die String Klasse, die ja ohnehin mit 
Vorsicht zu genießen ist und auf "double".

Nur das ist kompilierbar und liefert das richtige Eregbnis:
String foo1         =  list[i]["dt_txt"];
wetter[i].DateStamp =  foo1;

Das hier aber läuft durch. Ohne den Cast aber nicht.
wetter[i].WSpeed   =  3.6 * (double)list[i]["wind"]["speed"];

ich hoffe doch, dass die Json Klasse einen Destruktor hat, der den 
dynamischen Speicher wieder frei gibt, wenn die Routine verlassen wird. 
Due legt das nämlich auf dem Heap an wie ich meine und nicht auf dem 
Stack.

[Rufus Τ. F. (rufus)]

Christian J. schrieb:
> Ich hoffe doch, dass die Json Klasse einen Destruktor hat

Du hast die Sourcen - sieh sie Dir an.

[Christian J. (Gast)]

Naja, einfach geht es nicht mehr als hier:
https://arduinojson.org/v5/assistant/

Man trägt seinen Satz ein und bekommt alles geliefert mit fertigem Code, 
um kreuz und quer darauf zu zu greifen.....

[Rufus Τ. F. (rufus)]

Christian J. schrieb:
> Naja, einfach geht es nicht mehr als hier:

Was es nicht alles gibt. Faszinierend.