Wuerfel - Anweisung wird ignoriert

[Wuerfelspieler (Gast)]

Hallo,

ich bin grade dabei, einen "elektrischen Würfel" zu programmieren. Dafür 
habe ich neun LEDs als Matrix und folgenden Code.
Allerdings wird die Anweisung "PORTB = get_wuerfel(0, x);" ignoriert - 
wenn ich nun aber x im Code durch 0, 1 oder 2 ersetze, funktioniert 
soweit alles.

Ich suche nun schon seid etwa einer Stunde, finde aber partout keinen 
Fehler.
Kann mir jemand weiterhelfen?

#define F_CPU 4000000
#include <avr/io.h>
#include <avr/interrupt.h> 
#include <util/delay.h>

const int zeilen[3] = { 
    1 << PB0,
    1 << PB1,
    1 << PB2
};

const int spalten[3] = { 
    1 << PB3,
    1 << PB4,
    1 << PB5
};

int wuerfel[7][3][3] = {
  {
    { 1, 1, 1 },
    { 1, 0, 1 },
    { 1, 1, 1 }
  }
};

void delay_ms(int ms) {
  for(; ms > 0; ms--) {
    _delay_ms(1);
  }
}

void delay_us(int us) {
  for(; us > 0; us--) {
    _delay_us(1);
  }
}

int get_wuerfel(int ziffer, int linie) {
  int r = zeilen[linie];
  int i;

  for(i = 0; i < 3; i++) {
    if(wuerfel[ziffer][linie][i]) {
      r = r | spalten[i];
    }
  }

  r ^= 0b000111;
  return r;
}

int main (void) {
  DDRB = 0xFF;
  PORTB = 0xFF;

  while(1) {
    int x = 0;
    for(; x < 3; x++) {
      PORTB = get_wuerfel(0, x);
      delay_ms(1000);
    }
  }
  
  return 0;
}

[Wuerfelspieler (Gast)]

Selbst mit folgendem Code geht das ganze.
Aber das ist irgendwie nicht schön und auch nicht der Sinn von Variablen 
:o

    if(x == 2) {
        PORTB = get_wuerfel(0, 2);
      } else if(x == 1) {
        PORTB = get_wuerfel(0, 1);
      } else if(x == 0) {
        PORTB = get_wuerfel(0, 0);
      }

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

    int x = 0;
    for(; x < 3; x++) {
      :

Setz doch x mal nicht nur 1x zurück:

    int x;
    for(x = 0; x < 3; x++) {
      :

[Oliver (Gast)]

>Ich suche nun schon seid etwa einer Stunde, finde aber partout keinen
>Fehler.

Für solche Problemstellungen gibt es Debugger. Hast du damit schon mal 
nachgesehen, was genau passiert, und was wo schief geht?

Oliver

[Klaus (Gast)]

@Lothar: Heute morgen noch kein Kaffee gehabt? Deine beiden Codestücke 
mach doch genau das selbe ;)

[... (Gast)]

Das hier

int wuerfel[7][3][3] = {
  {
    { 1, 1, 1 },
    { 1, 0, 1 },
    { 1, 1, 1 }
  }
};

stimmt doch so nicht.

[Klaus (Gast)]

stimmt, die Initialisierung ist unvollständig. Aber solange get_wuerfel 
nur mit 0 als erstem Parameter aufgerufen wird, passiert noch nix 
schlimmes.

[Wuerfelspieler (Gast)]

Bisher rufe ich es nur mit "0" als Paramter auf. Auch wenn ich 
wuerfel[7][3][3] zu wuerfel[1][3][3] ändere, besteht das Problem 
weiterhin.

Einen Debugger oder entsprechende Hardware habe ich leider nicht..

[MeinerEiner (Gast)]

Debugger ist einer im AVR-Studio mit drin.

[Chris H. (wuselwutz)]

Warum 9 Leds? Hätten nicht 7 gereicht?

[Lothar M. (Firma: Titel) (lkmiller) (Moderator)]

Chris H. schrieb:
> Warum 9 Leds? Hätten nicht 7 gereicht?
Der Würfel kann doch auch mal um 90° gedreht zum Liegen kommen  ;-)

[Wuerfelspieler (Gast)]

Ja, sieben LEDs hätten auch  gereicht. Allerdings fand' ich die 
LED-Anordnung in H-Form optisch recht unansehlich und weil es 
matrixmäßig keinen Unterschied macht, ob da nun sieben oder neun LEDs 
sind, weil LEDs nicht soo viel kosten und weil der AtTiny2313 die Pins 
ja hat, habe ich mich für neun LEDs entschieden.

Habe das Problem übrigens gelöst. Wie? Mit einem neuen AtTiny2313 
grübel.