Tax rate code returning wrong output

[pejy69]

Hi
My name is PJ and i'm an adult student who requires a pointer in the right direction. Below is a snippett of the code that i believe i have something wrong in but cant work out what i have done wrong. The code is meant to:
Allow income to be entered and return a tax % rate.
<38000.00 meant to display 19.5%
>38000.00 but <60000.00 meant to display 33.0%
>60.000 meant to display 39.0%
It does not do this (it still displays dollar amounts) EG: enter 41000.00 as taxable income and the answer is 21525.00, and i have struggled with this for days, i need some guidance in the right direction, please
The code is as follows:

if (dectaxableInco me >= 38000)
dectax = dectaxableIncom e * 19.5m / 100;
if (dectaxableInco me > 38000 && dectaxableIncom e <= 60000)
dectax = dectaxableIncom e * 19.5m / 100;
dectax = dectax + dectaxableIncom e * 33m / 100;
if (dectaxableInco me > 60000)
dectax = dectaxableIncom e * 19.5m / 100;
dectax = dectax + dectaxableIncom e * 33m / 100;
dectax = dectax + dectaxableIncom e * 39m / 100;

[gpraghuram]

I think the first if should be if(dectaxableIn come <= 38000)


Raghu

[pejy69]

I think the first if should be if(dectaxableIn come <= 38000)


Raghu

I'd love to say yes, but no that does not work either, thanks for trying to help though

[r035198x]

What is the code

dectax = dectaxableIncome * 19.5m / 100;

supposed to be doing?

[pejy69]

What is the code

dectax = dectaxableIncome * 19.5m / 100;

supposed to be doing?

dividing every hundered dollars by 19.5%...... have a not quite understood this very well? I'm a first year student wanting to excel in this area, and we have had to develop this from pseudocode that was provided.

[JosAH]

Hi
My name is PJ and i'm an adult student who requires a pointer in the right direction. Below is a snippett of the code that i believe i have something wrong in but cant work out what i have done wrong. The code is meant to:
Allow income to be entered and return a tax % rate.
<38000.00 meant to display 19.5%
>38000.00 but <60000.00 meant to display 33.0%
>60.000 meant to display 39.0%
It does not do this (it still displays dollar amounts) EG: enter 41000.00 as taxable income and the answer is 21525.00, and i have struggled with this for days, i need some guidance in the right direction, please

Your code doesn't reflect the logic you just described; it should be something
like this:

if (income < 38000) percentage= 19.5;
else if (income < 60000) percentage= 33.0; // income is >= 38000 here
else percentage= 39.0; // income is >= 60000 here
 

btw, this is the C and C++ forum, not the C# forum but in this particular case
it doesn't matter much.

kind regards,

Jos

[r035198x]

For your description I would have expected something like ...

double rate = 0.0;
if(enteredAmount < 38000) {
   rate = 19.5;
}
else if(enteredAmount < 60000) {
    rate = 33;
}
else if ( .....

[pejy69]

For your description I would have expected something like ...

double rate = 0.0;
if(enteredAmount < 38000) {
   rate = 19.5;
}
else if(enteredAmount < 60000) {
    rate = 33;
}
else if ( .....

Thankyou for that.... i will work on that and alter things a little

[weaknessforcats]

I am moving this to the .NET forum.

In C++ the <, >, >=, etc.. operators don't work as expected for floating point. Perhaps in C# it's different.

[Curtis Rutland]

Do not double post your questions. If you are unsure where to post your question, the Mods can move it for you, but don't post it twice. It makes it hard for everyone to know what answers have been given and to keep track of information.

I have merged these threads.

MODERATOR