JSON parsen mit AVR

[Gerhard (Gast)]

Komplettes JSON passt wohl nicht so richtig in einen Mega88/168, aber 
letzendlich brauch ich das auch gar nicht. Sind auch nur sehr 
überschaubare Datensätze und leicht auseinander zu puzzeln. Allerdings 
kommt immer noch bisschen was dazu, und es unterscheidet sich auch von 
Gerätetyp zu Gerätetyp (RS485-Bus)

So funktioniert es bestens, gefällt mir aber nicht.
#define RD_cmd 1
#define VER_cmd 2
#define CONFIG_cmd 3
.
.
.
unsigned char parse_command (char * s)
{unsigned char command=0;
if (!(strncmpf (s, "\"RD\":", 5)))
   {command=RD_cmd;
    JsonPtr+=5;
   }
   else if (!(strncmpf (s, "\"VER\":", 6)))
      {...
      }
   .
   .

return (command);
}
Vergleicht also mit s mit <"RD":> (5 Zeichen) und erhöht bei Erfolg 
JsonPtr um 5. Und genau die magic number 5 ( oder 6, andere sind z.T. 
noch erheblich länger) sind mein  Problem. Lästig und fehleranfällig das 
abzuzählen....
Ideal wäre, auch den Vergleichsstring per #define anzugeben und der 
Compiler soll Vergleichslänge/offset selbst auskaspern. Geht das 
irgendwie?

[B. S. (bestucki)]

Gerhard schrieb:
> Compiler soll Vergleichslänge/offset selbst auskaspern. Geht das
> irgendwie?

static const char MyCommand[] = "Command";
static const size_t MyCommandSize = sizeof(MyCommand);
static const size_t MyCommandLength = sizeof(MyCommand) - 1; // == strlen(MyCommand)

Damit kannst du alle Magic Numbers erschlagen, auch die Strings selbst. 
Ich empfehle die Variablen als static zu deklarieren, da ich annehme, 
dass die Variablen nur in dieser Funktion benötigt werden. Ansonsten 
werden sie bei jedem Funktionsaufruf neu initialisiert. Die Alternative 
wäre, die Variablen global zu definieren, evt. auch mit static.

[Johann L. (gjlayde)]

Gerhard schrieb:
> Lästig und fehleranfällig das abzuzählen....

Nimm strlen zusammen mit Makro oder Inline-Funktion.

#define strncmpf(S, STR) \
    strncmp_P (S, PSTR (STR), strlen (STR))

bool func (const char *str)
{
    if (!strncmpf (str, "abc"))
        return true;
    return false;
}

Ein Problem ist, dass es wegen PSTR keine Inline-Funktion sein kann. 
Evtl. kannst du das dann noch ausbauen so dass str um strlen(...) erhöht 
wird.

[Johann L. (gjlayde)]

...etwa so in GNU-C99:
 

#define MATCH(S, STR)                                       \
  ({                                                        \
      match_len = strlen (STR);                             \
      match = ! strncmp_P (S, PSTR (STR), match_len);       \
      match;                                                \
  })

bool func (const char *str)
{
    bool match;
    size_t match_len;
    
    if (MATCH (str, "abc"))
    {
        str += match_len;
        
        if (MATCH (str, "blabla"))
            return true;
    }
    return false;
}

Makros sind zwar hässlich, aber an dieser Stelle braucht man nur 
eines...

[Gerhard (Gast)]

Be S. schrieb:
> Gerhard schrieb:
>> Compiler soll Vergleichslänge/offset selbst auskaspern. Geht das
>> irgendwie?
> static const char MyCommand[] = "Command";
> static const size_t MyCommandSize = sizeof(MyCommand);
> static const size_t MyCommandLength = sizeof(MyCommand) - 1; // ==
> strlen(MyCommand)
> Damit kannst du alle Magic Numbers erschlagen, auch die Strings selbst.
> Ich empfehle die Variablen als static zu deklarieren, da ich annehme,
> dass die Variablen nur in dieser Funktion benötigt werden. Ansonsten
> werden sie bei jedem Funktionsaufruf neu initialisiert. Die Alternative
> wäre, die Variablen global zu definieren, evt. auch mit static.

Perfekt, hätte ich selber drauf kommen können :-)
Danke.