Controller reagiert nicht immer auf Tasterdruck

[Thomas (Gast)]

Mein Arduino reagiert nicht auf jeden Tastendruck.



Woran kann das denn liegen? Es ist nicht immer so. Aber manchmal muss 
ich 10x drücken bis etwas passiert.


Das das auch schon jemand mal gehabt?


Tips und Vorschläge was man tun kann wären für mich sehr hilfreich.

[Bastler (Gast)]

z.B. alle delay() rauswerfen und durch gescheite Konstrukte ersetzen?

War nur mal so eine Annahme, soll aber bei Jüngern der Arduino IDE öfter 
vorkommen.

[6a66 (Gast)]

Thomas schrieb:
> Mein Arduino reagiert nicht auf jeden Tastendruck.

Mein Auto fährt nicht - woran liegt das? Kann das seind ass man da den 
Zündschlüssel rumdrehen muss oder die runden Dinger da dran sein müssen?

Wenn Du uns gute Informationen gibts können wir auch gute Antworten 
liefern!

rgds

[Karl M. (Gast)]

Thomas,

/was man tun kann ?/
Bitte stelle den gesamten Code ein !

[Ulrich F. (Gast)]

6a66 schrieb:
> Wenn Du uns gute Informationen gibts können wir auch gute Antworten
> liefern!

Meine Glaskugel stimmt der guten Antwort von Bastler zu!

[Peter D. (peda)]

Thomas schrieb:
> Woran kann das denn liegen?

Ne ordentliche Entprell-Lib (im Timerinterrupt) nehmen.

[oldmax (Gast)]

Hi
man kann aber auch den Taster dauernd festhalten, bis erwünschte 
Reaktion erfolgt. Dann die Pfoten von. und gut, Automatisiert... geht 
auch. Ein Modellauto einschalten, wenn Tastersignal erkannt, das fährt 
dann weg. Ist auch gut. Na ja, zum Betätigen halt das Auto wieder 
draufstellen...
Gruß oldmax

PS: das ihr aber auch immer so geil auf den geheimen Code seid..

[Harald W. (wilhelms)]

oldmax schrieb:

> PS: das ihr aber auch immer so geil auf den geheimen Code seid..

Genau, damit will sich Thomas schliesslich für den nächsten
Nobelpreis anmelden (Leider gibts keinen Nobelpreis für
Mathematik).

[Wolle G. (wolleg)]

Vielleicht hat der schnöde Taster eine Macke.
Ich hatte schon mehrere Tasterausfälle sogar an einer 
Markenwaschmaschine. Taster getauscht, fertig.

[Karl H. (kbuchegg)]

wolle g. schrieb:
> Vielleicht hat der schnöde Taster eine Macke.

Ist zwar nicht ausgeschlossen.

Aber wahrscheinlicher ist es, dass es ganz einfach wieder mal an einem 
Programm liegt, das mit Delay Aufrufen gespickt ist.

[Thomas (Gast)]

Hallo


hier der Code. Ich muss immer so ca. 3 Sek. warten, wenn ich dann einen 
der Taster betätige reagiert der Controller.

Kann man das noch optimierten?

#include <Wire.h>

//#include <LiquidCrystal.h>

unsigned char search_mode=0;

char string[8];

int b=0;
int c=0;

#define Button_next 30
#define Button_prev 31



unsigned char frequencyH=0;
unsigned char frequencyL=0;

unsigned int frequencyB;
double frequency=0;

float freq_available=0;



#define RST 12
#define CE  11
#define DC  10
#define DIN  9
#define CLK  8

