How can I get the selected value from radio button and comapre with correct option ?

Question

1.00/5 (1 vote)

See more:

I am getting values from radio buttons with id these radio button are in repeater . Correct option in rd4. I am comparing rd1,rd2, & rd3 with rd4 if any one matched then count+1 else count remains 0 then this count result i want to show in label .but result continuously shows 0 wether it is correct or not...

What I have tried:

 protected void Button1_Click(object sender, EventArgs e)
        {
            int count1=0 ;

           
            if (Repeater1.Items.Count > 0)
            {
                for (int count = 0; count < Repeater1.Items.Count; count++)
                {
                    RadioButton rd1 = (RadioButton)Repeater1.Items[count].FindControl("RadioButton1");
                    RadioButton rd2 = (RadioButton)Repeater1.Items[count].FindControl("RadioButton2");
                    RadioButton rd3 = (RadioButton)Repeater1.Items[count].FindControl("RadioButton3");
                    RadioButton rd4 = (RadioButton)Repeater1.Items[count].FindControl("RadioButton31");
    if (rd1.Checked)
                    {
                        if (rd1 == rd4)
                        {
                            count1++;
                        }
                        else
                        {
                            count1 = count1 + 0;
                        }


                    }
                    if (rd2.Checked)
                    {
                        if (rd2 == rd4)
                        {
                            count1++;
                        }
                        else
                        {
                            count1 = count1 + 0;
                        }

                    }
                    if (rd3.Checked)
                    {
                        if (rd3 == rd4)
                        {
                            count1++;
                        }
                        else
                        {
                            count1 = count1 + 0;
                        }

                    }

            Label1.Text = Convert.ToString(count1);
}

Posted 17-Oct-17 10:11am

XarDar XaaD

Updated 20-Oct-17 0:15am

Add a Solution

Comments

Richard Deeming 18-Oct-17 14:48pm

You're testing for reference equality. Two different instances of the RadioButton control will never be the same reference, so your tests will never succeed.

It's not clear why you're using another RadioButton for the correct answer. Particularly since the user will be able to inspect the control to see what the correct answer is.

Also, all of your else blocks which add zero to the variable are redundant. Adding zero to a number will not make any difference.

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)

Member 13286677 · Answer 1 · 2017-10-20T00:15:00

Solution 1

Why not simply assign a value, e.g "1" to the correct answer (it's radio button), then put that condition that adds the count for score if the value is 1.

e.g
if(rd=="1")
{
count += 1;
return
}
Label1.Text = Convert.ToString(count1);

Posted 20-Oct-17 0:15am

Member 13286677

Comments

Richard Deeming 20-Oct-17 11:04am

You can't apply the equality operator to a RadioButton and a string.

The return statement exits the current function without updating the label.

And the user would be able to view the page source to see which option was the correct answer.