sscanf liest ersten parameter nicht

[Seppel V. (seppelv)]

Hallo,

mit sscanf(STM32) will ich mehrere Parameter einlesen, das Problem ist 
dass er den ersten offensichtlich nicht einliest und der Parameter ist 
Null, sollte aber 1 sein. Die nachfolgenden werden problemlos 
eingelesen.

Woran kann das liegen?

Vielen Dank, Grüße, Seppel

---------------------------------
Gesendeter String: "PARAM_POS 1 2 33.3"

---------------------------------
Code:

uint16_t axis = 0U;
uint16_t iter = 0U;
float parameter = 0.0;

#include <string.h>

if (strncmp((const char *)usbRxBuffer,"PARAM_POS",9)==0U)
{
  sscanf(&usbRxBuffer[10], "%u %u %f", &axis, &iter, &parameter);
  SensorPosSetParam(axis,iter,parameter);
}

---------------------------------
Ergebnis, axis wird nicht eingelesen, aber warum?:

usbRxBuffer = PARAM_POS 1 2 33.3\0\0......

axis = 0

iter = 2

param = 33.2999992

[Clemens L. (c_l)]

%u ist unsigned int.

Für uint16_t brauchst du so etwas:

#include <inttypes.h>

sscanf(&buffer[10], "%"PRIU16" %"PRIU16" %f", &...);

[Seppel V. (seppelv)]

Clemens L. schrieb:
> %u ist unsigned int.
>
> Für uint16_t brauchst du so etwas:#include <inttypes.h>
> sscanf(&buffer[10], "%"PRIU16" %"PRIU16" %f", &...);

Hallo,

könntest Du bitte den Code vervollständigen und mir erklären warum das 
mit uint16_t wie auch uint32_t nicht funktioniert?

Ich denke dass ich nicht der einzige bin der darüber stolpert, da sind 
vollständige Posts, mit funktionalem Code und Erklärung sinnvoll. Auch 
die innen liegenden Anführungszeichen, was tun die?

Vielen Dank, Grüße Seppel

[J. S. (jojos)]

der Code funktioniert wenn ich den auf online gdb eingebe:

https://onlinegdb.com/nD89zYOS1D

#include <stdio.h>
#include <stdint.h>

const char usbRxBuffer[] = "PARAM_POS 1 2 33.3";

int main()
{
    printf("Hello World");
    
    uint16_t axis = 0U;
    uint16_t iter = 0U;
    float parameter = 0.0;

    #include <string.h>

    if (strncmp((const char *)usbRxBuffer,"PARAM_POS",9)==0U)
    {
        sscanf(&usbRxBuffer[10], "%u %u %f", &axis, &iter, &parameter);
        printf("result: %d %d %f", axis, iter, parameter);
    }


    return 0;
}

Bei uint16_t zwar mit Warnung, aber Ergebnis ist korrekt.

da würde ich davon ausgehen das der Inhalt von usbRxBuffer doch nicht so 
ist wie gedacht. Was sagt der Debugger? Oder ein printf vom usbRxBuffer?

[Rolf M. (rmagnus)]

Clemens L. schrieb:
> %u ist unsigned int.
>
> Für uint16_t brauchst du so etwas:
> #include <inttypes.h>
> sscanf(&buffer[10], "%"PRIU16" %"PRIU16" %f", &...);

Nicht PRIU16, sondern SCNu16. PRI -> printf-Familie, SCN -> 
scanf-Familie. Außerdem würde ich da Leerzeichen drumherum machen. In C 
ist das zwar nicht unbedingt nötig, aber in C++ bekommt man ohne eine 
Fehlermeldung.

Seppel V. schrieb:
> könntest Du bitte den Code vervollständigen und mir erklären warum das
> mit uint16_t wie auch uint32_t nicht funktioniert?

Weil die Größe nicht stimmt, zumindset bei uin16_t. %u ist ist für 
unsigned int, der auf einem ARM normalerweise 32 Bit ist. Du übergibst 
sscanf einen Zeiger auf einen 16-Bit-Wert, aber sscanf versucht 
entsprechend deiner Angabe, von dort stattdessen 32 Bit zu lesen, was 
natürlich falsch ist.
Für die typedefs aus stdint.h wie uint16_t gibt es entsprechende Makros, 
die den richtigen Format-Specifier für den jeweiligen Typen 
bereitstellen. Statt also "%u" musst du "%" SCNu16 schreiben, um den 
richtigen Format-Specifier für uint16_t in scanf zu bekommen.

[J. S. (jojos)]

angepasst:
https://onlinegdb.com/oAn-LR6e7

[Frank M. (ukw) (Moderator)]

Unabhängig vom obigen Size-Problem der Parameter:

Es empfiehlt sich (fast) immer, den Return-Wert von sscanf() zu prüfen. 
Hier wird die Anzahl der tatsächlich gefüllten Parameter zurückgegeben. 
Diese sollte man als erstes Kriterium für ein erfolgreiches Matching 
heranziehen.

[Seppel V. (seppelv)]

Hallo,

was funktioniert:

#include <stdio.h>
#include <stdint.h>

char charBuffer1[64];
uint16_t axis = 0U;
uint16_t iter = 0U;

sscanf(&usbRxBuffer[0], "%s %"PRIu16" %"PRIu16" %f", charBuffer1, &axis, &iter, &parameter);

Vielen Dank, Grüsse, Seppel

[Rolf M. (rmagnus)]

Ist trotzdem falsch. Es muss immer noch SCNu16 heißen, nicht PRIu16, und 
ja, das macht einen Unterschied.

[J. S. (jojos)]

nicht portabel, aber auf dem STM32 funktioniert das auch:

  uint16_t axis = 0U;
  uint16_t iter = 0U;
  float parameter = 0.0f;

  if (strncmp((const char *)usbRxBuffer,"PARAM_POS",9)==0U)
  {
      sscanf(&usbRxBuffer[10], "%hu %hu %f", &axis, &iter, &parameter);
  }

lässt man das h weg, dann schreibt sscanf Variablen auf dem Stack 
kaputt.