Switch().case funktioniert nicht wie gewünscht

[Atmega (Gast)]

Hallo,

ich habe auf meinen Atmega den Arduino Bootlader drauf gemacht und dann 
meinen Sketch aufgespielt. Eigentlich soll nach jedem Drücken des Taster 
eine andere Led erleuchten, aber irgendwie leuchtet gar keine. Insgesamt 
soll ein Motor entweder mit einem Poti gesteuert werden oder per Zufall.

Hier ist mein bisheriger Code

const int analogInPin = A0;  // Analog input pin for potentiometer
const int analogOutPin = 9; // Analog output pin for motor driver
const int direction1 = 12; // motor output 1
const int direction2 = 13; // motor output 2
const int button_mode = 11; // game mode
const int led1 = 8;
const int led2 = 7;
const int led3 = 6;
const int led4 = 5;
int sensorValue = 0;        // value read from the pot
int gamemode=1, mode_pushed=0;
long duration=0, lasttime=0, nowtime=0;
int mode_old=0;
void setup() {
pinMode(analogInPin, INPUT);
pinMode(analogOutPin, OUTPUT);
pinMode(direction1, OUTPUT);
pinMode(direction2, OUTPUT);
pinMode(led1, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(led3, OUTPUT);
pinMode(led4, OUTPUT);


}

void loop() {
  
mode_pushed = digitalRead(button_mode);
if(mode_old ==0 && mode_pushed==1)
{
gamemode+=1;
duration=0;
if (gamemode>4) gamemode=1;
}
mode_old=mode_pushed;  

switch(gamemode){
case 1: // manual speed selection
sensorValue = analogRead(analogInPin) / 4;
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
analogWrite(analogOutPin, sensorValue);   
analogWrite(led1, HIGH);        
break;

case 2: // random mode easy
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(37,65);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(42,65);    
}
analogWrite(analogOutPin, sensorValue);
analogWrite(led2, HIGH);
}  
break;

case 3: // random mode medium
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;        
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(40,100);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(47,100);       
}
}
analogWrite(analogOutPin, sensorValue);
analogWrite(led3, HIGH); 
break;

case 4: // random mode hard
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;        
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(40,200);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(50,150);       
}
}
analogWrite(analogOutPin, sensorValue);
analogWrite(led4, HIGH);
break;
default:
delay(5);
}
}

[Karl H. (kbuchegg)]

Atmega schrieb:

> mode_pushed = digitalRead(button_mode);

Bin ich blind?

Oder wo genau legst du für den Pin 'button_mode' fest, dass das ein 
Eingang sein soll?


Wenn du Code ein wenig übersichtlicher schreiben würdest, vielleicht 
nicht alles auf einmal sondern dich in Schritten dem Ziel näheren 
würdest, dann würdest du in Summe auch viel schneller zum Ziel kommen.

[Marcel P. (souko)]

Hi,
naja, HIGH wird bei Arduino soweit ich weiss als "1" gespeichert.

Du nimmst analogWrite um deine LED's anzusteuern. Das würde heissen, 
dein HIGH wird als

analogWrite(led0, 1);

 übersetzt.

und ein Analogwert von 1 sieht man nicht bei den LED's.
Ersetze das mal durch

digitalWrite(led0, HIGH);

dann klappts bestimmt auch mit dem leuchten ;-)

EDIT: Und das was kbuchegg schreibt, ist auch nicht zu verachten !

[Karl H. (kbuchegg)]

PS;
davon unabhängig erhebt sich dann auch noch die Frage, wie dein Taster 
angeschlossen ist und ob du an diesem Eingangspin einen Pullup (oder 
Pulldown) Widerstand brauchst.

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

Karl H. schrieb:
> Wenn du Code ein wenig übersichtlicher schreiben würdest
Das beste, was dafür je erfunden wurde, ist Indentation.
Auf Deutsch: Einrückungen.

[Atmega (Gast)]

Marcel P. schrieb:
> und ein Analogwert von 1 sieht man nicht bei den LED's.
> Ersetze das mal durch

Hi,
mit digitalWrite klappt das nun bedingt. Die Led leuchtet zwar, aber 
immer. Auch wenn ich den Taster gedrückt habe.

Karl H. schrieb:
> PS;
> davon unabhängig erhebt sich dann auch noch die Frage, wie dein Taster
> angeschlossen ist und ob du an diesem Eingangspin einen Pullup (oder
> Pulldown) Widerstand brauchst.

