Falscher Rückgabewert bei float Rückgabe

[Klaus (Gast)]

Hallo,
ich habe das Problem, dass meine Funktion sobald ein float oder double 
Wert zurückgegeben wird falsche Werte liefert.

int main(void)
{
  float temperatur;

  temperatur = read_temperature();
}

float read_temperature(uint8_t id)
{
  //do something
  temperatur *= 0.0625;
  return temperatur
}

Das liefert mir eine Temperatur 16807.

Änder ich das ganze zu bekomme ich die richtigen Werte...

int main(void)
{
  float temperatur;
  uint16_t buffer;

  buffer = read_temperature();
  temperatur = buffer*0.0625;
}

uint16_t read_temperature(uint8_t id)
{
  //do something
  return temperatur
}

Vielen Dank für eure Hilfe!

[Peter II (Gast)]

Klaus schrieb:
> Das liefert mir eine Temperatur 16807.

und welchen DatenType hat temperatur in

float read_temperature

bitte etwas mehr code zeigen, mit diesen kann man die werte nicht 
nachvollziehen weil man nicht weiss auf was temperatur gesetzt ist.

[(prx) A. K. (prx)]

Hast du Warnungen des Compilers ignoriert? Nimm mal -Wall rein (wenn 
GCC). Wass passiert, wenn du die Reihenfolge der Funktionen umkehrst?

[Ronny S. (duselbaer)]

Das read_temperature in deinem Code erwartet einen Parameter, beim 
Aufruf gibst Du keinen mit. Ist das nur ein Copy&Paste Fehler, oder ist 
das auch in deinem Code so?

[Klaus (Gast)]

Kann auch gerne die Funktion posten, dachte es ist so übersichtlicher.

float ow_read_temperature(uint8_t id)
{
  uint8_t i;
  uint8_t scratchpad[9]; //datenpuffer für auslesen des scratchpads
  float temperatur;
  
  // Start mit Master-Reset-Impuls u. Abfrage: Slave presence
  if(ow_reset())
  {
    return 0;
  }
  
  //Slave anwählen
  ow_match(id);
  
  // Abfrage-Befehl senden: "READ SCRATCHPAD" = 0xBE
  ow_wr_byte(0xbe);

  for (i=0; i<9; i++)
  {
    scratchpad[i] = ow_rd_byte();
  }
  
  if((scratchpad[1] & 0xF8) == 0x00) //Wenn sign bit == 0 -> positive temperatur
  {
    temperatur = ((scratchpad[1]<<8)+scratchpad[0]);
    temperatur *= 0.0625;
    return temperatur;
  }
  else
  {
    return 0; //Fehler
  }

[Klaus (Gast)]

Das mit dem Übergabeparameter ist ein Tippfehler das ist im Code 
richtig.

[Klaus (Gast)]

Jetzt funktioniert es. "Umdrehen" ging zwar nicht da in anderer c-Datei, 
ein extern float ow_read_temperature(uint8_t id) hat aber geholfen. 
Blöder Fehler.
Kann mir jetzt aber einer erklären warum es bei einem uint16_t 
Rückgabewert trotzdem funktioniert bei einem float jedoch nicht?

[(prx) A. K. (prx)]

Klaus schrieb:
> Kann mir jetzt aber einer erklären warum es bei einem uint16_t
> Rückgabewert trotzdem funktioniert bei einem float jedoch nicht?

Weil in uralter K&R-Tradition unbekannte Funktionen ein int als 
Rückgabewert haben. Der ist uint16_t hinreichend ähnlich.

[Karl H. (kbuchegg)]

Klaus schrieb:
> Jetzt funktioniert es. "Umdrehen" ging zwar nicht da in anderer c-Datei,
> ein extern float ow_read_temperature(uint8_t id) hat aber geholfen.
> Blöder Fehler.
> Kann mir jetzt aber einer erklären warum es bei einem uint16_t
> Rückgabewert trotzdem funktioniert bei einem float jedoch nicht?

Weil der Compiler bei allem was er nicht kennt Standardannahmen trifft. 
Und die lauten in deinem Fall:

Eine Funktion von der nichts bekannt ist, liefert einen int zurück.

Wenn das das komplette Programm ist

int main()
{
  double j;

  j = cos( 0.5 );
}

dann weiß der Compiler an dieser Stelle nichts von einer Funktion namens 
'cos'. Also gelten Standardannahmen. Zwei davon sind
* da du ein Argument benutzt und dieses Argument (0.5) den Datentyp 
double hat, vertrau ich meinem Progrtammierer und die Funktion cos wird 
angenommen als eine Funktion, die genau 1 Argument nimmt und dieses muss 
double sein
* da über den Datentyp des Returnwertes nichts bekannt ist, wird 
angenommen, dass die Funktion einen int zurückliefert.

Jetzt ...

double cos( double angle );

int main()
{
  double j;

  j = cos( 0.5 );
}

... sieht die Sache anders aus. Jetzt informiert der Prototyp der 
Funktion den Compiler darüber, wie sich die Sache mit der Funktion 'cos' 
wirklich verhält.

Steht denn diese 'implizit-int' Regel in den heutigen C-Büchern nicht 
mher drinnen? Dabei ist sie samt den Promotion Regeln so dermassen 
wichtig. Wie will man den anders verstehen, wie sich die 
Argumentübergabe bei variadischen Funktionen verhält? Oder hast du gar 
kein C-Buch? Gefährlich!

[Klaus (Gast)]

Danke, dass du dir die Mühe gemacht hast.