ATMEGA8 eingeschaltete LED Zählen

[Peter (Gast)]

if(PORTC & (BIT_1)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8  )
   count--; }
 if(PORTB & (BIT_2)){
  if( count <= 6 )
    count++;
    }else{
    if( count>8  )
    count--; }
 if(PORTB & (BIT_3)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8  )
   count--; }
 if(PORTC & (BIT_4)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8  )
   count--; }
if(PORTC & (BIT_5)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8  )
   count--;}
 if(PORTC & (BIT_6)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8 )
   count--;}
 if(PORTC & (BIT_7)){
   if( count <= 6 )
   count++;
   }else{
   if( count>8 )
   count--; }
   

if( count >= 1){
return 7;
if(count<=7){
  restart

Kann ich meine eingeschalteten LED so zählen?
oder habt ihr einen anderen Vorschlag?
Ich habe 7 LED und ich möchte einen bestimmten Vorgang "restart" 
einleiten, wenn 7 oder mehr LED an sind

[Max H. (hartl192)]

Peter schrieb:
> Ich habe 7 LED und ich möchte einen bestimmten Vorgang "restart"
> einleiten, wenn 7 oder mehr LED an sind
Wie können bei 7 LEDs mehr als 7 an sein?
Wenn die Zahl der LEDs unwichtig ist und es nur interessant it ob alle 7 
an sinf oder nicht sind geht es so schnellet

if(PORTB&0x0C)|(PORTC&0xF2)==0xFE)
  restart();

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> if( count>8 )
>    count--; }

 Kann niemals wahr werden.

