Codeschloss Atmega32 Fehler im Code

[Florian (Gast)]

Hi

Ich versuche ein Codeschloss zu bauen welches 2 Taster einliest. Wenn es 
die richtigen waren soll die Led an PA0 eingeschaltet werden.
Ich verstehe allerdings nicht wieso es nicht funktioniert.

#define F_CPU 8000000

#include <avr/io.h>
#include <util/delay.h>

int main(void)
{
  uint8_t z1;
  uint8_t z2;
  z1 = 0;
  z2 = 2;
  DDRA |= (1 << 0);
    while(1)
    {
    while(z1 < 1) {
         if(PINA & (1 << 1)) {
      _delay_ms(100);
      z1 = 1;
     }
     if(PINA & (1 << 2)) {
       _delay_ms(100);
       z1 = 2;
     }
    }
    while(z2 < 1) {
      if(PINA & (1 << 1)) {
        _delay_ms(100);
        z2 = 1;
      }
      if(PINA & (1 << 2)) {
        _delay_ms(100);
        z2 = 2;
      }
    }
  if (z1 == 1 && z2 == 1) {
    PORTA |= (1 << 0);
   }
    }
}

lgf

[Mike (Gast)]

Florian schrieb:
> Ich verstehe allerdings nicht wieso es nicht funktioniert.

Hast du den Ablauf im Simulator laufen lassen und dir angesehen, was 
anders läuft, als du es geplant hast?

[Nobby Nic (Gast)]

/* Ein Vorteil von Kommentaren im Source-Code ist, dass man nicht nur selbst den Code versteht sondern auch Dritte eine Möglichkeit haben, die Gedankengänge des Programmierers nachvollziehen zu können. */

[Mike (Gast)]

Nobby Nic schrieb:
> /* Ein Vorteil von Kommentaren im Source-Code ist, ...

Guter Code ist selbsterklärend. Da muss man nichts weiter zu sagen ;-)

[Marco (Gast)]

uint8_t z1;
  uint8_t z2;
  z1 = 0;
  z2 = 2;

in

  uint8_t z1;
  uint8_t z2;
  z1 = 0;
  z2 = 0;

ändern....

[Florian (Gast)]

Hab den Fehler gefunden:

> z2 = 2;

es muss

z2 = 0;

heissen.

Nobby Nic schrieb:
> /* Ein Vorteil von Kommentaren im Source-Code ist, dass man nicht nur
> selbst den Code versteht sondern auch Dritte eine Möglichkeit haben, die
> Gedankengänge des Programmierers nachvollziehen zu können. */

Den Code finde ich selbsterklärend

[Kollege (Gast)]

Mike schrieb:
> Nobby Nic schrieb:
> /* Ein Vorteil von Kommentaren im Source-Code ist, ...
>
> Guter Code ist selbsterklärend. Da muss man nichts weiter zu sagen ;-)

Nicht alle sprachen sind dafür ausdrucksstatk genug.

zB. sind Inline Kommentare bei Java Code ein "Smell", d.h. ein Anzeichen 
für schlechten Code.

Ob das in C auch so ist?

Kurze Methoden sollten aber auch in C möglich sein, dann nimmt man 
sprechende Namen für die Methoden.

[Nobby Nic (Gast)]

Florian schrieb:
> Den Code finde ich selbsterklärend

Wenn das so wäre, hättest du den Fehler viel eher entdeckt ;-)

[Wegstaben V. (wegstabenverbuchsler)]

Florian schrieb:
> Den Code finde ich selbsterklärend

dassteht aber in Widerspruch zu

Florian schrieb:
> Ich verstehe allerdings nicht wieso es nicht funktioniert.

[Kevin Lüttler (Gast)]

Kollege schrieb:
> zB. sind Inline Kommentare bei Java Code ein "Smell", d.h. ein Anzeichen
> für schlechten Code.
> Ob das in C auch so ist?

Empfiehlt sich zumindest wenn man Hardwareregister mit irgendwelchen 
Werten beschreibt. XYZSTAT |= (1<<ABC) | (1<<IRGENDWAS) ist nur mit 
danebenliegendem Datenblatt selbsterklärend.