While Schleife

Phil schrieb:
> Muss mich korrigieren! Es funktioniert nur von Schleife 1 zu 2 bei
> folgenden Code. Bei den oben aufgeführten überspringt er alle Schleifen.

Sollen wir jetzt "finde den Unterschied" spielen? Was ist am zweiten 
Code anders als am ersten?

> War es nicht so, dass die while solange ausgeführt wird wie die
> Bedienung true ist?

Ja.

Andere Frage: Wo ist die Entprellung?

Phil schrieb:
> Und nochmal Entschuldigung. Hab als zweiten Code den gleichen im
> Zwischenspeicher gehabt.

Häng das C-File als Attachment an, so wie es ist.

Rolf Magnus schrieb:

> Andere Frage: Wo ist die Entprellung?

Ich fürchte, das soll seine 'Entprellung' sein.

>   char Tag_c[0] = "0";

Ein Array der Länge 0?

Ich fürchte da wird ein

          sprintf(Tag_c,"%2d",Tag);

aber nicht reinpassen!


> Wenn ich das trotzdem falsch interpretiert habe mit
der Entprellung für den Fall, dann bitte ich um Aufklärung.

Dein Glück könnte sein, dass sprintf bzw. lcd Ausgabe auch Zeit 
brauchen. Aber abgesehen davon ist das ziemlicher Unsinn, was du da 
hast. Dein µC hachelt den Code schneller durch, als dein Taster auch nur 
ansatzweise an prellen denken kann.
In der Zeit, in der du mit Copy&Paste Codeverfielfältigung betrieben 
hast, hättest du auch eine ordentliche Entprellung übernehmen können. 
Wenn du dir wenigstens eine einzige Funktion für die Zahleneingabe 
gebaut hättest. Aber einfach Copy&Paste .... ts, ts, ts


Du hast ein LCD. AUf dem kann man sich auch Variablen ausgeben lassen, 
von denen nicht klar ist, welchen Wert sie haben. Und es ist auch nicht 
verboten, das LCD dazu zu benutzen sich zb Statusmeldungen des Programms 
auszugeben, damit der geneigte Programmierer nachvollziehen kann, was 
das Programm aus welchem Grund macht. Warum benutzt du das LCD nicht 
dazu? Hauptsache die Copyrightmeldung taucht am Anfang auf.

Phil schrieb:

> Zum Wesentlichen:
>
> Das Array passt nicht rein. Ist ja auch nur ein Feld groß. Für Tage und
> Monate brauch ich 2 Felder.

Nein. Du brauchst 3.
Um einen String "Hallo" abzuspeichern, brauchst du ein Array der Länge 
6. Denn hinter die Buchstaben kommt noch der für Strings obligatorische 
Abschluss in Form eines \0 zeichens.

und du brauchstz ein C-Buch.
So hab nämlich ich mit C angefangen: mit einem Buch und durcharbeiten 
der Übungen im Buch.
(Und PS: das war nachdem ich meinen ersten Compiler gebaut habe :-)


> Die Darstellung funktioniert allerdings
> wunderbar.

Das heißt noch lange nichts.

> Konnte es mir bislang nicht erklären. Ich brauch den Wert
> auch nicht im Array, sondern als int. Das Array ist nur für die LCD
> Ausgabe gedacht.

Ist wurscht.
der sprintf bügelt dir Speicher nieder. Ob das nur für jetzt ist und 
später wieder rauskommt oder nicht. Speicher überbügelt ist Speicher 
überbügelt. Danach kann alles mögliche passieren, je nachdem wie genau 
die Variablen im Speicher angeordnet sind.

Der Tip steht allerdings immer noch:
In deinem LCD ist zb die Ecke rechts oben in der ersten Zeile frei. Da 
kann man sich zb wunderbar mal laufend 'Programmablauf' ausgeben lassen 
und sich ansehen, wie sich der Wert verändert und überprüfen ob das so 
richtig ist oder nicht. Oder ZustandT1. Oder ZustandT2. Oder ZustandT3. 
Oder gleich alle 4 Variablen.

Phil schrieb:

> Ganz unabhängig von den array?

Solange dieser Fehler nicht korrigiert ist, nehme ich keine Wetten an. 
dein sprintf überschreibt alles mögliche im Speicher.

Phil schrieb:
> Karl Heinz Buchegger schrieb:
>> Der Tip steht allerdings immer noch:
>> In deinem LCD ist zb die Ecke rechts oben in der ersten Zeile frei. Da
>> kann man sich zb wunderbar mal laufend 'Programmablauf' ausgeben lassen
>> und sich ansehen, wie sich der Wert verändert und überprüfen ob das so
>> richtig ist oder nicht. Oder ZustandT1. Oder ZustandT2. Oder ZustandT3.
>> Oder gleich alle 4 Variablen.
>
> Alles klar. Das teste ich morgen gleich aus.

Blöd ist jetzt natürlich, dass du den Code einfach dupliziert hast. Gäbe
es nur 1 Funktion

uint16_t numberInput( char* Text, uint16_t Minimum, uint16_t Maximum, 
uint8_t Spalte )
{
  ...
}

dann bräuchte man nur an einer Stelle die Debug Ausdrücke einfügen und
hätte in einem Aufwasch für alle Zahleneingaben die Überprüfung drinnen.
So muss man jetzt jede Schleife einzeln 'pimpen' und nachher die Tests
und Ausgaben wieder aus jeder Schleife einzeln ausbauen.

Fazit: Es lohnt nicht, sich vermeintlich die Dinge 'einfach' zu machen, 
indem man zunächst auf vernünftige Codeorganisation verzichtet. Im 
Endeffekt hat man damit IMMER mehr Arbeit, als wie wenn man die Dinge 
gleich vernünftig gemacht hätte.

Diese if-else-if Monster mag sich keiner ansehen, weil viel zu 
unübersichtlich.
Für Statemachines gibt es schließlich switch-case.

Beim Entprellen lohnt es nicht, sich was selber auszudenken, ohne die 
Seiteneffekte zu beachten.
Man spart sich sehr viel Zeit, wenn man erprobte Lösungen im 
Timerinterrupt verwendet. Das Main kann dann dann bequem die 
Ereignisflags behandeln und hat keine Seiteneffekte (Delays, 
doppelte/verlorene Tastendrücke usw.) zu befürchten.