Peter schrieb:
> if( count >= 1){
> return 7;
> if(count<=7){
>   restart

 Wie willst du die untere Abfrage erreichen ?

 Und wo fängt was an, wo endet es ?

[Karl H. (kbuchegg)]

Peter schrieb:

Was soll denn der ganze Palawatsch mit ...

>    if( count <= 6 )
>    count++;
>    }else{
>    if( count>8  )
>    count--; }

Zähl halt einfach, wieviele Bits auf 1 sind

  count = 0;

  if( PORTB & (BIT_2) )
    count++;

  if( .... )
    count++;

  ...

nachdem du alle Bits geprüft hast, hast du die Anzahl der brennenden 
LEDs.

> Ich habe 7 LED und ich möchte einen bestimmten Vorgang "restart"
> einleiten, wenn 7 oder mehr LED an sind

Ja dann mach das doch

   if( count >= 7 )
     restart

wo ist denn da jetzt das Problem, bzw. warum muss man da so kompliziert 
rumtricksen während man die Portbits abklappert?

Ich persönlich würde ja ganz einfach bei jedem Einschalten einer LED 
einfach einen Zähler erhöhen bzw. beim jeweiligen Ausschalten der LED 
den Zähler wieder verringern, so dass ich zu jedem Zeitpunkt die Anzahl 
der brennenden LED in dieser globalen Zählvariable vorliegen habe. Aber 
das kann jeder halten wie er will.

[Karl H. (kbuchegg)]

Karl Heinz schrieb:

> Ich persönlich würde ja ganz einfach bei jedem Einschalten einer LED
> einfach einen Zähler erhöhen bzw. beim jeweiligen Ausschalten der LED
> den Zähler wieder verringern, so dass ich zu jedem Zeitpunkt die Anzahl
> der brennenden LED in dieser globalen Zählvariable vorliegen habe. Aber
> das kann jeder halten wie er will.



Überlesen. Aus dem OT
> Ich habe 7 LED und ich möchte einen bestimmten Vorgang
> "restart" einleiten, wenn 7 oder mehr LED an sind

Das ist eine extrem komplizierte Umschreibung für: Wenn alle LED 
brennen.
Das lässt sich aber leicht feststellen. Du kennst die Portbits aller LED 
und die müssen alle (offenbar) auf 1 sein. Ohne Ausnahme. Das ist eine 
simple Abfrage. Da muss man noch nicht einmal zählen.

[stefan us (Gast)]

Eingänge liest man über das Register PINC!

uint8_t mask=1;
uint8_t count=0;
for (uint8_t i=0; i<=6; i++) {
  if (PINC & mask) count++;
  mask<<=1;
}
printf("count=%i",count);

Was hast Du dir dabei gedacht?:

if( count>8 )
   count--;

Die Einrückungen sind sehr irreführend. Der Code mach beim zweiten 
hinschauen etwas ganz anderes, als auf den ersten Blick. Im Sinne der 
lesbarkeit rate ich Dir dazu, den Code so zu formatieren, dass die 
Einrückung der Klammerung entspricht - also so, wie es auch jeder Editor 
mit Auto-Format macht.

So ist es besser:

if(PORTC & (BIT_6)){
   if( count <= 6 ) {
     count++;
   }
}else{
   if( count>8 ) {
     count--;
   }
}

[Max H. (hartl192)]

stefan us schrieb:
> Eingänge liest man über das Register PINC!
Aber nur die an PORTC, die LEDs an PORTB kann er damit nicht einlesen. 
Wenn er die LEDs erst in Software durch setzten des entsprechenden bits 
in PORTx einschaltet kann er sie auch über das PORTx Register abfragen.

[Karl H. (kbuchegg)]

stefan us schrieb:
> Eingänge liest man über das Register PINC!

Das passt schon.
Er will wissen, wieviele Ausgänge vom Rest des Programms auf 1 gesetzt 
wurden. Das kann er mit dem jeweiligen PORT Register machen.

[stefan us (Gast)]

Mir ist gerade noch eine elegante Variante eingefallen, die mit einer 
Variablen weniger auskommt:

uint8_t mask=1;
uint8_t count=0;
do {
  if (PINC & mask) count++;
  mask<<=1;
}
while (mask<128);
printf("count=%i",count);

Es geht noch kompakter, aber ich finde es weniger gut lesbar:

uint8_t count=0;
for (uint8_t mask=1; mask<128; mask<<=1) {
  count++;
}
printf("count=%i",count);

[Marc V. (Firma: Vescomp) (logarithmus)]

stefan us schrieb:
> Eingänge liest man über das Register PINC!

 Ne, er schaltet die LEDs selber.

Karl Heinz schrieb:
> Das ist eine extrem komplizierte Umschreibung für: Wenn alle LED
> brennen.

 Ja.
 (PORTB & 0xFE) == 0xFE

[stefan us (Gast)]

> Aber nur die an PORTC, die LEDs an PORTB kann er damit nicht einlesen.

Ach, das habe ich ja ganz übersehen...

[Max H. (hartl192)]

Marc Vesely schrieb:
> Ja.
>  (PORTB & 0xFE) == 0xFE
Die LEDs an PORTC kann er damit nicht erfassen...

if((PORTB&0x0C)|(PORTC&0xF2)==0xFE)

[Marc V. (Firma: Vescomp) (logarithmus)]

Max H. schrieb:
> Die LEDs an PORTC kann er damit nicht erfassen...

 Ja, du hattest natürlich schon bei deiner ersten Antwort recht...
 Ich war ganz einfach verblüfft über die Unsinnigkeit - war aber
 nicht der einzige, Karl Heinz hat es auch erwischt ;-))

[Peter (Gast)]

uint8_t count = 0;

  if(PINB & (BIT_2)) count++;
  if(PINB & (BIT_3)) count++;
  if(PINC & (BIT_1)) count++;
  if(PINC & (BIT_4)) count++;
  if(PINC & (BIT_5)) count++;
  if(PINC & (BIT_6)) count++;
  if(PINC & (BIT_7)) count++;
    
    return count; 

if(count==7){
  restartGame();
}

Habe es nun so, aber es funktioniert dennoch nicht.

Habe ich irgendwo noch einen Fehler?

[Max H. (hartl192)]

Peter schrieb:
> Habe es nun so, aber es funktioniert dennoch nicht.
Was funktioniert nicht?
Wie ist BIT_2,... definiert?

[Karl H. (kbuchegg)]

Peter schrieb:

>     return count;
>
> if(count==7){
>   restartGame();
> }
>
>
>
> Habe es nun so, aber es funktioniert dennoch nicht.
>
> Habe ich irgendwo noch einen Fehler?

