0

我有一个游戏，它的main活动调用其他三个活动startActivityForResult，第一个 ( SignInActivity) 返回用户名，或者允许创建一个新的；第二个 ( LevelChooser) 用于getSharedPreferences查找具有该用户名的首选项文件，或创建一个新文件，显示用户到目前为止的进度（已解锁的级别、获得的星星），并允许用户选择玩任何未锁定的级别；如果在返回（通过）GameActivity之前成功完成关卡，第三个（ ）会更新用户的首选项文件。在我已覆盖将您返回到 SignInActivity；在它是被覆盖的，所以无论它如何发生，你都会回到那个。mainLevelChooserLevelChooseronBackPressedGameActivityonFinishLevelChooser

现在，十分之九这一切都完全按预期工作，但有时却并非如此：有时，不是看到用户的实际星星和级别，而是LevelChooser显示一组不正确且理论上不可能的值（例如，一个级别显示为锁定，但以三颗星完成）。如果您选择一个关卡然后在您第一次打开游戏时退出该关卡，这通常（但并非总是）发生：然后它将允许您玩显示为解锁的任何关卡，但GameActivity如果您完成级别，返回时显示相同的错误级别LevelChooser；或者，如果您退出LevelChooser并重新选择相同的用户名，它会恢复到预期的行为。我还设法通过反复启动关卡并退出它们来重现错误——如果你尝试了足够多的时间，它最终会出错。对于我自己的用户名（我认为，对于所有用户），错误信息总是相同的，即问题是间歇性的，但当它发生时，不是随机的。

我已经尝试过调试，但是由于某种原因，（a）问题只发生在我的手机上，而不是模拟器上，并且（b）在我的手机上调试（而不是运行）时，它要么正常工作，要么如果它出错，只是终止（AFAIR 甚至没有“X 已停止”对话框）而不是显示错误的关卡屏幕。我在调试中看到的唯一一件事是onCreate该LevelChooser活动有时会执行多次。

因为这个问题是间歇性的并且不能直接重现，我想知道我是否无意中假设某些异步过程已经/将会以及时、线性的方式完成，并且它通常（但不总是）有义务；否则，我认为我未能理解与 Activity 生命周期相关且重要的内容。否则，我很难过和猜测。

main Activity：

public class MainActivity extends AppCompatActivity {

public ImageView splash;
private int GET_USER_NAME_CODE = 0;
private int GET_LEVEL_CODE = 1;
private int PLAY_GAME_CODE = 2;
private String user;
private int level;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    // remove title
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    setContentView(R.layout.activity_splash_screen);

    splash = (ImageView) findViewById(R.id.splashView);

    splash.setOnClickListener(new View.OnClickListener() {
        @Override
        public void onClick(View view) {
            Intent sign_intent = new Intent(MainActivity.this, SignInActivity.class);
            startActivityForResult(sign_intent, GET_USER_NAME_CODE);
        }
    });

}

@Override
protected void onActivityResult(int requestCode, int resultCode, Intent data)
{
    super.onActivityResult(requestCode, resultCode, data);
    // check if the request code is same as what is passed
    if(requestCode==GET_USER_NAME_CODE)
    {
        user=data.getStringExtra("USER");
        Intent intent = new Intent(MainActivity.this, LevelChooser.class);
                intent.putExtra("user", user);
                startActivityForResult(intent, GET_LEVEL_CODE);
    }
    else {
        if(requestCode==GET_LEVEL_CODE) {
            level=data.getIntExtra("LEVEL", 0);
            if(level==-1) {
                Intent sign_intent = new Intent(MainActivity.this, SignInActivity.class);
                startActivityForResult(sign_intent, GET_USER_NAME_CODE);
            }
            else {
                Intent intent = new Intent(MainActivity.this, GameActivity.class);
                intent.putExtra("user", user);
                intent.putExtra("level", level);
                startActivityForResult(intent, PLAY_GAME_CODE);
            }
        }
        else {
            if(requestCode==PLAY_GAME_CODE) {
                user=data.getStringExtra("user");
               Intent intent = new Intent(MainActivity.this, LevelChooser.class);
                intent.putExtra("user", user);
                startActivityForResult(intent, GET_LEVEL_CODE);
            }
        }
    }
}

LevelChooser：

public class LevelChooser extends AppCompatActivity {

private String user;
private ImageView[] level;
private Boolean[] locked;
private int[] stars;
private SharedPreferences userprefs;
private SharedPreferences.Editor prefseditor;
private Boolean createNewPrefsFile = false;
private int tempResIdVisible;
private int tempResIdInvisible;
private ImageView tempView;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    setContentView(R.layout.levelchooser);
}

