Pointer Array schreiben / lesen

Schwer zu sagen, wenn man nicht weiß, wie sich bcd_ttostr verhält. 
Allerdings würde ich davon ausgehen, dass der Pointer, den die Funktion 
zurückgibt auf einen internen statischen Puffer zeigt, der 
logischerweise mit jedem Aufruf überschrieben wird. Also würde ich mal 
ein strrep um den Aufruf setzen.

Helmut A. schrieb:
> Hier ist die Funktion

Dann ist es so wie ich vermute habe. Der "time" Puffer wird bei jedem 
Aufruf überschrieben. Du musst ihn also kopieren bevor du die Funktion 
neu aufrufst. Ein Ansatz dafür wäre strrep.

Das bringt dich aber in die unangenehme Situation, dynamisch Speicher 
allozieren und freigeben zu müssen.

Entweder du änderst also deine Struktur so, dass du die RTC-Werte als 
Zahlen speicherst und dann nur für die Ausgabe in Text umwandelst, oder 
du erzeugst direkt für jedes Element einen ausreichend großen Puffer:

1

typedef struct  

2

{

3

  /*

4

  *  Zu einer bestimmten Uhrzeit gemessene Temperatur

5

  */

6

  int8_t temp;

7

8

  /*

9

  *  Uhrzeit des gemessenen Wertes

10

  */

11

  char time[9];

12

13

}tempStamp_t;

und machst dann

1

strcpy(stamp[0].time, (char*)bcd_ttostr(...));

oder, wenn du gerade dabei bist, kannst du dir die unnötige Kopieraktion 
auch sparen und die Funktion so ändern:

1

void bcd_ttostr(char* time, uint8_t hour, uint8_t min, uint8_t sec)

2

{

3

  /* BCD to ASCII */

4

  time[0] = (hour >> 4) + 48;

5

  time[1] = (hour & 0x0F) + 48;

6

  time[2] = ':';

7

  time[3] = (min >> 4) + 48;

8

  time[4] = (min & 0x0F) + 48;

9

  time[5] = ':';

10

  time[6] = (sec >> 4) + 48;

11

  time[7] = (sec & 0x0F) + 48;

12

  time[8] = '\0';

13

}

Und den Aufruf dann direkt so:

1

bcd_ttostr(stamp[0].time, 1, 2, 3);

"00:00:00" ist eine ein constantes char array, dass nicht verändert 
werden darf. Aus historischen gründen darf ein char* damit initialisiert 
werden. Der compiler wird vermutlich die beiden identischen 
Stringconstanten nur einmal speichern, wodurch stamp[0].time == 
stamp[1].time. Ändere entweder "char *time" zu "char time[9]", oder 
initialisere es mit einem compound literal (char[]){"00:00:00"}.

Helmut A. schrieb:
> Verstehe Ich jetzt mal nicht ganz.
> Ich möchte doch das der "time" Puffer überschrieben wird wenn sich die
> Uhrzeit ändert.

In deiner Version gibt es nur einen einzigen Time-Puffer, auf den sowohl 
der Eintrag für den Minimal, als auch der für den Maximalwert zeigt. Der 
einzelne Puffer kann logischerweise auch nur eine einzelne Uhrzeit 
beinhalten.

Du brauchst also mindestens zwei Puffer, um zwei Uhrzeiten speichern zu 
können.

@Helmut A. (Gast)

Ich hatte übersehen, dass bcd_ttostr nur einen Pointer auf einen anderen 
Buffer zurück gibt, und nicht den bestehenden übergeben bekommt und 
überschreibt. Sorry führ die Verwirrung.

Helmut A. schrieb:
> Sage Ich dem Compiler dadurch nicht das er mir für "00:00:00" 9 Bytes
> reservieren soll?

Nein. Das musst du schon in der Struktur machen.

Helmut A. schrieb:
> char *buff[2][9]
> Würde das gehen?

char buff[2][9] würde noch Sinn ergeben. Warum nimmst du nicht die 
Variante die ich oben vorgeschlagen habe?

Helmut A. schrieb:
> Könnte es mit einem Zeiger nicht auch funktionieren?

Der Zeiger nützt dir genau garnichts. Du weisst ja bereits, wo deine 
zwei Buffer sind, und wen du wie vorher wieder die Zeiger auf den 
gleichen Buffer zeigen lässt hasst du wieder das gleiche Problem. Du 
könntest aber memcpy verwenden, sofern du die Funktion bcd_ttostr nicht 
in unterschiedlichen Threads oder in Signal handlern nutzt.

1

typedef struct  

2

{

3

  int8_t temp;

4

  char time[9];

5

}tempStamp_t;

6

...

7

memcpy(stamp[0].time,bcd_ttostr(rtc.hour,rtc.minute,rtc.second),9);