Werteübergabe an Funktion()

Häh?

Verstehe kein Wort, sorry.

Wo soll denn der Rückgabewert herkommen?
Aus dem Code, den du versteckst?

Dicker Bug in Zeile 42.

Klaus W. schrieb:
> Wo soll denn der Rückgabewert herkommen?

Wie beschrieben aus dem Zuordnungs-Block.

Der Block ist nicht das Problem - daher weg gelassen.
Da bei aktivieren Testvariante (*2) alles Folgende funktioniert, liegt 
der Fehler vor (*2) davor!
Daher vermute ich, dass es sich um die Werteübergabe der Funktion 
handelt. Aber verstehen tuh ich nicht.

Frank M. schrieb:
> Dicker Bug in Zeile 42.

Gehts genauer?

__Son´s B. schrieb:
> Frank M. schrieb:
>> Dicker Bug in Zeile 42.
>
> Gehts genauer?

Nein. Weil Du die dafür relevanten Informationen nicht zeigst.

Jonathan schrieb:
> Naiv?

Weder sonderlich Niveau-, noch Humorvoll.

__Son´s B. schrieb:
> Frank M. schrieb:
>> Dicker Bug in Zeile 42.
>
> Gehts genauer?

Das ist Allgemeinwissen:
https://de.m.wikipedia.org/wiki/42_(Antwort%29

argc,argv schrieb:
> eigentlich müßte der Compiler meckern...
> ist main() bekannt, daß SpgUeberwach_Auswertung() als Argument
> ein uint16_t will?
>
> versuchs mal in main mit:
>
> StatusNeu = SpgUeberwach_Auswertung((uint16_t)MittelWert);

