在AS3.+上自定义lint优化你的代码

概述

lint是代码风格和语法规则的检查工具,不限于android平台,其他例如jslint,eslint..
在最新的稳定版本中,官方提供了342个定义好的lint规则,基本满足开发中的需要,基本介绍和使用方法参照官方教程

此外我们还需要某些特殊的场景下的检查,例如 序列化的内部类也需要序列化,如果没有序列化,我们要给出相应的提示

image

网上的demo大部分是3.0之前基于lamlok,使用aar依赖的,已经不适用于AS3.0
自己基于AS3.2, lint-checks版本26.2.0定义了一些lint

Demo地址

https://github.com/ukyo6/cuslint

下面介绍自定义lint的基本配置

Lint parser

Lint从第一个版本就选择了lombok-ast作为自己的AST Parser,并且用了很久。但是Java语言本身在不断更新,Android也在不断迭代出新,lombok-ast慢慢跟不上发展,所以Lint在25.2.0版增加了IntelliJ的PSI(Program Structure Interface)作为新的AST Parser。但是PSI于IntelliJ、于Lint也只是个过渡性方案,事实上IntelliJ早已开始了新一代AST Parser,UAST(Unified AST)的开发,而Lint也将于即将发布的25.4.0版中将PSI更新为UAST。

更多的介绍请参考这篇博客LintParser具体介绍, 简单的说从lambok->PSI->UAST,官方一直在优化lint的执行效率和扩展性,本文使用的是Lint26.2.0版本

配置

1. 创建java-library

在项目中新建java library, 添加最新稳定版lint依赖26.2.0

dependencies {
    compileOnly 'com.android.tools.lint:lint-api:26.2.0'
    compileOnly 'com.android.tools.lint:lint-checks:26.2.0'
}

2. 实现自定义的detector

detector是lint的核心, 每个规则都先继承抽象类 Detector,然后实现Detector.Scanner接口,详细的步骤会在接下来说明

3. 创建自定义的IssueRegistry, 在app中使用该Registry

继承IssueRegistry,添加自定义的detector

public class IssuesRegister extends IssueRegistry {
    @NotNull
    @Override
    public List<Issue> getIssues() {
        return new ArrayList<Issue>() {{
            //添加自定义的Detector
            add(SelfLogDetector.ISSUE);  
            add(NewThreadDetector.ISSUE);
            add(MessageObtainDetector.ISSUE);
            add(ViewIdCorrectnessDetector.ISSUE);
            add(LayoutNameDetector.ACTIVITY_LAYOUT_NAME_ISSUE);
            add(LayoutNameDetector.FRAGMENT_LAYOUT_NAME_ISSUE);
        }};
    }
}

在java moudle的build.gradle中声明自定义的IssueRegistry
``
jar {
manifest {
attributes("Lint-Registry-v2": "com.lintrules.IssuesRegister")
}
}

app中引入自定义的lint

  • Gradle Plugin3.0之前,用的是包装aar的方式引入自定义的lint, 参照 linkedIn方案

  • 3.0后增加 lintChecks, 无需再使用包装aar的方式

    New lintChecks dependency configuration allows you to build a JAR that defines custom lint rules, and package it into your AAR and APK projects. Your custom lint rules must belong to a separate project that outputs a single JAR and includes only compileOnly dependencies. Other app and library modules can then depend on your lint project using the lintChecks configuration:

    我们可以直接在项目的build.gradle里添加

    dependencies {
        lintChecks project(':lintrules') //这里添加java library
    }
    

实现Detector

detector是lint的核心,主要分为下面几步

  1. 每个规则都先继承抽象类Detector,然后实现Detector.Scanner接口
  2. 定义ISSUE的内容,严重级别,提示信息,提示位置
  3. 实现getApplicableXX和visitXX方法; 在getApplicableXX中定义检查的域,visitXX中定义检查的规则
  4. 调用JavaContext.reportIssue()提示异常

1.实现Scanner接口

Scanner包括以下几种

  • SourceCodeScanner 扫描 Java 或符合JVM规范的源码文件(如 kotlin)
  • ClassScanner 扫描编译后的 class 文件
  • BinaryResourceScanner 扫描二进制资源文件(如.png)
  • ResourceFloderScanner 扫描资源目录
  • XmlScanner 扫描 xml 文件
  • GradleScanner 扫描 Gradle 文件
  • OtherFileScanner 扫描其他文件

每个Scanner都实现了很多getApplicableXX和visitXX方法,这些方法都是成对使用的

2.定义ISSUE

ISSUE在每个Detector中定义,lint检查到相关项将ISSUE报告出来,示例:

