How do roundup a textbox

Question

0.00/5 (No votes)

See more:

I thought I was rounding up correctly but looks like I set textBox4 value before I use it in the division for getting textBox5 value. The Roundup needs to be in the first part of the code ad I cant figure it out. Can anyone help?

C#

var roundedInput = Math.Round(tbTrailer_Needed, 2, MidpointRounding.AwayFromZero);

if (!String.IsNullOrEmpty(tbSnp_Day.Text) && !string.IsNullOrEmpty(cbSnp_Uld_Trl.Text))
    tbTrailer_Needed.Text  =  (Convert.ToInt32(tbSnp_Day.Text) / Convert.ToInt32(cbSnp_Uld_Trl.Text)).ToString();

What I have tried:

Math.Round, MidpointRounding mode

Posted 7-Nov-16 11:47am

Member 12349103

Updated 9-Nov-16 10:22am

Patrice T

v7

Add a Solution

Comments

#realJSOP 7-Nov-16 18:20pm

Check the source code you pasted. It looks like it got all jumbled up.

Member 12349103 7-Nov-16 18:39pm

I have removed the Math.Ceiling from the second If statement and added it to the first and get an error textbox can not be used in this meathod

Philippe Mori 7-Nov-16 21:51pm

Write readable code... Maybe you would understand it or someone might help you...

an0ther1 7-Nov-16 23:52pm

Use your debugger, the values are not what you are actually expecting.
For instance - when you divide an int by an int you will always get an int (EG: 10/3 = 3, 9/3 = 3)
Math.Ceiling will round up, Math.Floor will round down & both require either a double or decimal value.

1 solution

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)

OriginalGriff · Accepted Answer · 2016-11-07T22:55:00

First off, rounding only works with floating point values - trying to round integers is not going to work.
Second, dividing any number (other than zero) by itself will always give the same value: one. And the Ceiling value of one is one.
Third, do yourself a favour, and stop using Visual Studio default names for everything - you may remember that "TextBox8" is the mobile number today, but when you have to modify it in three weeks time, will you then? Use descriptive names - "tbMobileNo" for example - and your code becomes easier to read, more self documenting, easier to maintain - and surprisingly quicker to code because Intellisense can get to to "tbMobile" in three keystrokes, where "TextBox8" takes thinking about and 8 keystrokes...

Do your conversions first - and use TryParse instead, as it allows you to respond to user input errors instead of your application crashing - then use the converted values, and finally generate your outputs:

C#

double myValue;
if (string.IsNullOrWhitespace(myTextBox.Text) || !double.TryParse(myTextBox.Text, out myValue))
   {
   // Report problem to user
   return;
   }
double myOtherValue;
...
//Use myValue and myOtherValue, and myYetAnotherValue here and generate your results.
double myResult = ...
//And finally output.
myOutputTextBox.Text = myResult.ToString();

It's a lot clearer what is going on, and a whole lot easier to debug where a problem is.