warning: comparison between pointer and integer

[Christoph E. (ello)]

Hallo,

Durch den Code unten soll eine Tasten-Entprellung stattfinden.
Es soll eine positive Flanke abgefragt werden und dann eine Aktion 
auslösen.
Den restlichen Teil des Programmes hab ich mal weggelassen.
Bei dieser Abfrage:

if (taster1 == 1) {

Kommt als Compilerwarnung: main.c:85: warning: comparison between 
pointer and integer.
Es funktioniert auch nicht auf dem µC.
Habt ihr eine Idee?
Hier ist mal der betroffene Programmcode:

#define TASTERPORT PIND   
#define TASTER1    PIND2   
#define TASTER2    PIND3  


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

void init(void)

{

  TCCR1B |= (1 << CS12) ; TIMSK |= (1 << OCIE1A);
  TCCR0 |= (1 << CS00) | (1 << CS02) ; TIMSK |= (1 << TOIE0);
  
// PORTD = 0b00000000; // Bei Ausgängen: Ausgangsbyte fest setzen. Bei Eingängen: Pullups aktivieren
  DDRD = 0b01100000; // Datenrichtungsregister: Eingang = 0, Ausgang =1 (Bit 0-4 und 7 = Eingang, Bit 5 und 6 = Ausgang) Led hängt an PD5 und PD6 gegen GND
  Handy_Init();
  
}

//Hauptprogramm
int main (void)
{
Handy_Init();
init();



//############################################################################
while (1)
{

char taster1(void)
{
  static char PinState;
  char NewState = TASTERPORT & ( 1 << TASTER1 );

  if( NewState == PinState )
    return 0;

  NewState = PinState;
  return NewState != 0;
}

char taster2(void)
{
  static char PinState;
   char NewState = TASTERPORT & ( 1 << TASTER2 );

  if( NewState == PinState )
    return 0;

  NewState = PinState;
  return NewState != 0;
}


if (taster1 == 1) {      // Pin 2 (Taster1) auf 1 abfragen
          PORTD |= (1 << PIN5);   // gelbe LED ein für SMS-Versand eingeleidet  
      GSM_Init();
            }    
      else {  
        PORTD &= ~(1 << PIN5);     // gelbe LED aus für SMS-Versand eingeleidet
      }
if (taster2 == 1) {      // Pin 3 (Taster2) auf 1 abfragen
      PORTD |= (1 << PIN6);     // gelbe LED ein für SMS-Versand eingeleidet
      Send_SMS(); 
      }
      else {
        PORTD &= ~(1 << PIN6);     // gelbe LED aus für SMS-Versand eingeleidet
      }      
}
return (1);
}

Schöne Grüße und Frohe Weihnachten

Ello

[Stefan B. (stefan)]

Ändere

if (taster1 == 1) {      // Pin 2 (Taster1) auf 1 abfragen

in

if (taster1() == 1) {      // Pin 2 (Taster1) auf 1 abfragen

Dein taster1 (und taster2) ist eine Funktion ;-)

[Lötlackl (Gast)]

Hi,

wenn taster1 & taster2 Funktionen sind, die Klammern dahinter nicht 
vergessen.

if (taster1() == 1)

mfg Lötlackl

[Johannes M. (johnny-m)]

BTW:
"Eingebettete" Funktionsdefinitionen sind zwar afaik zumindest unter C99 
möglich, ist aber grottenschlechter Programmierstil... Und dazu noch in 
einer while-Schleife.

Tu Dir selbst den Gefallen und mach das vernünftig und definiere die 
Funktionen außerhalb von main().

[Rolf Magnus (Gast)]

> "Eingebettete" Funktionsdefinitionen sind zwar afaik zumindest unter
> C99 möglich,

Nein. Sie sind eine compilerspezifische Erweiterung.

[Christoph E. (ello)]

Vielen Dank! Hat sofort geklappt!

MfG Ello

[Johannes M. (johnny-m)]

Rolf Magnus wrote:
>> "Eingebettete" Funktionsdefinitionen sind zwar afaik zumindest unter
>> C99 möglich,
>
> Nein. Sie sind eine compilerspezifische Erweiterung.
Aha. Danke, wieder was gelernt.