MVP模式临时数据存储：API加载场景下的变量存放及状态跟踪疑问

嘿，这个MVP模式下的变量存储问题我之前也纠结过，给你分享下我的实践思路：

关于详情页ID的存储位置 #

应该先存在View（也就是你的Fragment）里，再传递给Presenter，原因有这几点：

• 这个ID是通过Fragment的newInstance方法传入的Bundle来的，而Bundle属于Android组件（Fragment）的一部分，View层负责处理和Android生命周期相关的参数接收，Presenter不应该直接依赖Android的类（比如Bundle），这样能保持Presenter的独立性，方便单元测试。
• 具体做法：在Fragment的onCreate或者onViewCreated中从arguments取出ID，然后调用Presenter的方法（比如presenter.setDetailId(id)）把ID传递过去，之后Presenter就可以用这个ID去调用API了。

关于数据加载状态的跟踪变量 #

这个变量必须放在Presenter里，理由很简单：

• MVP里，Presenter是负责业务逻辑和数据状态管理的核心，View只需要负责UI展示。如果把加载状态放在View里，当Fragment因为屏幕旋转等原因重建时，状态会丢失，导致重复请求API；而Presenter如果是和View的生命周期正确绑定（比如用ViewModel持有Presenter，或者让Presenter在配置变化时保留），状态就能得以保留。
• 举个例子，Presenter里可以定义一个private var isDataLoaded = false变量，当调用API成功后把它设为true。之后在loadDetailData方法里先判断这个变量，如果已经加载过就直接返回缓存的数据，避免重复请求。

给你贴个简单的代码示例参考 #

Fragment（View层） #

class DetailFragment : Fragment(), DetailView {
    private lateinit var presenter: DetailPresenter
    private var detailId: Int = 0

    companion object {
        fun newInstance(id: Int): DetailFragment {
            val fragment = DetailFragment()
            val bundle = Bundle().apply {
                putInt("KEY_DETAIL_ID", id)
            }
            fragment.arguments = bundle
            return fragment
        }
    }

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        // 从arguments取出ID
        detailId = arguments?.getInt("KEY_DETAIL_ID") ?: 0
        // 初始化Presenter并传递ID
        presenter = DetailPresenter(this)
        presenter.setDetailId(detailId)
    }

    override fun onStart() {
        super.onStart()
        // 通知Presenter加载数据
        presenter.loadDetailData()
    }

    // 实现View层的回调方法，用于展示数据
    override fun showDetailData(data: DetailModel) {
        // 更新UI组件，比如TextView、ImageView等
    }

    override fun showError(message: String) {
        // 展示错误提示，比如Toast或者Snackbar
    }
}

Presenter层 #

class DetailPresenter(private val view: DetailView) {
    private var detailId: Int = 0
    private var isDataLoaded = false
    private var cachedData: DetailModel? = null
    // 假设你有一个API服务实例
    private val apiService = ApiClient.getService()

    fun setDetailId(id: Int) {
        this.detailId = id
    }

    fun loadDetailData() {
        if (isDataLoaded && cachedData != null) {
            // 数据已加载，直接用缓存更新UI
            cachedData?.let { view.showDetailData(it) }
            return
        }

        // 调用API加载数据
        apiService.getDetail(detailId).enqueue(object : Callback<DetailModel> {
            override fun onResponse(call: Call<DetailModel>, response: Response<DetailModel>) {
                response.body()?.let { data ->
                    isDataLoaded = true
                    cachedData = data
                    view.showDetailData(data)
                } ?: view.showError("数据加载失败")
            }

            override fun onFailure(call: Call<DetailModel>, t: Throwable) {
                view.showError(t.message ?: "网络请求失败")
            }
        })
    }
}

// View层的接口，定义Presenter需要通知View的方法
interface DetailView {
    fun showDetailData(data: DetailModel)
    fun showError(message: String)
}

额外的小建议 #

• 如果你担心Presenter在配置变化时被销毁，可以结合ViewModel来持有Presenter，这样Presenter的状态就能在屏幕旋转等场景下保留，避免重复请求。
• 永远不要让Presenter持有Android的上下文或者View的强引用，避免内存泄漏，最好用弱引用或者让View在销毁时解除绑定。

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