How to calculate total amt.

Question

2.50/5 (2 votes)

See more:

I want to calculate total. For this I am using following code undertxtDisc_TextChanged event.

qty = Convert.ToInt32(txtQty.Text)
        rate = Convert.ToInt32(txtRate.Text)
        disc = Convert.ToInt32(txtDisc.Text)

        total = qty * rate
        amt = total / disc
        total1 = total - disc

        txtAmt.Text = Convert.ToInt32(total1)

Code calculate correct total when i put disc > 0, i.e If I use disc = 0 then system throws an error :Arithmetic operation resulted in an overflow.

Another problem is If first time I put disc = 12 & before move forward I changed it to 10 then system throws an error : Input string is not in correct format.

Posted 10-Jan-13 3:55am

Yogi ,Pune

Updated 10-Jan-13 3:58am

Abhishek Pant

v2

Add a Solution

2 solutions

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)

**Logi Guna** · Answer 1 · 2013-01-10T04:23:00

You must ensure all input values were correct before performing calculation.
I suggest TryParse instead Try Catch block

VB

If Integer.TryParse(txtQty.Text, qty) AndAlso Integer.TryParse(txtRate.Text, rate) AndAlso Integer.TryParse(txtDisc.Text, disc) AndAlso (disc > 0) Then
    ' Do calculation here
    total = qty * rate
    amt = total / disc ' I suggest: amt = total \ disc
    total1 = total - disc
    txtAmt.Text = Convert.ToInt32(total1).ToString()
Else
    ' Something wrong in input parameter
    ' Do not calculate, you can give a MessageBox to tell user
End If

Nick Fisher (Consultant) · Answer 2 · 2013-01-10T04:01:00

Solution 1

Because you are dividing by zero?

Try:

VB

If total <> 0 And disc <> 0 Then
        amt = total / disc
Else
        amt = 0
End If

I don't get what you mean by the second question, '& before move forward I changed it to 10 ' - can you explain in more detail?

Posted 10-Jan-13 4:01am

Nick Fisher (Consultant)

Updated 10-Jan-13 4:13am

v4