Почему сравнение разных значений значимого типа возвращает истину?

Question

[Surface-ketch]

Доброго времени суток. Почему сравнение разных значений значимого типа возвращает истину в данном случае? Конкретно на 100 строке, условие все время возвращает true, хотя видно, что они не равны.

Скрины дебага

Скрин раз

Скрин два

Кусок кода

@Override
            public void onTextChanged(CharSequence charSequence, int i, int i1, int i2) {
                int int1 = editText.getText().toString().length();
                int int2 = currentWord.getEnglishWord().toString().length();

                String string1 = editText.getText().toString();
                String string2 = currentWord.getEnglishWord();

                if(string1.length() == string2.length());
                {
                    if(editText.getText().toString().equals(currentWord.getEnglishWord()))
                    {
                        Toast.makeText(getContext(), "WIIIIIIN!!!!", Toast.LENGTH_SHORT).show();
                    }else
                    {
                        int g =int1 + int2;
                        string1 +=string2;
                        Toast.makeText(getContext(), "LOOOOSE!!!!", Toast.LENGTH_SHORT).show();

                    }
                }
            }

P. S. Через Integer.equals тоже самое, все время истина, как-будто на 100 строке вообще условия нет.

Comments

[Dmitry Roo]

Вы крайне неудачно называете свои переменные.
Что за int1 и int2 ? Что за string1 и string2 ? Чем они отличаются? что обозначают?
Вот так, например, будет лучше:

String textFromEditText = editText.getText().toString();
String currentWord= currentWord.getEnglishWord();

int textLength= textFromEditText.length();
int currentWordLength = currentWord.length();

Answer 1

[Алексей Черемисин]

А зачем вам точка с запятой в конце выражения if?!!!!
Вы же понимаете, что в этом случае у вас оператор сравнения просто выполняется впустую!

Comments

[Surface-ketch]

Не заметил, решено, спасибо)

[Алексей Черемисин]

Surface-ketch, Бинго!

[Алексей Черемисин]

Surface-ketch, И не переносите открывающую скобку под if, делайте в конце!

if(string1.length() == string2.length()) {
                    if(editText.getText().toString().equals(currentWord.getEnglishWord())) {
                        Toast.makeText(getContext(), "WIIIIIIN!!!!", Toast.LENGTH_SHORT).show();
                    } else {
                        int g =int1 + int2;
                        string1 +=string2;
                        Toast.makeText(getContext(), "LOOOOSE!!!!", Toast.LENGTH_SHORT).show();
                    }
                }