In this program error input string was not in correct fora pls solve this problem

Question

0.00/5 (No votes)

See more:

C#

protected void Page_Load(object sender, EventArgs e)
{
    string score = Request.QueryString["Score"];
    Label1.Text = score;
    int value = Convert.ToInt32(score);
    if (value >= 10)
    {
        Label2.Text = "Cheers ! your score is satisfactory to promote you for the next level. All the best for the Next Move.";
    }
    else
    {
        Label2.Text = "Sorry ! Your score is not satisfactory to promote you for the next level. You can again attempt this level after Seven Days.";
    }
}
protected void Button1_Click(object sender, EventArgs e)
{
    int level = Convert.ToInt32(Request.QueryString["level"]);
    Response.Redirect("Toplist.aspx?level=" + level);
}
}

Input string was not in correct format

Posted 18-Apr-16 21:13pm

Member 12469537

Updated 18-Apr-16 21:50pm

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ)

v3

Add a Solution

Comments

Shambhoo kumar 19-Apr-16 3:19am

Again you asking same problem. why you are not checking your Score variable value.

Patrice T 19-Apr-16 3:20am

Where is the error, What is the value of the string ?

Shambhoo kumar 19-Apr-16 3:20am

Here is nothing wrong with your code please verify your score variable value. show your score variable value.

Jawad Ahmed Tanoli 19-Apr-16 3:21am

which line is generating error?
int value = Convert.ToInt32(score); may be there have you try debug what is in score string?

Tadit Dash (ତଡିତ୍ କୁମାର ଦାଶ) 19-Apr-16 3:50am

Debug and find the exact line causing the exception. Most probably, it is due to bad formatted data or null value in string. Check it out.

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 · 2016-04-18T21:50:00

Stop using Convert methods to change between datatypes: particularly when you are converting from strings to numbers you need to use TryParse instead. Convert methods throw an exception when they fail, where TryParse returns a value which allows you to report problems instead of your app (and website) failing.

C#

string score = Request.QueryString["Score"];
Label1.Text = score;
int value;
if (!int32.TryParse(score, out value))
    {
    // Report problem, or log it.
    ...
    return;
    }

But the reason why it's failing is very likely to be in the code which directed you to this page: use the if above to show you what exactly the query string contains, and then look at what it should be - since you are generating the query string, you are probably doing something very, very wrong there.

As a side note: if you don't want users to progress until they have a sufficient score, then you need to look at using Cookies or the Session object instead of query strings. Otherwise I can progress immediately just by editing the query string in the URL at the top of my browser and pressing ENTER...