Finde meinen Fehler einfach nicht

[Klaus (klaus_z)]

Hallo,

mein Programm soll einfach nur die LEDs von innen nach außen mit einem 
Schieberegister zum Leuchten bringen. In Visual Studio hat es auch ohne 
Probleme mit cout << "0"; und cout << "1" funktioniert (siehe Anhang). 
Also mein Programm in der IDE ist so gut wie identisch, unterschied ist 
halt nur das setzen der einzelnen Bits an PORTD. Ich versuche halt die 
Werte mit dem Umrechnen von Dezimal in Binär arbeiten zu lassen. Also 
irgendwo habe ich dort einen Fehler, den ich nicht sehe, bin für jede 
Hilfe dankbar.

#define F_CPU 8000000UL
#include <avr/io.h>
#include <util/delay.h>

//74HC595
#define sendValue PIND2  //Output zum Übergeben von High und Low ans Schieberegister, geht an Pin 14 Schieberegister
#define saveValue PIND0  // SHCP PIN11 am 74HC595. Zum schieben und eingeben 1 Bits im Shiftregister
#define saveData PIND1  // STCP PIN12 am 74HC595. Zum Ausgeben 1 Bytes.
const uint8_t delay = 200;

int main(void)
{
  uint8_t ausgabe[4] = {24, 36, 66, 129};//{00011000, 00100100, 01000010, 10000001}
  uint8_t zahl, rest;
  DDRD |= 0x07; //initialiseren des PORTDs als Ausgang an PD0 bis PD2
  PORTD |= (1<<sendValue);//PIN auf high

    while (1); 
    {
    for(uint8_t i = 0; i < 4; i++)//Erstmal möchte ich nur 4 Werte ausgeben ans SR
    {
      zahl = ausgabe[i]; //hier speicher ich den iten Wert in zahl
      for (uint8_t j = 0; j < 8; j++)
      {
        rest = zahl%2; //zahl durch 2 geteilt sollte entweder 1 oder 0 als Rest ergeben
        if(rest==1) //wenn 1 dann 1 ins Schieberegister schreiben
        {
          PORTD |= (1<<saveValue);
          PORTD &= ~(1<<saveValue);
        }
        else //ansonsten 0
        {
          PORTD &= ~(1<<sendValue);
          PORTD |= (1<<saveValue); 
          PORTD &= ~(1<<saveValue);
          PORTD |= (1<<sendValue); 
        }      
        zahl /= 2;  //Die Zahl wird durch zwei geteilt, bis die For-Schleife 8mal gelaufen ist
      }
      PORTD |= (1<<saveData); //Daten werden am Schieberegister ausgegeben
      _delay_ms(delay);      //mit Delay kann ich es sehen
      PORTD &= ~(1<<saveData);
    }
    }
}

[Foobar (asdfasd)]

Es wäre nicht schlecht, wenn du sagen würdest, was nicht geht.

Das Programm ist zwar ziemlich verkorkst, sollte aber funktionieren. 
Ein Problem ist der initiale Zustand der 3 Signale - du setzt nur das 
Datenbit auf 1, die beiden Clk sind undefiniert.

Ich vermute aber eher, dass die restlichen Signale (/OE=low und 
/SRCLR=high) nicht richtig gesetzt sind.

[Walter T. (nicolas)]

Es wird vermutlich gar nichts gehen, weil die "while(1);-Schleife" nur 
einmal ausgeführt wird.

[Thomas W. (Gast)]

kleines Semikolon, ganz gross...

Gruesse

Th.

[Dieter D. (Firma: Hobbytheoretiker) (dieter_1234)]

Lasst das mal besser ChatGPT machen und lösen. ;o)

[Rainer W. (rawi)]

Klaus schrieb:
> Also irgendwo habe ich dort einen Fehler, den ich nicht sehe, bin für jede
> Hilfe dankbar.

Wenn du in deiner IDE den Debugger benutzt und das Programm Schritt für 
Schritt ablaufen lässt, sollte sofort klar werden, wo der Programmablauf 
nicht deiner Erwartung entspricht.

[Falk B. (falk)]

Klaus schrieb:
> while (1);
>     {

So wird das nix.

[Falk B. (falk)]

Walter T. schrieb:
> Es wird vermutlich gar nichts gehen, weil die "while(1);-Schleife" nur
> einmal ausgeführt wird.

Nö. Die wird DAUERND ausgeführt!

[Alexander (alecxs)]

psst... er wollte doch keine Hilfe und von selbst draufkommen

[Klaus H. (hildek)]

Falk B. schrieb:
> Nö. Die wird DAUERND ausgeführt!

Richtig, uns sonst nichts mehr.
@Klaus: falls der Groschen noch nicht gefallen ist: das Semikolon muss 
weg!

[Klaus (klaus_z)]

Oh man. Habe echt lange gucken müssen, bis ich den Fehler gesehen habe. 
Es sind immer so blöde kleine Fehler.

Auf jedenfall danke für die Hilfe, ich hätte jetzt noch Stundenlang 
draufschauen können und es vielleicht nicht gefunden, aber nächstes mal 
mache ich es, wie es rawi geschrieben hat. Also:

Rainer W. schrieb:
> Wenn du in deiner IDE den Debugger benutzt und das Programm Schritt für
> Schritt ablaufen lässt, sollte sofort klar werden, wo der Programmablauf
> nicht deiner Erwartung entspricht.

[Stefan F. (Gast)]

Versuche mal eine andere IDE. Qt Creator warnt mich z.B. vor solchen 
Dummheiten.

http://stefanfrings.de/avr_ide/index.html#qtcreator

[Rolf M. (rmagnus)]

˙ɟ uɐɟǝʇs schrieb:
> Versuche mal eine andere IDE. Qt Creator warnt mich z.B. vor solchen
> Dummheiten.

Generell ist etwas LSP-fähiges im Zusamnenspiel mit z.B. clangd sinnvoll 
Qt Creator kann das inzwischen wohl optional tun.