I2C Display flackert

[H.M Murdock (Gast)]

Hallo zusammen,

ich bin – wie viele vll. hier – neu in Sachen Arduino Programmierung. 
Habe aber in div. anderen programmiertechnischen Sachen Erfahrung (SPS, 
VBA, kleine C++ Kenntnisse).
Bitte verzeiht mir also meine evtl. dumme Frage.
Mein Sketch sieht folgendermaßen aus und soll(te) eingentlich 
folgendermaßen ablaufen:
1.  Sobald das Raspi bereit ist, kommt ein Eingang auf dem Arduino an
2.  Ein Schrittmotor wird freigegeben und startet eine Referenzfaht 
solange bis er auf den Referenzschalter fährt
3.  Dann wird die „Fahrfreigabe“ weggenommen und über den 
Ultraschall-sensor soll geprüft werden ob ein Glas unter/vor dem Sensor 
steht.
Mein Problem ist, dass das Display zwischen dem Schritt 2 und 3 
permanent flackert  so als könnte es sich nicht entscheiden was 
angezeigt werden soll.
Der Sketch zeigt aktuell die letzte Variante. In den If-Bedingungen habe 
ich die „Distance“ schon selbst aufgenommen, hat aber auch nicht 
fuktioniert. Nun probierte ich es mit einer Art Merker, aber leider 
wieder ohne Erfolg.

Für Eure Unterstützung wäre ich dankbar – und ja wie ihr den 
Programmcode entnehmen könnt geht es um eine Cocktailmaschine 

/* Schrittkette
  // Variable "step" wird als Zustandsmerker verwendet
  // Zustände: 0 = Raspi noch nicht bereit (Eingang 3 auf µC noch nicht high)
  //           1 = Raspi bereit --> Referenzfahrt Motor
  //           2 = Raspi bereit, refpoint Maschine --> Warte auf Glas
             3 = Glas unter Dosiereinheit + Bestellung --> Bestellung wird zubereitet
*/


//##################################################################

#define trigger 9
#define echo 8
#define LED 2
#define pi_ready 3
#define teller_enable 12
#define clk_teller 5
#define refpoint_teller 7

//#################################################################
//Variablen
int step = 0;                           //Statusmerker
int ready = 0;                          //Merker für Automatikmodus
int  glada = 0;                          //Glas Da Merker
float time = 0, distance = 0;          //Variable zur Abstands-Messung
int pwm = 0;                           //PWM-Wert (0....255)

//###############################################################

//Bibliotheken einbinden
#include <LiquidCrystal_I2C.h>


LiquidCrystal_I2C lcd(0x27, 20, 4);


//#################################################################

void setup()
{
  lcd.init();
  Serial.begin(9600);

  pinMode(trigger, OUTPUT);
  pinMode(echo, INPUT);
  pinMode(LED, OUTPUT);
  pinMode(pi_ready, INPUT);
  pinMode(teller_enable, OUTPUT);
  pinMode(clk_teller, OUTPUT);
  pinMode(refpoint_teller, INPUT);

}
//####################################################################


void loop()
{
  measure_distance();

  if (digitalRead(pi_ready) == HIGH && (digitalRead(refpoint_teller) == LOW))        //Eingang Raspi bereit
  {
    step = 1;
    lcd_update();
    digitalWrite(teller_enable, HIGH);                                           //Antrieb für Flaschenteller freigeben
  }

  else {
    digitalWrite(teller_enable, LOW);
    step = 0;
    lcd_update();
  }

  if (digitalRead(pi_ready) == HIGH && (digitalRead(refpoint_teller) == HIGH ))      //Referenz Flaschenteller erreicht
  { digitalWrite(teller_enable, LOW);
    step = 2;
    ready = 1;
    lcd_update();
  }

  if (digitalRead(pi_ready) == HIGH && (digitalRead(refpoint_teller) == HIGH && (ready == 1))) {
    step = 3;
    lcd_update();
  }

  if (distance < 10)
  {
    digitalWrite(LED, HIGH);                //Eingang Glas da
  }


  Serial.print("Entfernung:  ");
  Serial.print(distance);
  Serial.print("     Schritt:  ");
  Serial.println(step);

}



//#############################################################

