Input string was not in a correct format when I pressed search button

Question

0.00/5 (No votes)

See more:

if (txt_search.Text == "" || int.Parse(txt_salary.Text)<=10000) { string display = "Enter the Employee ID to Complete Search"; ClientScript.RegisterStartupScript(this.GetType(), "Empty Text Box", "alert('" + display + "');", true); }

can anyone explain how can i remove this error i want to search employee having salary greater than 10000 when i press search button there was a error input string was not in a correct format.

What I have tried:

I have tried parse method to convert string

Posted 30-Jul-18 8:04am

Member 13931382

Updated 30-Jul-18 19:17pm

Add a Solution

Comments

Member 13931382 30-Jul-18 14:09pm

I want to make employee management system and search only those employees records whose salary is greater than 10000

CHill60 30-Jul-18 14:38pm

What is in txt_Salary.Text?

Member 13931382 30-Jul-18 14:40pm

Employee salary

CHill60 30-Jul-18 14:44pm

No - what value is in txt_Salary.Text when you get the error?

Member 13931382 30-Jul-18 14:48pm

according given condition I entered less than 10000 but an error message print on screen input string was not in a correct format

Patrice T 30-Jul-18 14:46pm

you are asked the exact contain of "txt_salary"

Member 13931382 30-Jul-18 14:50pm

exact contain?

CHill60 30-Jul-18 17:56pm

...comme ses pieds, non?

Patrice T 30-Jul-18 18:11pm

exactly :)

Patrice T 30-Jul-18 14:44pm

What is the value of 'txt_salary' ?

Member 13931382 30-Jul-18 14:52pm

Employee salary

CHill60 30-Jul-18 14:58pm

The actual value is what we need - it clearly isn't an integer value

Member 13931382 30-Jul-18 15:01pm

it is not an integer it was string value so that's why I have used parse method to conversion because I need integer

CHill60 30-Jul-18 17:53pm

You are making this really difficult and I don't know why ... what are the contents of the text box when you get the error... don't describe them, just tell us what the actual contents are! You are using int.Parse ... which is failing ... because the contents of the text box do not match what an integer would "look like"

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 · Answer 1 · 2018-07-30T19:18:00

The error message you get says "The string you tried to convert does not contain a number, so I can't convert it" - and that means that whatever the user has typed into txt_salary.Text isn't just digits - it may be "12 34", or "Hello", or "$10,000" , or perhaps "10000.00"

Instead of using int.Parse, at the top of the method use int.TryParse instead:

C#

int salary;
if !(int.TryParse(txt_salary.Text, out salary))
    {
    ... Bad input by user, report it to him ...
    return;
    }

And also validate all other entry fields. You can then use the validated values in salary and such line confident that they will work.