Nur genau Null zulassen

Treue Seele

19.06.2008, 18:23

Hi;

Follgende Bediengung soll nur ausgeführt werden wenn m_FX ganz genau Null ist:

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17	bool CNULLCalculate :: Fcalculate (float x) { m_fN = x; m_Fx = xxx; //Hier Funktion eintragen if (m_Fx == NULL) { return true; } else { return false; } }

Sie wird allerdings auch ausgeführt wenn m_Fx z.b irgend einen Wert zwischen 0.999.. und -0,999.. hat.

Jetzt ist die große Frage wie man das vermeiden kann..

Zum Seitenanfang

xardias

Community-Fossil

Beiträge: 2 731

Wohnort: Santa Clara, CA

Beruf: Software Engineer

2

19.06.2008, 18:37

Nun, es ist schwierig floats genau zu vergleichen. Bei Berechnungen mit floats schleichen sich sehr schnell Genauigkeitsfehler ein, daher macht man float vergleiche in der Regel immer mit einer gewissen Toleranz.
also z.B. x < 0.001f && x > -0.001f.

Und NULL sollte eigentlich nur für Pointer genutzt werden.

Und dein Fehler wird denke ich mal folgender sein:
NULL ist (meistens) als (void*)0 definiert, also ein ganzzahliger Typ.
Was macht C++, er wird dir dein m_Fx implizit in einen Integer casten, und da wird numal alles was <1 ist zu 0.

Also ein Vergleich mit 0.0f würde dein Problem lösen.

Zum Seitenanfang

Deviloper

Community-Fossil

Beiträge: 2 872

3

19.06.2008, 19:10

NULL ist zumindest als 0 definiert.

	C-/C++-Quelltext
1 2 3 4 5	const bool NULLCalculate::Fcalculate(const float x) { m_fN = x; return (m_Fx = x * x * x) == 0.0f; }

Devil Entertainment :: Your education is our inspiration
Der Spieleprogrammierer :: Community Magazin
Merlin - A Legend awakes :: You are a dedicated C++ (DirectX) programmer and you have ability to work in a team? Contact us!
Siedler II.5 RttR :: The old settlers-style is comming back!

Also known as (D)Evil

Zum Seitenanfang

David_pb

Community-Fossil

Beiträge: 3 886

Beruf: 3D Graphics Programmer

4

19.06.2008, 23:45

	C-/C++-Quelltext
1 2 3 4 5	const bool NULLCalculate::Fcalculate(const float x) { m_fN = x; return fabsf( (m_Fx = x * x * x) ) < FLT_EPSILON; }

@D13_Dreinig

Zum Seitenanfang

grek40

Alter Hase

Beiträge: 1 491

Wohnort: Dresden

5

19.06.2008, 23:50

@David

Es geht diesmal doch genau nicht darum, minimale Abweichungen zu tollerieren

Zum Seitenanfang

David_pb

Community-Fossil

Beiträge: 3 886

Beruf: 3D Graphics Programmer

6

19.06.2008, 23:53

Zitat von »"grek40"«

@David

Es geht diesmal doch genau nicht darum, minimale Abweichungen zu tollerieren

Oh, hu!

Überlesen!

@D13_Dreinig

Zum Seitenanfang

dot

Supermoderator

Beiträge: 9 757

Wohnort: Graz

7

20.06.2008, 11:56

Zitat von »"xardias"«

Und dein Fehler wird denke ich mal folgender sein:
NULL ist (meistens) als (void*)0 definiert, also ein ganzzahliger Typ.
Was macht C++, er wird dir dein m_Fx implizit in einen Integer casten, und da wird numal alles was <1 ist zu 0.

Also ein Vergleich mit 0.0f würde dein Problem lösen.

Ich denke du hast recht was das Problem anbelangt. Allerdings tritt es vermutlich schon weiter oben auf: Der Compiler darf (zumindest wenn er Standardkonform sein will) bei dem Vergleich im if m_Fx nicht in einen int casten, wenn m_Fx ein Gleitkommatyp ist, sondern müsste den Nullpointer (Nullpointerkonstanten sind von integralem Typ) in einen entsprechenden Gleitkommatyp umwandeln.

Die Frage ist: Von welchem Typ ist m_Fx. Ich denk mal dass m_Fx ein int ist. Dann werden schon bei der Zuweisung an m_Fx "die Kommastellen abgeschnitten".

Lösung: Mach aus m_Fx einen float. Dann musst du natürlich die, von den anderen bereits angesprochenen, Probleme beim Vergleich zweier Gleitkommazahlen berücksichtigen...

m4studios