Hi,
ich habe am MC PIN den Taster und einen 10k Widerstand. Der Widerstand 
ist mit GND verbunden und der andere Pol vom Taster mit +.

[Joe F. (easylife)]

Wie lange wartet denn "delay(5);" in "default:"?
5 Sekunden?

[Atmega (Gast)]

Karl H. schrieb:
> Oder wo genau legst du für den Pin 'button_mode' fest, dass das ein
> Eingang sein soll?

Habe ich vergessen, aber nun geändert. Problem besteht aber immer noch.

[Karl H. (kbuchegg)]

Atmega schrieb:
> Marcel P. schrieb:
>> und ein Analogwert von 1 sieht man nicht bei den LED's.
>> Ersetze das mal durch
>
> Hi,
> mit digitalWrite klappt das nun bedingt. Die Led leuchtet zwar, aber
> immer. Auch wenn ich den Taster gedrückt habe.

Gut, das ist jetzt nicht weiter verwunderlich.
Ich seh in deinem Programm, wenn ich mir die analogWrite durch 
entsprechende digitalWrite ersetzt denke, immer nur Stellen an denen du 
die LED einschaltest, aber nichts wo du sie jemals wieder ausschalten 
würdest.


Sagte ich eigentlich schon, dass es oft sehr zielführend ist, ein 
Programm in Schritten aufzubauen?

> ich habe am MC PIN den Taster und einen 10k Widerstand. Der Widerstand
> ist mit GND verbunden und der andere Pol vom Taster mit +.

Gut zu wissen.

Nichts desto trotz wird dein Taster noch prellen.
AUch wenn bei einem nackten AVR die Pins defaultmässig auf Input stehen, 
würde ich trotzdem nicht versäumen, das in Setup noch mal entsprechend 
zu wiederholen. Also, wo ist die entsprechende Einstellung für 
button_mode in setup()?

[Atmega (Gast)]

Karl H. schrieb:
> Nichts desto trotz wird dein Taster noch prellen.
> AUch wenn bei einem nackten AVR die Pins defaultmässig auf Input stehen,
> würde ich trotzdem nicht versäumen, das in Setup noch mal entsprechend
> zu wiederholen. Also, wo ist
 die entsprechende Einstellung für
> button_mode in setup()?
Habe ich bei mir schon hinzugefügt. Kann allerdings meinen ersten 
Beitrag ja nicht mehr editieren. :-)

Karl H. schrieb:
> Sagte ich eigentlich schon, dass es oft sehr zielführend ist, ein
> Programm in Schritten aufzubauen?

Ja.

Karl H. schrieb:
> Ich seh in deinem Programm, wenn ich mir die analogWrite durch
> entsprechende digitalWrite ersetzt denke, immer nur Stellen an denen du
> die LED einschaltest, aber nichts wo du sie jemals wieder ausschalten
> würdest.

Wo schalte ich die dann am besten wieder aus? In den anderen cases?

[Karl H. (kbuchegg)]

Ehe wir da jetzt noch ein paar mal hin und her hampeln.
Probier das mal

const int button_mode = 11; // game mode
const int led1 = 8;
const int led2 = 7;
const int led3 = 6;
const int led4 = 5;

uint8_t gamemode = 0;
uint8_t mode_old = 0;
uint8_t mode_pushed = 0;

void setup() {
  pinMode(button_mode, INPUT);

  pinMode(led1, OUTPUT);
  pinMode(led2, OUTPUT);
  pinMode(led3, OUTPUT);
  pinMode(led4, OUTPUT);
}


void loop() {
  
  mode_pushed = digitalRead( button_mode );
  if( !mode_old && mode_pushed )
  {
    gamemode++;
    if( gamemode > 4 )
      gamemode = 0;
  }
  mode_old = mode_pushed;  

  if( gamemode & 0x01 )
    digitalWrite( led1, HIGH );
  else
    digitalWrite( led1, LOW );

  if( gamemode & 0x02 )
    digitalWrite( led2, HIGH );
  else
    digitalWrite( led2, LOW );

  if( gamemode & 0x04 )
    digitalWrite( led3, HIGH );
  else
    digitalWrite( led3, LOW );

  delay( 10 );
}

wenn das nicht klappt, dann lass dir mal das Ergebnis vom digitalRead 
vom PinMode direkt auf eine LED ausgeben. Mglw. hast du den Taster am 
falschen Pin angeschlossen (oder die LED).

