zwei Rückgabewerte einer Funktion?!

[Peter(TOO)]

Hallo,

Was mache ich denn falsch, wenn ich mein Array nicht zurück kann ?
Ich möchte gerne "Uhrzeit" an die Funktion zurück geben...

Wie schon gesagt, musst du einen Zeger zurück geben.

Aber du hast noch zwei ganz grobe Fehler!

uint16_t TimeToString(uint8_t Stunden, uint8_t Minuten, uint8_t Sekunden)
{
    char Uhrzeit[9];

    Uhrzeit[0] = (Sekunden >> 4)  + 0x30;
    Uhrzeit[1] = (Sekunden & 0x0f) + 0x30;
    Uhrzeit[2] = ':';
    Uhrzeit[3] = (Minuten >> 4)  + 0x30;
    Uhrzeit[4] = (Minuten & 0x0f) + 0x30;
    Uhrzeit[5] = ':';        
    Uhrzeit[6] = (Stunden >> 4)  + 0x30;
    Uhrzeit[7] = (Stunden & 0x0f) + 0x30;
    Uhrzeit[8] = '\0';

    return(Uhrzeit);

}// Ende TimeToString

1.
char Uhrzeit[9];
Ist eine lokale Auto-Variable!

Beim Aufruf von TimeToString() wird diese auf dem Stack angelegt und nach verlassen von TimeToString() wird der Speicher wieder freigeben!
Wenn du nun eine andere Funktion aufrufst oder eine Interruptroutine ausgeführt wird, werden die Werte überschrieben und due hast nur noch Datensalat.
Also übe mal, wie man eine gute Salatsauce macht

Wenn du
static char Uhrzeit[9];
verwendest, umgehst du dieses Problem, dann wird Uhrzeit im allgemeinen Speicher angelegt.
Allerdings ändert jeder Aufruf von TimeToString() den Inhalt von Uhrzeit.


2. der Rückgabewert von TimeToString() ist nicht mehr
uint16_t
sondern
char *TimeToString(uint8_t Stunden, uint8_t Minuten, uint8_t Sekunden)

MfG Peter(TOO)

[schorsch_76]

Du kannst das auch in ein struct packen und dann dieses strukt zurück geben.

struct Uhrzeit {
 char sec;
 char min;
 char hour;
 char day;
 char month;
 char year;
};

...

Uhrzeit MyFunction()
{
 Uhrzeit ret;
 ret.min = ...;
 ret.sec = ...;

 ...
 return ret;
}

Gruß
Georg

[Peter(TOO)]

Hallo Georg,

Du hast jetzt aber immer noch den Fehler mit der auto Variablem.
Nach return wird der Speicher freigegeben!!

MfG Peter(TOO)

[schorsch_76]

Nein, denn es wird by value übergeben und nicht by reference.

EDIT:
Link
http://courses.washington.edu/css342...32/passby.html
http://de.wikipedia.org/wiki/Wertparameter

- - - Aktualisiert - - -

Ja, der Speicher wird freigegeben, aber es wird eine tiefe Kopie gemacht für den Aufrufer. Wenn ich

return &ret;

mache, dann habe ich ein Problem mit diesem Speicher.