Java if statement problem...

[Blazer06]

This is a very basic if statement in a much longer peice of code:

However, for some reason, it does not work. Can you see a reason why?

String strAccess = new String(rights1.getText());

if(strAccess == "1"){
	u1.setSelected(true);
	u2.setSelected(false);
	u3.setSelected(false);
}

strAccess will in fact = 1, but the if statement just won't run!


-Blaze

[vass0922]

The only thing that comes to mind is datatypes.

Not sure how java does it, it SHOULD treat that 1 as a string which is what you're comparing.. but the only that cames to mind is if its treating it as an integer whereas you'd want to test as strAccess = 1 (sans quotes)

You declared strAccess as a string, so I'd think it would convert whatever gets in as a string, but not sure if it will do that on the fly or if you need to do it explicitly.

[Blazer06]

I rewrote it to use spinners this morning, and got this code, which did the same thing.

String strAccess = new String(rights8SpinnerModel.getValue().toString());
if(strAccess == "1"){
	o1.setSelected(true);
	o2.setSelected(false);
	o3.setSelected(false);
}

So I musted around for awhile, and wrote this working code:

if(rights8SpinnerModel.getValue().toString() == "1"){
	o1.setSelected(true);
	o2.setSelected(false);
	o3.setSelected(false);
}

I'm still not sure why the later works and the former doesn't, but it works for now, and for now that's all I need.


-Blaze

[Blazer06]

Quote:

Originally Posted by vass0922

The only thing that comes to mind is datatypes.

Not sure how java does it, it SHOULD treat that 1 as a string which is what you're comparing.. but the only that cames to mind is if its treating it as an integer whereas you'd want to test as strAccess = 1 (sans quotes)

You declared strAccess as a string, so I'd think it would convert whatever gets in as a string, but not sure if it will do that on the fly or if you need to do it explicitly.

Eclipse is telling me that both are strings, no converting needed. Also, I can output strAccess, or set another text box to equal it or whatever the heck I want to do, but compare it to another string. See above post.


-Blaze