überleg mal, wie die Funktion wohl jeh zur Auswertung auf 7 kommen soll, 
wenn vorher ein return statt findet?

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> Habe ich irgendwo noch einen Fehler?

 Ja.
 PORTB und PORTC, nicht PINB und PINC.

[Peter (Gast)]

#define BIT_1 (1 << PC1)
#define BIT_2 (1 << PB2)
#define BIT_3 (1 << PB1)
#define BIT_4 (1 << PC3)
#define BIT_5 (1 << PC2)
#define BIT_6 (1 << PC4)
#define BIT_7 (1 << PC5)

das sind die Definitionen der Bits

[Karl H. (kbuchegg)]

Marc Vesely schrieb:
> Peter schrieb:
>> Habe ich irgendwo noch einen Fehler?
>
>  Ja.
>  PORTB und PORTC, nicht PINB und PINC.

Ich fände PORTx auch naheliegender, ist aber in diesem Fall egal. Im PIN 
Register spiegelt sich die Bitposition der PORT Register wieder, wenn 
die Pins auf Ausgang geschaltet sind. Es gibt beim setzen oder löschen 
eines Bits eine kleine Verzögerung von, wenn ich mich recht erinnere, 1 
oder 2 Takten. Aber das dürfte hier keine Rolle spielen.

[Bitflüsterer (Gast)]

Naja. Karl Heinz hat ja schon gesagt, dass man nicht mal zählen muß.
Also hier das nicht-zählen:

if ((PINC & 0xF7) == 0xF7) {
  restartGame();
}

Was Deine Frage betrifft:

>Habe ich irgendwo noch einen Fehler?

So ist die Antwort: Ja.

[Karl H. (kbuchegg)]

Bitflüsterer schrieb:
> Naja. Karl Heinz hat ja schon gesagt, dass man nicht mal zählen muß.
> Also hier das nicht-zählen:
>
>

> if ((PINC & 0xF7) == 0xF7) {
>   restartGame();
> }
>

>

haut nicht hin. Er hat die LED auf 2 Ports verteilt. Ganz oben ist schon 
mal eine saubere Abfrage gepostet worden.

[npn (Gast)]

stefan us schrieb:
> Es geht noch kompakter, aber ich finde es weniger gut lesbar:
> uint8_t count=0;
> for (uint8_t mask=1; mask<128; mask<<=1) {
>   count++;
> }
> printf("count=%i",count);

Wo fragst du hier den Port ab? Ich denke, da fehlt noch was, oder :-)

[Peter (Gast)]

leider wird der restart nicht durchgeführt, die LEDs bleiben an.
wenn ich den restart direkt an den Anfang der while schelife setze, 
funktioniert er aber, also muss es noch ein Problem geben

[Max H. (hartl192)]

Peter schrieb:
> #define BIT_1 (1 << PC1)
> #define BIT_2 (1 << PB2)
> #define BIT_3 (1 << PB1)
> #define BIT_4 (1 << PC3)
> #define BIT_5 (1 << PC2)
> #define BIT_6 (1 << PC4)
> #define BIT_7 (1 << PC5)
Wenn das so ist funktioniert meine Lösung aus dem ersten Post nicht.

P.S: Ich finde es so verwirrend... Ich glaube so gut wie jeder hat 
gedacht, das BIT_1 (1<<1), BIT_2 (1<<2),..., BIT_n (1<<n) ist. 
Korrigiert nicht wenn ich falsch liege…

Peter schrieb:
> leider wird der restart nicht durchgeführt, die LEDs bleiben an.
> wenn ich den restart direkt an den Anfang der while schelife setze,
> funktioniert er aber, also muss es noch ein Problem geben
Setzt das return nach dem if

[Bitflüsterer (Gast)]

Karl Heinz schrieb:
> Bitflüsterer schrieb:
> <Irgendwas Falsches>
> haut nicht hin.

Danke für den Hinweis. Sorry.

[Peter (Gast)]

uint8_t count = 0;
  if((PORTB&0x0C)|(PORTC&0xF2)==0xFE) {
    restartGame();
  }

