Ceiling function using C# Anyone?

Question

2.75/5 (4 votes)

See more:

Ok guys this is the code.
I need the ceiling function in calculations so that the text displayed does not exceed the space available in the textbox.
I have done this in VB.Net. I have no clue as to how to perform the same in C#. I’m NOT a programmer; this is a hobby for me if you be so kind as to lend a helping hand.
Thanks again.
Lukann

C#

private void GoButt_Click(object sender, RoutedEventArgs e)
{
    try
    {
        double VL = Convert.ToDouble(LO.Text);
        double VA = Convert.ToDouble(AO.Text);
        double VTS = Convert.ToDouble(VL * VA);
        //
        TSF.Text = VTS.ToString();
        TMS.Text = (VL * 12 / 16).ToString();
        TMT.Text = (VL * 2 / 10).ToString();
        TMSc.Text = (VL * 12 / 16 * 4).ToString();
        int idxGSL = GSL.SelectedIndex;
        int idxCSL = CSL.SelectedIndex;
        double tmp = (((idxCSL + 1) * VTS) / (32 + 4 * idxGSL));
        TGB.Text = String.Format("{0}",tmp);
        TGBS.Text = (VTS * 1.5 / 258).ToString();
    }
    catch(Exception)
    {
        //Do something meaningful here
    }
}

Posted 3-Mar-11 9:16am

Lukann

Updated 4-Mar-11 3:26am

Toli Cuturicu

v4

Add a Solution

Comments

wizardzz 3-Mar-11 15:24pm

What is VTS or CSL? You need to provide more information for me to help.

Sergey Alexandrovich Kryukov 3-Mar-11 15:45pm

Who cares?

wizardzz 3-Mar-11 15:50pm

I figured he had tried using Math.Ceiling(), yet was still having trouble, possibly due to variable values. I didn't realize he made to effort on his own and that his question was so simple.

Sorry about the double post, my browser froze. Deleted initial comment.

Sergey Alexandrovich Kryukov 3-Mar-11 21:56pm

No trouble at all; and thanks for your note.
I did not notice any effort from OP except formula not using Ceiling. I don't understand how one can make an issue out of this.
--SA

Sergey Alexandrovich Kryukov 3-Mar-11 15:47pm

Lukann, did your try just to search in help? You won't make any progress if you're not learning to find answers by yourself; even the best Answer's won't help you.
--SA

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)

Dima Popov · Accepted Answer · 2011-03-03T22:33:00

I understand that programming is your hobby, but it does not mean that you shouldn't improve your skills. To start with I strongly suggest using better naming rules (prefixes for controls, meaningful variable names etc.).

C#

private void GoButt_Click(object sender, RoutedEventArgs e)
        {
            double VL = 0;
            double VA = 0;
            int idxGSL = GSL.SelectedIndex;
            int idxCSL = CSL.SelectedIndex;
            if (!Double.TryParse(LO.Text, out VL))
            {
                // Optionally display a parsing error (e.g. using MessageBox.Show)
                return;
            }
            if (!Double.TryParse(AO.Text, out VA))
            {
                // Optionally display a parsing error (e.g. using MessageBox.Show)
                return;
            }
            if (idxGSL == -1 || idxCSL == -1)
            {
                return;
            }
            double VTS = VL * VA;
            double tmp = (((idxCSL + 1) * VTS) / (32 + 4 * idxGSL));
            TSF.Text = VTS.ToString();
            TMS.Text = (VL * 12 / 16).ToString();
            TMT.Text = (VL * 2 / 10).ToString();
            TMSc.Text = (VL * 12 / 16 * 4).ToString();
            TGBS.Text = (VTS * 1.5 / 258).ToString();

            // Is this what you want to achieve?
            TGB.Text = Math.Ceiling(tmp).ToString();
        }

AspDotNetDev · Accepted Answer · 2011-03-03T09:33:00

Solution 1

C#

TGB.Text = (Math.Ceiling(VTS / 32 * (CSL.SelectedIndex +1))).ToString();

Posted 3-Mar-11 9:33am

AspDotNetDev

Updated 3-Mar-11 11:48am

v2

Comments

Sergey Alexandrovich Kryukov 3-Mar-11 15:45pm

Sure, my 5.
--SA

Lukann 3-Mar-11 17:46pm

double' does not contain a definition for 'ToSring' and no extension method 'ToSring' accepting a first argument of type 'double' could be found

AspDotNetDev 3-Mar-11 17:50pm

That was a typo, which I have fixed. Did you compare the differences between your code and the code I posted? That you could not figure out the problem was a typo indicates you may not be trying as hard as you should be.

Henry Minute 3-Mar-11 17:52pm

Oh yes it does. Look here http://msdn.microsoft.com/en-us/library/3hfd35ad%28v=VS.90%29.aspx

AspDotNetDev 3-Mar-11 18:17pm

My code originally said "ToSring" rather than "ToString". That is why Lukann was confused. Silly bugger couldn't work out that was a typo.

Henry Minute 3-Mar-11 20:28pm

I regret to say I missed your deliberate mistake. :)

Dima Popov 4-Mar-11 3:53am

I wanted to joke about this, "sayz no method tosring????" but thought it would be too stupid. Oh well...

Olivier Levrey 4-Mar-11 9:27am

My 5.