Question

I am currently creating an android application with different options. One of the option would be to have a button that would show “Activate” as default. When the application would be running, clicking on it would change it to “Disable” and then to “activate” if clicked again. I believe that all I have to do is to .getText with a string variable then use this variable in a if statement but it seems like it is not reacting to any of my conditions…

        final Button button = (Button) findViewById(R.id.bSensor);


    button.setOnClickListener(new View.OnClickListener() {
        public void onClick(View v) {
            // Perform action on click

            String buttonText = button.getText().toString();    
            if (buttonText == "@string/Disable") {
                button.setText(R.string.Enable);
            }
            else if (buttonText == "@string/Enable"){
                button.setText(R.string.Disable);
            }

        }
    });

Thanks for help

Phyzikk

Answer 1

You shouldn’t use the == operator when comparing strings in Java. Source

You should either use the .equals() method of the string, or alternatively you could keep a global boolean state flag to determine which value is set. This way you won’t need to do a string compare every time you need to figure out if it’s active or disabled.