通过构造器向多个Fragment传递对象是否属于Android开发不良实践？

是否属于不良实践？ #

当前实现不算完全不可用，但存在多个明确的问题，属于需要优化的不良实践：

1. 可变引用传递的安全风险：你全程传递的是同一个OrderItem对象的引用，任何地方调用setter修改属性，所有持有该引用的位置（比如RecyclerView适配器里的原始列表数据）都会同步变更，一旦出现非预期修改，排查成本极高，还可能出现列表展示数据和实际数据不一致的问题。
2. Fragment 实现不符合官方规范：你通过私有构造器给BottomSheetOrderFragment传对象、直接持有MainActivity的引用调用方法，这两个操作都有问题：
   • 系统触发Fragment重建（比如屏幕旋转、系统资源回收）时，会调用Fragment的无参构造方法，你通过构造器传入的OrderItem会丢失，大概率触发空指针崩溃，官方要求Fragment参数必须通过setArguments传递Parcelable序列化对象。
   • Fragment直接持有宿主Activity的强引用，会导致Fragment完全无法复用，还可能出现内存泄漏风险。
3. 传递链路冗余：Adapter→Activity→BottomSheet→Activity→ViewModel→Repo→DB的传递链路太长，后续如果要修改OrderItem的属性，整条链路上的代码都可能需要调整，维护成本高。

更优实现方案 #

可以根据项目规模选择对应的优化方案：

轻量优化方案（适合小项目） #

解决最核心的风险点即可：

• 传递对象时做深拷贝，所有修改操作都作用在副本上，不要修改原始数据源的对象，避免污染全局数据。
• 用官方Fragment Result API做Fragment和Activity的通信，不要直接持有Activity引用，示例代码如下：
  1. BottomSheet修改完数据返回结果：

  // BottomSheetOrderFragment内
  @OnClick(R.id.btn_edit_order)
  void onEditOrderBtnClick(){
      // 先copy修改后的对象再返回
      OrderItem modifiedCopy = new OrderItem(this.orderItem);
      Bundle result = new Bundle();
      result.putParcelable("modified_order", modifiedCopy);
      getParentFragmentManager().setFragmentResult("order_edit_finish", result);
      dismiss();
  }
  

  2. MainActivity监听结果处理后续逻辑：

  // MainActivity onCreate内注册监听
  getSupportFragmentManager().setFragmentResultListener("order_edit_finish", this, (requestKey, bundle) -> {
      OrderItem modifiedOrder = bundle.getParcelable("modified_order");
      // 直接传给ViewModel处理存库逻辑即可
      mainViewModel.saveOrder(modifiedOrder);
  });
  

统一状态管理方案（适合中大型项目） #

用绑定MainActivity生命周期的共享ViewModel做全局状态托管，彻底切断组件间的直接依赖：

• 所有和OrderItem相关的状态都存在共享ViewModel中，提供选中、修改、保存的统一操作方法。
• 流程简化为：
  • 点击RecyclerView条目：调用ViewModel的selectOrder(orderItem)方法，保存当前选中的订单
  • BottomSheet观察ViewModel中的选中订单数据，渲染页面
  • 用户修改订单：调用ViewModel的updateSelectedOrder(xxx)方法，修改ViewModel内的订单数据
  • 提交保存：ViewModel直接调用Repo层的方法完成数据库写入，不需要跨组件传递对象
    这种方案所有数据变更都收敛在ViewModel层，调试成本低，组件之间完全解耦，后续迭代维护成本极低。

进阶优化 #

如果项目架构分层明确，可以把数据库实体类和UI层使用的模型类分开定义，中间做字段映射，避免UI层意外修改不该操作的数据库字段，进一步提升数据安全性。

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