如何从Activity A点击按钮向Activity B的RecyclerView添加食材数据？

Hey there! Let's work through this problem step by step to get those ingredients added to your Shopping List RecyclerView smoothly. I'll break down the key fixes and best practices to make this work reliably.

You mentioned you have a menu button, but let's confirm the implementation is complete so the click triggers your ingredient loop properly:

First, create a menu resource file (if you haven't already) at res/menu/recipe_menu.xml:

<menu xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto">
    <item
        android:id="@+id/action_add_to_shopping_list"
        android:title="Add to Shopping List"
        app:showAsAction="ifRoom"/>
</menu>

Then, wire this menu into Activity A and handle the button click:

@Override
public boolean onCreateOptionsMenu(Menu menu) {
    // Inflate the menu resource
    getMenuInflater().inflate(R.menu.recipe_menu, menu);
    return true;
}

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    if (item.getItemId() == R.id.action_add_to_shopping_list) {
        // Run your existing loop to add ingredients
        for (int x = 0; x < recipeIngredients.length; x++) {
            ShoppingList.insertItem(recipeIngredients[x]);
        }
        // Optional: Give the user feedback
        Toast.makeText(this, "Ingredients added to shopping list!", Toast.LENGTH_SHORT).show();
        return true;
    }
    return super.onOptionsItemSelected(item);
}

The biggest issue here is likely that your insertItem method isn't properly updating the RecyclerView's adapter, or you're relying on static data that doesn't persist across Activity lifecycle changes. Let's fix that:

Option 1: Basic Fix (Using Static List with Lifecycle Checks) #

If you want to stick with your current static approach, update Activity B to ensure the adapter refreshes when data changes:

public class ShoppingList extends AppCompatActivity {
    private RecyclerView recyclerView;
    private ShoppingListAdapter adapter;
    // Static list to hold items (note: this can cause memory leaks if not handled carefully)
    public static ArrayList<String> shoppingItems = new ArrayList<>();

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_shopping_list);
        
        // Initialize RecyclerView and adapter
        recyclerView = findViewById(R.id.shopping_recycler);
        adapter = new ShoppingListAdapter(shoppingItems);
        recyclerView.setLayoutManager(new LinearLayoutManager(this));
        recyclerView.setAdapter(adapter);
    }

    // Updated insertItem method
    public static void insertItem(String ingredient) {
        shoppingItems.add(ingredient);
        // Optional: Prevent duplicates
        // if (!shoppingItems.contains(ingredient)) shoppingItems.add(ingredient);
    }

    // Refresh the list when the Shopping List screen is opened/resumed
    @Override
    protected void onResume() {
        super.onResume();
        if (adapter != null) {
            adapter.notifyDataSetChanged();
        }
    }
}

Option 2: Better Approach (Use ViewModel to Avoid Memory Leaks) #

Static lists can cause memory leaks if Activity B is destroyed. A ViewModel is designed to safely hold data across lifecycle changes. Here's how to implement it:

First, create a shared ViewModel:

import androidx.lifecycle.ViewModel;
import java.util.ArrayList;

public class ShoppingListViewModel extends ViewModel {
    private ArrayList<String> shoppingItems = new ArrayList<>();

    public void addIngredient(String ingredient) {
        // Add a check to avoid duplicates
        if (!shoppingItems.contains(ingredient)) {
            shoppingItems.add(ingredient);
        }
    }

    public ArrayList<String> getShoppingItems() {
        return shoppingItems;
    }
}

Update Activity A's button click to use the ViewModel:

@Override
public boolean onOptionsItemSelected(MenuItem item) {
    if (item.getItemId() == R.id.action_add_to_shopping_list) {
        ShoppingListViewModel viewModel = new ViewModelProvider(this).get(ShoppingListViewModel.class);
        for (int x = 0; x < recipeIngredients.length; x++) {
            viewModel.addIngredient(recipeIngredients[x]);
        }
        Toast.makeText(this, "Ingredients added!", Toast.LENGTH_SHORT).show();
        return true;
    }
    return super.onOptionsItemSelected(item);
}

Then, update Activity B to use the ViewModel:

public class ShoppingList extends AppCompatActivity {
    private RecyclerView recyclerView;
    private ShoppingListAdapter adapter;
    private ShoppingListViewModel viewModel;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_shopping_list);
        
        recyclerView = findViewById(R.id.shopping_recycler);
        viewModel = new ViewModelProvider(this).get(ShoppingListViewModel.class);
        
        // Initialize adapter with data from ViewModel
        adapter = new ShoppingListAdapter(viewModel.getShoppingItems());
        recyclerView.setLayoutManager(new LinearLayoutManager(this));
        recyclerView.setAdapter(adapter);
    }

    @Override
    protected void onResume() {
        super.onResume();
        adapter.notifyDataSetChanged();
    }
}

• Use Efficient Adapter Updates: Instead of notifyDataSetChanged(), use notifyItemInserted(shoppingItems.size() - 1) in your insertItem method for smoother animations.
• Handle Duplicates: Add a check before adding ingredients to avoid cluttering the shopping list with duplicates (as shown in the ViewModel example).
• Consider LiveData for Real-Time Updates: If you want the shopping list to update automatically while Activity B is open, modify the ViewModel to use MutableLiveData and observe changes in Activity B.

内容的提问来源于stack exchange，提问作者wilrus