Kurze Frage zu Codeoptimierung

Lukas B. schrieb:
> ... da die Uhr nicht sonderlich genau geht. (ca. 5 Sekunden in
> einer Woche

Das läßt sich besser hinkriegen, wenn der Quarz nicht wilden 
Temperaturschwankungen ausgesetzt ist.
Beitrag "Die genaue Sekunde / RTC"

if ((millisekunden%50)==0)   //Rest der Division=0?

Wenn die Ungenauigkeit normal ist, stört sie mich nicht. Auf ein Jahr 
hochgerechnet sind das ja gerade mal ~4 Minuten pro Jahr, damit kann ich 
leben, zumal der Wecker sowieso alle 6 Monate gestellt werden muss 
(Sommer-, Winterzeit).
Für eine RTC habe ich leider keinen Port mehr frei, zumindest keinen von 
den Interrupt-Ports.

@crazy horst: Danke, das sieht besser aus.

Lukas B. schrieb:
> Wenn die Ungenauigkeit normal ist, stört sie mich nicht.
zumal der Wecker sowieso alle 6 Monate gestellt werden muss
(Sommer-, Winterzeit)
Warum "hängst" du dich nicht an die Netzfrequenz? Die EVUs sorgen schon 
dafür, dass Uhren, die die Netzfrequenz als Normal nutzen, nicht "falsch 
gehen".

@Εrnst B✶: Also doch wieder if(...||...,usw.)? Oder wie sieht deine 
Lösung in der Praxis aus? Ich kann mir das nicht so ganz vorstellen.

@RTC: Ich gucke es mir mal an.

braucht du denn die Millisekunden überhaupt? Wenn nein warum arbeits du 
nicht in 50 Millisekunden schritten. Also in Vielfachen von 50ms.

Nein, ich brauche die Millisekunden nicht. Also müsste ich den Timer 
anders einstellen, sodass er nur alle 50ms aufgerufen wird, richtig?

Lukas B. schrieb:
> Also müsste ich den Timer
> anders einstellen, sodass er nur alle 50ms aufgerufen wird, richtig?

ja oder wenn das nicht geht (wegen teiler) dann kannst du auch im timer 
bis 50 zählen und dann erst die 50ms-Variable hochzählen.

Dann kannst du auch gleich aus dem 16bit wert ein 8bit wert machen, dann 
rechnet sich dann für den µC einfacher.

Peter II schrieb:
> Dann kannst du auch gleich aus dem 16bit wert ein 8bit wert machen, dann
> rechnet sich dann für den µC einfacher.

Wie mache ich das denn?

Lukas B. schrieb:
> Wie mache ich das denn?

u_int18 millisekunden  -> u_int8 millisekunden

Momentan ist der Datentyp unsigned int, aber wenn ich ihn auf uint8_t 
(etwas anderes erkennt der Compiler nicht) ändere, wird nichts mehr 
hochgezählt.

Lukas B. schrieb:

1

> if(millisekunden == 950 || millisekunden == 900 || millisekunden == 850 || 

2

>      millisekunden == 800 || millisekunden == 750 || millisekunden == 700 ||  

3

>        millisekunden == 650 || millisekunden == 600 || millisekunden == 550 ||  

4

>      millisekunden == 500 || millisekunden == 450 || millisekunden == 400 ||  

5

>      millisekunden == 350 || millisekunden == 300 || millisekunden == 250 || 

6

>      millisekunden == 200 || millisekunden == 150 || millisekunden == 100 ||  

7

>      millisekunden == 50  || millisekunden == 0)

ist schon deswegen ungünstig, weil die Laufzeit für
den Fall "millisekunden == 950" anders ist als für den Fall
"millisekunden == 0" (wegen "short-circuit evaluation" von logischen 
Ausdrücken in C).

Spielt vielleicht im vorliegenden Fall nicht so die Rolle, aber hässlich
aussehen tut´s in jedem Fall.

Murx

Lukas B. schrieb:
> Für eine RTC habe ich leider keinen Port mehr frei, zumindest keinen von
> den Interrupt-Ports.

Seit wann braucht Software eine Port?
Ein Click ...

@Michael: Das war nur ein Missverständnis, ich dachte bei RTC an einen 
externen Baustein.

@Murx: Ich habe es jetzt so gelöst, dass die ISR nur alle 50ms 
aufgerufen wird, ich denke, das ist die eleganteste Lösung. Nur das 
Problem mit den 16-bit Variablen ist irgendwie seltsam.

Lukas B. schrieb:
> Momentan ist der Datentyp unsigned int, aber wenn ich ihn auf uint8_t
> (etwas anderes erkennt der Compiler nicht) ändere, wird nichts mehr
> hochgezählt.

uint8_t geht ja auch nur bis 255 - da passen deine 1000ms nicht rein. 
Aber wenn du in 50ms schritten arbeitets dann brauchst du nur 0-200 und 
das passt rein.

Zeig uns doch mal das ganze programm, wenn du immer int verwendest, hast 
du noch viel mehr optimierungsmöglichkeiten als die IF abfrage.

Stimmt, du hast recht, jetzt funktioniert es. :D

Der gesamte Code befindet sich im Anhang. Wenn ihr noch Fehler oder 
Optimierungspotential findet, sagt Bescheid.

P.S. Das ist mein erstes richtiges Programm, ich bitte um Nachsicht. ;)