Android静态代码代码检测

项目大了,想要让项目自动化管理工具。查阅资料了解到detekt这个可以实现。该库是基于ktLint实现的。

项目官方地址

配置gradle

buildscript {
    repositories {
        google()
        mavenCentral()
        gradlePluginPortal()
    }
    dependencies {
        classpath "com.android.tools.build:gradle:<agp_version>"
    }
}

plugins {
    id "com.android.application"
    id "org.jetbrains.kotlin.android" version "<kotlin_version>"
    id "io.gitlab.arturbosch.detekt" version "1.23.0-RC1"
}

repositories {
    mavenCentral()
}

添加commit hook
在git流中添加代码检测钩子
那么如何在git flow中[添加hook]?(https://www.atlassian.com/git/tutorials/git-hooks)

  1. 打开项目所在的Git仓库,并进入项目根目录。
  2. 进入.git/hooks目录,这个目录中包含了一些默认的Git Hook模板。
  3. 选择你需要添加的Hook类型,例如pre-commit。
  4. 将你的Hook脚本文件放置在该目录中,命名为pre-commit(或其他Hook类型的名称)。下面的代码就是pre-commit代码。
#!/usr/bin/env bash
echo "Running detekt check..."
OUTPUT="/tmp/detekt-$(date +%s)"
./gradlew detekt > $OUTPUT
EXIT_CODE=$?
if [ $EXIT_CODE -ne 0 ]; then
  cat $OUTPUT
  rm $OUTPUT
  echo "***********************************************"
  echo "                 Detekt failed                 "
  echo " Please fix the above issues before committing "
  echo "***********************************************"
  exit $EXIT_CODE
fi
rm $OUTPUT
  1. 添加可执行权限,使用命令chmod +x pre-commit。
  2. 编辑Hook脚本文件,根据需要添加代码,例如运行自动化测试或代码规范检查。
  3. 提交你的Hook脚本文件到Git仓库中,可以口头约定手动去操作,但这看起来好傻,我们是程序员,当然懒是第一生产力。
    写一段task执行自动注入pre-commit。

tips:下面的detekt-pre-commit.sample其实就是上面我贴的pre-commit代码。

task detektInit(){
    if(!file("${project.rootDir}/.git/hooks/pre-commit").exists()){
        print("commit hooks not exists,create it")
        copy {
            from "${project.rootDir}/config/detekt/detekt-pre-commit.sample"
            into "${project.rootDir}/.git/hooks/"
            rename { String fileName->
                return "pre-commit"
            }
        }
    }
}

默认的应该是基于ktLint,可以根据自己需要添加对应的规则。
忽略警告排除
baseline

  • 也可以直接用官方的rule

下面我列了2个官方规则,
其中下面这个是twitter的compose代码检测规则

detektPlugins "io.gitlab.arturbosch.detekt:detekt-formatting:1.22.0"
detektPlugins "com.twitter.compose.rules:detekt:0.0.26"

另外我们可以通过ktLint官网查看检测规则,输出相应的规则文档。

遇到的哪些坑?
1、根据官方文档添加自定义规则忽略文件
detekt {
        config = files("${project.rootDir}/config/detekt/detekt-config.yml")
 }

平平无奇的代码始终编译报错,路径完全没错。
最后发现是因为文件名中带了中划线 - 导致的,将名字改成detektConfig.yml搞定。

2、如果项目在build文件产生变动的情况下commit的话,会报java文件编译错误。
image.png

对项目进行一次build,就不会出错了,搞定。

3、文件存在依旧报错

image.png

查看文件对应的属性存在,这里报错又是莫名其妙。
然后查看官方依赖插件版本稳定性
image.png

因为我的版本是Java1.8,kt 1.8. gradle 7.3.3,agp7.2.2
所以对detekt版本进行降级从1.22.0降级到1.19.0尝试问题依然存在。
但是check速度下降了。把版本重新升到1.22.0。
在github上搜关键字autoCorrect
发现是格式输出有问题,修改规则:

formatting:
  active: true
  NoTrailingSpaces:
    active: false
    autoCorrect: false
  Indentation:
    active: false
    autoCorrect: false
  NoBlankLineBeforeRbrace:
    active: false
    autoCorrect: false

重新提交


image.png

最终提交通过。
搞定。

4、git commit的时候hooks没有生效?

查看.git\hooks\pre-commit 文件是否为可执行文件。如果不是可执行文件,可能因为编码问题变成了txt文本文件,那么hooks也不会自动执行。

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

推荐阅读更多精彩内容