youth5201314 / banner

🔥🔥🔥Banner 2.0 来了!Android广告图片轮播控件,内部基于ViewPager2实现,Indicator和UI都可以自定义。
Apache License 2.0
12.92k stars 2.51k forks source link

希望把一些方法修饰为包级可见,不然会有使用误导 #1118

Closed ZhZhZhen closed 10 months ago

ZhZhZhen commented 2 years ago

我所遇见的问题: 我在设置指示器的时候,创建完Indicator,会直接获取config然后进行属性修改。最后再通过setIndicator设置该指示器。但是setIndicator()->initIndicator()->initIndicatorAttr()会将我设置给config的值给覆盖。导致我的设置失效。

看了代码后,我觉得事实上这个indicatorConfig更像是给Banner的一系列setIndicatorXXX()方法和initIndicatorAttr()去做内部设置的。如果该config只是给库内部使用的,那我觉得不应该让使用者访问到该config。

可能的解决方法 1、把config的访问禁用了 2、提供对config的修改,并且config的修改能影响到使用效果,而不是被Banner的属性给覆盖了,导致使用者设置失效