[C] sgn-Funktion verhält sich komisch (falsche Rückgabe)

[fkerber]

Hi!

Entweder stehe ich mächtig auf dem Schlauch oder irgendwas ist hier faul.
Es geht um die Implementierung der SGN-Funktion. Folgender Versuch scheitert ganz massiv:

Code:

int sgn(float p)
{
   if(p=0) 
   return 0;

   if(p>0)
   return 1;

   if(p<0) 
   return -1;
}

Es kommt folgendes bei raus:

Code:

sgn(-9)  --> -1
sgn(0)   --> 7842
sgn(6)   --> 7842

Lasse ich das letzte if weg (also immer -1 returnen, wenn er an die stelle kommt) oder schreibe noch irgendein return dahinter ist alles in Butter.

Meine Vermutung ist, dass es irgendwas mit undefiniertem Funktionswert zu tun haben könnte, aber ich verstehe nicht wieso. Es sind ja alle Eingaben eigentlich abgedeckt...


Ciao, Frederic

[Dax]

[edit]snip, desregard that :/[/edit]

[3_of_8]

Ähem!

Code:

   if(p=0) 
   return 0;

Wie heißt der Gleichheitsoperator in C?

[Florian H]

Zitat von 3_of_8:

Ähem!
Wie heißt der Gleichheitsoperator in C?

Joah, böse Falle für Pascal'er
Wie kommt allerdings das seltsame Ergebnis dann zustande?
(p=0) ergibt doch eigentlich immer true, sodass eine 0 zurückgegeben wird und p danach 0 ist ?

[3_of_8]

p=0 gibt 0 zurück, 0 wird als false interpretiert, der Rückgabewert ist daher undefiniert. (Also Stackmüll)

[fkerber]

ARGH

Es ist nicht mein Code, aber ich habe ihn jetzt bestimmt 2h angeschaut und kam nicht drauf woran es hängt.
Dann ist der Fehler aber klar.
Es wird 0 zu gewiesen, dann gibt es dafür aber keinen Else-Fall mehr und somit undef. irgendwas als Ergebnis...

Danke!

[3_of_8]

Das passiert jedem mal, vor allem, wenn man nicht allzu viel mit C macht, und selbst wenn... Da ist eine strenge Sprache wie Java doch von Vorteil, da kann sowas nur passieren, wenn man Booleans vergleicht, und wann macht man das schon.

Im übrigen, die übersichtliche Version wäre ja die folgende:

Code:

int sgn(float x) {
  return (x<0)?-1:((x>0)?1:0));
}

[fkerber]

Ob ein Programmieranfänger dir da zustimmen würde?
Aber grundsätzlich hast du recht!

[3_of_8]

Da war auch eine gewisse Ironie dabei.

[Der Jan]

Zitat von 3_of_8:

Im übrigen, die übersichtliche Version wäre ja die folgende:

Code:

int sgn(float x) {
  return (x<0)?-1:((x>0)?1:0));
}

Ich bezweifle, dasss das ein Pascaler für übersichtlich halten wird, aber ich liebe diesen Operator *sich duckt und wegschleicht*