Return Wert erst wenn if-Anweisung TRUE

[Alex (Gast)]

Hallo,

wie kann ich den Rückgabewert so einfügen, dass er nur zurückgegeben 
wird wenn die If-Anweisung:

  // Wenn Taste länger gedrückt wird
  if (pInput->pressedCycles >50)
  {

TRUE ist?

Sollte die If-Anweisung FALSE sein, soll der Rückgabewert nicht 
zurückgegeben werden!

So bekomme ich die Warnung:
main.h:674: warning: control reaches end of non-void function

uint8_t TasterDimmenAufAb (struct inputDesc* pInput)
{


  //Steigende Flanke abfragen
  pInput->bR_TRIG       = Eingang[pInput->EingangArr] && !pInput->Hilfsmerker;  
  //Fallende Flanke abfragen
  pInput->bF_TRIG       = !Eingang[pInput->EingangArr] && pInput->Hilfsmerker;  
  pInput->Hilfsmerker   = Eingang[pInput->EingangArr];

    // Ausschalten wenn nur kurz gedrückt wurd, damit nicht ausgeschaltet
    // wird wenn hochgedimmt wurde!
    if (pInput->bF_TRIG && pInput->pressedCycles < 50 && (!pInput->TasteIstGedrueckt) && licht_status[pInput->bitStatus])
    {
    licht_schalten[pInput->bitStatus] = 1;
    }
    
    //Einschalten wenn noch nicht an ist  
    if (pInput->bR_TRIG && (!licht_status[pInput->bitStatus]))
    {
    licht_schalten[pInput->bitStatus] = 1;
    pInput->TasteIstGedrueckt = 1;
    }

    //Beim Loslassen des Tasters Speicher "TasteGedrückt" löschen  
    if (pInput->bF_TRIG)
    {
    pInput->TasteIstGedrueckt = 0;
    }
    //Länge des Tastendruckes aufzeichnen
    if (Eingang[pInput->EingangArr])
    pInput->pressedCycles++;
    else
    pInput->pressedCycles = 0;
    
  //#######################################################################
  //Dimmen
  //#######################################################################

  // Wenn Taste länger gedrückt wird
  if (pInput->pressedCycles >50)
  { 
    //Nach Zeit Dimmwert erhöhen      
    pInput->takt++;
     
    if (pInput->takt == 5)
    {         

      if (u8SpeicherDimmwert[pInput->bitStatus] == 100)
      pInput->aufab = 1;
      if (u8SpeicherDimmwert[pInput->bitStatus] == 5)
      pInput->aufab = 0;
      
      //Wenn unter 100%, dann plus
      if (u8SpeicherDimmwert[pInput->bitStatus] <100 && !pInput->aufab)    
      u8SpeicherDimmwert[pInput->bitStatus]++;          
    
      else if (u8SpeicherDimmwert[pInput->bitStatus] >5 && pInput->aufab)
      u8SpeicherDimmwert[pInput->bitStatus]--;                      
          
    pInput->takt = 0;
       
    }   
   
  
    return   u8SpeicherDimmwert[pInput->bitStatus];  
  }
  
}

[Hmm (Gast)]

>So bekomme ich die Warnung:
>main.h:674: warning: control reaches end of non-void function

Der tiefere Sinn ist der: Du kannst keine non-void Funktion 
beenden/verlassen ohne einen Wert zurückzugeben.

Das
>Sollte die If-Anweisung FALSE sein, soll der Rückgabewert nicht
>zurückgegeben werden!

geht also unter keinen Umständen.

[Dietrich L. (dietrichl)]

Alex schrieb:
> Sollte die If-Anweisung FALSE sein, soll der Rückgabewert nicht
> zurückgegeben werden!

Das geht nicht. Eine (nicht-void) Funktion muss immer einen Wert 
zurückgeben.
Du kannst das Ergebnis der If-Abfrage nur in dem Rückgabewert kodieren, 
z.B. bei FALSE gibst Du 0 zurück.

Gruß Dietrich

[Hmm (Gast)]

Stell Dir vor Du rufst die Funktion auf:

if (TasterDimmenAufAb (Input) = ...)

unint_t a;
a = TasterDimmenAufAb (Input);

Woher soll Dein nachfolgender Code wissen, das kein Wert zurückgegeben 
wurde?

[Andreas B. (andreasb)]

Kann sein das ich die Frage falsch verstehe (ist nicht so ganz klar..)

Aber ggf. ist ja das deine Lösung:

// Wenn Taste länger gedrückt wird

// Bemerke: Semikolon am Ende, und vergleich > durch <= ersetzt.
while(pInput->pressedCycles <= 50);

return TRUE;

> Sollte die If-Anweisung FALSE sein, soll der Rückgabewert nicht
> zurückgegeben werden!

Ist auf jeden Fall erfüllt:-)


mfg Andreas