So what am i doing wrong now? (RichEdit problem in C++ builder)

Ok, its fairly simple: Condition 1 + Condition 2 = add the lines "hello world" in richedit. But its not working
(i have 4 conditions in one button click)

Should sound simple but its giving me headackes for example. (wonder what im doing wrong here)

Code:

---

void __fastcall TForm1::Button1Click(TObject *Sender)
{
if(CheckBox1->Checked==true & CheckBox2->Checked==true);
{
RichEdit1->Lines->Add("hello world");
}

if(CheckBox3->Checked==false & CheckBox4->Checked==false);
{
RichEdit1->Lines->Add("goodbye world");
}
}

---

Gives hello world
goodbye world

on the same buttonclick , it totally disregards that its supposed to be bounded to the if statement and the given conditions,

Clicking on the button should give only: Hello World or Goodbye world not both

what gives? I mean i used checkboxes, and they seemed to work perfectly fine when the conditions where given, when you use it for adding lines in richedit the whole thing goes haywire making me wonder what im doing wrong?

The thing should only add lines when conditions are being met, why isn't it doing so?

& is a bitwise and, but you want a logical and. Can you figure it out from there?

That doesn't work.

using && as the logical and as you describe it , gives the same problem.

Its not nice, i need assistance on this one.

What states are the four checkboxes in?

all are false by default.

So when all are false it outputs "hello world"?

>> Clicking on the button should give only: Hello World or Goodbye world not both
Your code would give both if boxes 1 & 2 are checked and 3 & 4 are not checked.

yep that's the problem , its giving both indiscriminatly.
Have a look what happens if i press the button.

http://blogphotohost.com/show.php?id...f7858B56d40C8c

I changed the link, hopefully it will show up now.

"Unauthorized referer" when I click that link.

Check is a property, not a function, right?

You changed it to &&, right?

Regardless on wether if i use & or && it doesn't work,

what i noticed tho is that i should remove the ; behind my if statement. then it works for the following. http://blogphotohost.com/show.php?id...5eBfbcS7f78332

but if i try this same thing with my RichEdit i get the goodbye hello world mumbo jumbo again =\

>> Regardless on wether if i use & or && it doesn't work
That's not the point. The point is you have to use && or it will never work. If you're trying to get it to work with & then you are wasting time. Make sure you are using &&, then continue on to see what else needs to be fixed.

>> what i noticed tho is that i should remove the ; behind my if statement. then it works for the following.
I didn't notice those. You must remove the semi-colons or it will never work. The semi-colons end the if statement, so the block of code after it will always be run.

>> but if i try this same thing with my RichEdit i get the goodbye hello world mumbo jumbo again =\
Please copy and paste the code (with the two fixes above) that still causes the problem.

BTW, your new picture is too small to see anything.

Amen i will go along with using && , i have absolutely no problems with that as long as it will get my program to work. I removed the semi colons, and i re-hosted the pictures, hopefully you can view them now.

My code as now would be this.

Code:

---


void __fastcall TForm1::Button1Click(TObject *Sender)
{

if(CheckBox1->Checked==true && CheckBox2->Checked==true)
{
 RichEdit1->Lines->Add("hello world");
}

if(CheckBox3->Checked==false && CheckBox4->Checked==false)
{
 RichEdit1->Lines->Add("goodbye world");
}

---

it gives

Hello world
goodbye world

as it ignores my if statements

urgh, im seeing whats going wrong now, in this example.

of course if id checked 1 and 2 in order to get hello world,

i didn't checked 3 + 4 , so in other words the false statement is true and therefore goodbye world is printed also.

You can also use else if if you always want good bye to be ignored when hello is printed. Glad it's working better.