Android根据RecyclerView列表状态控制Spinner触摸监听失效解决求助

Got it, let's break down why your current setup only works once, and get it fixed properly.

The Root Problem #

When you call v.setEnabled(false) on the Spinner, it stops receiving all touch events entirely. That means your OnTouchListener won't fire again—even after the RecyclerView list becomes empty. So once it's disabled, there's no way to trigger the code that would re-enable it.

Solution 1: Interact Instead of Disable (Quick Fix) #

Instead of disabling the Spinner outright, keep it enabled but intercept touch events when the list isn't empty. This way, the Spinner always receives touch events, so you can check the list state every time the user taps it.

Here's the adjusted code:

sGames.setOnTouchListener(new View.OnTouchListener() { 
    @Override 
    public boolean onTouch(View v, MotionEvent event) { 
        if (pokemonCardsGridAdapter.getItemCount() == 0) { 
            // List is empty: let the Spinner behave normally
            v.performClick();
            return false; // Allow the event to pass through to the Spinner
        } else { 
            // List isn't empty: block the Spinner and show a toast
            Toast.makeText(PokemonTeambuilderEditorActivity.this, "The list is full", Toast.LENGTH_SHORT).show();
            return true; // Intercept the event so the Spinner doesn't respond
        } 
    } 
});

If you want the Spinner to look disabled when the list is full, add a visual cue like adjusting its alpha:

// Inside the else block
v.setAlpha(0.5f);
// Inside the if block
v.setAlpha(1.0f);

Solution 2: Data-Driven State Updates (Better Practice) #

A more reliable approach is to tie the Spinner's state directly to the RecyclerView adapter's data changes. This way, the Spinner automatically enables/disables whenever the list's item count changes—no need to rely on touch events.

First, register an observer on your RecyclerView adapter to listen for data changes:

// Add this right after you initialize pokemonCardsGridAdapter
pokemonCardsGridAdapter.registerAdapterDataObserver(new RecyclerView.AdapterDataObserver() {
    @Override
    public void onChanged() {
        super.onChanged();
        updateSpinnerState();
    }

    @Override
    public void onItemRangeInserted(int positionStart, int itemCount) {
        super.onItemRangeInserted(positionStart, itemCount);
        updateSpinnerState();
    }

    @Override
    public void onItemRangeRemoved(int positionStart, int itemCount) {
        super.onItemRangeRemoved(positionStart, itemCount);
        updateSpinnerState();
    }
});

Then create a helper method to update the Spinner's state:

private void updateSpinnerState() {
    boolean isListEmpty = pokemonCardsGridAdapter.getItemCount() == 0;
    sGames.setEnabled(isListEmpty);
    // Optional visual feedback
    sGames.setAlpha(isListEmpty ? 1.0f : 0.5f);
    
    // Add toast feedback only when user taps a disabled Spinner
    if (!isListEmpty) {
        sGames.setOnTouchListener((v, event) -> {
            if (event.getAction() == MotionEvent.ACTION_DOWN) {
                Toast.makeText(PokemonTeambuilderEditorActivity.this, "The list is full", Toast.LENGTH_SHORT).show();
            }
            return false;
        });
    } else {
        sGames.setOnTouchListener(null); // Remove the listener when enabled
    }
}

This approach is cleaner because it's data-driven: whenever items are added or removed from the list, the Spinner's state updates automatically. No more relying on touch events to trigger state changes.

Final Recommendation #

Go with Solution 2—it's more maintainable and aligns with Android's best practices for UI state management. It ensures the Spinner's state always reflects the current list state, even if the list changes without user interaction (like from a background task).

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