Pointer Array schreiben / lesen

[Helmut A. (Gast)]

typedef struct  
{
  /*
  *  Zu einer bestimmten Uhrzeit gemessene Temperatur
  */
  int8_t temp;
  
  /*
  *  Uhrzeit des gemessenen Wertes
  */
  char *time;
  
}tempStamp_t;

tempStamp_t stamp[] =
{
  { -12 , "00:00:00" },  // Niedrigst gemessene Temperatur
  { 12  , "00:00:00" },  // Höchst gemessene Temperatur
};

Hallo,

wenn Ich folgendes versuche..

    if ( sts3x.actual < sts3x.lowest )
    {
      sts3x.lowest = sts3x.actual;
      stamp[0].temp = sts3x.lowest;
      stamp[0].time = (char*)bcd_ttostr(rtc.hour,rtc.minute,rtc.second);
    }  
    else if ( sts3x.actual > sts3x.highest )
    {
      sts3x.highest = sts3x.actual;
      stamp[1].temp = sts3x.highest;
      stamp[1].time = (char*)bcd_ttostr(rtc.hour,rtc.minute,rtc.second);
      
      //stamp[1].time = "16:10:00";
    }

Und das ganze dann auslese, habe Ich in stamp[1].time genau die gleiche 
Uhrzeit wie in stamp[0]. Was mache Ich hier falsch?
Irgendetwas stimmt mit meiner Zeiger Thematik nicht so ganz..

[Felix U. (ubfx)]

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. (Gast)]

Felix U. schrieb:
> 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.

Hier ist die Funktion

/* contains the time string */
char time[9];
char date[14];


char *bcd_ttostr(uint8_t hour, uint8_t min, uint8_t sec)
{
  /* BCD to ASCII */
  time[0] = (hour >> 4) + 48;
  time[1] = (hour & 0x0F) + 48;
  time[2] = ':';
  time[3] = (min >> 4) + 48;
  time[4] = (min & 0x0F) + 48;
  time[5] = ':';
  time[6] = (sec >> 4) + 48;
  time[7] = (sec & 0x0F) + 48;
  time[8] = '\0';
  
  return time;
}

[Felix U. (ubfx)]

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:

typedef struct  
{
  /*
  *  Zu einer bestimmten Uhrzeit gemessene Temperatur
  */
  int8_t temp;
  
  /*
  *  Uhrzeit des gemessenen Wertes
  */
  char time[9];
  
}tempStamp_t;

und machst dann

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:

void bcd_ttostr(char* time, uint8_t hour, uint8_t min, uint8_t sec)
{
  /* BCD to ASCII */
  time[0] = (hour >> 4) + 48;
  time[1] = (hour & 0x0F) + 48;
  time[2] = ':';
  time[3] = (min >> 4) + 48;
  time[4] = (min & 0x0F) + 48;
  time[5] = ':';
  time[6] = (sec >> 4) + 48;
  time[7] = (sec & 0x0F) + 48;
  time[8] = '\0';
}

Und den Aufruf dann direkt so:

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

[Helmut A. (Gast)]

Felix U. schrieb:
> 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.

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

[Daniel A. (daniel-a)]

"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"}.

[Felix U. (ubfx)]

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)]

Daniel A. schrieb:
> "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[8]", oder
> initialisere es mit einem compound literal (char[]){"00:00:00"}.
Meinst du so?

tempStamp_t stamp[] =
{
  { 0 , (char[]){"00:00:00"} },
  { 0 , (char[]){"00:00:00"} },      
};

[Helmut A. (Gast)]

Felix U. schrieb:
> 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.
Ich dachte das tue Ich hiermit

tempStamp_t stamp[ ] =
{
  { -12 , "00:00:00 " },  // Niedrigst gemessene Temperatur
  { 12  , "00:00:00 " },  // Höchst gemessene Temperatur
};

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

[Daniel A. (daniel-a)]

@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.

[Felix U. (ubfx)]

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. (Gast)]

Felix U. schrieb:
> 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.
Wie kann man das noch machen?

char *buff[2][9]

Würde das gehen?

[Felix U. (ubfx)]

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. (Gast)]

Felix U. schrieb:
> 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?

Diese Funktion benutze Ich schon an anderen Stellen erfolgreich so muss 
Ich nicht ständig noch einen Parameter übergeben und setze so nur auf 
einen Puffer. Jetzt wirds mir auch klar wieso das nicht funktioniert.

[Helmut A. (Gast)]

Könnte es mit einem Zeiger nicht auch funktionieren?

typedef struct  
{
  /*
  *  Zu einer bestimmten Uhrzeit gemessene Temperatur
  */
  int8_t temp;
  
  /*
  *  Uhrzeit des gemessenen Wertes
  */
  char time[9];
  
  /*
  *  Zeiger auf den Temperatur String
  */
  char *ptr;
  
}tempStamp_t;

tempStamp_t stamp[2] =
{
  { 0 , "00:00:00" , stamp[0].time },
  { 0 , "10:00:00" , stamp[1].time },
};

[Daniel A. (daniel-a)]

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.

typedef struct  
{
  int8_t temp;
  char time[9];
}tempStamp_t;
...
memcpy(stamp[0].time,bcd_ttostr(rtc.hour,rtc.minute,rtc.second),9);

[steve41993 (Gast)]

Hallo Helmut A.

ich würde es folgendermaßen versuchen, wenn du die Funktion 'bcd_ttostr' 
nicht verändern willst:

typedef struct  
{
  /*
  *  Zu einer bestimmten Uhrzeit gemessene Temperatur
  */
  int8_t temp;
  
  /*
  *  Uhrzeit des gemessenen Wertes
  */
  char time[9];
  
}tempStamp_t;

tempStamp_t stamp[] =
{
  { -12 , "00:00:00" },  // Niedrigst gemessene Temperatur
  { 12  , "00:00:00" },  // Höchst gemessene Temperatur
};

    if ( sts3x.actual < sts3x.lowest )
    {
      sts3x.lowest = sts3x.actual;
      stamp[0].temp = sts3x.lowest;
      memcpy(stamp[0].time, (char*)bcd_ttostr(rtc.hour,rtc.minute,rtc.second), 9);
    }  
    else if ( sts3x.actual > sts3x.highest )
    {
      sts3x.highest = sts3x.actual;
      stamp[1].temp = sts3x.highest;
      memcpy(stamp[1].time, (char*)bcd_ttostr(rtc.hour,rtc.minute,rtc.second), 9);
      //stamp[1].time = (char*)bcd_ttostr(rtc.hour,rtc.minute,rtc.second);
      //so würde dein stamp[1].time Pointer auf den Buffer zeigen. Der Inhalt wird ja nicht in stamp[1].time kopiert.
      
      //stamp[1].time = "16:10:00";
    }

Dein Problem ist, dass nach dem beide stamp[0].time und stamp[1].time 
gesetzt wurden diese auf den Buffer zeigen. Somit repräsentieren diese 
immer den aktuellen Inhalt des Buffers.

Grüße
steve41993

[Helmut A. (Gast)]

So klappt es jetzt ja auch..

strcpy ( stamp[1].time , bcd_ttostr(rtc.hour,rtc.minute,rtc.second) );