|
|
You should not be trying to do this; float/double values are inherently imprecise so two variables which nominally contain the same value will not necessarily be exactly the same. This is due to the binary representation of floating point numbers. There are many papers on the internet including this one[^] that deal with it.
It's time for a new signature.
|
|
|
|
|
You need to compare the difference to some epsilon. Check float.h for DBL_EPSILON or FLT_EPSILON .
"One man's wage rise is another man's price increase." - Harold Wilson
"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons
"Man who follows car will be exhausted." - Confucius
|
|
|
|
|
thx all for your support..
nvm it i solved it ..stupid ME ...
just added this method:
bool classname::isStringEquals(float fNewNumber,float fOldNumber)
{
CString value1;
CString value2;
value1.Format( _T("%.4f"), fNewNumber);
value2.Format( _T("%.4f"), fOldNumber);
if(value1==value2)
return true;
else
return false;
cheers from canada
|
|
|
|
|
|
How am i killing performance, well all i wanted to check is if the other float value is close or 0.10 and that code does check that i cannot use greater or less than since both float can be changed might be >0 or <0 , so rounding off values check if it's close to 0.10 or not because i do not have problem in my math if value if 0.0999978 or 0.10 both will give me same answer since it just checks value and then do not use them for any calculation, i will paste my code later on here and then you can have a look at it and you might be able to give me better suggestion,
thx for reply tho
cheers from canada.
|
|
|
|
|
Performance is killed because CString will new and delete memory to hold the chars for both instances.
If this is in a highly used piece of code, you could fragment the heck out of your heap.
The Epsilon isEqualish(float, float, float) approach is the best.
An alternative would be to use a Fixed Point numeric class that would exactly represent a decimal value, but that might also have performance implications. It all depends on what you are doing.
Skip the CString.Format even though it seems convenient.
|
|
|
|
|
You're kinda doing it wrong - at least in general. You code may work fine for your requirements!
Try:
BOOL IsEqualish (double d1, double d2, double dEpsilon)
{
ASSERT(dEpsilon > 0.0);
d1 -= d2;
if (d1 < 0.0)
d1 = -d1;
return (d1 <= dEpsilon)
}
or for consise:
BOOL IsEqualish (double d1, double d2, double dEpsilon) { return abs (d1 - d2) <= dEpsilon; }
Where dEpsilon is the error margin. For 4 dps, dEpsilon = 0.0001, and so on.
Iain.
I am one of "those foreigners coming over here and stealing our jobs". Yay me!
|
|
|
|
|
I didn't get your code why would i use epsilon ? and it has to be FLOAT
thx for reply.
cheers from canada
|
|
|
|
|
mohit`12 wrote: I didn't get your code why would i use epsilon ?
You yourself brought up the problem of floating point numbers being not perfect - if you read some of the links given you, you will realise that testing for equality is basically a waste of time. Digital computers simply do not do perfect representations of non-integers. Unless you're very lucky, and it's 1/(2^n). Two numbers can be reeeeeeeally close, and not equal. The simple fact you asked your original question shows that you know this already.
mohit`12 wrote: and it has to be FLOAT
So change double to float . Or whatever FLOA Ts your boat. Not the hardest of challenges!
Iain.
ps, My code will also cope (as it should) with negative floating values.
I am one of "those foreigners coming over here and stealing our jobs". Yay me!
|
|
|
|
|
I think you want
fabs
for C/C++.
|
|
|
|
|
viaducting wrote: I think you want fabs
You'd be right, too! That's what happens when you code in a text editor...
Not to worry - the OP is convinced all I typed was rubbish anyway!
But good to point out an error, in case it helps someone else.
Iain.
I am one of "those foreigners coming over here and stealing our jobs". Yay me!
|
|
|
|
|
I'm not a great one for going mental about code's performance but that's just horrible on so many levels. Why not do something like:
bool near_enough_equals( double a, double b )
{
return (b - a) < ( (b + a) / 2000.0 );
}
Cheers,
Ash
|
|
|
|
|
I am dealing with float and there is not way i can use DOUBLE for that particular code and my value might be negative so it might return negative value using your code and also b will be always less than a so negative
thx for reply
cheers from canada
|
|
|
|
|
Floats are converted to doubles before any serious arithmetic is done on them. The only reason you'd use a float is if you really had to save the 4 bytes (on most implementations). You actually introduce more arithmetic errors by saving everything as a float between calculations than doing everything as a double.
As for the negative problem, use std::fabs or something on the values before processing them:
bool are_these_two_near_enough( double a, double b )
{
a = std::fabs( a ); b = fabs( b );
return std::fabs( a - b ) < ( ( a + b ) / 2000.0 );
}
and if you're really worried about the difference between float and double use this as the function signature:
template< typename T >
bool are_these_two_floating_point_values_near_enough( T a, T b )
Edited as I didn't have all the HTML escapes properly sorted
modified on Tuesday, August 24, 2010 5:54 AM
|
|
|
|
|
mohit`12 wrote: if(value1==value2)
return true;
else
return false;
Ok, setting aside your real problem, why not simply code this as:
return (value1 == value2);
Call me anal if you like, but that's so much simpler and easier to read.
Once you agree to clans, tribes, governments...you've opted for socialism. The rest is just details.
|
|
|
|
|
Why use strings? Just round the values.
double a = 1.00003;
double b = 0.99997;
a *= 100;
b *= 100;
a += 0.5;
b += 0.5;
if (::floor(a) == ::floor(b))
{
}
|
|
|
|
|
floor will convert 0.099999787 to 0.10 ? and it has to be float value will it work ?
thx
cheers from canada.
|
|
|
|
|
a = 0.09999;
a *= 100;
a += 0.5;
floor(a);
b = 0.10001;
b *= 100;
b += 0.5;
floor(b);
|
|
|
|
|
... and use "ceil" to round up value
for example,
printf("ceil of 2.3 is %.1lf\n", ceil (2.3));
output
ceil of 2.3 is 3.0
to use the floor/ceil should be included math.h
Sorry for my poor english.
Best Regards,
Alexander S.
|
|
|
|
|
|
float a = 1.555;
float b = floor(a);
float c = ceil(a);
a = 0.09999;
b = floor(a);
c = ceil(a);
modified on Tuesday, August 24, 2010 1:56 AM
|
|
|
|
|
fasked wrote: Why add 0.5?
To turn floor() into a round()
|
|
|
|
|
using strings for numeric information when you don't need strings is never the right approach. For numerical problems, use a numeric solution.
Here are some possibilities:
1. you might put an upper limit on the deviation, similar to:
float is=0.999999877543;
float closeTo=1.0;
bool closeEnough=abs(is-closeTo)<0.01;
that would be close enough within 1% (since closeTo itself is one).
2. you might opt for a relative deviation (assuming the ideal value isn't zero):
float is=12.999999877543;
float closeTo=13.0;
bool closeEnough=abs((is-closeTo)/closeTo)<1.0e-6;
that would be close enough when within 1 ppm (pars pro million)
|
|
|
|
|
as i mentioned in previous post those value can be changed so and it may vary from negative to zero and zero to +2.707 E06 so not sure if that would work.
|
|
|
|