C Rückgabewert

[Janiiix3]

Der Kompiler will mir weiß machen, dass es kein Rückgabewert gibt, stimmt ja auch teilweise.
Wie kann ich den Compiler denn befriedigen?



CodeBox C

float read_vrms(uint8_t phase)
{
  
   static uint8_t cnt;
  
   ADE7758.vrms[0] += ((float)ADE7758_read_uint24(REG_AVRMS) / FULL_SCALE_VRMS) * RMS_SPANNUNG_MAX;
   ADE7758.vrms[1] += ((float)ADE7758_read_uint24(REG_BVRMS) / FULL_SCALE_VRMS) * RMS_SPANNUNG_MAX;
   ADE7758.vrms[2] += ((float)ADE7758_read_uint24(REG_CVRMS) / FULL_SCALE_VRMS) * RMS_SPANNUNG_MAX;  
  
   if (cnt++ >= AVG)
   {
     cnt = 0;
    
     ADE7758.state |= (1<<0); // voltage average value ready
    
     ADE7758.vrms[0] /= AVG;
     ADE7758.vrms[1] /= AVG;
     ADE7758.vrms[2] /= AVG;
    
     return ADE7758.vrms[phase];    
   }
// Hier gehört normalerweiße das "return" hin.
}

CodeBox C

Warning     control reaches end of non-void function [-Wreturn-type]

 

[Dirk]

Der Kompiler will mir weiß machen, dass es kein Rückgabewert gibt, stimmt ja auch teilweise.
Wie kann ich den Compiler denn befriedigen?

Der Compiler meckert, da eine Rückgabe nur bei einer Bedingung erfolgt.
Was wird zurückgegeben, wenn die Bedingung nicht wahr ist?

Eleganter ist es, wenn du eine Rückgabevariable am Anfang der Funktion definierst und ihr einen Initialwert zuweist und diese dann am Ende der Funtion zurückgibst. In dem if-Block kannst du dieser Variablen ja einen neuen Wert zuweisen.

Dirk

 

[TommyB]

Ich würde es in VB.Net so machen dass ich in der if ein else if (elif) einbinde und dann "Nothing" bzw. in C# "Null" zurück gebe, damit ich es auch erkennen kann ob es einen Rückgabewert gibt oder nicht. In wie weit das in C möglich ist kA. Man könnte auch einen definitiven Wert zurückgeben der nie auftreten kann, sagen wir -32767 (Minimalwert für Int16).
Ok, ob es geklappt hat oder nicht erkennst du an deinem State, aber der Compiler weißt dich auf eine Unschönheit in deinem Quelltext hin.

Edit: Richtig wäre natürlich ein else, nicht else if. Kommt davon wenn man andere Dinge nebenbei macht…

 

[Janiiix3]

@TommyB sehr gute Idee mit der else if. Man denkt einfach zu kompliziert.