public static final Issue ISSUE = Issue.create(
    "InnerClassSerializable", //问题 Id
    "内部类需要实现Serializable接口", //问题的简单描述
    "内部类需要实现Serializable接口",//问题的详细描述
    Category.SECURITY, //问题类型  
    5, // 0-10严重级别
    Severity.ERROR, //问题严重程度
    new Implementation(SerializableDetector.class,
    Scope.JAVA_FILE_SCOPE); //Detector 和 Scope 的对应关系

3.实现getApplicableXX和visitXX方法方法

  • 例如实现SourceCodeScanner接口,我们applicableSuperClasses()中指定需要检查的父类名列表,visitClass()当检测到你指定的父类名列表时,就会进入该方法, 根据参数JavaContext, Uclass可以很方便的获取Class的继承关系,名字等信息
    @Nullable
    @Override
    public List<String> applicableSuperClasses() {
        //指定检查"java.io.Serializable"
        return Collections.singletonList(CLASS_SERIALIZABLE);
    }

    /**
     * 扫描到applicableSuperClasses()指定的list时,回调该方法
     */
    @Override
    public void visitClass(JavaContext context, UClass declaration) {
        //只从最外部开始向内部类递归检查
        if (declaration instanceof UAnonymousClass) {
            return;
        }
        sortClass(context, declaration);
    }
  • 再比如XmlScanner, 可以利用getApplicableElements()和visitElement()方法来进行xml中节点的扫描
@Override
    public Collection<String> getApplicableElements() {
        return Arrays.asList( //指定检查这几个控件的命名规范,可自行扩展
                TEXT_VIEW,
                IMAGE_VIEW,
                BUTTON,
                EDIT_TEXT,
                CHECK_BOX
        );
    }
    
    /**
     * 扫描到getApplicableElements()指定的xml节点时,回调该方法
     */
    @Override
    public void visitElement(XmlContext context, Element element) {
        //这个detector只扫描android:id属性,其他属性跳过
        if (!element.hasAttributeNS(ANDROID_URI, ATTR_ID)) return;

        Attr attr = element.getAttributeNodeNS(ANDROID_URI, ATTR_ID);
        String value = attr.getValue();
        if (value.startsWith(NEW_ID_PREFIX)) {
        ....
        }
    }

4.报告ISSUE

在需要报告ISSUE的地方调用context.report()

context.report(ISSUE, //定义好的ISSUE
            uClass.getNameIdentifier(), //ISSUE对应的AST节点
            context.getLocation(uClass.getNameIdentifier()), //ISSUE提示的位置
            String.format("内部类 `%1$s` 需要实现Serializable接口", uClass.getName())); //ISSUE的描述

使用

除了在代码中的静态waring,error提示,还可以输出报告来查看
输入 gradlew 工程名:lintDebug 可以在build/reports目录下查看lint报告,demo中检测到的问题

image

也可以查看详细的信息

image

lint调试

开发完自定义lint规则后,可能需要对代码进行验证,调试方式如下

  • 在自定义的lint代码中打好断点
  • 选择 Run -> Eidt Configurations, 在Android Studio的运行参数(Run Configurations)中添加一个Remote类型,都取默认值即可(端口号5005)


    image
  • 在Teminal窗口中输入gradlew lintDebug -Dorg.gradle.daemon=false -Dorg.gradle.debug=true


    image
  • 选择刚才配置的remote参数,点击debug的console看到以下输出: Connected to the target VM, address: 'localhost:5005', transport: 'socket' 就可以进行调试了

Lint构建优化

随着官方lint-checks的更新,detector的数量越来越多,构建一次linttask的速度肯定是很慢的(即使每个detector指定了scope,都需要扫描项目内所有该scope的文件...)
lint的开发者也在google论坛上给出了lint构建的建议 Lint Performance Tips,大概是以下几点:

  • 使用graldew lintDebug, gradlew lintRelease代替gradlew lint, 因为后者会执行lint lintDebug lintRelease,造成成倍的开销
  • 使用gradlew 模块名:lintDebug可以检查指定的模块
  • lintOptions配置checkTestSources和ignoreTestSources
  • 不要使用checkAllWarnings,因为有一些lint检查,特别是id为“WrongThreadInterprocedural”的,非常慢

参考资料

最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 206,214评论 6 481
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 88,307评论 2 382
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 152,543评论 0 341
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 55,221评论 1 279
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 64,224评论 5 371
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 49,007评论 1 284
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 38,313评论 3 399
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 36,956评论 0 259
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 43,441评论 1 300
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 35,925评论 2 323
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 38,018评论 1 333
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 33,685评论 4 322
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 39,234评论 3 307
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 30,240评论 0 19
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 31,464评论 1 261
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 45,467评论 2 352
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 42,762评论 2 345

推荐阅读更多精彩内容