Kleine Aufgabe und AvR Rechnet falsch?

Patrick90 schrieb:
> up:
[...]
> goto up;
Pfui, nimm ne while(true){} , ist ganz schlechter stil

Ebenso rast dein Programm dadurch, ich denke mal, dass du es noch 
nichtmals flackern siehst, sondern die LEDs nur auf einer gewissen 
helligkeit sind?!

1. Tipp:
kein goto
2. Tipp: nach dem setzten von Portb _delay_ms(XX); verwenden
3. Tipp: bit schubserein verwenden, ich denke das ist eleganter als 
deine jetztige methode
4. anstatt int wüde ich byte verwenden

Nun danke für die Tipps aber die sind nun banal für mich btw bei der 
2ten schleife soll <=1 stehen nicht 3

es ist nur verwirrent das avr das nicht richtig berechnen kann =/

nimm bitoperationen

bsp. für 10x shiften:

1

unsigned char value = 0x0f;

2

unsigned char upperbit;

3

int i;

4

for (i=0; i<10; i++)

5

{

6

    upperbit = value & 0x80;        // oberstes bit merken

7

    value <<= 1;                    // nach links shiften

8

    if (upperbit) value |= 1;       // wenn oberstes bit gesetzt war, unterstes bit setzen

9

10

}

Schon mal was von einrücken gehöhrt?
Schon mal der erste Fehler:

1

int f[3];

2

f[3]=0;

Die Sache mit Arrays und Indizes solltest Du Dir nochmal anschauen.
An welcher Stelle im Program erhällst Du 223. Am Besten Du nimmst Dir 
Bleistift und Papier und spielst selber mal µC. Btw., Du weist aber 
schon, daß int 16 Bit hat?

// schiebt ein Byte bitweise nach links und fügt links herausfallende 
Bits
// rechts wieder ein.

uint8_t i;
i = irgendetwas;

while(1)
  {
  if (i & 0x80)    // das Bit.7 testen, wenn gesetzt
    {
    i << 1;    // nach links schieben und
    i++;    // das links rausgefallende Bit rechts wieder einfügen
    }
  else      // wenn Bit.7 nicht gesetzt,
    {
    i << 1;    // dann nur schieben
    }

  machwas_mit_dem_Byte;
  }

Noch ne Frage. Wie kommst Du auf "240-15"??? Das kommt in Deinem 
Programm nicht vor.

fehler gefunden ^^ bei ersten while bedingung 3 anstatt 4

Patrick90 schrieb:
> 11110000
> 11100001

da du den datentyp int geaehlt hast, ist f[] 2 byte gross. ausserdem 
wird eine multiplikation niemals durch das carry rotieren.
sonst wuerde ja aus einer gerade zahl (LSB = 0) irgendwann eine ungerade 
werden (LSB = 1)

deine erste schleife wird 5 mal durchlaufen, das ist dir klar?

debugge das programm schritt fuer schritt und beobachte die 
registerinhalte. an welcher stelle ist f[1] == 223? die info solltest du 
noch dazuschreiben.

das goto eine todsuende ist, wurde ja schon geschrieben.

f[3] könnte problematisch werden, wenn danach noch eine variable kommt. 
Dein Index geht nur von 0 - 2.

Mein Gott, ihr könnt es aber auch kompliziert machen.

  MOVLW  B'00001111'
  MOVWF  MeineZelle        ; MeineZelle vorbelegen

zyklus:
  RLF    MeineZelle,W      ; MSB ins Carry
  RLF    MeineZelle,F      ; Carry ins LSB
  GOTO   zyklus

Sowas braucht beim PIC nur 800 ns, wovon der GOTO die Hälfte frißt.

Apropos GOTO, Ich halte das GOTO Statement sowohl bei C als auch bei 
Pascal durchaus nicht für schlechten Stil. Wo man eine Ewigschleife zu 
drehen beabsichtigt, da ist GOTO genau richtig!

Auf alle Fälle ist das besser und lesbarer als sowas wie while(true) 
{....}

oder gar while (1)....


W.S.

Patrick90 schrieb:
> fehler gefunden ^^ bei ersten while bedingung 3 anstatt 4

Ich liege am Boden.
Dein Code macht überhaupt nicht, was er soll. Er ist umständlich, 
langsam, fast unlesbar, und funktioniert nur mehr oder weniger zufällig 
für Zahlen <=15 als Startwert.
Arrays scheinen dir auch sehr zu gefallen, einfache Variablen sind ja 
schließlich langweilig.

Dann geben dir 2 Leute Tips, wie man Linksshifts mit Carry korrekt und 
auf einfache Weise implementiert, das interessiert dich aber nicht die 
Bohne.

Ich bin weg. Luft abgelassen.