Frage zu Entprellungsprogramm

[Frank (Gast)]

Hallo,
bin gerade dabei ein Programm für eine einfache Tasterentprellung zu 
schreiben. Leider bekomme ich beim Compilieren bisher immer noch einen 
Übersetzungsfehler. Hier ist mein bisheriges Programm:

#include <avr/io.h>
#include <util/delay.h>  
#include "lcd-routines.h"

char buffer[20];


void delay_ms(uint16_t ms)
{
  while(ms>0)
  {
  _delay_ms(1);
  ms--;
  }
}


bool entprellung(uint8_t port)
{
  
  
  if(!(PIND & (1<<port)))    //wenn Taster gedrückt, also PD0 auf low
  {  
    delay_ms(10);    //10ms warten, danach abfragen ob Pin PD0 immer noch auf low ist
    
    if(!(PIND & (1<<port)))
    {  lcd_setcursor(0,1);
        lcd_string("buffer");          
      return 1;
      
    }
    return 0;
  }
}



int main(void)
{

  DDRD = 0x00;  //alle Ports als Eingang
  DDRB = 0xFF;  //alle Ports als Ausgang
  PORTB = 0xFF;  //alle LEDs aus

  lcd_init();
  lcd_setcursor(0,1);
  lcd_string("Hallo");




  while(1)
  {
    if(entprellung(PD0))
    {
      PORTB &= ~(1<<PB7);  //LED7 an, wenn taster an  PD0 gedrückt
    }
    
    
      
  }



}

Der Fehler muss auf jeden Fall an meiner Funktion entprellung liegen. 
Ich möchte, dass diese bei gedrücktem Taster ein True und bei nicht 
gedrückem ein False zurückgibt und somit bestimmt ob die LED7 an PD0 
eingeschaltet wird.

Wer kann mir sagen, was ich codetechnisch ändern muss, damit das 
Programm fehlerfrei übersetzt wird?

Gruß

[Herr Biber (Gast)]

Ich versteh dein Probelm nicht, normalerweise teilt dir ein Compiler 
doch mit in welcher Zeile der Fehler liegt...

[Frank (Gast)]

hier ist die Fehlermeldung:

../Tasterentprellung.c:18: error: expected '=', ',', ';', 'asm' or 
'__attribute__' before 'entprellung'


Leider wird mir die Stelle im Programm beim anklicken der Fehlermeldung 
grad nicht angezeigt. Weiß auch nicht warum das des AVR Studio da nicht 
macht. Normal geht des nämlich schon.

[Krapao (Gast)]

> Leider bekomme ich beim Compilieren bisher immer noch einen
> Übersetzungsfehler.

Welchen? Dein Quellcode bietet mehrere Möglichkeiten.

[STK500-Besitzer (Gast)]

Frank schrieb:
> Leider bekomme ich beim Compilieren bisher immer noch einen
> Übersetzungsfehler

Und was meldet der Compiler genau?

[Booool (Gast)]

C - kein bool Datentyp ( standardmäßig )
C++ - hat bool

Das sollte alles erklären.

[Thomas E. (thomase)]

Frank schrieb:
> Der Fehler muss auf jeden Fall an meiner Funktion entprellung liegen.
Nein. Deine Fehlermeldung sagt dir doch, wo der Fehler liegt:

> ../Tasterentprellung.c:18: error: expected '=', ',', ';', 'asm' or
> '__attribute__' before 'entprellung'
>>before 'entprellung'
Da fehlt VORHER was.

#include <avr/io.h>
#include <util/delay.h>
#include "lcd-routines.h"

die io.h und die delay.h gehören zur libc, die lcd-routines.h und die 
vermutliche lcd-routines.c sind von dir.
Wahrscheinlich fehlt irgendwo ein Semikolon.
Entscheide selber, wo du anfängst zu suchen.

mfg.

[Frank (Gast)]

Booool schrieb:
> C - kein bool Datentyp ( standardmäßig )
> C++ - hat bool

Was gibt es dann für ne alternative Möglichkeit das Ganze in C zu 
programmieren?

[Thomas E. (thomase)]

Frank schrieb:
> Booool schrieb:
>> C - kein bool Datentyp ( standardmäßig )
>> C++ - hat bool
>
> Was gibt es dann für ne alternative Möglichkeit das Ganze in C zu
> programmieren?

Keine Ahnung, was der will. Wahrscheinlich hat er letzte Nacht zuviel 
getrunken.

mfg.

[Frank (Gast)]

Thomas Eckmann schrieb:
> die io.h und die delay.h gehören zur libc, die lcd-routines.h und die
> vermutliche lcd-routines.c sind von dir.
> Wahrscheinlich fehlt irgendwo ein Semikolon.
> Entscheide selber, wo du anfängst zu suchen

Also einen Fehler in den LCD-Unterprogrammen kann ich ausschließen. Die 
wurden in der Vergangenheit schon ausgiebig getestet und verwendet. 
Falls also wirklich ein Semikolon fehlen sollte, kann es nur im 
dargestellten Code sein.

