Komische Werte im char array

Andi schrieb:
> Weiß jemand woher dieses Zeichen kommt?

ja weil du keine ende vom String hast. Es fehlt ein 0 Byte.

Aber das ganze ist viel zu kompliziert!

das ganze umkopieren ist überhaupt nicht notwenig, atoi hört von selber 
auf zu lesen wenn kein Digit mehr kommt.

Peter II schrieb:
> Andi schrieb:
>> Weiß jemand woher dieses Zeichen kommt?
>
> ja weil du keine ende vom String hast. Es fehlt ein 0 Byte.
>
> Aber das ganze ist viel zu kompliziert!
>
> das ganze umkopieren ist überhaupt nicht notwenig, atoi hört von selber
> auf zu lesen wenn kein Digit mehr kommt.

Warum ist dann beim Array foo2 nie so ein Zeichen am Ende?

Danke für den Hinweis mit atoi! Da kann ich mir die Hälfte sparen ;)

1

#include "string.h"

2

3

#define SERIAL_BUFFER_SIZE 10

4

5

int main(void) {

6

  char data[SERIAL_BUFFER_SIZE] = {'1','2','3','4','5','6','.','7','8','\0'};

7

  uint8_t indexDot;

8

9

  dot = strnchr (data, SERIAL_BUFFER_SIZE, '.');

10

  data [dot] = '\0';

11

12

  uint32_t integer_foo1 = atol(data);

13

  uint32_t integer_foo2 = atol(&data[dot + 1]);

Fehlerbehandlung fehlt natürlich noch.

Wobei atol natürlich auch ein wenig blöd ist, wenn Du den vollen 
Wertebereich von uint32_t nutzen willst. Aber gut. Das Ganze ist sowieso 
ein Blödsinn - äh, ein Übungscode zum wegschmeissen, da schadet das nun 
auch nichts.

A. H. schrieb:
> An dieser Stelle sollte Dein Compiler laut meckern:
>   char foo1[indexDot];
>
> Wenn Du ein Feld definieren willst muss die Größe zur Compilezeit
> bekannt sein. indexDot ist aber eine Variable. Definiere das Feld auch
> zu SERIAL_BUFFER_SIZE und probiere es noch mal.
>
> Die schon erwähnte Bemerkung mit dem fehlenden '\0' Zeichen stimmt
> natürlich auch. Du müsstest mindestens indexDot + 1 als Feldgröße
> anlegen.

Der Compiler sagt dazu nichts...seltsam. Ich dachte die Terminierung mit 
dem '\0'-Zeichen wird nur für die Interpretation als String benötigt?

A. H. schrieb:
> An dieser Stelle sollte Dein Compiler laut meckern:
>   char foo1[indexDot];
>
> Wenn Du ein Feld definieren willst muss die Größe zur Compilezeit
> bekannt sein.

nein muss es nicht. Im Aktuellen C und C++ ist das zulässig.

Andi schrieb:
> Ich dachte die Terminierung mit
> dem '\0'-Zeichen wird nur für die Interpretation als String benötigt?

sie immer notwendig wenn man in C mit Strings arbeitet. Woher soll denn 
sonst atoi wissen wo schluss ist.

Bitflüsterer schrieb:
> #include "string.h"
>
> #define SERIAL_BUFFER_SIZE 10
>
> int main(void) {
>   char data[SERIAL_BUFFER_SIZE] =
> {'1','2','3','4','5','6','.','7','8','\0'};
>   uint8_t indexDot;
>
>   dot = strnchr (data, SERIAL_BUFFER_SIZE, '.');
>   data [dot] = '\0';
>
>   uint32_t integer_foo1 = atol(data);
>   uint32_t integer_foo2 = atol(&data[dot + 1]);
>
> Fehlerbehandlung fehlt natürlich noch.

data [dot] = '\0';

ist nicht notwendig, kann sogar stören. Wenn man mit const arbeitet.

Peter II schrieb:
> nein muss es nicht. Im Aktuellen C und C++ ist das zulässig.

1

#include <stdint.h>

2

3

int main(void)

4

{

5

  uint8_t i;

6

  uint8_t array[i];

7

  for(i = 0; i < 10; i++)

8

  {

9

    array[i] = 0;

10

  }

11

  return 0;

12

}

mit gcc 4.8.1, -Wall -Werror -pedantic -pedantic-errors -std=c++11 
compiliert gibt:

1

error: ISO C++ forbids variable length array 'array' [-Wvla]

Scheint also doch nicht ganz so zulässig zu sein.

Andi schrieb:
> Der Compiler sagt dazu nichts...seltsam.
Der compiler sagt da nur was zu wenn du mit -pedantic compilierst. Ohne 
-pedantic hält er schön die klappe.

Grüße

Kaj schrieb:
> mit gcc 4.8.1, -Wall -Werror -pedantic -pedantic-errors -std=c++11
> compiliert gibt:error: ISO C++ forbids variable length array 'array'
> [-Wvla]
> Scheint also doch nicht ganz so zulässig zu sein.

bei C geht es

-Wall -Werror -pedantic -pedantic-errors -std=c99

test.c:6:11: error: variable 'array' set but not used 
[-Werror=unused-but-set-variable]
   uint8_t array[i];

Peter II schrieb:
> bei C geht es
Ja, habs grad festgestellt, liegt an der stelle am c++ compiler :-/

Tom K. schrieb:
> in C++ bisher nur als gcc-Erweiterung
Danke für den hinweis