zhouhuandev / basedemo Goto Github PK

这个框架内容很丰富，周末简单体验了一下，挺不错的，学到了一些东西，但也存在一些bug说一说（因为我这边clone下来的已经魔改太多了，就不pr了）

• lib_base#view#BaseActivity#initCommonView

if (enableAllowFullScreen()) {
    window.setFlags(
        WindowManager.LayoutParams.FLAG_FULLSCREEN,
        WindowManager.LayoutParams.FLAG_FULLSCREEN
    )
    requestWindowFeature(Window.FEATURE_NO_TITLE);
}

这段允许全屏的代码放在setContentView(R.layout.activity_root)之后会报错，需要放前面

• lib_base#view#BaseActivity(BaseFragment)#initToolbar

  ivToolbarRight = view.findViewById(R.id.iv_toolbar_right)

两个基类都缺失了上一行代码中的视图绑定，缺失该行将导致toolbar右侧图标和监听无法加载

• memory leak

fragment在进行切换和销毁时存在内存泄漏，目前比较严重的一个是在fragment中使用toolbar，本质是绑定activity的actionBar，fragment销毁并不会自动解绑，还是需要手动解绑。我重写了BaseFragment#onDestroyView：

override fun onDestroyView() {
    super.onDestroyView()
    mActivity.setSupportActionBar(null)
}

这能避免小黄鸟弹窗，但日志上还是会存在可能泄漏的提示，这我就不清楚为什么了

• Binding instance cleared.

  好像来回切换的时候requireBinding会报这个exception，我有空再去研究一下为什么

java.lang.IllegalStateException: Binding instance cleared.

错误，目前我是这么处理的，但是感觉存在内存泄露风险

注释位置打开就会报以下错：