Display Arduino

[Katrina N. (Gast)]

Hey Leute,

ich steh gerade glaub ich mega auf dem Schlauch....
Mein Display ist zur Auswahl von Cocktails da. Ich habe bis jetzt 6 
Strings in denen verschiedene stehen. Jetzt soll über die Taster Down 
und Up alle angezeigt werden.
Ich dachte es mir so:
   case btnUP:
     {

     lcd.setCursor(0,0);
     lcd.print(myStrings[i]);
     i--;
     lcd.setCursor(0,1);
     lcd.print(myStrings[i]);
     i--;
     break;
     }
   case btnDOWN:
     {

    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    break;

     }



Nur leider zählt er beim zweiten nicht hoch und nicht runter.... er 
zeigt mir in der ersten Zeile String 1 an und in der 2. Zeile String 2, 
dass funktioniert.


Hat jemand eine Idee????



Vielen vielen Dank

[Hubert (Gast)]

Probier mal lcd.clear(); vor der Cursorzuweisung.

[Katrina N. (Gast)]

bringt leider nichts ...

[Katrina N. (Gast)]

und warum braucht mein Display zweimal den RESET Button, wenn ich wieder 
zur Anfangsbedingung will?

[Mick (Gast)]

Poste mal den ganzen Code - oder wenigstens die Deklaration von 
myStrings.

[Katrina N. (Gast)]

// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
char* myStrings[]={"GinTonic        ", "JackyCola       ", "WodkaBull 
",
"WodkaLemon      ", "CaptainCola     ","Wasser          "};

void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("Cocktailwahl    ");

}

void loop() {
  int x;
  int i;
  x = analogRead (0);
  lcd.setCursor(0,0);
  if (x < 60) {
    lcd.print ("Abbruch         ");
  }
  else if (x < 200) {

    lcd.print ("GIN             ");
  }
  else if (x < 400) {
    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
   }

  else if (x < 600){

    lcd.print ("Bestaetigung    ");
  }
  else if (x < 800){

    lcd.print ("Welcome         ");
  }
}

[Carl D. (jcw2)]

Und welche Hardware hält an "A0"?

[Mandi (Gast)]

es sieht so aus, ohne den ganzen Code zu kennen, als ob nur
einmal btnDOWN bzw. btnUP angesprungen wird.
Der Fehler liegt wahrscheinlich in der Tastenabfrage oder
Aufruf des Unterprogramms.

  case btnUP:
    {
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i--;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i--;
    if (i==0) i=6;  // Nach Menüpunkt 1 dann Menüpunkt 6
    break;
    }

  case btnDOWN:
    {
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    if (i==7) i=1;   // Nach Menüpunkt 6 dann Menüpunkt 1
    break;
    }

[Hubert (Gast)]