static const byte ASCII[][5] =
{
 {0x00, 0x00, 0x00, 0x00, 0x00} // 20  
,{0x00, 0x00, 0x5f, 0x00, 0x00} // 21 !
,{0x00, 0x07, 0x00, 0x07, 0x00} // 22 "
,{0x14, 0x7f, 0x14, 0x7f, 0x14} // 23 #
,{0x24, 0x2a, 0x7f, 0x2a, 0x12} // 24 $
,{0x23, 0x13, 0x08, 0x64, 0x62} // 25 %
,{0x36, 0x49, 0x55, 0x22, 0x50} // 26 &
,{0x00, 0x05, 0x03, 0x00, 0x00} // 27 '
,{0x00, 0x1c, 0x22, 0x41, 0x00} // 28 (
,{0x00, 0x41, 0x22, 0x1c, 0x00} // 29 )
,{0x14, 0x08, 0x3e, 0x08, 0x14} // 2a *
,{0x08, 0x08, 0x3e, 0x08, 0x08} // 2b +
,{0x00, 0x50, 0x30, 0x00, 0x00} // 2c ,
,{0x08, 0x08, 0x08, 0x08, 0x08} // 2d -
,{0x00, 0x60, 0x60, 0x00, 0x00} // 2e .
,{0x20, 0x10, 0x08, 0x04, 0x02} // 2f /
,{0x3e, 0x51, 0x49, 0x45, 0x3e} // 30 0
,{0x00, 0x42, 0x7f, 0x40, 0x00} // 31 1
,{0x42, 0x61, 0x51, 0x49, 0x46} // 32 2
,{0x21, 0x41, 0x45, 0x4b, 0x31} // 33 3
,{0x18, 0x14, 0x12, 0x7f, 0x10} // 34 4
,{0x27, 0x45, 0x45, 0x45, 0x39} // 35 5
,{0x3c, 0x4a, 0x49, 0x49, 0x30} // 36 6
,{0x01, 0x71, 0x09, 0x05, 0x03} // 37 7
,{0x36, 0x49, 0x49, 0x49, 0x36} // 38 8
,{0x06, 0x49, 0x49, 0x29, 0x1e} // 39 9
,{0x00, 0x36, 0x36, 0x00, 0x00} // 3a :
,{0x00, 0x56, 0x36, 0x00, 0x00} // 3b ;
,{0x08, 0x14, 0x22, 0x41, 0x00} // 3c <
,{0x14, 0x14, 0x14, 0x14, 0x14} // 3d =
,{0x00, 0x41, 0x22, 0x14, 0x08} // 3e >
,{0x02, 0x01, 0x51, 0x09, 0x06} // 3f ?
,{0x32, 0x49, 0x79, 0x41, 0x3e} // 40 @
,{0x7e, 0x11, 0x11, 0x11, 0x7e} // 41 A
,{0x7f, 0x49, 0x49, 0x49, 0x36} // 42 B
,{0x3e, 0x41, 0x41, 0x41, 0x22} // 43 C
,{0x7f, 0x41, 0x41, 0x22, 0x1c} // 44 D
,{0x7f, 0x49, 0x49, 0x49, 0x41} // 45 E
,{0x7f, 0x09, 0x09, 0x09, 0x01} // 46 F
,{0x3e, 0x41, 0x49, 0x49, 0x7a} // 47 G
,{0x7f, 0x08, 0x08, 0x08, 0x7f} // 48 H
,{0x00, 0x41, 0x7f, 0x41, 0x00} // 49 I
,{0x20, 0x40, 0x41, 0x3f, 0x01} // 4a J
,{0x7f, 0x08, 0x14, 0x22, 0x41} // 4b K
,{0x7f, 0x40, 0x40, 0x40, 0x40} // 4c L
,{0x7f, 0x02, 0x0c, 0x02, 0x7f} // 4d M
,{0x7f, 0x04, 0x08, 0x10, 0x7f} // 4e N
,{0x3e, 0x41, 0x41, 0x41, 0x3e} // 4f O
,{0x7f, 0x09, 0x09, 0x09, 0x06} // 50 P
,{0x3e, 0x41, 0x51, 0x21, 0x5e} // 51 Q
,{0x7f, 0x09, 0x19, 0x29, 0x46} // 52 R
,{0x46, 0x49, 0x49, 0x49, 0x31} // 53 S
,{0x01, 0x01, 0x7f, 0x01, 0x01} // 54 T
,{0x3f, 0x40, 0x40, 0x40, 0x3f} // 55 U
,{0x1f, 0x20, 0x40, 0x20, 0x1f} // 56 V
,{0x3f, 0x40, 0x38, 0x40, 0x3f} // 57 W
,{0x63, 0x14, 0x08, 0x14, 0x63} // 58 X
,{0x07, 0x08, 0x70, 0x08, 0x07} // 59 Y
,{0x61, 0x51, 0x49, 0x45, 0x43} // 5a Z
,{0x00, 0x7f, 0x41, 0x41, 0x00} // 5b [
,{0x02, 0x04, 0x08, 0x10, 0x20} // 5c ¥
,{0x00, 0x41, 0x41, 0x7f, 0x00} // 5d ]
,{0x04, 0x02, 0x01, 0x02, 0x04} // 5e ^
,{0x40, 0x40, 0x40, 0x40, 0x40} // 5f _
,{0x00, 0x01, 0x02, 0x04, 0x00} // 60 `
,{0x20, 0x54, 0x54, 0x54, 0x78} // 61 a
,{0x7f, 0x48, 0x44, 0x44, 0x38} // 62 b
,{0x38, 0x44, 0x44, 0x44, 0x20} // 63 c
,{0x38, 0x44, 0x44, 0x48, 0x7f} // 64 d
,{0x38, 0x54, 0x54, 0x54, 0x18} // 65 e
,{0x08, 0x7e, 0x09, 0x01, 0x02} // 66 f
,{0x0c, 0x52, 0x52, 0x52, 0x3e} // 67 g
,{0x7f, 0x08, 0x04, 0x04, 0x78} // 68 h
,{0x00, 0x44, 0x7d, 0x40, 0x00} // 69 i
,{0x20, 0x40, 0x44, 0x3d, 0x00} // 6a j 
,{0x7f, 0x10, 0x28, 0x44, 0x00} // 6b k
,{0x00, 0x41, 0x7f, 0x40, 0x00} // 6c l
,{0x7c, 0x04, 0x18, 0x04, 0x78} // 6d m
,{0x7c, 0x08, 0x04, 0x04, 0x78} // 6e n
,{0x38, 0x44, 0x44, 0x44, 0x38} // 6f o
,{0x7c, 0x14, 0x14, 0x14, 0x08} // 70 p
,{0x08, 0x14, 0x14, 0x18, 0x7c} // 71 q
,{0x7c, 0x08, 0x04, 0x04, 0x08} // 72 r
,{0x48, 0x54, 0x54, 0x54, 0x20} // 73 s
,{0x04, 0x3f, 0x44, 0x40, 0x20} // 74 t
,{0x3c, 0x40, 0x40, 0x20, 0x7c} // 75 u
,{0x1c, 0x20, 0x40, 0x20, 0x1c} // 76 v
,{0x3c, 0x40, 0x30, 0x40, 0x3c} // 77 w
,{0x44, 0x28, 0x10, 0x28, 0x44} // 78 x
,{0x0c, 0x50, 0x50, 0x50, 0x3c} // 79 y
,{0x44, 0x64, 0x54, 0x4c, 0x44} // 7a z
,{0x00, 0x08, 0x36, 0x41, 0x00} // 7b {
,{0x00, 0x00, 0x7f, 0x00, 0x00} // 7c |
,{0x00, 0x41, 0x36, 0x08, 0x00} // 7d }
,{0x10, 0x08, 0x08, 0x10, 0x08} // 7e ←
,{0x78, 0x46, 0x41, 0x46, 0x78} // 7f →
};

