我应该在 onSaveInstanceState 中保留数据对象吗

Question

我在我的安卓应用程序中使用greenDAO 来显示RecyclerView 中的对象列表。我有一个 RecyclerView.Adapter 的子类，它接受一个作为 greenDAO 实体的对象列表。

我在onCreate做的是：

1. 为我的列表创建一个传递null 的适配器实例。这只是为了让下面的RecyclerView 知道适配器。
2. 使用布局和适配器初始化RecyclerView。
3. 调用一个使用 greenDAO 异步查询数据的方法，并在成功后使用实际的对象列表更新适配器，以便显示它们。

这是相关代码：

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

    ...

    mLayoutManager = new LinearLayoutManager(this, LinearLayoutManager.VERTICAL, false);
    mListAdapter = new MyRecyclerAdapter(null);
    mList.setHasFixedSize(true);
    mList.setLayoutManager(mLayoutManager);
    mList.setAdapter(mListAdapter);
    refreshItems();
}

public void refreshItems()
{
    AsyncSession asyncSession = ((App)getApplication()).getDaoSession().startAsyncSession();
    asyncSession.setListenerMainThread(new AsyncOperationListener()
    {
        @Override
        public void onAsyncOperationCompleted(final AsyncOperation operation)
        {
            if (operation.isCompletedSucessfully())
                mListAdapter.setItems((List<Item>) operation.getResult());
        }
    });

    asyncSession.loadAll(Item.class);
}

这很好用。现在我注意到，当然，每次我旋转活动或从另一个活动返回到它时，都会调用通过 greenDAO 查询数据库的方法。这很清楚，因为我是从onCreate 调用该方法的。

我的问题是：最好的做法是像我这样做（每次都重新查询 DAO）还是让我的对象可打包并将我在onSaveInstanceState 中的列表保存在onRestore 中而不是重新查询 DAO？

Answer 1

您所做的是完全有效的，您不需要将查询到的数据保存在onSaveInstanceState()，使用内存缓存或任何其他优化（即使 GreenDAO 没有内部缓存）。

事实上，您完全可以，因为您异步执行查询 - GreenDAO 的创建者声称在大多数情况下查询可以在 UI 线程上执行（我很难同意）。

我还建议您在onStart()而不是onCreate()中执行数据查询。我个人认为 onCreate() 应该仅用于您将在构造函数中执行的操作（例如字段初始化）。在onStart() 中执行此查询的另一个原因是，如果用户离开您的应用程序很长时间然后又回到它，数据可能会过时（例如，由于SyncAdapter 的后台同步）并且您会想要刷新它。

您可能要添加的最后一部分是“数据更改通知”。如果您查询和显示给用户的数据可以在没有用户交互的情况下更改（例如，由于SyncAdapter 的后台同步），您将希望此机制到位。这个概念很简单 - Activity 在 onCreate() 中注册有关数据更改的通知，如果收到通知，您执行重新查询以确保用户看到最新数据。

我不能说以上是“最佳实践”，但它们是行之有效的良好实践。

惰性列表：

正如@pskink 在他的评论中所建议的，您也可以使用LazyList。但请注意，它并不能排除对数据异步查询的需要。 LazyList 的使用允许您像往常一样执行查询，但以按需方式将结果加载到内存中。如果您希望查询产生大量数据，这可能很有用。

然而，在我看来，只有在观察到实际性能问题时才应该优化代码。因此，除非您提前知道特定查询会产生数千个结果，否则我说您不需要LazyList。