Android预订APP中ListView嵌套GridView的Checkbox数据重复问题咨询

Hey there! Let's break down how to fix your booking app's data duplication issue and properly handle selected venue + date data transfer—here's a step-by-step solution tailored to your ListView nested GridView setup:

First: Fix Data Duplication in Multi-Adapters #

Data duplication usually happens when adapters are reused across list items or your data references aren't properly isolated. Here's how to fix it:

1. Avoid Reusing GridView Adapter Instances
   Each ListView item (date) needs its own unique GridView adapter. If you share the same adapter across multiple date items, scrolling will cause data overlap as views get reused.

2. Isolate Data Sources
   Ensure each date's venue list is a separate instance (not a reference to the same list). If you're fetching venue data dynamically, return a new list for each date instead of reusing the same collection.

3. Strictly Use ViewHolder Pattern
   Both ListView and GridView adapters must implement ViewHolder to prevent incorrect view-data binding during scroll reuse.

Second: Manage Selected Venue + Date Data #

You need a centralized way to track which venues are selected for each date. Use a Map<String, List<String>> where:

• Key: The date string (e.g., "2024-05-20")
• Value: List of selected venue IDs (e.g., ["UH", "U1"])

Example Code Implementation

1. ListView Adapter (Date Items)

public class DateListAdapter extends BaseAdapter {
    private List<String> dateList;
    private Map<String, List<String>> selectedBookings;

    public DateListAdapter(List<String> dateList, Map<String, List<String>> selectedBookings) {
        this.dateList = dateList;
        this.selectedBookings = selectedBookings;
    }

    @Override
    public int getCount() {
        return dateList.size();
    }

    @Override
    public Object getItem(int position) {
        return dateList.get(position);
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        DateViewHolder holder;
        String currentDate = dateList.get(position);

        if (convertView == null) {
            convertView = LayoutInflater.from(parent.getContext())
                    .inflate(R.layout.item_date_list, parent, false);
            holder = new DateViewHolder();
            holder.dateText = convertView.findViewById(R.id.tv_date);
            holder.venueGrid = convertView.findViewById(R.id.gv_venues);
            convertView.setTag(holder);
        } else {
            holder = (DateViewHolder) convertView.getTag();
        }

        // Set date text
        holder.dateText.setText(currentDate);

        // Create a unique GridView adapter for this date
        VenueGridAdapter gridAdapter = new VenueGridAdapter(
                getVenuesForDate(currentDate), // Fetch venues for this specific date
                currentDate,
                selectedBookings
        );
        holder.venueGrid.setAdapter(gridAdapter);

        return convertView;
    }

    // Replace with your actual venue data fetch logic
    private List<String> getVenuesForDate(String date) {
        return Arrays.asList("UH", "U1", "U2", "U3");
    }

    static class DateViewHolder {
        TextView dateText;
        GridView venueGrid;
    }
}

2. GridView Adapter (Venue Items)

public class VenueGridAdapter extends BaseAdapter {
    private List<String> venueList;
    private String currentDate;
    private Map<String, List<String>> selectedBookings;

    public VenueGridAdapter(List<String> venueList, String currentDate, Map<String, List<String>> selectedBookings) {
        this.venueList = venueList;
        this.currentDate = currentDate;
        this.selectedBookings = selectedBookings;

        // Initialize empty list for the date if it doesn't exist
        if (!selectedBookings.containsKey(currentDate)) {
            selectedBookings.put(currentDate, new ArrayList<>());
        }
    }

    @Override
    public int getCount() {
        return venueList.size();
    }

    @Override
    public Object getItem(int position) {
        return venueList.get(position);
    }

    @Override
    public long getItemId(int position) {
        return position;
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        VenueViewHolder holder;
        String venueId = venueList.get(position);

        if (convertView == null) {
            convertView = LayoutInflater.from(parent.getContext())
                    .inflate(R.layout.item_venue_grid, parent, false);
            holder = new VenueViewHolder();
            holder.venueText = convertView.findViewById(R.id.tv_venue_id);
            holder.selectCheckbox = convertView.findViewById(R.id.cb_select);
            convertView.setTag(holder);
        } else {
            holder = (VenueViewHolder) convertView.getTag();
        }

        // Set venue ID and restore selected state
        holder.venueText.setText(venueId);
        holder.selectCheckbox.setChecked(
                selectedBookings.get(currentDate).contains(venueId)
        );

        // Update selected data on checkbox toggle
        holder.selectCheckbox.setOnCheckedChangeListener((buttonView, isChecked) -> {
            List<String> selectedVenues = selectedBookings.get(currentDate);
            if (isChecked) {
                if (!selectedVenues.contains(venueId)) {
                    selectedVenues.add(venueId);
                }
            } else {
                selectedVenues.remove(venueId);
            }
        });

        return convertView;
    }

    static class VenueViewHolder {
        TextView venueText;
        CheckBox selectCheckbox;
    }
}

3. Handle Delete Actions

// Initialize the selected bookings map in your Activity/Fragment
private Map<String, List<String>> selectedBookings = new HashMap<>();

// Delete selected venues for a specific date
public void deleteSelectedForDate(String targetDate) {
    selectedBookings.remove(targetDate);
    dateListAdapter.notifyDataSetChanged(); // Refresh ListView
}

// Delete all selected bookings across all dates
public void deleteAllSelected() {
    selectedBookings.clear();
    dateListAdapter.notifyDataSetChanged();
}

// Get selected data for API calls or further processing
public Map<String, List<String>> getSelectedBookings() {
    return new HashMap<>(selectedBookings); // Return a copy to avoid external modification
}

Key Takeaways #

• No adapter reuse: Each ListView date item gets its own GridView adapter to prevent data overlap.
• Centralized state tracking: The selectedBookings map keeps all selected venue-date pairs in one place, making deletion and data transfer trivial.
• State restoration: Always re-set checkbox states in GridView's getView to preserve selections during scroll.

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