das programm springt jetzt in den resart. allerdings nicht, wenn alle 7 
LED an sind, sondern beim drücken von bestimmten tastern

[Marc V. (Firma: Vescomp) (logarithmus)]

Karl Heinz schrieb:
> die Pins auf Ausgang geschaltet sind. Es gibt beim setzen oder löschen
> eines Bits eine kleine Verzögerung von, wenn ich mich recht erinnere, 1
> oder 2 Takten.
 Atmel sagt, 1 NOP zwischen OUT und IN ist nötig.

>  Aber das dürfte hier keine Rolle spielen.
 Ja. Ich meine Nein. Ich meine du hast Recht.

[npn (Gast)]

Max H. schrieb:
> P.S: Ich finde es so verwirrend... Ich glaube so gut wie jeder hat
> gedacht, das BIT_1 (1<<1), BIT_2 (1<<2),..., BIT_n (1<<n) ist.
> Korrigiert nicht wenn ich falsch liege…

Habe ich auch erst gedacht, aber wenn er seine LEDs so verdrahtet hat, 
dann ist es eben so. Vielleicht sollte man dann die Definitionen nicht 
"BIT_1" "BIT_2" usw. nennen, sondern "LED_1" "LED_2". Das bringt weniger 
Verwirrung.

[Karl H. (kbuchegg)]

Peter schrieb:
> leider wird der restart nicht durchgeführt, die LEDs bleiben an.
> wenn ich den restart direkt an den Anfang der while schelife setze,
> funktioniert er aber, also muss es noch ein Problem geben

weil zuvor ein return steht!

Lies doch mal deinen Code

[Peter (Gast)]

