divide value

Question

0.00/5 (No votes)

See more:

Questions:
Very easy but can't work in program, need help my brain is over used.
I am trying to calculate the value and get the percentage but I end up getting a 0.

int a = 122333; (value here could expand to hundred thousand)
int b = 128222; value here could expand to hundred thousand)

int answ = 0;

answ = (a - b);
answ = -5889;
answ = ((answ / a) * 100);

answ = 0 = definitely wrong
CORRECT ANSWER SHOULD BE: (-4.81)

Please enhance my basic code to get the right value because I always get 0

help :-<</xml>

Posted 5-Sep-10 8:18am

dabuskol

Add a Solution

3 solutions

Solution 2

answ = ((answ / a) * 100);

The integer (-5889 / 122333) is 0, 0 * 100 is 0 so your program is correct.
The integer (100 * answ / a) would evaluate to -588900 / 122333 which is -4 nearer to your wish.

cheers,
AR

Posted 5-Sep-10 8:41am

Alain Rist

Comments

dabuskol 5-Sep-10 14:51pm

I haven't tried your answer because I went out of the office to think load but monitoring my mails. How can I retain the negative value? Is it correct to use Math.Sign(answ) either the output is + or -.
thanks

Alain Rist 5-Sep-10 15:00pm

Use float, double or decimal C# types to get an usable value :)

Solution 3

To make it simple.

Change

C#

int answ = 0;

Into

C#

double answ = 0;

then

MIDL

answ = ((answ / a) * 100);//= -4,81390957468549

Posted 6-Sep-10 0:02am

Per Söderlund

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard MacCutchan · Accepted Answer · 2010-09-05T08:31:00

Solution 1

answ = ((answ / a) * 100);

-5889 / 122333 in integer arithmetic is zero. If you want the real values then convert your numbers to doubles.

Posted 5-Sep-10 8:31am

Richard MacCutchan

Comments

Dalek Dave 5-Sep-10 16:17pm

good answer

Abhishek Sur 5-Sep-10 18:55pm

Reason for my vote of 5
Good answer.

#realJSOP 6-Sep-10 6:09am

If you want the most accurate response, you want to use decimal.