Second Chance GUI

Prime b

This is what I have done so far

            double minPayRate = 7.5;
            double maxPayRate = 49.99;
            double inputNumber;
            int count = 0;

            while (true)
            {
                inputNumber = Convert.ToDouble(inPutTextBox.Text);
                if (inputNumber >= minPayRate && inputNumber <= maxPayRate) break;
                count++;
                if (count == 2)
                    outPutLabel.Text = " You had used your two tries to enter valid hourly payment";
                return;
            }
            outPutLabel.Text = " Hourly pay rate {0:C}, while weekly pay rate is {1:C}", inputNumber, inputNumber * 40);

My problem is , how do i set button invisible after 2 tries? And I also have an error on last line for some reason, please help

17 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes14y ago
Accepted answer
I imagine that the code you've posted is in a button Click handler and that you want to make that button invisible after the user has had two tries to input a valid pay rate.

If that's the case, then try something like this:

private int count = 0; // this needs to be a field rather than a local variable

private void button1_Click(object sender, EventArgs e)
{
double minPayRate = 7.5;
double maxPayRate = 49.99;
double inputNumber;
inputNumber = double.TryParse(inPutTextBox.Text, out inputNumber);

if (inputNumber >= minPayRate && inputNumber <= maxPayRate)
{
outPutLabel.Text = String.Format("Hourly pay rate {0:C}, while weekly pay rate is {1:C}", inputNumber, inputNumber * 40);
count = 0;
return;
}

count++;

if (count == 2)
{
outPutLabel.Text = "You have used your two tries to enter valid hourly payment";
count = 0;
button1.Visible = false;
}
else
{
outPutLabel.Text = "Invalid hourly payment, try again";
inPutTextBox.Focus();
}
}
Sam Hobbs14y ago
Vulpes is British. It is past midnight for Vulpes. I will look at the other thread later.
Prime b14y ago
wow, that's unfortunate, Sam could you look at my other post the twitter one? I thought Vulpes is here but apparently he is not here.
Sam Hobbs14y ago
I wrote a long response and a problem with this forum caused it to be lost.
Vulpes14y ago
One year of my degree included a computer science course and I wrote my first programs then in a language called ALGOL, a fore-runner of PASCAL.

There are some library functions that you tend to use all the time. In .NET, these include the methods and properties of the String, Array and the various collection classes.
Prime b14y ago
haha nice, getting code out of your head before you go to bed, thats's awesome. So how did u learn to program, and whats commoner library functions
Vulpes14y ago
It's a long time ago now but my degree's in Maths rather than Computer Science as I wanted to be an actuary at the time.

It won't be long before you start thinking in code - it helps if you can remember the commoner library functions. However, the problem can be getting the code out of your mind when it's time for bed ;)
Prime b14y ago
Thank you Vuples, also what degree you have, if you don't mind me asking you, or shall we just pm each other? And its awesome that you able to write code in your head without compiler, omg, i cant wait when i will be able to do that. Practice makes it perfect
Vulpes14y ago
Due to lack of time, I sometimes do write the source code off the top of my top without the aid of a compiler which means that I'm prone to making the occasional mistake.

I've done that here since this line:

inputNumber = double.TryParse(inPutTextBox.Text, out inputNumber);

should have simply been:

double.TryParse(inPutTextBox.Text, out inputNumber);

Although double.TryParse does return a bool to indicate whether the parsing was successful or not, we're not interested in it here as the 'out' parameter, inputNumber, will be given a value of zero in that event which is an invalid pay rate anyway :)

Strictly speaking you only need to use curly brackets in if/else statements if there's more than a single statement in the 'if' or 'else' block of code which is the case here. However, you'll sometimes see it done even when there's only a single statement, just for clarity.
Prime b14y ago
Vulpes, do you write the source code in your head without compiler?
I also have a problem, the inputNumber = double.TryParse(inPutTextBox.Text,out inputNumber); line gives me an error saying " Cannot implicitly convert type 'bool' to 'double' " , how can I fix that?

Also, why we always put curly brackets around every if and else statements?
Prime b14y ago
help? and why
Sam Hobbs14y ago
Please respond to my first reply.
Prime b14y ago
help
Prime b14y ago
lol Sam , I guess i did.
But this is what I am talking about

3. a. Write a console-based program that prompts a user for
an hourly pay rate. If the user enters values less than $7.50
or greater than $49.99, prompt the user again. If the user
enters an invalid value again, display an appropriate error
message. If the user enters a valid value on either the
fi rst or second attempt, display the pay rate as well as the
weekly rate, which is calculated as 40 times the hourly
rate. Save the program as SecondChancePayRate.cs.
b. Create a GUI application similar to the one described in
Exercise 3a. Allow the user to click a button to submit a
rate. After the user has made a valid entry, or after two
invalid entries, make the button invisible. Save the project
as SecondChancePayRateGUI.

This is not homework, I am just practicing before 2nd C# semester will start.
I have done 3a, vulpes helped me, you can see it here http://www.c-sharpcorner.com/Forums/Thread/154660/second-chance.aspx
But I need help in 3b, to make button hidden after 2 times wrong entry!
Sam Hobbs14y ago
Yes, you killed the forum.
Sam Hobbs14y ago
What button? I don't see a button, so it is obviously already hidden. Unhide it by setting its Visible property to true and hide it by setting the Visible to false.

I don't know what you are trying to do in the last line so I do not know how to fix it.

Note that in your while loop, you are calling return so the loop will always be executed only once. You probably need to use curly brackets ("{}") for your if statement so the return is executed only when count is 2.
Prime b14y ago
lol forum is dead