ein Int-Array einer Funktion

[oderlachs]

Hallo ,
ich Suche mal wieder Hilfe oder Anregung.
ich habe vor 3 Messwerte(MW) in einem Intarray MW[] beim Funktionsaufruf zu speichern und dieses Array als Return zurück zu bekommen, es geht nicht. Was mache ich falsch:

int mess()
{
int MW[3];  
MW[0]  = analogRead(IR_RFL_V) ;
MW[1]  = analogRead(IR_RFL_L) ;
MW[2]  = analogRead(IR_RFL_R) ;
return     MW[];
}

Hier der Aufruf im Programm:

void loop()
{....
IR_RFL_I[] = mess();
....
}

IR_RFL_I ist so definiert:

int IR_FL_I[3];

Ich finde nicht den Fehler, warum es nicht zu kompilieren geht....

Gruss Gerhard

[Peter(TOO)]

Hallo Gerhard,

1. int MW[3]; ist eine automatische lokale Variable und wird auf den Stack abgelegt wenn die Funktion aufgerufen wird.
Wenn die Funktion mit return beendet wird, wird aber er Stack wieder abgeräumt, inklusive der Variablen MW.
Abhilfe:
entweder
a) static int MW[3];
b) MW als globale Variable deklarieren
c) Die Funktion mess() so schreiben, dass das Array als Zeiger auf das Array übergeben wird. Dann muss der Speicherplat für MW im AUfrufenden Teil deklariert werden.

2. Die Funktion soll einen int zurückgeben.
MW[] ist aber ein Zeiger auf eine Array aus int.
MW[] entspricht &MW[0]
Du müsstest also
int *mess()
schreiben, dann sollte das Compilieren gehen, aber das zurückgelieferte Array kann dann irgendwelche Random Werte enthalten. Es kann dann manchmal die gewünschten Werte enthalten und, wenn z.B. ein Interrupt auftritt, nur noch irgendwelchen Schrott.

MfG Peter(TOO)

[Sisor]

oder so:

...
int MW[3];//globales Variablenfeld
...
void updateMW(){
MW[0]  = analogRead(IR_RFL_V) ;
MW[1]  = analogRead(IR_RFL_L) ;
MW[2]  = analogRead(IR_RFL_R) ;
}
...
void loop() {
....
updateMW();
....
}

[Peter(TOO)]

Hallo,
Ist zwar die einfachste, aber leider auch unschönste Variante:

- Seiteneffekte sind nun mal unschön:
In einem grösseren Projekt, wird schnell übersehen, dass update() die globale Variable verändert.
Jeder darf sich ausmalen, was man da an Zeit zur Fehlersuche aufwenden kann, besonders wenn ein Fremder etwas am Code ändern muss.

- unflexibel
Am flexibelsten ist die Variante bei welcher das Array über einen Zeiger übergeben wird.
Man kann beliebig viele Messwerte abspeichern. Zudem ist es auch egal, ob man die Messwerte Global. lokal oder auf dem Heap ablegen will, update muss dazu nicht verändert werden.

MfG Peter(TOO)

[oderlachs]

Ich habe es auch so erst ein mal gelöst..:

void mess()
{
 
IR_FL_I[0]  = analogRead(IR_RFL_V);
IR_FL_I[1]  = analogRead(IR_RFL_L);
IR_FL_I[2]  = analogRead(IR_RFL_R);

}

int IR_FL_I[3] ist in meiner Lib zum Projekt global deklariert...

praktische Erfahrungen habe ich noch nicht damit..muss noch was löten, montieren...usw..

Danke für Eure Hilfe..
ich werde berichten wie ich weiter komme...

Gerhard

Hier mein bisheriger CODE zum Projekt

[Wsk8]

void mess(int *arr) {
arr[0]  = analogRead(IR_RFL_V) ;
arr[1]  = analogRead(IR_RFL_L) ;
arr[2]  = analogRead(IR_RFL_R) ;
}

void loop()
{....
mess(MW);
....
}

Sollte in deinem Fall die beste Lösung sein. Globale Variablen sollte man wenn möglich vermeiden.
Funktion ist natürlich nicht sicher, da keine Überprüfung des Arrays stattfindet.

mfg