¿Qué quiero lograr?

  • Quiero guardar valores de x, y y score, para que cuando el usuario gire la pantalla, los valores se recuperen y no se regeneren.

¿Qué es lo que tengo?

  1. Probablemente el código correcto onSaveInstanceState que no funciona probablemente debido a que algunos @Override entran en conflicto con él

¿Qué necesito?

  • Algunas correcciones que me ayudarán a guardar valores de variables

Código

MainActivity.java (sin onSaveInstanceState)

import androidx.appcompat.app.AppCompatActivity;

import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.EditText;
import android.widget.TextView;
import java.util.Random;

public class MainActivity extends AppCompatActivity {

    private EditText textBox;
    private int z;
    public int score = 0;



    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        ((TextView) findViewById(R.id.scoreText)).setText(R.string.nullScore);
        textBox = findViewById(R.id.editText);
        textBox.requestFocus();
        setActual();
    }

    private void setActual() {
        Random random = new Random();
        int x = random.nextInt(10) + 1; //you get a random number between 0 and 9 then add 1 to get 1 - 10, no need to use array..
        int y = random.nextInt(10) + 1;
        z = x * y;
        String hintStr = x + " × " + y;
        ((TextView) findViewById(R.id.hintTextView)).setText(hintStr);
    }


            public void onCheck(View view) {

        int answer = Integer.parseInt(textBox.getText().toString());
        String res = "Incorrect";
        if (answer == z) {
            res = "Correct";
            score++;
            Log.d("TAG", "Correct."); // right answer
            textBox.setText("");
            setActual(); // let's regenerate values
        } else if (score == 0) {
            Log.d("TAG", "Incorrect."); // wrong answer
            textBox.setText("");
            setActual();
            ((TextView) findViewById(R.id.scoreText)).setText(R.string.nullScore);
        } else {
            Log.d("TAG", "Incorrect."); // wrong answer
            textBox.setText("");
            score--;
            setActual();
        }
        ((TextView) findViewById(R.id.scoreText)).setText("Score:" + ' ' + score);
        ((TextView) findViewById(R.id.result_text_view)).setText(res);
    }

}

MainActivity.java (con onSaveInstanceState)

import androidx.appcompat.app.AppCompatActivity;

import android.os.Bundle;
import android.util.Log;
import android.view.View;
import android.widget.EditText;
import android.widget.TextView;
import java.util.Random;

public class MainActivity extends AppCompatActivity {

    private EditText textBox;
    private int z;
    public int score = 0;
    private int x;
    private int y;

    public void onSaveInstanceState(Bundle savedInstanceState) {
        savedInstanceState.putInt("x", x);
        savedInstanceState.putInt("y", y);
        savedInstanceState.putInt("score", score);
    }


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        textBox = findViewById(R.id.editText);
        textBox.requestFocus();
        setActual();
        if (savedInstanceState != null) {
            x = savedInstanceState.getInt("x");
            y = savedInstanceState.getInt("y");
            score = savedInstanceState.getInt("score");
        }
    }

    private void setActual() {
        Random random = new Random();
        x = random.nextInt(10) + 1; //you get a random number between 0 and 9 then add 1 to get 1 - 10, no need to use array..
        y = random.nextInt(10) + 1;
        z = x * y;
        String hintStr = x + " × " + y;
        ((TextView) findViewById(R.id.hintTextView)).setText(hintStr);
    }


            public void onCheck(View view) {

        int answer = Integer.parseInt(textBox.getText().toString());
        String res = "Incorrect";
        if (answer == z) {
            res = "Correct";
            score++;
            Log.d("TAG", "Correct."); // right answer
            textBox.setText("");
            setActual(); // let's regenerate values
        } else if (score == 0) {
            Log.d("TAG", "Incorrect."); // wrong answer
            textBox.setText("");
            setActual();
            ((TextView) findViewById(R.id.scoreText)).setText(R.string.nullScore);
        } else {
            Log.d("TAG", "Incorrect."); // wrong answer
            textBox.setText("");
            score--;
            setActual();
        }
        ((TextView) findViewById(R.id.scoreText)).setText("Score:" + ' ' + score);
        ((TextView) findViewById(R.id.result_text_view)).setText(res);
    }

}

0
Kaper365 13 feb. 2020 a las 16:57

2 respuestas

La mejor respuesta

Debería considerar llamar a super.onSaveInstanceState ()

@Override
protected void onSaveInstanceState(@NonNull Bundle outState) {
    // save your state
    super.onSaveInstanceState(outState);
}

Además, para evitar recrear la actividad cada vez que gira la pantalla, puede manejar el cambio de configuración usted mismo al agregar esto a su declaración de Actividad en el manifiesto de la aplicación:

android:configChanges="keyboardHidden|orientation|screenSize"

Y anular:

@Override
public void onConfigurationChanged(Configuration newConfig) {
    super.onConfigurationChanged(newConfig);
}

Podría ahorrar tiempo para el usuario si su aplicación no necesita ser recreada después de la orientación o la pantalla Cambios de tamaño ...

1
avianey 13 feb. 2020 a las 15:29

¿Estás seguro de que los valores no se recuperan? Creo que el problema aquí es que siempre se llama a setActual () en onCreate () que genera los valores x e y y asigna el valor z a los x e y generados (¡incluso cuando se guardanInstanceState! = Null)

Mejor manera de manejar las cosas es

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    textBox = findViewById(R.id.editText);
    textBox.requestFocus();

    if (savedInstanceState != null) {
        x = savedInstanceState.getInt("x");
        y = savedInstanceState.getInt("y");
        score = savedInstanceState.getInt("score");
        z = x * y;
        String hintStr = x + " × " + y;
        ((TextView) findViewById(R.id.hintTextView)).setText(hintStr);
    } else { setActual(); }
}

También debe llamar a super.onSaveInstanceState (savedInstanceState) así

@Override
protected void onSaveInstanceState(Bundle savedInstanceState) {
    super.onSaveInstanceState(savedInstanceState);
    savedInstanceState.putInt("x", x);
    savedInstanceState.putInt("y", y);
    savedInstanceState.putInt("score", score);
}
0
Sara Talaat 13 feb. 2020 a las 15:33