__Son´s B. schrieb:
> _AUSZUG_
> int main()
> { ...
>   uint16_t MittelWert; //Bereich 0-1023

Deklaration steht im main()
Reicht das nicht?

argc,argv schrieb:
> noch nie was debugged?

Leider nein!
Auch mit dem Simulator habe ich mich noch nicht beschäftigt.

Benutze einfach den Debugger. Lass dein Programm bis zur entsprechenden 
Stelle laufen. Mit einem Breakpoint hälst du ihn direkt in der Zeile vor 
dem Aufruf der Funktion an. Dann lässt du das Programm Zeile für Zeile 
weiterlaufen und schaust dir an, was in den jeweiligen Variablen drin 
steht.

Es kann passieren, dass dir der Compiler die Variablen wegoptimiert, 
dann musst du mit volatile arbeiten.

Nun kannst du sehen, was dein Programm genau macht. Denn es kann sein, 
dass es etwas völlig anderes (logisches) macht, was du nicht 
vorhergesehen hast.


Wenn du möchstest das man dir hier Hilfe (oder überhaupt erst helfen 
kann) dann reduziere deinen Code soweit, bis nur noch der Code übrig 
ist, der das Problem verursacht (und auch weiterhin noch verursachen 
muss). Dann poste diese Code komplett als Compilierbares Beispiel, dann 
können auch andere sich das ganze anschauen und Compilieren/Debuggen.
Bei deinem Code oben fehlen Beispielsweise alle wichtigen Stellen. Ich 
bin mir nämlich ziemlich sicher, dass der Fehler in den Zeilen zu finden 
ist, von denen du glaubst, das er dort nicht zu finden sei!

argc,argv schrieb:
> aber nicht die Deklaration der Funktion

uint8_t SpgUeberwach_Auswertung(uint16_t MittelWert)

Und wie verhält sich dein Programm, wenn du schreibst:

1

uint8_t SpgUeberwach_Auswertung(uint16_t MittelWert)

2

{

3

  static uint8_t LedAktuell; //Bereich 1-3

4

  uint8_t StatusNeu;  //Bereich 1-5

5

6

  // Hier folgt ein Block der die 

7

  // Eingangswerte (0-1023), 5 Zuständen (1-5) zuordnet.

8

9

  return StatusNeu;

10

}

11

12

int main()

13

{ ...

14

  uint16_t MittelWert; //Bereich 0-1023

15

  uint8_t StatusNeu; //Bereich 1-5

16

  ...

17

  MittelWert=700;  //Testvariante 0-1023(*1) 

18

  StatusNeu = SpgUeberwach_Auswertung(MittelWert);

19

  SpgUeberwach_Ausgang(StatusNeu);

20

}

Chris L. schrieb:
> Wenn du möchstest das man dir hier Hilfe (oder überhaupt erst helfen
> kann) dann reduziere deinen Code soweit, bis nur noch der Code übrig
> ist, der das Problem verursacht (und auch weiterhin noch verursachen
> muss). Dann poste diese Code komplett als Compilierbares Beispiel, dann
> können auch andere sich das ganze anschauen und Compilieren/Debuggen.

Siehe Anhang.
Danke schon mal!

Michael B. schrieb:
> Und wie verhält sich dein Programm, wenn du schreibst:uint8_t

>   MittelWert=700;  //Testvariante 0-1023(*1)
>   StatusNeu = SpgUeberwach_Auswertung(MittelWert);
>   SpgUeberwach_Ausgang(StatusNeu);

Genau hier (*1) wirft das Prog einen falschen Wert aus.
"MittelWert=700;" in der Funktion SpgUeberwach_Auswertung() (siehe *2) 
klappt einwandfrei! Daher kann ich mir nur ein Übergabeproblem 
vorstellen >>> aber wo ist der Fehler???

__Son´s B. schrieb:
> Siehe Anhang.

Ach du Kagge, immer noch das Programm dass sich auf 5 Zeilen kürzen 
lässt
und in dem unser bersison den Überblick verloren hat weil er aus 5 
Anweisungen mal eben 50 Anweisungen machte.

argc,argv schrieb:
> uint8_t SpgUeberwach_Auswertung(uint16_t MittelWert)
> {
>   static uint8_t LedAktuell; //Bereich 1-3
>   uint8_t StatusNeu;  //Bereich 1-5
>
>  printf("Mittelwert=%d\r\n",Mittelwert);
>
>  /* Rest */

Habe keine Terminal-Ausgabe. Bekomme ich auch nicht so schnell hin.

Die Funktion void SpgUeberwach_Ausgang(uint8_t StatusNeu) nimmt eine 
Byte Variable, wird aber mit "MittelWert" gefüttert welches als uint16_t 
deklariert ist. Da fällt dir nix auf?

Cyblord -. schrieb:
> Die Funktion void SpgUeberwach_Ausgang(uint8_t StatusNeu) nimmt eine
> Byte Variable, wird aber mit "MittelWert" gefüttert

Nö, wird sie nicht, sondern mit uint8_t StatusNeu.

Aber er hat sich völlig in den überzogenen if's in 
SpgUeberwach_Auswertung vergaloppiert und den Überblick verloren.

Da wird ein static uint8_t LedAktuell verglichen, das nie gesetzt wurde, 
und entweder 0 oder zufällig ist aber zu Beginn nie 1 oder 2 oder 3 hat.

Er weigert sich den Code so wie in diesem Forum schon gezeigt 
aufzuräumen und wundert sich über Fehler.

Dem Mann ist nicht zu helfen.

Cyblord -. schrieb:
> Die Funktion void SpgUeberwach_Ausgang(uint8_t StatusNeu) nimmt eine
> Byte Variable, wird aber mit "MittelWert" gefüttert welches als uint16_t
> deklariert ist. Da fällt dir nix auf?

Nicht wirklich...

"void SpgUeberwach_Ausgang(uint8_t StatusNeu)" wird mit StatusNeu (8bit) 
gefüttert. Die Funktion hat keinen Rückgabewert.
Im main() wird die Var. "uint8_t StatusNeu;" deklariert.
Was hat "StatusNeu" mit dem "MittelWert" zu tun?

Michael B. schrieb:
> Da wird ein static uint8_t LedAktuell verglichen, das nie gesetzt wurde,
> und entweder 0 oder zufällig ist aber zu Beginn nie 1 oder 2 oder 3 hat.

Ok!
Dein Hinweis zu LedAktuell ist richtig - die Zeile wurde bei 
Code-Umstrukturieren versehentlich gelöscht. Werde mich morgen drum 
kümmern.
(Rest deiner Behauptung ist daneben - zukünftig bitte einfach weg 
lassen!)

Da kommt man auch ganz durcheinander bei der wirren Namensgebung. Das 
ist eher ein Logikrätsel, als ein übersichtliches Programm.

__Son´s B. schrieb:
> (Rest deiner Behauptung ist daneben - zukünftig bitte einfach weg
> lassen!)

Tja, Wahrheiten werden von dir nicht gerne gehört.

Cyblord -. schrieb:
> Da kommt man auch ganz durcheinander bei der wirren Namensgebung. Das
> ist eher ein Logikrätsel, als ein übersichtliches Programm.

Ich habe noch nicht mal überhaupt ein Programm gesehen :-)

Doh! schrieb:
> Mal ganz davon abgesehen, dass static LedAktuell in der Auswertung nie
> gesetzt wird, aber immer überprüft ...

Programm läuft jetzt (fast) einwandfrei!!!
(Nur definierter Startzustand ist noch fehlerhaft.)

Wie geschrieben, wurde ein großes lauffähiges main(), in viele kleine 
Funktionen gesplittet. Hier sind 3 Var. (u.a. LedAktuell) in 2 
Funktionen dauernd übersehen worden.

Daher an dieser Stelle ein großes DANKESCHÖN für euren Fingerzeig 
darauf.

Echte Computer sind PCs überlegen schrieb:
> Systematisches Vorgehen beim Programmieren macht Debugger so gut wie
> überflüssig.

Deswegen sind die von Dir favorierten Sprachen ja auch so irrwitzig 
erfolgreich; praktisch niemand mehr programmiert in den fehlerträchtigen 
C/C++-Sprachen.




Nein?

Ach. Dies ist nicht der Ort für Programmiersprachen-Flamewars, die 
wurden bereits andernorts ad nauseam ausgefochten.

__Son´s B. schrieb:
> Programm läuft jetzt (fast) einwandfrei!!!
> (Nur definierter Startzustand ist noch fehlerhaft.)

ALLES läuft jetzt!