Gruß

[Peter II (Gast)]

deine funktion delay_ms ist auch sehr ungünstig. jeder schlaue compiler 
wird sie einfach ignorieren.

[Thomas E. (thomase)]

Frank schrieb:
> Also einen Fehler in den LCD-Unterprogrammen kann ich ausschließen.
Das sagen alle. Aber dann würde es auch laufen.

Frank schrieb:
> Falls also wirklich ein Semikolon fehlen sollte, kann es nur im
> dargestellten Code sein.
Da ist kein Fehler drin.

Peter II schrieb:
> jeder schlaue compiler wird sie einfach ignorieren
Warum sollte er das tun?

mfg.

[Krapao (Gast)]

>> Frank:
> Was gibt es dann für ne alternative Möglichkeit das Ganze in C zu
> programmieren?

Einen in C bekannten Datentyp nehmen.

Also statt
bool entprellung(uint8_t port)

z.B.
#include <stdint.h> // für uint8_t
...
uint8_t entprellung(uint8_t port)

>>  Peter II:
> deine funktion delay_ms ist auch sehr ungünstig. jeder schlaue compiler
> wird sie einfach ignorieren.

IMHO stimmt das nicht. delay_ms hat als Seiteneffekt einen Aufruf 
Funktion _delay_ms, die ihrerseite volatile gekennzeichnete Anweisungen 
enthält. Der Aufruf von delay_ms darf nicht "wegoptimiert" werden.

[Frank (Gast)]

Peter II schrieb:
> deine funktion delay_ms ist auch sehr ungünstig. jeder schlaue compiler
> wird sie einfach ignorieren.

Ich habs gerade mit meinem LCD getestet. Die Funktion delay_ms() wird 
ausgeführt.


Jetzt hab ich das Programm leicht abgeändert und jetzt funktioniert es 
wie gewünscht:

#include <avr/io.h>
#include <util/delay.h>  
#include "lcd-routines.h"

char buffer[20];


void delay_ms(uint16_t ms)
{
  while(ms>0)
  {lcd_setcursor(0,1);
    lcd_string("Zeitverzoegerung");
  _delay_ms(1);
  ms--;
  }
}


inline uint8_t entprellung(uint8_t port)
{
  
  
  if(!(PIND & (1<<port)))    //wenn Taster gedrückt, also PD0 auf low
  {  
    delay_ms(10);    //10ms warten, danach abfragen ob Pin PD0 immer noch auf low ist
    
    if(!(PIND & (1<<port)))
    {            
      return 1;
      
    }
    return 0;
  }
}



int main(void)
{

  DDRD = 0x00;  //alle Ports als Eingang
  DDRB = 0xFF;  //alle Ports als Ausgang
  PORTB = 0xFF;  //alle Pins auf high => alle LEDs aus

  lcd_init();
  lcd_setcursor(0,1);
  lcd_string("Hallo");




  while(1)
  {
    if(entprellung(PD0))
    {
      PORTB &= ~(1<<PB7);  //LED7 an, wenn taster an PD0 gedrückt
    }
    
    
      
  }



}

das "inline uint8_t" vor der Funktion entprellung hab ich aus einem 
Beispielprogramm von dieser Seite "geklaut":

http://www.mikrocontroller.net/articles/Entprellung

Jetzt lässt sich das Programm fehlerfrei übersetzen und funktioniert 
auch so wie ich will.

Jetzt ist nur mein Problem, dass ich die Bedeutung von dem "inline 
uint8_t" in Verbindung mit der Funktion entprellung nicht verstehe bzw 
kenne. Es muss sich wohl auf den Rückgabewert der Funktion beziehen

Kann mir vielleicht jemand die Bedeutung in diesem Zusammenhang 
erklären?

Gruß

[Frank (Gast)]

Krapao schrieb:
> z.B.
> #include <stdint.h> // für uint8_t
> ...
> uint8_t entprellung(uint8_t port)

Hab ich jetzt im Nachhinein auch ausprobiert. Lässt sich zwar fehlerfrei 
compilieren. Das Programm funktioniert aber dann nicht mehr wie 
gewünscht, da die LED dann dauerhaft leuchtet, auch dann schon wenn ich 
den Taster noch gar nicht gedrückt habe.

Gruß

[Thomas E. (thomase)]

Krapao schrieb:
> bool entprellung(uint8_t port)
Hast recht. Das ist der Fehler.
Nehme alles zurück.
Booool (Gast) ist nicht betrunken.

#include <stdbool.h> und dann geht's.

mfg.

[Thomas E. (thomase)]

Frank schrieb:
> Das Programm funktioniert aber dann nicht mehr wie gewünscht,

Deine Funktion ist unvollständig.
Sie handelt nur den Fall ab, wenn die Taste gedrückt wurde. Was macht 
sie, wenn die Taste nicht gedrückt wurde?

