getactivity / androidcodestandard Goto Github PK

Resource IDs will be non-final in Android Gradle Plugin version 5.0, avoid using them in switch case statements
对于资源id还是避免用switch case 好

1、layout_marginStart、layout_marginHorizontal在style中书写时存在使用局限。
2、关于全局变量命名不提倡使用mInstrument这种形式，模型使用如果同时出现set、get方法时，使用public更便捷。理由参考《代码整洁之道》。
3、资源ID命名上不建议带上模块名称（layout除外）。存在重复使用的string，带有模块名称并不合适。建议使用资源的属性功能去命名，这样虽然可能损失了某种程度上的美观，但是总体来说，提升代码开发效率，不必花时间去翻找重复的资源。
4、部分规范不必完全遵守，应根据实际出发。

• 如果用了 app:layout_constraintLeft_toLeftOf="parent"的话，那么约束右边的则用，app:layout_constraintRight_toRightOf="parent"，不要使用 app:layout_constraintEnd_toEndOf="parent"

当然：还是如作者所讲应当首推start和end属性

颜色命名这里我觉得有些不合理, 项目主题色, 按钮颜色这些项目固定用色还能用 功能用途 命名, 产品想改也是统一改, 其他颜色就该用颜色的rgb码命名, 设计那边出图给的也是rgb码, 不然如果一个颜色用在了一个textview的背景又用在了一个edittext的边框, 用描述怎么命名?

图中说的"别人不知道我那个色值是干什么的, 看到我有写就引用了", 不应该就是这样吗? 颜色就是颜色本身, 没有额外描述是主题色, 按钮颜色的都是表明这个颜色本身的作用就是显示该颜色.

期待看到回复!

如果跳转的 Activity 需要传递参数，应该在目标的 Activity 中定义静态的 start 又或者 newIntent 方法。

在Activity中定义静态方法，会不会引起内存泄漏，因为静态方法的生命周期和APP一样长，导致Activity无法被回收。

按我的理解应该以:"isXX：方法返回值为 boolean 型的请使用 is 或 check 为前缀标识"为准吧, 这两处有点冲突