Karl Heinz, das return habe ich bereits gelöscht, dank deinem Hinweis :)

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> if((PORTB&0x0C)|(PORTC&0xF2)==0xFE) {
>     restartGame();

 if((PORTB&0x06)==0x06) & ((PORTC&0x3E)==0x3E) {

[Peter (Gast)]

Marc, dieser Code gibt 2 Fehler aus,

Error  1  lvalue required as unary '&' operand  C:\Dokumente und 
Einstellungen\sams\Desktop\Interaktive LED\Programm\interaktive LED.c 
255  24  interaktive LED

Error  2  expected ';' before '{' token  C:\Dokumente und 
Einstellungen\sams\Desktop\Interaktive LED\Programm\interaktive LED.c 
255  45  interaktive LED

[Max H. (hartl192)]

So sollten die Klammern passen:

if(((PORTB&0x06)==0x06) && ((PORTC&0x3E)==0x3E))

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> Marc, dieser Code gibt 2 Fehler aus,

 Ja, dann schreibe es doch richtig, ich wollte dir nur zeigen
 welche Werte du vergleichen must...

[Daniel H. (Firma: keine) (commander)]

Wie wäre es so?

uint8_t i;
uint8_t count = 0;

for(i = 0; i < 8; i++) {
    count += (PORTB >> i) & 1;
}

[Karl H. (kbuchegg)]

Peter schrieb:
> Marc, dieser Code gibt 2 Fehler aus,

Zeig doch mal, was du geschrieben hast.
Marcs Code ist ok. Ist im Grunde derselbe wie vorher, nur dass die 
Vergleichswerte andere sind und den tatsächlichen Bitpositionen 
entsprechen (wenn ich mich im Kopf nicht vertan habe)

Man könnte das auch noch ein wenig anders schreiben, so dass die Gefahr 
sich an dieser Stelle in den Bits zu vertun etwas gemildert ist, aber 
syntaktisch ist da erst mal alles in Ordnung in Marcs Code.

[Peter (Gast)]

uint8_t getNubmerOfLighteningLed(){
  uint8_t count = 0;
  if(((PORTB&0x06)==0x06) && ((PORTC&0x3E)==0x3E)) {
    restartGame();


7 LEDs brennen, es passiert nichts..

[Max H. (hartl192)]

Daniel H. schrieb:
> Wie wäre es so?
> uint8_t i;
> uint8_t count = 0;
>
> for(i = 0; i < 8; i++) {
>     count += (PORTB >> i) & 1;
> }
Die LEDs sin immer noch auf PORTB und PORTC verteilt.

[Daniel H. (Firma: keine) (commander)]

Max H. schrieb:
> Die LEDs sin immer noch auf PORTB und PORTC verteilt.

Autsch, hatte ich übersehen, danke.

[Marc V. (Firma: Vescomp) (logarithmus)]

Max H. schrieb:
> if(((PORTB&0x06)==0x06) && ((PORTC&0x3E)==0x3E))

 Also, wenn ich die Klammern für ihn zählen muß...
 Aber, AndAlso habe ich wirklich übersehen.

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> 7 LEDs brennen, es passiert nichts..

 Wie ware es, wenn du die LEDs auch mal ausmachst ?

[Peter (Gast)]

uint8_t count = 0;
  if(!((PORTB&0x06)==0x06) && ((PORTC&0x3E)==0x3E)) {
    restartGame();
  }

der Code startet das Spiel neu, wenn man erst alle LED an macht und dann 
wieder aus, also schon mal sehr gut, nur dass es bereits geschehen soll, 
wenn alle LED an sind

[Karl H. (kbuchegg)]

Peter schrieb:
> #define BIT_1 (1 << PC1)
> #define BIT_2 (1 << PB2)
> #define BIT_3 (1 << PB1)
> #define BIT_4 (1 << PC3)
> #define BIT_5 (1 << PC2)
> #define BIT_6 (1 << PC4)
> #define BIT_7 (1 << PC5)


ehe da beim Umrechnen der Bitpositionen noch mehr Fehler passieren.

Ergänze da mal

...
#define BIT_7 (1 << PC5)

#define ALL_ON_B   ( BIT_2 | BIT_3 )
#define ALL_ON_C   ( BIT_1 | BIT_4 | BIT_5 | BIT_6 | BIT_7 )

und in der Funktion heisst es dann

void checkForRestart()
{
  if( ( ( PORTB & ALL_ON_B ) == ALL_ON_B ) &&
      ( ( PORTC & ALL_ON_C ) == ALL_ON_C ) )
    restartGame();
}

WEnn dann der Restart immer noch nicht ausgelöst wird, obwohl alle LED 
brennen, solltest du dir mal überlegen, ob deine LED wirklich genau dann 
brennen, wenn am Port ein 1 Bit ausgegeben wird oder ob das nicht 
umgekehrt ist.

[Marc V. (Firma: Vescomp) (logarithmus)]

Karl Heinz schrieb:
> void checkForRestart()
> {
>   if( ( ( PORTB & ALL_ON_B ) == ALL_ON_B ) &&
>       ( ( PORTC & ALL_ON_C ) == ALL_ON_C ) )
>     restartGame();
> }
>
> WEnn dann der Restart immer noch nicht ausgelöst wird, obwohl alle LED
> brennen, solltest du dir mal überlegen, ob deine LED wirklich genau dann
> brennen, wenn am Port ein 1 Bit ausgegeben wird oder ob das nicht
> umgekehrt ist.

 Ich glaube, er schaltet die LEDs in restartGame() nicht aus. Da fehlt
 ganz einfach so etwas wie AllLedOff(), oder so.

[Peter (Gast)]

Karl Heinz vielen Dank.

Es besteht noch das Problem, dass der Restart erst ausgeführt wird, wenn 
ich alle LED ausschalte. Was muss ich ändern, sodass dieser ausget wird, 
wenn alle an sind?

[Max H. (hartl192)]

Wie hast du die LEDs verschalten?

[Marc V. (Firma: Vescomp) (logarithmus)]

Peter schrieb:
> Es besteht noch das Problem, dass der Restart erst ausgeführt wird, wenn
> ich alle LED ausschalte. Was muss ich ändern, sodass dieser ausget wird,
> wenn alle an sind?

 Er wird ausgeführt, nur du merkst es nicht, weil du die LEDs in Restart
 nicht ausmachst.
 Meiner Meinung nach.

[Karl H. (kbuchegg)]

Marc Vesely schrieb:

>  Er wird ausgeführt, nur du merkst es nicht, weil du die LEDs in Restart
>  nicht ausmachst.
>  Meiner Meinung nach.

Klingt vernünftig.