Problem mit Variablen bei C Uhr

Anfänger schrieb:
> Hat jemand einen Tipp?
Ich würde zur Sicherheit auf >= 59 abfragen.
Zeig mal ein bisschen mehr vom Code und verwende dabei die C-Code 
Formatierung

1

[c]C-Code[/c]

Poste mal den gesamten Code (als anhang). In dem Codeausschnitt kann ich 
nichts sehen, was falsch wäre. Ich würde evtl. if(sek>=59)  benützen, um 
sicher zu gehen, aber sollte eig. kein Problem sein.

Nein, kein Tipp.
Aber wenn du uns den Sourcecode zeigst und ggf noch sagst welche 
Umgebung du nutzt und vielleicht sogar noch erwähnst welche Art Display, 
Library usw. dann könnte ein Tipp dabei rausspringen

holger schrieb:
> Deine Minute war 59. Danach wird 0 ausgegeben.
> 59 -> 09. Sieht das ungefähr so aus?

kann sein wenn er nur 0 statt 00 ausgibt, beliebter Fehler :-)

Wer sagt denn das die IF Abfrage zeitnah erfolgt?

Anfänger schrieb:
> if(sek==59){
> sek=0;
> min++;

was ist wenn sek schon bei 60 ist?

Max H. schrieb:
> Ich würde zur Sicherheit auf >= 59

eben usw.

Anfänger schrieb:
> beim umspringen von 59 auf 1
Eine Uhr sollte eigentlich von 59 auf 0 und nicht auf 1 springen.

> lcd_print_itoa(sekunde);
Die Funktion gibt keinen führenden Nullen aus, stimmt's?

BTW: Die unteren if würde ich weiter einrücken, dann sieht man auf 
Anhieb, dass es verschachtelte if sind.

Anfänger schrieb:
> die Funktion habe ich aus der vorhandenen lib und sieht so aus:
Sieht nicht so aus als wäre eine führende null.

> Die Uhr springt ja jeweils durch rücksetzen sekunde=0; auf Null oder
> muss ich das anderst realisieren?
Die if-Abfrage sollte auf >59 abfragen und das print sollte erst nach 
der Abfrage sein.

Max H. schrieb:
> print sollte erst nach
> der Abfrage sein.

und immer 2-stellig erfolgen ! 00 statt 0

sprintf 02d

oder per strcpy

> Sieht nicht so aus als wäre eine führende null.

Jetzt muß ich doch mal fragen wo du das erkennst.
Der TO benutzt lcd_print_itoa,
zeigt uns aber die Beschreibung von itoa. Was lcd_print_itoa ist, das 
wissen wir nicht.
Oder habe ich was übersehen ?

Anfänger schrieb:
> also wie gesagt das flag wird in der Iterrupt Routine gezählt.

Das ewige und meist auch sinnvolle Dogma mit der möglichst kurzen ISR 
besagt nicht, dass man darin gar nichts machen darf. Zähl deine Uhr in 
der ISR hoch. Soviel Zeit muss sein. Und die ist auch vorhanden. Dann 
tritt auch das Problem, dass ein Inkrement verschluckt wird, gar nicht 
erst auf. Du zählst einfach bis 60 bzw. 24 und setzt dann auf 0.

Nur Ausgabe auf das Display machst du natürlich nicht in der ISR. Sowas 
ist meist tödlich.

mfg.

Und wie sieht die lcd_print_itoa aus ?
Wo kommt die her ?
Was macht die ?

Troll schrieb:
> Na, na, na, es reicht doch die Sekunden in der ISR zu inkrementieren.

Ja. Aber in einer Long-Variable. Das reicht für 136 Jahre. Die Uhrzeit 
rechnet man erst zur Ausgabe aus.

mfg.

Wenn das C ist, dann ist lcd_print_itoa eine Funktion. Die macht 
anscheinend irgendwas  mit itoa und gibt was auf das LCD aus.
Ob sie das mit/ohne 0 oder Leerzeichen macht, das wissen wir nicht.

Wahrscheinlich/anscheinend nicht - aber wenn uns der TO nicht sagt
welche Libraries er verwendet, dann ist alles nur geraten und stochern 
im Nebel.

P.s. Die LCD Funktion solle nicht aus dem Interrupt sondern in der 
Hauptprogrammschleife aufgerufen werden.
Aber das ist das 2.te Problem.

Anfänger schrieb:
> char buffer [1000];
Ist das nicht ein bisschen übertreiben?

Eine führende Null für eine eistellige Zahl könnte man so einbauen:

1

    itoa(zahl, buffer, 10); 

2

    if(buffer[1] == 0x00)

3

      lcd_data('0');

4

5

    lcd_string(buffer);

Warum immer so kompliziert?

1

void lcd_print_time(void)

2

{

3

  char buffer [12];

4

  lcd_set_cursor(1,1);

5

  sprintf( buffer, "%2u:%02u.%02u",

6

           (uint16_t)hour,

7

           (uint16_t)min,

8

           (uint16_t)sec );

9

  lcd_string(buffer);

10

}

Machst du die LCD Ausgabe mittlerweile in der Hauptprogrammschleife oder 
noch vom Interrupt aus ?

Troll schrieb:
> Hoffentlich reicht der Speicher ... ;-)

Auf nem ATtiny2313 könnts knapp werden. Es sollte schon ein ATmega88 
sein.