Warning Code

[Execute0426]

Hi guys, I have a warning with this code:

		    if (comboBox2.SelectedValue == Cash)
            {
                comboBox3.Enabled = false;
            }
            else
            {
                comboBox3.Enabled = true;
            }

it says in the warning that Possible unintended reference comparison; to get a value comparison, cast the left hand side to type 'string' .

I hope you can help me. Thnks

[Factory Factory]

is "Cash" supposed to be a string, or is it the name of a variable or constant that contains other data? I'd bet that you derped, it's supposed to be the string "Cash," and you meant to write:

 

if (comboBox2.SelectedValue == "Cash")

[Execute0426]

I try that and the warning is almost the same.

[Factory Factory]

Oh, I think I see now: I think it's saying that 

comboBox2.SelectedValue

is a pointer reference, and you need to dereference it (like by casting it to a string)  if you want to check its value rather than check that it points to a particular memory address.

 

I could be wrong. I am Bad At Pointers.

[vorticalbox]

6 hours ago, Factory Factory said:

Oh, I think I see now: I think it's saying that 

comboBox2.SelectedValue

is a pointer reference, and you need to dereference it (like by casting it to a string)  if you want to check its value rather than check that it points to a particular memory address.

 

I could be wrong. I am Bad At Pointers.

comboBox2.SelectedItem.ToString() == "Cash"

[Nuluvius]

You are getting the warning because ListControl.SelectedValue (inherited by ComboBox) returns an object. When == is used on an expression of type object it will resolve to Object.ReferenceEquals.

 

The implementation provided by @vorticalbox is correct.