Bei jedem Tastedruck sollte sich das LED Muster verändern (es stellt die 
Binärzahl dar)

[Atmega (Gast)]

Karl H. schrieb:
> Ehe wir da jetzt noch ein paar mal hin und her hampeln.
> Probier das mal

Vielen Dank für die Mühe, aber der Code war etwas zu ungewohnt für mich 
und deswegen habe ich mich noch mal selber rangesetzt.

const int analogInPin = A0;  // Analog input pin for potentiometer
const int analogOutPin = 9; // Analog output pin for motor driver
const int direction1 = 12; // motor output 1
const int direction2 = 13; // motor output 2
const int button_mode = 11; // game mode
const int led1 = 8;
const int led2 = 7;
const int led3 = 6;
const int led4 = 5;
int sensorValue = 0;        // value read from the pot
int gamemode=1, mode_pushed=0;
long duration=0, lasttime=0, nowtime=0;
int mode_old=0;
void setup() {
pinMode(analogInPin, INPUT);
pinMode(button_mode, INPUT);
pinMode(analogOutPin, OUTPUT);
pinMode(direction1, OUTPUT);
pinMode(direction2, OUTPUT);
pinMode(led1, OUTPUT);
pinMode(led2, OUTPUT);
pinMode(led3, OUTPUT);
pinMode(led4, OUTPUT);


}

void loop() {
  
mode_pushed = digitalRead(button_mode);
delay(100);
if(mode_old ==0 && mode_pushed==1)
{
gamemode+=1;
duration=0;
if (gamemode>4) gamemode=1;
}
mode_old=mode_pushed;  

switch(gamemode){
case 1: // manual speed selection
sensorValue = analogRead(analogInPin) / 4;
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
analogWrite(analogOutPin, sensorValue);   
digitalWrite(led4, LOW);  
digitalWrite(led1, HIGH);        
break;

case 2: // random mode easy
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(37,65);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(42,65);    
}
analogWrite(analogOutPin, sensorValue);
}  
digitalWrite(led1, LOW);        
digitalWrite(led2, HIGH);        
break;

case 3: // random mode medium
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;        
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(40,100);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(47,100);       
}
}
analogWrite(analogOutPin, sensorValue);
digitalWrite(led2, LOW);  
digitalWrite(led3, HIGH);  
break;

case 4: // random mode hard
nowtime=millis();
if(nowtime>lasttime+duration){
lasttime=nowtime;        
if(random(5)>0){
duration=random(3000,15000);
digitalWrite(direction1, HIGH);
digitalWrite(direction2, LOW);
sensorValue=random(40,200);
}
else{
duration=random(1500,5000);
digitalWrite(direction2, HIGH);
digitalWrite(direction1, LOW);
sensorValue=random(50,150);       
}
}
analogWrite(analogOutPin, sensorValue);
digitalWrite(led3, LOW);
digitalWrite(led4, HIGH);  
break;
default:
delay(5);
}
}

So funktioniert es jetzt. Das delay(100) musste ich einsetzen weil sonst 
oft ein Tastendruck doppelt oder zweifach gezählt wurde.

Vielleicht hilft es ja irgendjemandem.^^

[Cpp (Gast)]

Atmega schrieb:
> mich noch mal selber rangesetzt
Die Formatierung hättest du aber behalten können.

> Vielleicht hilft es ja irgendjemandem.^^
Lies dir mal was zur Entprellung durch.

[Cpp (Gast)]

Cpp schrieb:
>> Vielleicht hilft es ja irgendjemandem.^^
Upps, da hab ich wohl die flasche Zeile(n) Zitiert :-( Die hier sollte 
es sein:

> So funktioniert es jetzt. Das delay(100) musste ich einsetzen weil sonst
> oft ein Tastendruck doppelt oder zweifach gezählt wurde.

[Amateur (Gast)]

Also beim Lesen des Codes von "Karl Heinz" wird man nicht HIGH.
Im Gegensatz zu dem Geschreibsel von "Atmini".

[Arduino (Gast)]

Cpp schrieb:
> Die Formatierung hättest du aber behalten können.

Man hätte auch einfach mal in der IDE die Taste Strg-T drücken können.

[Atmega (Gast)]

Arduino schrieb:
> Man hätte auch einfach mal in der IDE die Taste Strg-T drücken können.

Oh, danke für den Tip!