void measure_distance()                   //Ultraschall-Abstands-Messung
{
  lcd_update();
  digitalWrite(trigger, LOW);
  delayMicroseconds(2);
  digitalWrite(trigger, HIGH);
  delayMicroseconds(10);
  digitalWrite(trigger, LOW);
  delayMicroseconds(2);
  time = pulseIn(echo, HIGH);

  distance = time * 340 / 20000;
}

//############################################################

void lcd_update()                       //div Statis im LCD anzeigen lassen
{
  switch (step) {
    case 0:                             //Raspi noch nicht bereit
      lcd.backlight();
      lcd.setCursor(0, 0);
      lcd.print(" >> Warte auf Raspi");
      lcd.setCursor(0, 1);
      lcd.print("    Raspi bereit");
      lcd.setCursor(0, 2);
      lcd.print("    Referenzfahrt  ");
      lcd.setCursor(0, 3);
      lcd.print("    Warte auf Glas  ");
      break;

    case 1:                             //Raspi bereit, Referenzfahrt Flaschenteller
      lcd.backlight();
      lcd.setCursor(0, 0);
      lcd.print("    Warte auf Raspi");
      lcd.setCursor(0, 1);
      lcd.print(" >> Raspi bereit");
      lcd.setCursor(0, 2);
      lcd.print(" >> Referenzfahrt  ");
      lcd.setCursor(0, 3);
      lcd.print("    Warte auf Glas");
      break;

    case 2:                               //Referenzpunkt erreicht, warte auf Glas
      lcd.backlight();
      lcd.setCursor(0, 0);
      lcd.print("    Warte auf Raspi");
      lcd.setCursor(0, 1);
      lcd.print(" >> Raspi bereit");
      lcd.setCursor(0, 2);
      lcd.print(" >> Referenzfahrt  ");
      lcd.setCursor(0, 3);
      lcd.print("    Warte auf Glas  ");
      break;

    case 3:                               //Glas da, Bestellung kann entgegengenommen werden
      lcd.backlight();
      lcd.setCursor(0, 0);
      lcd.print("    Warte auf Raspi   ");
      lcd.setCursor(0, 1);
      lcd.print("    Raspi bereit     ");
      lcd.setCursor(0, 2);
      lcd.print("    Warte auf Glas     ");
      lcd.setCursor(0, 1);
      lcd.print(" >> Glas da        ");
      break;

    default:
      lcd.backlight();
      lcd.setCursor(0, 0);
      lcd.print(" >>Initialisierung");
      break;

  }
}

[Martin O. (ossi-2)]

Es sieht für mich so aus, als würdest Du das LC-Display immer 
aktualisieren.
Das kann Flimmern verursachen. Also LC-Display immer nur 1x 
aktualisieren wenn sich was geändert hat.

[H.M Murdock (Gast)]

Hallo Martin,

vielen Dank für die schnelle Antwort.

Aus diesem Grund habe ich die Funktion lcd_update geschrieben um nur 
dann zu aktualisieren wenn sich in den IF-Bedingungen was geändert hat.

oder wie kann ich das sonst nur 1x aktualisieren?

Danke schon mal im Voraus.

Gruß
Murdock

[Martin O. (ossi-2)]

Ich glaube Du rufst lcd_update mehrfach auf, selbst wenn sich nichts 
geändert hat.

Du könntest Dir in lcd_update merken, mit welchem Wert von step der 
letzte Aufruf erfolgte, und die Aktualisierung nur machen, wenn ein 
neuer Wert von step kommt.

[H.M Murdock (Gast)]

Hallo Martin,

habe jetzt den Fehler gefunden :-) manchmal wird man richtig blind wenn 
man solange vor dem Bildschirm sitzt..

der Fehler lag bzw. begann in der 2.IF-Bedingung... Wenn diese erfüllt 
ist, wird die Variable "ready" auf 1 gesetzt und somit ist die 2. und 3. 
if-Bedingung "wahr" somit ist es logisch, dass er nicht weiß was er 
anzeigen soll/darf und muss :-)

vielen Dank für die schnelle Hilfe...

Da muss ich mir noch was anderes einfallen lassen :-)

Gruß und schönen Abend noch

Murdock