7 Segmentanzeige - multiplexen Integer mit Taster inkrementieren

[Peter N. (gizmo2308)]

Hallo zusammen,

ich möchte einen Integer mittels Taster inkrementieren.
Habe folgenden Code geschrieben, aber der Wert wird ohne Tastendruck 
einfach hochgezählt.
Könnte mir jemand einen Tipp geben, was ich Falsch gemacht habe?

// Prozessor : Atmega8
#define  readout    PORTD
#define digit_1  0b00000000
#define digit_2  0b00010000
#define digit_3  0b00100000
#define digit_4  0b00110000
#define digit_5  0b01000000
#define digit_6  0b01010000
#define digit_7  0b01100000
#define digit_8  0b01110000
#define  value    PORTB
#define time     1
volatile uint16_t counter;

//----------------------------------------------------------------------

void initTimer0()
{
  TCCR0=0x03;              // prescaler 64
  TIMSK=0x01;              // mask INT Timer-Overflow
}

void initPorts()
{
  // PD4...PD6 Demultiplexer
  DDRD |= (1<<PD4) | (1<<PD5) | (1<<PD6);
  // PB0...PB4 Decoder
  DDRB |= (1<<PB0) | (1<<PB1) | (1<<PB2) | (1<<PB3) | (1<<PB4);

  DDRD  = 0b01110000;                // PortD = Input
  PORTD = 0b00000100;                // pull-up

}

void SectionToSegment(uint16_t ivalue)
{
  char ziffer1 = 0;
  char ziffer2 = 0;
  char ziffer3 = 0;

  while(ivalue>=100)
  {
    ziffer1+=1;
    ivalue+=-100;
  }
  while(ivalue>=10)
  {
    ziffer2+=1;
    ivalue+=-10;
  }

  ziffer3=ivalue;

  value = ziffer1;
  readout = digit_1;
  waitMs(time);

  value = ziffer2;
  readout = digit_2;
  waitMs(time);

  value = ziffer3;
  readout = digit_3;
  waitMs(time);

}

//----------------------------------------------------------------------
main ()                // main program, startet bei Power ON und Reset
{
  initPorts();          // Initialize Ports
  sei();                       // enable interrupts
  do 
  {

    if (!(PIND&0x04))
      PORTB = counter++;

    SectionToSegment(counter);
  }
  while (true);          // End of loop mainloop
}

[Karl H. (kbuchegg)]

Peter Nick schrieb:

> Könnte mir jemand einen Tipp geben, was ich Falsch gemacht habe?

Du hast da einen ziemlichen Pallawatsch drinnen.
An manchen Stellen bin ich mir nicht wirklich sicher, iob du weißt, was 
die Zeichen bedeuten, die du hingeschrieben hast.

Zum Problem

    if (!(PIND&0x04))
      PORTB = counter++;
    SectionToSegment(counter++);

wenn du hier beim Funktionsaufruf von SectionToSegment dem Argument 
counter ein ++ nachhängst, dann wird counter selbstverständlich erhöht. 
So wie immer, wenn einer Variablen ein ++ angehängt wird. Das ++ 
bedeutet: um 1 erhöhen.
Und dieses Eerhöhen hier beim Funktionsaufruf ist in keiner Weise davon 
abhängig ob am PIND das bewusste Bit auf 0 oder auf 1 ist. Ergo: Es wird 
immer erhöht.

[Karl H. (kbuchegg)]

  while(value>=100)
  {
    ziffer1+=1;
    value+=-100;

man darf eine Subtraktion ruhig als

    value -= 100;

schreiben.

Auch würde ich dir empfehlen, mit Leerzeichen nicht so geizig zu sein. 
Seit Kindesbeinen hast du dein Gehirn darauf trainiert, dass zwischen 
Wörtern ein Leerraum steht. Dein Gehirn ist darauf trainiert, Wörter in 
einem Text genau an diesem Leerraum schnell zu erkennen. Was passiert, 
wenn man diesen Leerraum nicht macht, 
dasistganzschnellaneinemBeispieldemonstriert. Hattest du 
Schweirigkeiten, den letzten Satzteil zu lesen? War das anstrengend und 
hattest du das Gefühl, das dir da jemand absichtlich Prügel zwischen die 
Beine wirft? Warum machst du dann genau dasselbe in deinem Programmen? 
Auch ein Programm besteht aus Wörtern. Du hast schon mit der Bedeutung 
der Wörter bzw. der Bedeutung dessen, was die 'Wortzusammenstellungen' 
aussagen genug zu kämpfen. Du musst dir nicht auch noch Prügel zwischen 
die Beine werfen, indem du es dir selbst schwer machst, erst mal die 
Wörter in dem Zeichenwust überhaupt zu finden.

  while( value >= 100 )
  {
    ziffer1 += 1;
    value -= 100;
  }

um wieviel leichter ist das zu lesen, wenn man ausnutzt, dass dein 
Gehirn auf Leerräume trainiert ist.

[Peter N. (gizmo2308)]

> wenn du hier beim Funktionsaufruf von SectionToSegment dem Argument
> counter ein ++ nachhängst, dann wird counter selbstverständlich erhöht.
> So wie immer, wenn einer Variablen ein ++ angehängt wird. Das ++
> bedeutet: um 1 erhöhen.
> Und dieses Eerhöhen hier beim Funktionsaufruf ist in keiner Weise davon
> abhängig ob am PIND das bewusste Bit auf 0 oder auf 1 ist. Ergo: Es wird
> immer erhöht.

Hallo Karl Heinz,
habe ich soeben bemmerkt, hatte einen Copy Paste Fehler.

[Karl H. (kbuchegg)]

Peter Nick schrieb:

> Hallo Karl Heinz,
> habe ich soeben bemmerkt, hatte einen Copy Paste Fehler.

Als Fragesteller ist es aber keine gute Idee, wenn du nachträglich dein 
Posting editierst und damit das Gefragte komplett umdrehst. Tippfehler 
zu korrigieren ist ok, nachträglich Ergänzungen anbringen ist ok. Aber 
deine Veränderung dreht die Frage komplett um. Denn jetzt ist es nicht 
mehr der Funktionsaufruf, in dem der ++ vorkommt, sondern du hast eine 
2.te Abfrage drinnen, die vorher nicht da war. Aber selbst bei dieser 
Abfrage muss ich mich fragen: Ja, was erwartet er eigentlich?
Da steh

    if( PO ....

und jetzt hast du nochmal ausgebessert, wobei du der Frage die komplette 
Grundlage entzogen hast. Jetzt siehst du dämlich aus, weil du im 
Begleittext nach etwas fragst, was sich in der jetzigen Version im Code 
nicht mehr wiederfindet. Und ich seh auch dämlich aus, weil ich auf 
etwas eingehe, was in deinem Code nicht mehr vorkommt.

So funktioniert das nicht. Wenn du änderst, dann darfst du nicht grob 
sinnentstellend ändern. Vor allen Dingen wenn du bemerkst, dass da schon 
jemand darauf eingegangen ist.
Ich bin raus.