void LcdXY(int x, int y)
{
  LcdWriteCmd(0x80 | x); //Column
  LcdWriteCmd(0x40 | y); //Row
}

void LcdWriteString(char *characters)
{
  while(*characters) LcdWriteCharacter(*characters++);
}

void LcdWriteCharacter(char character)
{
  for(int i=0; i<5; i++) LcdWriteData(ASCII[character - 0x20][i]);
  LcdWriteData(0x00);
}

void LcdWriteData(byte cmd)
{
digitalWrite(DC, HIGH);// DC pin is High for Data
digitalWrite(CE, LOW);
shiftOut(DIN, CLK, MSBFIRST, cmd);//transmit serial data
digitalWrite(CE, HIGH);
}

void LcdWriteCmd(byte cmd)
{
digitalWrite(DC, LOW);// DC pin is low for commands
digitalWrite(CE, LOW);
shiftOut(DIN, CLK, MSBFIRST, cmd);//transmit serial data
digitalWrite(CE, HIGH);
}



//LiquidCrystal lcd(12, 11, 5, 4, 3, 2); 

void setup()   { 

  Wire.begin();
  //lcd.begin(16, 2);


  pinMode(RST, OUTPUT);
  pinMode(CE, OUTPUT);
  pinMode(DC, OUTPUT);
  pinMode(DIN, OUTPUT);
  pinMode(CLK, OUTPUT);
  digitalWrite(RST, LOW);
  digitalWrite(RST, HIGH);


  LcdWriteCmd(0x21); //LCD extended commands
  LcdWriteCmd(0xB8); // set LCD Vop (contrast)
  LcdWriteCmd(0x04); // set temp coefficent
  LcdWriteCmd(0x14); //LCD bias mode 1:40
  LcdWriteCmd(0x20); //LCD basic commands
  LcdWriteCmd(0x0C); //LCD normal video mode

  //LcdXY(0,0);

  /// buttons  

  pinMode(Button_next, INPUT);
  digitalWrite(Button_next, HIGH); //pull up resistor

  pinMode(Button_prev, INPUT);
  digitalWrite(Button_prev, HIGH); //pull up resistor

  frequency=94.7; //starting frequency

  

  frequencyB=4*(frequency*1000000+225000)/32768; //calculating PLL word

  frequencyH=frequencyB>>8;

  frequencyL=frequencyB&0XFF;

  //delay(1);

  Wire.beginTransmission(0x60);   //writing TEA5767

  Wire.write(frequencyH);
  Wire.write(frequencyL);
  Wire.write(0xB0);
  Wire.write(0x10);
  Wire.write(0x00);
  Wire.endTransmission();

  //delay(1);

}

