使用 AsyncTask 将数据从 Activity 传递到 Fragment - Android

Question

我正在尝试将 ArrayList 从 MainActivity 中的 AsyncTask 传递到一个片段，但我得到了一个NullPointerException用于调用CategoryAdapter.getItemCount()即使我在 BroadCastReceiver 调用之后传递数组。

我究竟做错了什么？

主要活动

 class GetBooksAsync extends AsyncTask<Void, Void, Void> {
        LocalBroadcastManager manager = LocalBroadcastManager.getInstance(getApplicationContext());
    @Override
    protected Void doInBackground(Void... voids) {
        for (ECategories category : ECategories.values()) {
            try {
                categories.add(new Category(category.toString(), apiClient.getBooks(category)));
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        return null;
    }
    @Override
    protected void onPostExecute(Void aVoid) {
        super.onPostExecute(aVoid);
        Intent intent = new Intent("com.android.mainapp");
        intent.putExtra("categories", categories);
        manager.sendBroadcast(intent);
        replaceFragment(new HomeFragment());
    }
}

主页片段

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container,
                         Bundle savedInstanceState) {
    initBroadCastReceiver();
    categoryAdapter = new CategoryAdapter(categories,getContext());
    View view = inflater.inflate(R.layout.fragment_home, container, false);
    recyclerView = view.findViewById(R.id.parent_rv);
    recyclerView.setAdapter(categoryAdapter);
    recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
    categoryAdapter.notifyDataSetChanged();
    return view;
}

private void initBroadCastReceiver() {
    manager = LocalBroadcastManager.getInstance(getContext());
    MyBroadCastReceiver receiver = new MyBroadCastReceiver();
    IntentFilter filter = new IntentFilter();
    filter.addAction("com.android.mainapp");
    manager.registerReceiver(receiver,filter);
}

class MyBroadCastReceiver extends BroadcastReceiver {

    @Override
    public void onReceive(Context context, Intent intent) {
        //get the categories from the intent
        categories = new ArrayList<Category>();
        categories = (ArrayList<Category>) intent.getSerializableExtra("categories");
    }
}

我也试过从 OnReceive 方法附加 recyclerView，但它没有被附加。 先感谢您！

Answer 1

• Category 是否序列化？
• 可以把BroadcastReceiver当作内部的class，然后Adpater收到数据的时候更新数据，因为代码运行速度很快，不需要注册监听，马上处理。

Answer 2

我想您将数据从MainActivity传递到HomeFragment的方式不正确。

你期待什么

1. 调用MainActivity#GetBooksAsync
2. 等到onPostExecute被调用
3. HomeFragment准备接收广播消息，然后更新UI
4. 将消息从MainActivity广播到片段

这里发生了什么

1. 调用MainActivity#GetBooksAsync
2. 等到onPostExecute被调用
3. 从MainActivity广播消息。 没有接收者可以接收此消息！
4. HomeFragment准备接收广播消息，然后更新UI

---

那你应该如何传递数据呢？

有几种方法。

1. 像您所做的那样在 UI 组件之间广播数据。 但是您需要注意组件的生命周期。 也就是说，当您广播数据时，接收器必须已经初始化并且 UI 组件处于活动状态。

2. 建一个 singleton class 来存放数据。 您的活动和片段将 singleton class 视为数据存储的公共位置。

3. 如果数据大小足够小，请使用Intent和额外属性来传递数据。

4. 使用实时数据。 我相信这是社区推荐的最现代的方式。 虽然我不确定它是如何工作的。

---

为了验证这是一个生命周期问题，

您可以尝试在发送广播消息之前添加延迟。

class GetBooksAsync extends AsyncTask<Void, Void, Void> {
...
    @Override
    protected void onPostExecute(Void aVoid) {
        super.onPostExecute(aVoid);
        Intent intent = new Intent("com.android.mainapp");
        intent.putExtra("categories", categories);
        
        TimerTask task = new TimerTask() {
            @Override
            public void run() {
                manager.sendBroadcast(intent);
            }
        };

        Timer timer = new Timer();
        timer.schedule(task, 5 * 1000); // Delay the broadcast after 5 seconds

        
        replaceFragment(new HomeFragment());
    }

Answer 3

你的 Adapter 应该这样写。

    class CategoryAdapter extends RecyclerView.Adapter<CategoryAdapter.VHolder>{
        
        private ArrayList<Category> list = new ArrayList<Category>();

        public void setList(ArrayList<Category> list) {
            this.list = list;
            notifyDataSetChanged();
        }

        public CategoryAdapter(Context context) {
            // Do not pass a list in the constructor, because the list may be empty
        }
         
        class VHolder extends RecyclerView.ViewHolder {

            public VHolder(@NonNull View itemView) {
                super(itemView);
            }
        }
        
        ......
}

您的片段应该有一个 BroadcastReceiver 的全局适配器来更新数据

public class Test extends Fragment {

    // Create a global Adapter for BroadcastReceiver to call and update data
    private CategoryAdapter adapter;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState) {
        adapter = new CategoryAdapter(getContext());
        initBroadCastReceiver();
        View view = inflater.inflate(R.layout.fragment_home, container, false);
        recyclerView = view.findViewById(R.id.parent_rv);
        recyclerView.setAdapter(categoryAdapter);
        recyclerView.setLayoutManager(new LinearLayoutManager(getContext()));
        return view;
    }

    private void initBroadCastReceiver() {
        manager = LocalBroadcastManager.getInstance(getContext());
        MyBroadCastReceiver receiver = new MyBroadCastReceiver();
        IntentFilter filter = new IntentFilter();
        filter.addAction("com.android.mainapp");
        manager.registerReceiver(receiver,filter);
    }

    class MyBroadCastReceiver extends BroadcastReceiver {

        @Override
        public void onReceive(Context context, Intent intent) {
            //get the categories from the intent
            ArrayList<Category> categories = (ArrayList<Category>) intent.getSerializableExtra("categories");
            adapter.setList(categories);
        }
    }
}

Answer 4

我认为您的代码存在几个问题：

1. 您的任务在与 UIThread（调度任务并处理结果）不同的线程中运行。 这意味着它很可能在不同的处理器/内核上运行。 处理后的值（例如您的集合）缓存在处理器中，并且在执行后的某个时候将数据写回 RAM。 但这可能会在调用onPostExecute方法之后发生，该方法也会将集合带到另一个处理器缓存中。 但是，当在集合从任务返回到 RAM 之前完成此操作时，它仍然是空的。 这就是所谓的竞争条件。

现在有几种方法可以解决这个问题。 最简单的一种是使用Collections.synchronizedList(categories)这可以防止处理器缓存列表值并始终将其返回到 RAM（或使用在所有处理器/内核之间共享的 L3 缓存）。

2. 我不确定你传递给集合的到底是什么。 意图（及其数据）需要可序列化，而您添加到集合中的内容可能不可序列化。

然后我会使用AsyncTask参数：

class GetBooksAsync extends AsyncTask<ECategories, Void, Collection<Category>> {
        LocalBroadcastManager manager = LocalBroadcastManager.getInstance(getApplicationContext());
    @Override
    protected Void doInBackground(ECategories... eCategories) {
        Collection<Category> categories = [whatever you want to use];
        for (ECategories category : eCategories) {
            try {
                categories.add(new Category(category.toString(), apiClient.getBooks(category)));
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        return categories;
    }
    @Override
    protected void onPostExecute(Collection<Category> categories) {
        super.onPostExecute(categories);
        Intent intent = new Intent("com.android.mainapp");
        intent.putExtra("categories", categories);
        manager.sendBroadcast(intent);
        replaceFragment(new HomeFragment());
    }
}

并注意AsyncTask和LocalBroadcastManager已弃用。