Kürzungsmöglichkeiten?

Steffen T. Gast #576377
Lesenswert?
• ▲
▼ 
Hallo,

mein "Progrämmchen" ist etwas zu groß geworden, vllt findet einer von 
euch noch eine Möglichkeit etwas zu kürzen... Ich habe schon Stunde in h 
abgekürzt, damit es etwas weniger Speicher frisst, aber es reicht noch 
nicht... (also eine abgespeckte Version, wenn z.B. wieder von 0,0,0 
anstatt von der aktuellen Uhrzeit gestellt wird habe ich schon)

Bevor ich es vergesse... das Programm steuert eine LCD Anzeige an und 
frägt zu Beginn die aktuelle Uhrzeit ab, danach läuft die Uhr und man 
kann sie wieder stellen, wenn man nochmals den Taster drückt...

Vielleciht findet auch noch einer eine Verbesserung, denn im Moment ist 
es ja so, dass die Uhrzeit nicht ununterbrochen läuft sondern, imer 
wieder aussetzt, wenn Daten an das LCD-Display geschickt werden...

Hier der Code:

1
#include "lcd.c"

2
#include <avr/interrupt.h>

3


4
volatile unsigned char time_ist; 

5
unsigned char Buffer[20];

6
unsigned int std=0;

7
unsigned int min=0;

8
unsigned int sek=0;

9


10
ISR (TIMER1_COMPA_vect  ) {

11
  time_ist++;

12
}

13


14
void timer (unsigned long time_soll) {

15
  time_ist = 0;

16
  TCCR1A = 0;      

17
  TCCR1B = 1 <<CS10;   

18
  TCCR1B |= 1 <<WGM12;

19
  OCR1A = 64000 ;     

20
  TIMSK = 1 << OCIE1A;   

21
  sei();

22
  while (time_soll >= time_ist) {}

23
}

24


25
int zeit (int einheit, int umbruch,const char* worte){

26
  lcd_clrscr ();

27
  lcd_puts (worte);

28
  itoa( einheit, Buffer, 10 );

29
  lcd_puts (Buffer);

30
  timer(50);

31
  while(PIND & (1<<PIND4) )

32
    if(!(PIND & (1<<PIND3) )) {  

33
    if(++einheit == umbruch) einheit = 0; 

34
    lcd_clrscr ();

35
    lcd_puts (worte);

36
      itoa( einheit, Buffer, 10 );

37
    lcd_puts (Buffer);

38
    timer(50);

39
  }

40
  return einheit;

41
};

42


43
void stellen(){

44
  std = zeit(std,24,"h: ");

45
  min = zeit(min,60,"min: "); 

46
  sek = zeit(sek,60,"s: ");

47
}

48


49
int main (void) {

50


51
  PORTD= 0x18;

52
  DDRD = 0x18; 

53
  lcd_init (LCD_DISP_ON);

54
  stellen();

55
    

56
  for (;;){  

57
    timer (250);

58
  if(++sek == 60){

59
  sek = 0;

60
      if(++min == 60){

61
      min = 0;

62
        if(++std == 24){ 

63
    std = 0;

64
    }

65
      }

66
    }

67
    lcd_clrscr ();

68
  lcd_puts ("    ");

69
    if(std < 10) lcd_puts("0");

70
    itoa( std, Buffer, 10 );

71
  lcd_puts (Buffer);

72
  lcd_puts(":");

73
    if(min < 10) lcd_puts("0");

74
    itoa( min, Buffer, 10 );

75
  lcd_puts (Buffer);

76
  lcd_puts(":");

77
  if(sek < 10) lcd_puts("0");

78
    itoa( sek, Buffer, 10 );

79
  lcd_puts (Buffer);

80
    if(!(PIND & (1<<PIND4))) stellen();

81
  };

82
}


Vielen Dank schon einmal :)
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden
yalu Gast #576400
Lesenswert?
• ▲
▼ 
Das Stück Code, was du gepostet hast, ist so kurz, dass es auch von
einem kleineren AVR nur einen Bruchteil des Flash-Speichers füllt. Es
bringt also nicht allzu viel, hier etwas zu optimieren. Wenn es nur um
ein paar Bytes geht, kannst du folgendes probieren:

- Compiler-Option -Os, falls du das noch nicht hast

- alle Funktionen mit Ausnahme des Interrupthandlers und main als
  static deklarieren, dies führt bei aktivierter Optimierung dazu,
  dass sie geinlinet wedren

- 16-Bit-Variablen, wo es der Wertebereich erlaubt, durch
  8-Bit-variablen ersetzen

- evtl. brint es etwas itoa durch eine eigene Funktion zu ersetzen,
  die auf Basis 10 und zwei Ziffern beschränkt ist

Ansonsten gibt es vielleicht in lcd.c noch etwas zu tun.

Aber es wundert mich dennoch, wie du mit diesem kleinen Programm 1 KB
vollbekommst, es sei denn, du hast die Optimierung deaktiviert und
lcd.c verwendet Dinge wie die Delays aus delay.h. Dann sollte -Os
auf jeden Fall die Lösung sein.
Antwort Zitieren Bearbeiten Löschen Beitrag melden
Thread beobachten

Antwort schreiben

Hinweis: der ursprüngliche Beitrag ist mehr als 6 Monate alt. Bitte hier nur auf die ursprüngliche Frage antworten, für neue Fragen einen neuen Beitrag erstellen.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.