Katrina N. schrieb:
> else if (x < 400) {

Ich würde zusätzlich noch folgende Bedingung hinzufügen:

else if ((x<400) && (x>200)) {

Genauso natürlich auch bei den anderen Schleifen:

else if ((x<600) && (x>400)) {

else if ((x<800) && (x>600)) {

Ansonsten sind bei einem Wert von sagen wir mal x=500 die Bedingungen 
von 3 Schleifen erfüllt, ob dies gewünscht ist, ist fraglich.

[Katrina N. (Gast)]

@Mandi

vielen Dank, das denke ich auch, dass es nur einmal aufgerufen wird.
Mit deiner Erweiterung funktioniert es jedoch auch noch nicht.


@Carl Drexler
an "A0" hängen die 1er Eingänge der Schalter UP, Right, Down

[Katrina N. (Gast)]

Ja stimmt Hubert, diese Absicherung sollte ich auf jeden Fall machen.

[SloJo (Gast)]

Hubert schrieb:
> Ich würde zusätzlich noch folgende Bedingung hinzufügen:
>
> else if ((x<400) && (x>200)) {

Die sind ueberfluessig und auch noch falsch, weil ein Wert von 200 nun 
gar keinem Bereich mehr zugeordnet ist.

[SloJo (Gast)]

Katrina N. schrieb:
> Hat jemand eine Idee????

Ja, fang nochmal von vorne an:

 - Poste den aktuellen, kompletten Sourcecode und nicht Fragmente aus 
verschiedenen Ansaetzen.
 - Erklaere genau, was die Anforderungen sind und wie die Loesung 
funktionieren soll.
 - Erklaere genau, wie du testest und was dabei passiert.

Haeltst du dich daran, kann man dir helfen. Aber wahrscheinlich findest 
du das Problem dann sogar selbst.

[doedel (Gast)]

Du darfst die i Variable nicht innerhalb von Loop deklarieren. Die wird 
logischerweise sonst bei jedem Schleifendurchlauf wieder auf 0 gesetzt.

[Loddaar (Gast)]

i ist eine lokale Variable, du willst aber eine globale

du solltest auch die Warnungen des Compilers nicht ignorieren

[Katrina N. (Gast)]

Alles klar also nochmal von Anfang an:
hier mein aktueller Code für das Display mit Ansteuerung vom Arduino 
Mega:
// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
char* myStrings[]={"GinTonic        ", "JackyCola       ", "WodkaBull 
",
"WodkaLemon      ", "CaptainCola     ","Wasser          "};

void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("Cocktailwahl    ");

}

void loop() {
  int x;
  int i;
  x = analogRead (0);
  lcd.setCursor(0,0);
  if (x < 60) {
     lcd.clear();
    lcd.print ("Abbruch         ");
  }
  else if (x < 200) {
     lcd.clear();
    lcd.print ("GIN             ");
  }
  else if (x < 400) {

    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    if (i==7) i=1;
   }

 else if (x < 600) {
    lcd.clear();
    lcd.print ("Bestaetigung    ");
  }
  else if (x < 800) {
   lcd.clear();
    lcd.print ("Welcome         ");
  }
}



Ziel ist:

Eine Auswahl von Cocktails, die auf meinem 16*2 Display mit den Buttons 
Up and Down angezeigt werden sollen.

Problem:
Laden funktioniert, die Buttons Right, Left, Up und Select 
funktionieren.
Bei Down zeigt er mir aktuelle nur den String 1 in Zeile 1 an und String 
2 in Zeile 2 bei erneutem rücken von Down sollte eigentlich dann String 
3 in Zeile 1 und String 4 in Zeile 2 stehen.


Vielen Dank schon mal

[Katrina N. (Gast)]

Der Compiler zeigt keine Warnungen an. Habe das i jetzt als globale 
gesetzt, jetzt hängt sich das Display nach betätigen von Button Down 
auf.

[Mark B. (markbrandis)]

Du hast die Variable i zwar angelegt, aber nicht initialisiert. Damit 
ist ihr Wert unbestimmt (wenn es eine lokale Variable ist). Das passt so 
nicht.

Als globale Variable sollte ihr Initialwert laut C-Standard gleich Null 
sein, wenn man keine explizite Zuweisung vornimmt. Trotzdem ist es 
besser wenn man einen sinnvollen Startwert hinschreibt. Zumal als 
Anfänger(in) sollte man sich das angewöhnen :-)

[Katrina N. (Gast)]

Ja das ist mir schon klar mit der 0, die ist auch da gewesen.

[Mark B. (markbrandis)]

Katrina N. schrieb:
> Ja das ist mir schon klar mit der 0, die ist auch da gewesen.

Dann postest Du also nicht den Code, der aktuell bei Dir läuft. Somit 
ist es sinnlos Dir helfen zu wollen. Schade auch. :-(

[Katrina N. (Gast)]

Naja doch schon, habe nach dem geschrieben wurde, dass ich es als lokale 
Variable deklariert, es als globale, mit int i = 0, geändert.

[Katrina N. (Gast)]

das ist jetzt der aktuelle:

// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
char* myStrings[]={"GinTonic        ", "JackyCola       ", "WodkaBull 
",
"WodkaLemon      ", "CaptainCola     ","Wasser          "};
int i=0;
void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("Cocktailwahl    ");

}

void loop() {
  int x;

  x = analogRead (0);
  lcd.setCursor(0,0);
  if (x < 60) {
     lcd.clear();
    lcd.print ("Abbruch         ");
  }
  else if (x < 200) {
     lcd.clear();
    lcd.print ("GIN             ");
  }
  else if (x < 400) {

    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    if (i==7) i=1;
   }

 else if (x < 600) {
    lcd.clear();
    lcd.print ("Bestaetigung    ");
  }
  else if (x < 800) {
   lcd.clear();
    lcd.print ("Welcome         ");
  }
}

[Heiko G. (Gast)]

Ich würde erstmal das Ergebnis von analogRead ausgeben, um 
sicherzustellen das die A/D-Wandlung die erwarteten Werte liefert.

Weiter wird im Bereich x zwischen 200 und 400 die variable I so schnell 
inkrementiert wie die Loop()-Funktion aufgerufen wird, so das beim 
loslassen dann eigentlich nur ein zufälliger Wert in I steht. Vermutlich 
soll doch bei jedem Betätigen die Variable I nur einmal weitergestellt 
werden.

Was gibt analogRead zurück wenn keine Taste betätigt wird ?

Da nur 6 Strings im Array sind werden diese von 0 .. 5 indiziert. Daher 
muss es

if (i==6) i=0;

heissen

[Katrina N. (Gast)]

Heiko G. schrieb:
> Was gibt analogRead zurück wenn keine Taste betätigt wird ?

hier gibt er 1023 aus

[Katrina N. (Gast)]

Ja bei betätigen soll i nur einmal weitergestellt werden

[Hubert (Gast)]

SloJo schrieb:
> else if ((x<400) && (x>200)) {

Richtig, es muss wie folgt heißen: else if ((x<400) && (x>=200)) {

Jedoch musst du mir noch erklären, warum es deiner Meinung nach nicht 
gebraucht wird.

[Mandi (Gast)]

zum testen:

// mit Arduino-Monitor Ausgabe
// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

int i = 0;

char* myStrings[]={
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};

void setup() {
  Serial.begin(9600);
  for (i = 0; i < 6; i++){
    Serial.println(myStrings[i]);  // für Testzwecke
    delay(1);
  }

    // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
    // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("Cocktailwahl:   ");
  }

void loop() {

  int x=0;                         // evtl. x=1023
  x = analogRead (0);

  if (x > 0) && (x < 61) {         //  1-60
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Abbruch         ");
  }

  else if (x > 60) && (x < 201) {  // 61-200
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("GIN             ");
  }

  else if (x > 200) && (x < 401) { // 201-400
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    if (i==6) i=0;                 // Index 0 für GinTonic
   }

  else if (x > 400) && (x < 601) { // 401-600
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Bestaetigung    ");
  }

  else if (x > 600) && (x < 801) { // 601-800
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Welcome         ");
  }

  else if (x > 800) && (x < 1023) { // 801-1022
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("801-1022        ");
  }
  delay(250);
}

// bei x gleich 1023 überspringt er die 'if..else if' Abfragen

[Hubert (Gast)]

Mandi schrieb:
> else if (x > 60) && (x < 201) {  // 61-200

Fehlen da nicht die äußeren Klammern? Ansonsten sieht es passend aus.

[SloJo (Gast)]

Hubert schrieb:
> Richtig, es muss wie folgt heißen: else if ((x<400) && (x>=200)) {
>
> Jedoch musst du mir noch erklären, warum es deiner Meinung nach nicht
> gebraucht wird.

Ich muss zwar nicht, aber ich tue es trotzdem: Er kommt sowieso nicht in 
den else-Zweig, wenn x < 200 ist. Also brauchst du es da nicht mehr 
abpruefen.

[Mandi (Gast)]

@Hubert: Klammern vergessen
Korrektur mit Klammern:

// mit Arduino-Monitor Ausgabe
// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

int i = 0;

char* myStrings[]={
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};

void setup() {
  Serial.begin(9600);
  for (i = 0; i < 6; i++){
    Serial.println(myStrings[i]);  // für Testzwecke
    delay(1);
  }

    // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
    // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("Cocktailwahl:   ");
  }

void loop() {

  int x=0;                         // evtl. x=1023
  x = analogRead (0);

  if ((x > 0) && (x < 61)) {         //  1-60
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Abbruch         ");
  }

  else if ((x > 60) && (x < 201)) {  // 61-200
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("GIN             ");
  }

  else if ((x > 200) && (x < 401)) { // 201-400
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    if (i==6) i=0;                 // Index 0 für GinTonic
   }

  else if ((x > 400) && (x < 601)) { // 401-600
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Bestaetigung    ");
  }

  else if ((x > 600) && (x < 801)) { // 601-800
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("Welcome         ");
  }

  else if ((x > 800) && (x < 1023)) { // 801-1022
    Serial.print("AnalogRead: ");Serial.println(x);  // für Testzwecke
    lcd.clear();
    lcd.print ("801-1022        ");
  }
  delay(250);
}

// bei x gleich 1023 oder 0 überspringt er die 'if..else if' Abfragen

[Katrina N. (Gast)]

Hey ja super vielen dank, probiere ich gleich mal aus :) Ihr seid echt 
klasse

[Katrina N. (Gast)]

Leider immer noch kein Erfolg......

[Karl (Gast)]

Mandi schrieb:
> delay(250);

Der Ansatz das Ganze mit delay zu lösen ist doch mist. So wie es jetzt 
ist, wird 4 mal pro Sekunde weitergeschaltet, dass ist für die meisten 
zu schnell. Und was Passiert eigentlich, wenn der Aufruf zu einem 
Zeitpunkt einer steigenden/fallenden Flanke erfolgt? dann wird aufeinmal 
Abbruch gemacht, obwohl Button Down gedrückt wurde!

Mach eine ordentliche Routine, wo die Tastenabfrage gemacht wird. Diese 
wird in einem festen Zyklus ausgeführt (ca. 50 bis 100 ms). In der 
Routine wird der ADC eingelesen und aus dem Ergebnis eine Zuordnung zu 
Button gemacht und mit dem Wert aus dem vorherigen Zyklus verglichen. 
Wenn die gleich sind, wird das Ergebnis weiterverarbeitet. Den Zyklus 
erzeugst du am besten mit einem Timer (keine Angst, da gibt es eine Lib 
für). Im Interrupt setzt du ein Flag. Das Flag wird in der Loop dann 
einfach über eine if-Abfrage abgefragt, und wenn es gesetzt wurde wird 
die Einleseroutine ausgeführt und anschließend das Flag zurückgesetzt.

[Karl (Gast)]

Katrina N. schrieb:
> Leider immer noch kein Erfolg......

Trenn erstmal deine Aufgaben. Kümmer dich erstmal darum, dass das 
einlesen der Taster funktioniert. Mach das in einer eigenen Funktion.
Dann mach eine Funktion die auf die Taster reagiert.
Teste das alles mit Ausgabe der wichtigsten Variablen über 
Serial.println.
Zum Schluss baust du deinen LCD Text mit ein und fertig.

[Katrina N. (Gast)]

Karl schrieb:
> Trenn erstmal deine Aufgaben. Kümmer dich erstmal darum, dass das
> einlesen der Taster funktioniert. Mach das in einer eigenen Funktion.
> Dann mach eine Funktion die auf die Taster reagiert.
> Teste das alles mit Ausgabe der wichtigsten Variablen über
> Serial.println.
> Zum Schluss baust du deinen LCD Text mit ein und fertig.

Vielen Dank Karl.

Meine Taster funktionieren ja nur nicht die Ausgabe wenn ich den Taster 
zwei mal drücke, das sich dann der Text ändert.

[djuri (Gast)]

Hey Katrina
Eine kleine Start Hilfe.
LG.

//
//  Beitrag "Display Arduino"
//
// ====================================================
// Test mit:
// Arduino uno R3
// LCD Keypad Shield 'ROBOT' [ lcd(8, 9, 4, 5, 6, 7); ]
// ====================================================
//
// Die Werte von meinem, [LCD Keypad Shield -ROBOT-] sind:
// --------------------------------------------------------------------
// Dazu R_Netz-       RIGHT,     UP,   DOWN,  LEFT, SELECT, NONE
// werk  neu:   ?!==> R1=3k, R2=330, R3=620, R4=1k,  R5=3k,
// Gerechnet:   U_AD0  =  0,  0.496,  1.203,  1.97,  3.113, (4.995)Volt
// --------------------------------------------------------------------
// Gemessene:             0,     98,    254,   406,    640, 
(1023)[10bit]
// +20  dazu:  ADC     = 20,    118,    274,   426,    660,  1022
// +50  dazu:  ADC     = 50,    148,    304,   456,    690,  1022
// ====================================================================
//
// include the library code:
#include <LiquidCrystal.h>

int test = 1;        //   = 0; // keine 'Serial Monitor' Ausgabe

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

int i = 0;             // === Global ===

char* myStrings[]={    // So uebersichtlich
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};  // nach ; // nur noch Kommentar

//----------------------
void setup() {
  Serial.begin(9600);
//=== Serial Monitor === ----------------------------------------
if (test) { Serial.println();      //
  for (int i = 0; i < 6; i++){     // i = === Lokal ===
    Serial.println(myStrings[i]);  // Serial Monitor -> 9600 baud
  } // ende test
//    delay(1);                    // nicht notwendig
    }
//---------------------------------------------------------------
// set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
//  lcd.clear();                    // nicht notwendig
// Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print(" W e l c o m e  ");
//lcd.print("1234567890123456");    // nur zur Orientierung

//  delay(1000);                    // nicht notwendig
} // ende setup()
//---------------------------------------------------------------
//
//===============================================================
void loop() {
  int x=0;                          // x = 0 bis 1022
  x = analogRead (0);

// [RIGHT]   ADC wert = 0
  if ((x >= 0) && (x < 50)) {       // x >= 0
if (test) { // ,,,,,,,,,,,,,,,,,,,,,,,,,,,,
    Serial.print("AnalogRead: |RIGHT| = ");
    Serial.println(x);
}
    lcd.setCursor(0,1);
    lcd.print ("RIGHT  =        ");
  //lcd.print ("1234567890123456");
    lcd.setCursor(9,1);
    lcd.print (x);
  } // ende |RIGHT|

// [UP]   ADC wert = 98
  else if ((x >= 50) && (x < 148)) {
    Serial.print("AnalogRead: |UP|    = ");
    Serial.println(x);
    lcd.setCursor(0,1);
    lcd.print ("UP     =        ");
  //lcd.print ("1234567890123456");
    lcd.setCursor(9,1);
    lcd.print (x);
  } // ende |UP|

// =========================================================
// Menue Angebot (myStrings
// ========================
// [DOWN]   ADC wert = 254
  else if ((x >= 148) && (x < 304)) {
    Serial.print("AnalogRead: |DOWN|  = ");
    Serial.println(x);
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;                                // Global
   } // ende |DOWN|
// =========================================================

// [LEFT]   ADC wert = 406
  else if ((x >= 304) && (x < 456)) {
    Serial.print("AnalogRead: |LEFT|  = ");
    Serial.println(x);
    lcd.setCursor(0,1);
    lcd.print ("LEFT   =        ");
  //lcd.print ("1234567890123456");
    lcd.setCursor(9,1);
    lcd.print (x);
  } // ende |LEFT|

// [SELECT]   ADC wert = 640
  else if ((x >= 456) && (x < 690)) {
    Serial.print("AnalogRead: |SELECT|= ");
    Serial.println(x);
    lcd.setCursor(0,1);
    lcd.print ("SELECT =        ");
  //lcd.print ("1234567890123456");
    lcd.setCursor(9,1);
    lcd.print (x);
  } // ende |SELECT|

// [NONE]   ADC wert = 1022
  else if ((x >= 690) && (x <= 1023)) {
    Serial.print("AnalogRead: |NONE|   = ");
    Serial.println(x);
    lcd.setCursor(0,1);              // es wird ueberschrieben
    lcd.print ("NONE   =        ");  // =
  //lcd.print ("1234567890123456");  // =
    lcd.setCursor(9,1);              // =
    lcd.print (x);                   // =

    lcd.setCursor(0,1);              // und weiter gets,
    lcd.print ("  Cocktailwahl  ");  // =
  //lcd.print ("1234567890123456");
  } // ende |NONE|

  if (i==6) i=0;        // Beim Cocktailwahl BLEIBEN!
// ==============       // sonst ist er in nirwana
// ===================================================
  delay(250);           // ersaetzen, ???
} // ende loop()

// =========================================================
// Die 'if' schleifen {Innereien} in Funktionen verbahnen,
// oder mit 'switch' und Funktion den 'delay()' loswerden.
// Cocktail Auswahl aufbauen,
// Auswaehlen, bestaetigen, zahlen,
// danken und weiter mit Welkome.
// =========================================================
// Achte auf die Gueltigkeit der Variablen.
// Zuruecksetzen nicht vergessen.
// ==============================================================
//

[Mark B. (markbrandis)]

djuri schrieb:
> // Die 'if' schleifen

So etwas gibt es nicht.

Was es gibt, sind bedingte Verzweigungen.

[Dieter F. (Gast)]

Da würde ich mich eher daran

http://www.dfrobot.com/wiki/index.php?title=Arduino_LCD_KeyPad_Shield_(SKU:_DFR0009)

halten und dort meine Logik einbauen.

Bei den Tasten, wo nichts passieren soll, belässt Du es in der 
CASE-Anweisung einfach bei einem BREAK.

Ich würde auch nicht jedesmal ein lcd.clear() hinpacken, das macht die 
Display-Ausgabe nur "unruhig". Einfach überschreiben (mit ausreichend 
Leerzeichen am Ende) tut es auch und verhindert das "Flackern".

[Mick (Gast)]

Schau dir das hier mal an: https://code.google.com/p/u8glib/wiki/tmenu

Abfragen der Taster könnte wie folgt aussehen. Dabei wird jeweils nur 
ein "Click" erkannt. Bevor eine weitere Taste erkannt wird, muss der 
gedrückte Schalter zuerst losgelassen werden. Die entsprechenden 
Analogwerde musst du halt noch anpassen.

/*-----( Declare Constants )-----*/
#define btnRIGHT  0
#define btnUP     1
#define btnDOWN   2
#define btnLEFT   3
#define btnSELECT 4
#define btnNONE   5

/*-----( Declare Variables )-----*/
int adc_key_prev  = 0;
int adc_key_current  = 0;

void loop(){
  adc_key_in = read_LCD_buttons();
  if(adc_key_prev != adc_key_current){
    /*
    dein code hier
    */
    adc_key_prev = adc_key_current;
  }
}

int read_LCD_buttons(){
  int adc_key_in = analogRead(0);      // read the value from the sensor 
  delay(5); //switch debounce delay. Increase this delay if incorrect switch selections are returned.
  int k = (analogRead(0) - adc_key_in); //gives the button a slight range to allow for a little contact resistance noise
  if (5 < abs(k)) return btnNONE;  // double checks the keypress. If the two readings are not equal +/-k value after debounce delay, it tries again.
  // my buttons when read are centered at these valies: 0, 144, 329, 504, 741
  // we add approx 50 to those values and check to see if we are close
  if (adc_key_in > 1000) return btnNONE; // We make this the 1st option for speed reasons since it will be the most likely result
  if (adc_key_in < 50)   return btnRIGHT;  
  if (adc_key_in < 195)  return btnUP; 
  if (adc_key_in < 380)  return btnDOWN; 
  if (adc_key_in < 555)  return btnLEFT; 
  if (adc_key_in < 790)  return btnSELECT;   
  return btnNONE;  // when all others fail, return this...
}

[Katrina N. (Gast)]

Hey danke an euch alle bin jetzt schon einen rießen Großen Schritt 
weitergekommen das mit Down geht bis auf das letzte up hängt sich noch 
auf. hier mein Aktuelles Skript:
// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);
char* buttons[]={"Abbruch         ","Bestaetigung    ","Welcome 
"};
char* myStrings[]={    // So uebersichtlich
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};
int i=0;
void setup() {
  // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
  // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("    WELCOME     ");
  lcd.setCursor(0,1);
  lcd.print("  Cocktailwahl  ");

}

void loop() {
  int x;

  x = analogRead (0);
  lcd.setCursor(0,0);
  if ((x >= 0) && (x < 50)) {
     lcd.setCursor(0,1);
    lcd.print (buttons[0]);
  }
  else if ((x >= 50) && (x < 148)) {
  lcd.print ("UP              ");
//    lcd.setCursor(0,0);
//    lcd.print(myStrings[i]);
//    i--;
//    lcd.setCursor(0,1);
//    lcd.print(myStrings[i]);
//    i--;
  }
  else if ((x >= 304) && (x < 456)) {
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;

   }

 else if ((x >= 456) && (x < 690)) {
   lcd.setCursor(0,1);
    lcd.print (buttons[1]);
  }
  else if ((x >= 148) && (x < 304)) {
  lcd.setCursor(0,1);
    lcd.print (buttons[2]);
  }
   if (i==6)
   {    i=0;
        lcd.setCursor(0,0);              // und weiter gets,
        lcd.print ("  Cocktailwahl  ");
   }
}

[Dieter F. (Gast)]

Katrina N. schrieb:
> hier mein Aktuelles Skript

O.K., dann bin ich hier raus.

Ich verstehe nicht, warum Du das recht gut strukturierte Beispiel aus

http://www.dfrobot.com/wiki/index.php/LCD_KeyPad_Shield_For_Arduino_SKU:_DFR0009

nicht nutzt - aber Du wirst schon Deine Gründe haben.

[Katrina N. (Gast)]

Ich war jetzt einfach nur froh, dass es geklappt hat @Dieter F.
Eine Frage wie kann ich es verhindern, das er bei längerem Drücken 
trotzdem weiterzählt? Wollen die Taster jetzt über Digitale Eingänge 
schalten, ist es dann weg?

[Dieter F. (Gast)]

Katrina N. schrieb:
> Eine Frage wie kann ich es verhindern, das er bei längerem Drücken
> trotzdem weiterzählt?

In diesem Fall würde ich ganz einfach nach jeder vollständigen Ausgabe 
ein Delay mit 500 - 1000 ms setzen, also z.B.

...
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;    
    delay(500);
...

Dann hat derjenige, der auf die Taste drückt, nach der vollständigen 
Ausgabe des neuen Display-Inhaltes (er sieht das Ergebnis des 
Tastendrucks) noch 500 ms Zeit, den Finger von der Taste zu nehmen. Die 
erforderliche Zeit musst Du halt ausprobieren.

[Katrina N. (Gast)]

Stimmt auf die einfache Idee bin ich gar nicht gekommen :D

[Mandi (Gast)]

für die UP-Taste:

...
  else if ((x >= 50) && (x < 148)) {
  lcd.print ("UP              ");
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i--;
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i--;
    if (i==0) i=6;     // nach Menüpunkt 1 wieder Menüpunkt 6
  }
...

[Mandi (Gast)]

Ganzes Beispiel:
(nicht getestet, keine Arduino-IDE auf Internet-PC)

// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

char* buttons[]={
 "Abbruch         ",
 "Bestaetigung    ",
 "Welcome         "};

char* myStrings[]={    // So uebersichtlich
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};

int i=0;

void setup() {
    // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
    // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("    WELCOME     ");
  lcd.setCursor(0,1);
  lcd.print("  Cocktailwahl  ");
}

void loop() {
  int x;
  x = analogRead (0);

  if ((x >= 0) && (x < 50)) {          // Abbruch
    lcd.setCursor(0,1);
    lcd.print (buttons[0]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 50) && (x < 148)) {   // UP
    if (i<1) i=6;
    i--;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i--;
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 304) && (x < 456)) {  // Down
    if (i>5) i=0;
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    lcd.setCursor(0,1);
    lcd.print(myStrings[i]);
    i++;
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 456) && (x < 690)) {   // Bestaetigung
    lcd.setCursor(0,1);
    lcd.print (buttons[1]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 148) && (x < 304)) {  // Welcome
    lcd.setCursor(0,1);
    lcd.print (buttons[2]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }

// wird gebraucht?
//    {    i=0;
//         lcd.setCursor(0,0);            // und weiter gehts,
//         lcd.print ("  Cocktailwahl  ");
//    }
} // Ende Loop

[Mandi (Gast)]

kleine Korrektur!
(vom letzten Beispiel)

// in UP:
    if (i<1) i=6;
ändern zu
    if (i<2) i=6;

// in Down:
    if (i>5) i=0;
ändern zu
    if (i>4) i=0;

[Eric B. (beric)]

Mal 2 dumme Fragen:
1. Es stehen im Display immer 2 Getränke zum Auswahl, welche von Beiden 
krieg ich serviert wenn ich auf "Bestätigung" drücke?

2. Was passiert wenn mal eine ungerade Anzahl an Getränken zur Auswahl 
stehen soll (also z.B. 5 oder 7, statt 6)?

[Mandi (Gast)]

@Eric B.
 Ausgabe nur eines Getränks in Zeile 1, mit Up und Down bedienbar.
Sollte nur als erweiterbares Beispiel für Katrina N. dienen.

// include the library code:
#include <LiquidCrystal.h>

// initialize the library with the numbers of the interface pins
LiquidCrystal lcd(8, 9, 4, 5, 6, 7);

char* buttons[]={
 "Abbruch         ",
 "Bestaetigung    ",
 "Welcome         "};

char* myStrings[]={    // So uebersichtlich
 "GinTonic        ",
 "JackyCola       ",
 "WodkaBull       ",
 "WodkaLemon      ",
 "CaptainCola     ",
 "Wasser          "};

int Max_Anzahl_Cocktail=6;  // Anzahl Cocktails in myStrings[]
int Taste=0;
int i=0;

void setup() {
    // set up the LCD's number of columns and rows:
  lcd.begin(16, 2);
    // Print a message to the LCD.
  lcd.setCursor(0,0);
  lcd.print("    WELCOME     ");
  lcd.setCursor(0,1);
  lcd.print("  Cocktailwahl  ");
}

void loop() {
  int x;
  x = analogRead (0);

  if ((x >= 0) && (x < 50)) {          // Abbruch
    lcd.setCursor(0,1);
    lcd.print (buttons[0]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 50) && (x < 148)) {   // UP
    if (Taste==0) i--;
    if (i<1) i=Max_Anzahl_Cocktail;
    i--;
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    Taste=1;
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 304) && (x < 456)) {  // Down
    if (Taste==1) i++;
    if (i>Max_Anzahl_Cocktail-1) i=0;
    lcd.setCursor(0,0);
    lcd.print(myStrings[i]);
    i++;
    Taste=0;
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 456) && (x < 690)) {   // Bestaetigung
    lcd.setCursor(0,1);
    lcd.print (buttons[1]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }
  else if ((x >= 148) && (x < 304)) {  // Welcome
    lcd.setCursor(0,1);
    lcd.print (buttons[2]);
    delay(500);   // Wartezeit für nächsten Tastendruck
  }

// wird gebraucht?
//    {    i=0;
//         lcd.setCursor(0,0);            // und weiter gehts,
//         lcd.print ("  Cocktailwahl  ");
//    }
} // Ende Loop

[Karl (Gast)]

Dieter F. schrieb:
> In diesem Fall würde ich ganz einfach nach jeder vollständigen Ausgabe
> ein Delay mit 500 - 1000 ms setzen, also z.B.

Genau, am besten gleich 1000 ms, dann dauert es bei 10 Getränken min 10 
s bis ich habe, was ich will. Schon mal ein PC/Smartphone benutzt, wo 
man nach jeder Aktion 0,5 bis 1 s warten muss, bis etwas passiert? Das 
würde jeder Nutzer als unbrauchbar wegfeuern!

[Dieter F. (Gast)]

Karl schrieb:
> Genau, am besten gleich 1000 ms, dann dauert es bei 10 Getränken min 10
> s bis ich habe, was ich will. Schon mal ein PC/Smartphone benutzt, wo
> man nach jeder Aktion 0,5 bis 1 s warten muss, bis etwas passiert? Das
> würde jeder Nutzer als unbrauchbar wegfeuern!

Bei 500 ms dauert es 5 Sekunden einmal durchzuscrollen. Außerdem 
solltest Du wenigstens vollständig zitieren:

Dieter F. schrieb:
> Dann hat derjenige, der auf die Taste drückt, nach der vollständigen
> Ausgabe des neuen Display-Inhaltes (er sieht das Ergebnis des
> Tastendrucks) noch 500 ms Zeit, den Finger von der Taste zu nehmen. Die
> erforderliche Zeit musst Du halt ausprobieren.

Aber Du kannst ja gerne eine bessere Lösung vorschlagen :-)

[avr (Gast)]

Dieter F. schrieb:
> Aber Du kannst ja gerne eine bessere Lösung vorschlagen :-)

Alle Delays raus und dann eine anständige Tastenentprellung rein. Alles 
andere ist Pfusch.

[Dieter F. (Gast)]

avr schrieb:
> Alle Delays raus und dann eine anständige Tastenentprellung rein. Alles
> andere ist Pfusch.

Ah ja, dann zeigt doch mal ein Beispiel der Entprellung für das "LCD 
Keypad Shield", welches die Tastendrücke anhand des Einflusses des 
Widerstandswerts (liegt eine "Dekade" dahinter) auswertet. Ich bin 
gespannt ...

[Eric B. (beric)]

Dieter F. schrieb:
> Ah ja, dann zeigt doch mal ein Beispiel der Entprellung für das "LCD
> Keypad Shield", welches die Tastendrücke anhand des Einflusses des
> Widerstandswerts (liegt eine "Dekade" dahinter) auswertet. Ich bin
> gespannt ...

In Pseudo code:

var tasten_status = NO_KEY
    entpreller = 0
    adcwert = 0
    alter_adcwert = 0

const ENTPRELL_LIMIT = 8 // Experimentell ermitteln!

loop:
  adcwert = adc lesen
  adcstatus = ermittele_tasten_status(adcwert)

  if adcstatus == alter_adcstatus:
    entpreller ++
    if entpreller == ENTPRELL_LIMIT:
      tastenstatus = adcstatus
      entpreller = 0
    end if
  else
    entpreller = 0
  end if
  alter_adc_status = adc_status

  tue_was_mit(tasten_status)

[avr (Gast)]

Dieter F. schrieb:
> Ah ja, dann zeigt doch mal ein Beispiel der Entprellung für das "LCD
> Keypad Shield", welches die Tastendrücke anhand des Einflusses des
> Widerstandswerts (liegt eine "Dekade" dahinter) auswertet. Ich bin
> gespannt ...

Was man entprellt ist doch völlig egal. Man hat hier eine Information, 
die angibt welcher Taster gedrückt ist. Diese Information kann aber 
"prellen". Sobald man eindeutig, den Tastendruck und das Loslassen 
erkennt, kann man mit diesen Event das Menü aufbauen.

Pseusocode habt ihr ja schon wieder bekommen. Ich finde bei solchen 
Sachen wie Entprellung, zu dem es auch sehr viel im Internet schon zu 
lesen gibt, hätte man sich auch selbst mal Gedanken machen können.

[djuri (Gast)]

Hallo, Katrina
Die Lösung ist nicht ganz, in deinem sine, hoffe aber das Dir weiter 
hilft.
Es geht wunderbar, auch OHNE:  delay()   und   Tastenentprellung!
Finde heraus:  Welcher  Taster?  Für was gut ist?

Gruß.