AVR_ Lauflicht stoppen

[Serieousl L. (stephan2013)]

Hallo

Ich habe ein Lauflicht und soll es um folgende Funktionen erweitern.
(1)

Stoppen des Lauflichtes wenn PortB/ Pin2 auf LOW gesetzt wird, soll das 
LL anhalten. Geht PortB.2 wieder auf High , läuft des LL weiter.

mit low meint man, wenn es auf 0 gesetzt wird oder ?

Code

         int16_t i=0;
         DDRB=0x00;   // input Port
         PORTB=0xFF;  // alles aus
    
         DDRA=0xFF;   // Port A als Output setzen
         PORTA=0x00;   // alles ausschalten
    while (i<=7)
    {
            PORTA = ~(0x01 << i);
            if (PINB &(0x01 <<1)==PINB2))
            {
                PORTA=0xFF;
            }
            else
                i++;
    }

[Eumel (Gast)]

hisham moustafa schrieb:
> mit low meint man, wenn es auf 0 gesetzt wird oder ?

Ja

[Serieousl L. (stephan2013)]

Und stimmt jetzt die Umsetzung(Code) ?

[Dennis Assenmacher (Gast)]

Schau doch einfach ob es klappt :D. Muss da nicht irgendwo ein delay 
rein? Sonst wirds weniger Lauf mehr Standlicht

[Serieousl L. (stephan2013)]

Das Problem ist dass ich es nur mit dem Simulator zuhause ausprobieren 
kann.

Ja Wartfunktion is auch dabei:
Aber mit dem Simulator hab ichs versucht aber da liegt irgendwo ein 
Fehler

   int16_t i=0;
    DDRB=0x00;   // input Port
    PORTB=0xFF;  // alles aus
    
    DDRA=0xFF;   // Port A als Output setzen
    PORTA=0xFF;   // alles ausschalten
    while (i<=7)
    {
      PORTA = ~(0x01 << i);
      WaitMs(500);
      if (PINB &(0x01 <<1)==0)
      {
        PORTA=0xFF;
      }
      else
      {
         i++;
      }         
    }

[Serieousl L. (stephan2013)]

??????????

[Edi R. (edi_r)]

Was passiert, wenn i größer als 7 geworden ist?

[Serieousl L. (stephan2013)]

dann beginnt der lauflicht von vorne

[Eumel (Gast)]

hisham moustafa schrieb:
> dann beginnt der lauflicht von vorne

Und welchen Wert hat i dann?

[Gerhard W. (gerhard86)]

hisham moustafa schrieb:
> dann beginnt der lauflicht von vorne

Eher nicht, i muss auf 0 zurückgesetzt werden, sonst endet das Programm 
nach einem Durchlauf

[Serieousl L. (stephan2013)]

ja stimmt:

while (i<=7)
    {
     PORTA = ~(0x01 << i);
     WaitMs(500);
      if (PINB &(0x01 <<1)==PINB2)
      {
        i++;
      }
      else
      {
        PORTA=0xFF;
         
      }
      if (i>7)
        i=0;         
    }

[Karl H. (kbuchegg)]

Da erhebt sich jetzt natürlich sofort die Frage....

...
    while (i<=7)
    {
      ....

      if (i>7)
        i=0;         
    }

.... wie soll eigentlich i jemals größer als 7 werden, damit diese 
Abfrage hier

    while (i<=7)

jemals eine Chance hat, FALSE zu werden.

i kann durch dieses Konstukt hier

      if (i>7)
        i=0;

und dem Inkrement innerhalb der Schleife niemals größer als 7 werden. 
Das bedeutet aber auch, dass die Abfrage von i beim while eine sinnlose 
Abfrage ist, denn die Bedingung kann nie eintreten.
Was du aber machst, ist: du verwirrst dich selbst und einen Leser deines 
Programmcodes, indem du ihm durch die reine Anwesenheit der Abfrage 
suggerierst, da würde etwas dahinterstecken.

Tut es aber nicht

  while( 1 )
  {

     ....

  }

und gut ists. Jeder Programmierer erkennt das als Endlosschleife, für 
die es keine Abbruchbedingung gibt.



Und darüber

     if (PINB &(0x01 <<1)==PINB2)

solltest du auch noch mal nachdenken.
In C braucht es keinen expliziten Vergleich an dieser Stelle. Das if wil 
leinfach nur einen arithmetischen Ausdruck. Ergibt die Auswertung des 
Ausdrucks 0, dann gilt das als logisches FALSE. Ergibt die Auswertung 
etwas anderes als 0, dann gilt das als logisch TRUE. Fertig, Mehr 
braucht es nicht.
Der Ausdruck

     PINB & ( 1 << PB1 )

ergibt 0, wenn der Pin PB1 auf 0 liegt. Und er ergibt etwas anderes als 
0, wenn der Pin selber auf 1 liegt. Welcher Wert in letzterem Fall genau 
heraus kommt, interessiert uns nicht. Uns interessiert nur, dass da 
etwas rauskommt, was nicht 0 ist. UNd das reicht dann schon.

Abfrage auf Pin High

   if( PINB & ( 1 << PB1 ) )             // wenn Pin PB1 auf 1 ist

Abfrage auf Pin Low

   if( !( PINB & ( 1 << PB1 ) ) )       // wenn Pin PB1 auf 0 ist
  ...

oder auch so

Abfrage auf Pin High

   if( PINB & ( 1 << PB1 ) != 0 )       // ungleich 0, egal was - hauptsache ungleich 0

Abfrage auf Pin Low

   if( PINB & ( 1 << PB1 ) == 0 )       // wenn Pin PB1 auf 0 ist

Aber gewöhn dir diese Vergleich mit expliziten Zahlenwerten ausser 0 ab. 
Das sind Stolpersteine. Drücke deine Vergleiche als Vergleiche mit 0 
aus, oder lass sie überhaupt gleich weg - denn in Wirklichkeit braucht 
die keiner.

[Serieousl L. (stephan2013)]

Vielen Dank.

Und wegen das Anhalten von dem lauflicht:
Wenns auf PINB2 auf Low ist schalte ich alle LEDS aus oder ??
sonst erhöhe ich den zähler um eins

                 if (PINB &(1 <<PINB2) == 0)
      {
        PORTA=0xFF;
      }
      else
      {
        i++; 
      }

[AVRfreak (Gast)]

Hallo,

> Wenns auf PINB2 auf Low ist schalte ich alle LEDS aus oder ??

"Stoppen" würde ich als "nicht weiter schieben", aber nicht als 
"Ausschalten der LEDs" interpretieren ...

[Serieousl L. (stephan2013)]

Ja wie soll ich dann das lauflicht anhalten

[Karl H. (kbuchegg)]

hisham moustafa schrieb:
> Ja wie soll ich dann das lauflicht anhalten

indem zb i nicht mehr erhöht wird :-)

Wenn i nicht erhöht wird, dann gibt

     PORTA = ~(0x01 << i);

immer dasselbe Bitmuster aus. Wird immer dasselbe ausgegeben, dann 
änderen sich auch die LED nicht. LEDs an einem Port, die sich nicht 
verändern sehen für einen Betrachter wie aus?

(alternativ kannst du natürlich auch auf die Ausgabe in diesem Fall 
komplett verzichten. Denn wozu etwas an die Ports ausgeben, was sowieso 
schon an den Ports anliegt? Ein Ausgabe-Port behält ja ganz von alleine 
solange seinen Wert in allen Bits, solange bis du etwas anderes zuweist)

[Serieousl L. (stephan2013)]

Ja stimmt ^^
Nochmals vielen Dank !