bool entprellung(uint8_t port)
{
  if(!(PIND & (1<<port)))  {
    delay_ms(10);

    if(!(PIND & (1<<port)))
    {  lcd_setcursor(0,1);
        lcd_string("buffer");
      return 1;

    }
    return 0;
  }
return 0;  //Damit sollte es gehen.
}

[Frank (Gast)]

Thomas Eckmann schrieb:
> Deine Funktion ist unvollständig.
> Sie handelt nur den Fall ab, wenn die Taste gedrückt wurde. Was macht
> sie, wenn die Taste nicht gedrückt wurde?
>
> bool entprellung(uint8_t port)
> {
>   if(!(PIND & (1<<port)))  {
>     delay_ms(10);
>
>     if(!(PIND & (1<<port)))
>     {  lcd_setcursor(0,1);
>         lcd_string("buffer");
>       return 1;
>
>     }
>     return 0;
>   }
> return 0;  //Damit sollte es gehen.
> }

Stimmt. Jetzt hab ich das Programm mal folgendermaßen abgeändert und 
jetzt funktioniert alles wie gewünscht:

#include <avr/io.h>
#include <util/delay.h>  
#include "lcd-routines.h"
#include <stdbool.h>

char buffer[20];


void delay_ms(uint16_t ms)
{
  while(ms>0)
  {lcd_setcursor(0,1);
    lcd_string("Zeitverzoegerung");
  _delay_ms(1);
  ms--;
  }
}


bool entprellung(uint8_t port)
{
  
  
  if(!(PIND & (1<<port)))    //wenn Taster gedrückt, also PD0 auf low
  {  
    delay_ms(10);    //10ms warten, danach abfragen ob Pin PD0 immer noch auf low ist
    
    if(!(PIND & (1<<port)))
    {            
      return 1;
      
    }
    return 0;
  }
  return 0;
}



int main(void)
{

  DDRD = 0x00;  //alle Ports als Eingang
  DDRB = 0xFF;  //alle Ports als Ausgang
  PORTB = 0xFF;  //alle Pins auf high => alle LEDs aus

  lcd_init();
  lcd_setcursor(0,1);
  lcd_string("Hallo");




  while(1)
  {
    if(entprellung(PD0))
    {
      PORTB &= ~(1<<PB7);  //LED7 an, wenn taster an PD0 gedrückt
    }
    
    
      
  }



}

Interessanterweise funktioniert das Programm nicht richtig, wenn ich das 
zweite return 0 in der Funktion entprellung weglasse. Dann leuchtet 
nämlich die LED auch dann schon, wenn ich noch gar nicht den Taster 
gedrückt habe.
Ich kann mir dieses Phänomen nur so erklären, dass die Funktion 
entprellung dann auch bei nicht gedrücktem Taster ein True-Ergebnis 
zurückmeldet. Kann diese Einschätzung jemand bestätigen?

Gruß

[Yalu X. (yalu) (Moderator)]

Frank schrieb:
> Interessanterweise funktioniert das Programm nicht richtig, wenn ich das
> zweite return 0 in der Funktion entprellung weglasse.

Weil dann der Rückgabewert bei nicht gedrückter Taste undefiniert ist.
Schaltest du mit -Wall die Ausgabe von Compiler-Warnungen ein, wird dir
dies mit der Meldung

warning: control reaches end of non-void function

mitgeteilt.

Übrigens hätte bei deinem ursprünglichen Problem ein etwas aktuellerer
GCC (4.6+) folgende Fehlermeldung ausgegeben, wodurch sich deine erste
Frage wahrscheinlich erübrigt hätte:

../Tasterentprellung.c:18:1: error: unknown type name 'bool'

Noch etwas: Wenn eine Funktion vom Typ bool ist, sollte sie konsequen-
terweise nicht 0 oder 1, sondern false oder true zurückgeben. Diese
Konstanten sind ebenfalls in stdbool.h definiert.

[Purzel H. (hacky)]

Das Programm ist leider kompletter Muell.

1) eine Delay funktion ist der Beginn allem Uebels.
   Denn sie wartet nicht nur, sondern macht das
   Programm auch total unuebersichtlich. Das ist
   die sogenannte Procedurale Programmierung.
   Irgendwann weiss man nicht mehr genau wo das
   Programm denn steht. Und dann wird es schwierig.
2) Man sollte sich angewoehnen in main genau an
   einer Stelle zu warten, und den Rest mit einer
   Zustandsmaschine zu machen. Das oben beschriebene
   Problem ist hier viel einfacher, denn man weiss
   wo im code Das Programm wartet, man weiss nur
   den State nicht, un den kann man dann einfach
   mitteilen.
3) arbeite mit einem Timer, der einen Tick vorgibt.
   zB 10ms, oder aehnlich. Der Timer setzt ein Flag,
   und auf das reagiert man dann im Main.