void loop()
{

  unsigned char buffer[5];

  //lcd.setCursor(0, 0);

  Wire.requestFrom(0x60,5); //reading TEA5767

  if (Wire.available()) 

  {
    for (int i=0; i<5; i++) {

      buffer[i]= Wire.read();
    }

    freq_available=(((buffer[0]&0x3F)<<8)+buffer[1])*32768/4-225000;

     LcdWriteString("FM ");
    //lcd.print("FM ");
    LcdWriteString(dtostrf(freq_available/1000000,5,2,string));

    //lcd.print((freq_available/1000000));

    frequencyH=((buffer[0]&0x3F));

    frequencyL=buffer[1];

    if (search_mode) {

      if(buffer[0]&0x80) search_mode=0;

    }

    //if (search_mode==1) LcdWriteString(" SCAN");//lcd.print(" SCAN");
    //else {
      //LcdWriteString("NO SCAN");//lcd.print("       ");
    //}

    //lcd.setCursor(0, 1);
    LcdXY(0, 1);
    

    //LcdWriteString("Level: ");//lcd.print("Level: ");
    //LcdWriteCharacter((buffer[3]>>4));
    //LcdWriteString("/16");
    //lcd.print((buffer[3]>>4));
    //lcd.print("/16 ");

    if (buffer[2]&0x80) LcdWriteString("STEREO   ");//lcd.print("STEREO   ");
    else LcdWriteString("MONO     ");//lcd.print("MONO   ");

  }

  ///// buttons read

  //////////// button_next////////// 
  if (!digitalRead(Button_next)&&!b) {

    frequency=(freq_available/1000000)+0.10;

    frequencyB=4*(frequency*1000000+225000)/32768+1;

    frequencyH=frequencyB>>8;
    frequencyL=frequencyB&0XFF;   

    Wire.beginTransmission(0x60);   

    Wire.write(frequencyH);
    Wire.write(frequencyL);
    Wire.write(0xB0);
    Wire.write(0x1F);
    Wire.write(0x00); 

    Wire.endTransmission(); 

    //////////////////////

    b=100;

  };

  if (!digitalRead(Button_next)&&b==1) {

    ///scannnn UP

    search_mode=1;

    Wire.beginTransmission(0x60);   

    Wire.write(frequencyH+0x40);
    Wire.write(frequencyL);
    Wire.write(0xD0);
    Wire.write(0x1F);
    Wire.write(0x00); 

    Wire.endTransmission();

    /////////////////

    b=100;

  };    

  if (!b==0) b--;

  //////////// button_prev////////// 
  if (!digitalRead(Button_prev)&&!c) {

    frequency=(freq_available/1000000)-0.10;

    frequencyB=4*(frequency*1000000+225000)/32768+1;

    frequencyH=frequencyB>>8;
    frequencyL=frequencyB&0XFF;

    Wire.beginTransmission(0x60);   

    Wire.write(frequencyH);
    Wire.write(frequencyL);
    Wire.write(0xB0);
    Wire.write(0x1F);
    Wire.write(0x00); 

    Wire.endTransmission(); 

    c=100;

  };

  if (!digitalRead(Button_prev)&&c==1) {

    ///scannnn DOWN

    search_mode=1;

    Wire.beginTransmission(0x60);   

    Wire.write(frequencyH+0x40);
    Wire.write(frequencyL); 

    Wire.write(0x50);
    Wire.write(0x1F);
    Wire.write(0x00);
    Wire.endTransmission();   

    c=100;

  };          

  if (!c==0) c--;

  ////////////////////

}

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

