java - 在 Android 中使用 static 关键字保存 Tic-tac-toe 棋盘的状态

Question

昨天我问了this网络上关于如何在关闭和打开 fragment 后恢复简单的井字游戏的问题。正如我在问题中所说:

I'm learning Android development by creating a test project: Tic-tac-toe. My Tic-tac-toe app starts with a Main Menu activity with a Button that says New Game. After clicking New Game, an activity with a fragment containing a Tic-Tac-Toe game (in a GridLayout) launches. The user can play the game, but when they go back to the Main Menu, the game state is not saved.

I want to change this so that when the user goes back to the Main Menu, they will see a new Button called "Continue" (in addition to the "New Game" Button). Once the user clicks "Continue", the game they were playing before continues. If the click the "New Game" button, a new game will be launched like before.

通过两种不同的方法提出了三个答案:利用 SharedPreferences 或使用 startActivityForResult()。这两种方法似乎都工作得很好，但我偶然发现了一个更干净的解决方案，只需将 Tic-Tac-Toe 对象声明为静态:

public static TicTacToe t = new TicTacToe();

这似乎可以完成我需要的一切，因为现在无论我创建或销毁 fragment 的频率如何，我都只会有一个 t 实例。因此，在方向改变时，返回主菜单并再次打开游戏，甚至在关闭并重新启动应用程序时，游戏状态保持不变，就像我想要的那样。然而，由于没有人提出这个解决方案，而且我是一个Android开发新手，我必须问:使用这个方法有什么问题吗？

这是我的完整代码，仅做了一个更改:

我的MainMenu类:

public class MainMenu extends ActionBarActivity {

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


    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        getMenuInflater().inflate(R.menu.main_menu, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        int id = item.getItemId();
        if (id == R.id.action_settings) {
            return true;
        }
        return super.onOptionsItemSelected(item);
    }

    public void startGame(View view) {
        Intent intent = new Intent(this,MainGame.class);
        startActivity(intent);
    }

}

带有相应的xml文件activity_main_game:

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <Button
        android:id="@+id/button1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="@string/new_game"
        android:onClick="startGame"

        />

</RelativeLayout>

现在我的 MainGame.class 看起来像这样:

public class MainGame extends ActionBarActivity {

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main_game);
        if (savedInstanceState == null) {
            getSupportFragmentManager().beginTransaction()
                    .add(R.id.container, new BoardFragment()).commit();
        }
    }

    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        getMenuInflater().inflate(R.menu.main_game, menu);
        return true;
    }

    @Override
    public boolean onOptionsItemSelected(MenuItem item) {
        int id = item.getItemId();
        if (id == R.id.action_settings) {
            return true;
        }
        return super.onOptionsItemSelected(item);
    }

    public static class BoardFragment extends Fragment {

        public static TicTacToe t = new TicTacToe();  //A class I wrote that launches a simple TicTacToe game

        @Override
        public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {

            if (savedInstanceState != null) {
                t = new TicTacToe(savedInstanceState.getInt("TicTacToeData"),
                );
            }

    //Graphics stuff here: variable rootView which contains the TicTacToe grid is defined
    //and onClickListeners are added to the ImageViews in the GridLayout which makes corresponding
    //changes to t.

            return rootView;
        }

        @Override
        public void onSaveInstanceState(Bundle savedInstanceState) {
            savedInstanceState.putInt("TicTacToeData", t.getGameData());
        }

    }
}

Answer 1

is there something wrong with using this method?

首先，一般来说，应谨慎使用静态数据成员，因为它们代表故意的内存泄漏。此 TicTacToe 对象中的任何内容都无法进行垃圾回收，并且此 TicTacToe 对象可以间接访问的任何内容都无法进行垃圾回收。在经典的 Java 开发中，static 数据成员被认为是较差的形式。在 Android 中，它们往往会被更多地使用，但最好仅由经验丰富的开发人员使用。

与此相关的是为工作选择正确的工具。我们可以通过斩首来治愈癌症，因为肿瘤会停止生长。然而，这集中体现了“治疗方法比疾病本身更糟糕”。在这种情况下，针对特定问题有更好的解决方案，例如应考虑处理配置更改(例如屏幕旋转)来代替静态数据成员。

其次，静态数据成员不会永远存在。当您的流程消失时，它们就会消失。一旦您不再位于前台，您的进程就可以随时消失，并且当您的进程终止时，您的 TicTacToe 对象会噗。而且，由于您的进程可以在进入后台后几秒钟内终止(取决于设备上发生的其他情况)，因此您的数据模型面临风险。

这又回到了用户的期望。

如果用户期望“继续”选项在任何时候都可用，那么虽然您可以使用 TicTacToe 作为某种进程内缓存，但真实的数据模型需要持久数据存储:SharedPreferences、数据库或任意文件。

在光谱的另一端，如果您不关心用户的游戏可能会失败，例如，当他们接听电话并且您进入后台时，那么就说您的数据模型纯粹是在内存当然是一个选择。

还有“中间立场”方法(例如，保存实例状态)。

您的其他问题中关注 startActivityForResult() 的解决方案恕我直言，是不合适的。