Why is this code not working?

Question

0.00/5 (No votes)

See more:

Hello, i am new to c# programming
but whenever i type Adam in the textbox the messagebox shows up even though i did not click button1, But the checkbox works normally with button1 please help me.

C#

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace C_tutorials
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            int i=0;

            if (textBox1.Text == "Adam" || checkBox1.Checked)

                MessageBox.Show("hello");


        }


    }
}

Posted 8-Feb-15 11:27am

fun Tertain

Add a Solution

Comments

Kenneth Haugland 8-Feb-15 17:40pm

https://msdn.microsoft.com/en-us/library/6373h346.aspx

fun Tertain 8-Feb-15 17:45pm

lol i figure that the events of the textbox and checkbox were wired to the button1 function if you know what i mean. So checkChanged and textChanged both ran the button1 function

Kenneth Haugland 8-Feb-15 17:49pm

From your question I guess you want this one instead;
https://msdn.microsoft.com/en-us/library/2a723cdk.aspx
&&
:)

fun Tertain 8-Feb-15 17:53pm

i guess the link answers my question partially as of why checkbox1 did not run right away until i clicked button1. but it seemed to work fine after i erased the fact that textbox1 and checkbox1 were connected to the button 1 function

Kenneth Haugland 8-Feb-15 18:04pm

Sounds like you are a little confused. The || operator will, given that the function Button1_Click() is started, check if either the Text is Adam or the checkbox is checked. It just needs one of them to be true in order to proceed to show the message. What else you are saying dosnt relly make any sence to me, but the again, I don't see all your code either.

fun Tertain 8-Feb-15 18:17pm

That is actually all my code but i do get what you are saying and that
the | operator will check the first operand, and if that is false it won't proceed to the next operand. and i already know what the || operand does.

CLilium 9-Feb-15 4:10am

Wrong, read what is in first link Kenneth Haugland posted, you are using OR operator, if either of the conditions is true, then the result is true, so if your first conditions is true (not false as you mentioned) it doesn't need to check the other one, because it already knows the result.
If you need both conditions to be true to show the message you have to use AND operator, posted as 2nd link.
Other than that I don't know why you bind textbox1 and checkbox1 to that button1 event funtion, but if there is still something wrong just say it, don't know if you solved everything you needed.

fun Tertain 8-Feb-15 19:17pm

I forgot to mention i am using microsoft visual basic c# 2010

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)

fun Tertain · Answer 1 · 2015-02-08T13:17:00

Solution 1

button1 event was wired to textbox1 and checkbox1

Posted 8-Feb-15 13:17pm

fun Tertain

Comments

[no name] 8-Feb-15 23:36pm

As per your question posted here if the checkbox is checked or the textbox is having the text as "Adam" then only your messagebox should pop out. There is another scenario where you can face the same situation. That is if you are having the same method name refered in the textbox text changed event your messagebox is going to display.