Was steht da dick und fett direkt über der Texteingabebox?

Antwort schreiben
Wichtige Regeln - erst lesen, dann posten!

* Längeren Sourcecode nicht im Text einfügen,
  sondern als Dateianhang

BTW: wie schnell ist dein "Wire"?

[Alf (Gast)]

Ich kann nicht erkennen, was du da für wichtige Wire.Transmissions
aussendest, aber:

Eine sichere Tastenerkennung ohne einen Timer-Interrupt,
der z.B. alle 1...50 ms den Zustand der Tasten abfragt, ist
unzuverlässig, wenn der µC auch manchmal für längere Zeit mit
was ganz anderem beschäftigt ist.

Wenn man die Tastenerkennung in diesem Timer-Interrupt
fix bearbeitet, stört das kaum bei den anderen Aufgaben.
Du musst nur ein paar Register bereitstellen, in denen die
Tasten-Aktionen für die Hauptschleife festgehalten werden.

In der Hauptschleife wird irgendwann in den Registern nachgeschaut,
was angefordert wurde - und darauf reagiert. - Und das bearbeitete
Register gelöscht, sonst wird es eine Endlos-Reaktion...

[Karl H. (kbuchegg)]

Erklär doch mal, was hier

  if (!digitalRead(Button_next)&&!b) {

....
    b=100;

  };

  if (!digitalRead(Button_next)&&b==1) {

....
    b=100;

  };    

  if (!b==0) b--;

der Sinn der Sache ist. So ganz durchschau ich das noch nicht, welchen 
Zweck hier b erfüllen soll.
Ich seh aber das, wenn b erst mal auf 100 ist, es mindestens 99 
Durchläufe durch loop benötigt, bis b wieder entweder 0 oder 1 ist. 99 
(bzw. 100) Durchläufe in denen offenbar jedesmal per I2C ein Empfänger 
nach seiner Frequenz gefragt wird, die dann mittels Floating Point 
Arithmetik umgerechnet und aufs Display gegeben wird. Was beides auch 
nicht gerade das schnellste zu sein scheint. Und vor allen Dingen: 99 
Durchläufe, in denen eine gedrückte Taste zu keiner Aktion führt.

"Kann man das optimieren" ist gut. Man könnte zuallererst mal eine 
vernünftige Tastendruckerkennung einbauen.