Радиогруппа, рассчитывающая ответы дважды

Question

У моей проблемы должен быть простой ответ, и я уверен, что упускаю что-то очевидное... Может ли кто-нибудь указать мне на это?

Когда я запускаю приложение на своем телефоне, радиогруппа ведет себя должным образом (то есть 0 баллов за неправильный ответ и 1 за правильный ответ), однако, если я нажимаю неправильный ответ, затем возвращаюсь к правильному ответу и повторно отправляю, он добавляет один больше на счет. Таким образом, один и тот же вопрос покажет две точки только для того, чтобы дважды щелкнуть правильный ответ...

package com.example.android.englishquizl3;

import android.os.Bundle;
import android.support.v7.app.AppCompatActivity;
import android.view.View;
import android.widget.RadioButton;
import android.widget.RadioGroup;
import android.widget.Toast;

import static com.example.android.englishquizl3.R.id.radioButton2;
import static com.example.android.englishquizl3.R.id.radioButton3;

public class MainActivity extends AppCompatActivity {

int baseScore = 0;
private RadioGroup radioGroup1;
private RadioGroup radioGroup2;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    // Initialize Radio Group and attach click handler
    radioGroup1 = (RadioGroup) findViewById(R.id.radioGroup1);
    radioGroup2 = (RadioGroup) findViewById(R.id.radioGroup2);
    radioGroup1.clearCheck();
    radioGroup2.clearCheck();

    // Attaches checkedChangeListener to radio group
    radioGroup1.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {

        @Override
        public void onCheckedChanged(RadioGroup group, int checkedId) {
            RadioButton your = (RadioButton) findViewById(radioButton2);

            if (your.isChecked()) {
                baseScore = baseScore + 1;
            } else {
            }

            int score = 1;
            if (your.isChecked()) score += 1;

            switch (score) {
                case R.id.radioButton2:
                    if (your.isChecked()) ;
                    score++;
                default:
                    break;
            }
        }
    });

    radioGroup2.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
        @Override
        public void onCheckedChanged(RadioGroup group, int checkedId) {
            RadioButton theyre = (RadioButton) findViewById(radioButton3);

            if (theyre.isChecked()) {
                baseScore = baseScore + 1;
            } else {
            }

            int score = 1;
            if (theyre.isChecked()) score += 1;

            switch (score) {
                case R.id.radioButton3:
                    if (theyre.isChecked()) ;
                    score++;
                default:
                    break;
            }
        }
    });

}

public void onSubmit(View v) {
    RadioButton rb1 = (RadioButton) radioGroup1.findViewById(radioGroup1.getCheckedRadioButtonId());
    RadioButton rb2 = (RadioButton) radioGroup2.findViewById(radioGroup2.getCheckedRadioButtonId());
    Toast.makeText(MainActivity.this, "Score: " + baseScore, Toast.LENGTH_SHORT).show();
}
}

И мой xml

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="wrap_content"
android:orientation="vertical"
android:padding="20dp">

<TextView
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_marginBottom="8dp"
    android:text="1) I think _______ cat is lovely."
    android:textColor="#000000" />

<RadioGroup
    android:id="@+id/radioGroup1"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:orientation="vertical">

    <RadioButton
        android:id="@+id/radioButton1"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:text="you're" />

    <RadioButton
        android:id="@+id/radioButton2"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:text="your" />
</RadioGroup>

<TextView
    android:layout_width="wrap_content"
    android:layout_height="wrap_content"
    android:layout_marginBottom="8dp"
    android:text="2) _______ going to shop tomorrow."
    android:textColor="#000000" />

<RadioGroup
    android:id="@+id/radioGroup2"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:orientation="vertical">

    <RadioButton
        android:id="@+id/radioButton3"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:text="They're" />

    <RadioButton
        android:id="@+id/radioButton4"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:text="Their" />
</RadioGroup>

<LinearLayout
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:orientation="horizontal">

    <Button
        android:id="@+id/submitBtn"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:onClick="onSubmit"
        android:text="Submit" />

</LinearLayout>

Answer 1

Вы только добавляете 1 к baseScore поэтому каждый раз, когда вы меняете состояние и получаете правильный ответ, вы добавляете еще один. Вам придется уменьшить baseScore, если вы проверите неправильный номер или просто установите его на 1

Answer 2

@ В вашем вопросе много ошибок в случае переключения, вы добавили точку с запятой в условие if, чего я не знаю почему, потому что в этом случае нет смысла иметь такое выражение if.

А на проверенных изменениях вы добавляете в переменную, которая уже имеет 1 балл.

Я бы посоветовал вам взять глобальную переменную, чтобы отслеживать результаты.