菜品列表刷新闪退修复记录

记录日期：2026-08-31

项目仓库：/Users/liang/AndroidStudioProjects/CusumptionMachine

实施分支：fix-menu-dish-list-refresh-crash

问题原因

主页面下拉刷新时会先清空 mGetDishList，而 DishListAdapter 与页面共同持有该列表。RecyclerView 中仍存在旧的 ViewHolder 和预取位置时，数据数量已经变为 0，可能触发 Invalid view holder adapter position；用户点击尚未消失的旧菜品时，也可能使用旧位置访问空列表并触发 IndexOutOfBoundsException。网络响应越慢，界面与数据不同步的时间窗口越长，因此更容易触发，但网络慢不是根本原因。

修复方案

• DishListAdapter 在主页面初始化时只创建并绑定一次，不再在每次展示菜品时重新创建。
• Adapter 持有独立的菜品数据副本，页面不能再通过修改外部列表直接改变 Adapter 的数据。
• 新增 updateData() 作为唯一列表更新入口，复制新数据后调用 notifyDataSetChanged()，确保 RecyclerView 状态与条目数量同步。
• 下拉刷新开始时保留当前菜品和分类缓存，接口成功后再整体提交新数据；接口失败时继续显示旧菜品。
• 点击事件先校验 Adapter 位置是否有效，再直接回传当前 DishBean，不再由 MainActivity 使用旧位置二次访问外部列表。
• 接口确认菜品为空时，通过 Adapter 更新入口清空界面数据。

影响文件

• app/src/main/java/com/cpt/cusumption/adapter/DishListAdapter.java
• app/src/main/java/com/cpt/cusumption/activity/MainActivity.java

自动验证

• 执行 JAVA_HOME=/Users/liang/Library/Java/JavaVirtualMachines/corretto-1.8.0_482/Contents/Home bash gradlew :app:compileDebugJavaWithJavac：BUILD SUCCESSFUL。
• 执行 JAVA_HOME=/Users/liang/Library/Java/JavaVirtualMachines/corretto-1.8.0_482/Contents/Home bash gradlew :app:testDebugUnitTest :app:assembleDebug：BUILD SUCCESSFUL。
• Debug 单元测试共 22 个，跳过 0、失败 0、错误 0。
• Debug APK：app/build/outputs/apk/debug/app-debug.apk。
• APK SHA-256：984d4343e1c5297fe0296bd9dc08b2fb2dd3c4aa8343eddfc99f5a4cb0364289。

真机验收要求

• 在菜品列表持续滚动时执行下拉刷新，确认刷新过程中和刷新完成后均不闪退。
• 使用慢网或延迟接口环境，下拉刷新后连续点击当前菜品，确认点击有效且不访问空列表。
• 模拟接口失败，确认刷新结束后旧菜品仍可正常显示和点击。
• 模拟接口返回空菜品，确认列表隐藏、空状态显示且无 RecyclerView 位置异常。
• 连续切换菜品分类、营养筛选和排序，确认列表内容与点击菜品一致。

交付边界

• 本次只调整主页面菜品 RecyclerView 的 Adapter 生命周期、数据更新和点击安全逻辑。
• 支付、购物车结算、菜品接口协议、筛选规则和其他页面没有改变。
• 未安装 APK，未执行 Git 暂存、提交或推送。