zwei Rückgabewerte einer Funktion?!

[Janiiix3]

Okay, dass scheint zu klappen. Danke !

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

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

[askazo]

Du kannst in C ein Array nicht direkt als Rückgabewert einer Funktion verwenden.
Die gängige Lösung ist eigentlich, einen Zeiger auf das Array zurückzugeben.
Dafür solltest Du Dich aber mit Zeigern auskennen.

Allerdings stellt Dir die C-Bibliothek schon eine Funktion zur Verfügung, die genau das kann, was Du mit dieser Funktion machen möchtest...

#include <stdio.h>

uint8_t Stunden;
uint8_t Minuten;
uint8_t Sekunden;

char Uhrzeit[9];

// Hier kommt die Berechnung der Uhrzeit hin
...

printf(Uhrzeit,"%02u:%02u:%02u",Stunden,Minuten,Sekunden);

Siehe auch http://www.mikrocontroller.net/artic...ntf_und_printf
und http://home.htw-berlin.de/~junghans/cref/FUNCTIONS/format.html

Gruß,
askazo

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