OnTitleBarListener 接口的回调方法上需添加不可空注解，这对 kotlin 开发者将会很友好 - Githubissues

getActivity / TitleBar

Android 标题栏框架，从此布局属性不用记

Apache License 2.0

1.75k stars 217 forks source link

OnTitleBarListener 接口的回调方法上需添加不可空注解，这对 kotlin 开发者将会很友好 #57

Closed anjiemo closed 2 years ago

anjiemo commented 2 years ago

【警告：请务必按照 issue 模板填写，不要抱有侥幸心理，一旦发现 issue 没有按照模板认真填写，一律直接关闭】

建议收集

issue 是否有人曾提过类似的问题？【必答】无
框架文档是否有提及到此问题？【必答】：否
你觉得框架有什么不足之处？【必答】OnTitleBarListener 接口的 onLeftClick、onTitleClick、onRightClick 方法参数未添加不可空注解（@NoNull）。
你觉得该怎么去完善会比较好？【非必答】我查看了框架的源码，发现这几个回调方法的地方参数都为 TitleBar 类本身，不可能出现为 null 的情况，所以建议添加 @NoNull 注解修饰该参数，这样在 kotlin 开发者覆写这几个回调方法的时候 IDE 就不会识别为可空类型，开发者也不用去判空或者修改这个可空类型为不可空。这对 kotlin 开发者将会非常很友好。

getActivity commented 2 years ago

小伙子，经过认真思考，这个建议不给予采纳，具体原因如下：

@NonNull 注解需要添加 Support 或者 AndroidX 相关的依赖库才可以，这样会导致使用的人需要处理这两个库冲突的问题。
加不加 @NonNull 注解都不影响在 Kotlin 的使用，Kotlin 针对 Java 方法中没有添加 @NonNull 注解的参数都会默认加上 ?，如果你不喜欢 ?，可以直接去除掉，这并不会导致编译报错。

anjiemo commented 2 years ago

okay，那么是否可以在接口注释上加以说明呢？不然只能查看调用处才能知道这个参数是否为 null ，如果加在接口注释上的话应该会比较方便一点。

getActivity commented 2 years ago

getActivity commented 2 years ago

@anjiemo 这个可以有，已经加上了，多谢建议。