@Override
protected void onStart() {
    super.onStart();


        level = new ImageView[21];
        locked = new Boolean[21];
        stars = new int[21];

        user = getIntent().getStringExtra("user");
        userprefs = getSharedPreferences(user, MODE_PRIVATE);
        prefseditor = userprefs.edit();

        //level numbers for views etc start from 1 to match images etc
        level[0] = null;
        locked[0] = null;
        stars[0] = 0;

        locked[1] = false;
        level[1] = (ImageView) findViewById(R.id.level1);
        level[1].setOnClickListener(new LevelClickListener(level[1], 1));
        if (!userprefs.contains("stars1")) {
            createNewPrefsFile = true;
        }
        stars[1] = userprefs.getInt("stars1", 0);
        if (stars[1] != 0) {
            for (int j = 1; j < 4; j++) {
                tempResIdInvisible = getResources().getIdentifier("stars" + j + "_1", "id", getPackageName());
                tempView = (ImageView) findViewById(tempResIdInvisible);
                tempView.setVisibility(View.INVISIBLE);
            }
            tempResIdVisible = getResources().getIdentifier("stars" + stars[1] + "_1", "id", getPackageName());
            tempView = (ImageView) findViewById(tempResIdVisible);
            tempView.setVisibility(View.VISIBLE);
        }

        for (int i = 2; i < 21; i++) {
            locked[i] = userprefs.getBoolean("locked" + i, true);
            if (locked[i]) {
                tempResIdVisible = getResources().getIdentifier("padlock" + i, "id", getPackageName());
                tempResIdInvisible = getResources().getIdentifier("level" + i, "id", getPackageName());
            } else {
                tempResIdVisible = getResources().getIdentifier("level" + i, "id", getPackageName());
                tempResIdInvisible = getResources().getIdentifier("padlock" + i, "id", getPackageName());
                level[i] = (ImageView) findViewById(tempResIdVisible);
                level[i].setOnClickListener(new LevelClickListener(level[i], i));
            }
            tempView = (ImageView) findViewById(tempResIdVisible);
            tempView.setVisibility(View.VISIBLE);
            tempView = (ImageView) findViewById(tempResIdInvisible);
            tempView.setVisibility(View.INVISIBLE);

            stars[i] = userprefs.getInt("stars" + i, 0);
            if (stars[i] != 0) {
                for (int j = 1; j < 4; j++) {
                    tempResIdInvisible = getResources().getIdentifier("stars" + j + "_" + i, "id", getPackageName());
                    tempView = (ImageView) findViewById(tempResIdInvisible);
                    tempView.setVisibility(View.INVISIBLE);
                }
                tempResIdVisible = getResources().getIdentifier("stars" + stars[i] + "_" + i, "id", getPackageName());
                tempView = (ImageView) findViewById(tempResIdVisible);
                tempView.setVisibility(View.VISIBLE);
            }
        }

        if (createNewPrefsFile) {
            for (int i = 1; i < 21; i++) {
                prefseditor.putBoolean("locked" + i, locked[i]);
                prefseditor.putInt("stars" + i, stars[i]);
                prefseditor.commit();
            }
        }
}

GameActivity：

public class GameActivity extends AppCompatActivity implements TextToSpeech.OnInitListener {

//TTS Object
private TextToSpeech myTTS;
//TTS status check code
private int MY_DATA_CHECK_CODE = 0;
private int level;
private String user;
private PhonemeGroup levelGroup;

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    requestWindowFeature(Window.FEATURE_NO_TITLE);
    Toolbar toolbar = (Toolbar) findViewById(R.id.toolbar);
    setSupportActionBar(toolbar);

    level = getIntent().getIntExtra("level", 0);
    user = getIntent().getStringExtra("user");
    levelGroup = initializeLevels(level);

    Intent checkTTSIntent = new Intent();
    checkTTSIntent.setAction(TextToSpeech.Engine.ACTION_CHECK_TTS_DATA);
    startActivityForResult(checkTTSIntent, MY_DATA_CHECK_CODE);
}

@Override
public void finish() {
    Intent intent = new Intent();
    intent.putExtra("user", user);
    setResult(2, intent);

    super.finish();
}

@Override
public void onStop() {
    if (myTTS != null) {
        myTTS.stop();
    }
    super.onStop();
}

@Override
public void onDestroy() {
    if (myTTS != null) {
        myTTS.shutdown();
    }
    Button ok_button = (Button) findViewById(R.id.button);
    ok_button.setOnClickListener(null);
    ImageView tickImageView = (ImageView) findViewById(R.id.tickImageView);
    tickImageView.setOnClickListener(null);
    ImageView starsView = (ImageView) findViewById(R.id.starsImageView);
    starsView.setOnClickListener(null);

    super.onDestroy();

    unbindDrawables(findViewById(R.id.GameParentView));
    System.gc();
}

正确显示截图：没有错误的LevelChooser截图

上面选择Level 1然后退出后的截图：有错误的 LevelChooser 的屏幕截图

4

1 回答 1

0

事实证明，问题不在于getSharedPreferences()这样，而在于user返回的变量getIntent().getStringExtra()有时——在我发现难以确定或可靠再现的情况下——null就像在这个问题中一样。然后将该字符串用作 的参数getSharedPreferences()，从而为用户生成一组保存的结果null。每次问题再次出现时，都会显示这些结果（显示在问题的第二张图片中）。

解决方案——尽管它仍然没有解释为什么getStringExtra()应该返回——是通过将以下代码放入需要有效用户的两个活动中来null对附加内容进行空保护：onCreate

if(getIntent().getExtras()==null || getIntent().getStringExtra("user")==null) {
    Intent intent = new Intent();
    setResult(0, intent);
    finish();
    }

如果这是在加载新活动的布局之前执行的，它会返回到主活动并再次尝试（可以这么说），而用户不会看到任何问题。

于 2016-07-16T21:10:12.467 回答