如何写好kotlin代码

关于kotlin

kotlin作为一个JVM语言,它在高度兼容Java代码的前提之下,对Java语言进行了极大的简化和升级,在保留了面向对象的编程语言的优势前提下,拔高语言中函数的地位并拓展函数的能力,为开发者提供了函数式的编程框架,各种函数用法使kotlin代码极其简洁,甚至乍一看已经和Java完全不同了,,以上种种保证它可以不影响现有代码的前提下,提高后续的代码开发效率。因此kotlin这几年在迅速普及,据我所知,大量的前后端开发团队都已经在使用kotlin了。

既然kotlin大家都在用,那么我们如何写好kotlin代码呢?最佳选择当然是参考官方教程或者官方代码规范指导等等文档来写。比如作为Android开发的推荐编程语言,Android官方提出了一些代码规范:Kotlin代码风格指导,而kotlin自己的官方语言教程里面其实也有各种各样的代码规范。

也就是说,想要自己写出更规范和健壮(稳定)的kotlin代码,就需要熟悉所有相关的规则规范和kotlin一些语言教程的文档

这好像是一句正确的废话...

image

当然,熟读完并且记住所有的规则规范然后写代码的时候不出错,显然有些强人所难了,因此,我们需要一个插件来记住所有的代码规则规范,然后扫描我们的代码,然后帮助我们分析自己的kotlin代码。

kotlin编码规范最全的还是 ktlint,ktlint项目不仅参考了Android官方的Kotlin style guide,而且还有kotlin官方的编码规范等等。因此ktlint是一个不错的选择。

但是,一段好的代码不仅仅是代码格式的规范,代码本身应该具备稳定性,即代码运行没有性能或崩溃隐患,所以代码分析插件最好能指出哪些代码片段会影响代码稳定性,而ktlint主要能力还是检查编码规范,无法扫描kotlin代码中的性能问题和隐患代码,因此不算完美。

好了,不卖关子了,介绍下detekt。目前来讲,它相对更全面的Kotlin代码分析插件。

detekt是一款kotlin静态代码分析工具,它可以检查kotlin的编码规范,也能检查代码的性能开销和隐患,而且,你可以自定义自己的代码分析规则体系,这样,你就不必全盘接受代码所有代码规范,可以去掉那些无关紧要的规则检查,让开发相对灵活一些。

detekt使用简介

安装

参考官网,其实安装使用的方式有很多种,这里介绍一种。

  • 单独gradle文件集成

以下配置写在一个单独的gradle文件中


repositories {
    // if you 'gradle install' all detekt modules
    mavenLocal()
    // or when all modules should be provided
    maven {
        url  "http://dl.bintray.com/arturbosch/code-analysis"
    }
}

configurations {
    detekt
}

task detekt(type: JavaExec) {
    main = "io.gitlab.arturbosch.detekt.cli.Main"
    classpath = configurations.detekt
    def input = "$project.projectDir.absolutePath"
    // detekt.yml就是我们可自定义的代码分析规则的配置文档,后面会讲
    def config = "$project.projectDir/detekt.yml"
    def filters = ".*test.*"
    ...
    ...
    // 输出结果的文件,可选html,xml,或txt等其他格式
    def output = "$project.projectDir/reports/detekt/detekt-report.html"
    def outputXml = "$project.projectDir/reports/detekt/detekt-report.xml"
    def params = [ '-p', input, '-c', config, '-f', filters, '-r', "html:$output",'-r', "xml:$outputXml"]
    args(params)
}

dependencies {
    detekt 'io.gitlab.arturbosch.detekt:detekt-cli:1.0.0.[CURRENT_MILESTONE]'
    detekt 'io.gitlab.arturbosch.detekt:detekt-formatting:1.0.0.[CURRENT_MILESTONE]'
}


然后命令行运行gradle detekt即可,默认输出文件在build文件夹下,当然我们也可以自己配置输出目录地址。

代码分析规则

上面提到的输入文件detekt.yml就是detekt分析kotlin代码的分析规则依据,我们可以自定义编写规则,detekt可以定义的规则类型分为9类

  • comments 注释文档规范检查
  • complexity 复杂度检查
  • empty-blocks 空block检查
  • exceptions 表达式规范检查
  • formatting 格式规范检查
  • naming 命名规范检查
  • performance 性能检查
  • potential-bugs 潜在bug检查
  • style 代码风格检查

举个例子,以下是糗百项目中使用的部分规则配置(其实主要是根据默认配置文件来选择自己需要的规则配置)


test-pattern: # Configure exclusions for test sources
  active: true
  patterns: # Test file regexes
  - '.*/test/.*'
  - '.*/androidTest/.*'
  - '.*Test.kt'
  - '.*Spec.kt'
  - '.*Spek.kt'
  exclude-rule-sets:
  - 'comments'
  exclude-rules:
  - 'NamingRules'
  - 'WildcardImport'
  - 'MagicNumber'
  - 'MaxLineLength'
  - 'LateinitUsage'

build:
  maxIssues: 1  # 发现超过多少个问题时,task返回结果失败
  weights:  #比重
  # complexity: 2
  # LongParameterList: 1
  # style: 1
  # comments: 1


comments:
  active: false  // 是否激活该规则
  CommentOverPrivateFunction:
    active: false
  CommentOverPrivateProperty:
    active: false
  EndOfSentenceFormat:
    active: false
    endOfSentenceFormat: ([.?!][ \t\n\r\f<])|([.?!]$)
  UndocumentedPublicClass:
    active: false
    searchInNestedClass: true
    searchInInnerClass: true
    searchInInnerObject: true
    searchInInnerInterface: true
  UndocumentedPublicFunction:
    active: false

complexity:
  active: true
  ComplexCondition:  #条件是否过于复杂
    active: true
    threshold: 4  临界值
  ComplexInterface:
    active: false
    threshold: 10
    includeStaticDeclarations: false
  ComplexMethod:  #方法复杂难懂
    active: false
    threshold: 10
    ignoreSingleWhenExpression: false
    ignoreSimpleWhenEntries: false

  NestedBlockDepth: # 嵌套深度检查
    active: true
    threshold: 4  #阈值
  StringLiteralDuplication:
    active: false
    threshold: 3
    ignoreAnnotation: true
    excludeStringsWithLessThan5Characters: true
    ignoreStringsRegex: '$^'
  TooManyFunctions: # 单个文件/对象内部的方法数限定,功能单一原则
    active: false
    thresholdInFiles: 11
    thresholdInClasses: 11
    thresholdInInterfaces: 11
    thresholdInObjects: 11
    thresholdInEnums: 11
    ignoreDeprecated: false
    ignorePrivate: false
    ignoreOverridden: false

empty-blocks:
  active: true
  EmptyCatchBlock:
    active: true
    allowedExceptionNameRegex: "^(_|(ignore|expected).*)"
  EmptyClassBlock:  # 空的class,不需要{}
    active: true


exceptions:
  active: true
  ExceptionRaisedInUnexpectedLocation:
    active: false
    methodNames: 'toString,hashCode,equals,finalize'
  ThrowingNewInstanceOfSameException:
    active: false
  TooGenericExceptionCaught: #异常捕获范围太广
    active: false
    exceptionNames:
    - ArrayIndexOutOfBoundsException
    - Error
    - Exception
    - IllegalMonitorStateException
    - NullPointerException
    - IndexOutOfBoundsException
    - RuntimeException
    - Throwable
    allowedExceptionNameRegex: "^(_|(ignore|expected).*)"


formatting:
  active: true
  android: false
  autoCorrect: true
  MaximumLineLength:   #  单行最长的长度,暂时不启用检查
    active: false
    maxLineLength: 120
  ModifierOrdering:    #  修饰符一致性
    active: true
    autoCorrect: true
  NoBlankLineBeforeRbrace:
    active: false
    autoCorrect: true
  NoConsecutiveBlankLines:  #  没有连续的空行
    active: true
    autoCorrect: true

  NoWildcardImports:  # 没有通配符的倒导入
    active: true
    autoCorrect: true
#  PackageName:
#    active: true
#    autoCorrect: true
  ParameterListWrapping:
    active: true
    autoCorrect: true
    indentSize: 4


naming:
  active: true
  ClassNaming: # class name 首字母大写
    active: true
    classPattern: '[A-Z$][a-zA-Z0-9$]*'

  FunctionMinLength:
    active: false
    minimumFunctionNameLength: 3
  FunctionNaming:  #fun命名规则
    active: true
    functionPattern: '^([a-z$][a-zA-Z$0-9]*)|(`.*`)$'
    excludeClassPattern: '$^'
    ignoreOverridden: true
  FunctionParameterNaming:  #fun param 命名规则 驼峰
    active: true
    parameterPattern: '[a-z][A-Za-z0-9]*'
    excludeClassPattern: '$^'
    ignoreOverriddenFunctions: true
  MatchingDeclarationName: # kt文件中,只有一个类的情况下,最好保持类名文件名一致,一个文件多个类的情况下,选择能描述这些类的文件名称
    active: true
  MemberNameEqualsClassName: # 成员名和类名混淆,暂时不起用
    active: false
    ignoreOverriddenFunction: true
  ObjectPropertyNaming:  # 对象属性命名规则
    active: true
    constantPattern: '[A-Za-z][_A-Za-z0-9]*'
    propertyPattern: '[A-Za-z][_A-Za-z0-9]*'
    privatePropertyPattern: '(_)?[A-Za-z][_A-Za-z0-9]*'
  VariableNaming:
    active: true
    variablePattern: '[a-z][A-Za-z0-9]*'
    privateVariablePattern: '(_)?[a-z][A-Za-z0-9]*'
    excludeClassPattern: '$^'
    ignoreOverridden: true

performance:
  active: true
  ArrayPrimitive:  # 数组类型避免开装箱开销
    active: true
  ForEachOnRange: # ForEach循环代码的检查
    active: true
  SpreadOperator:
    active: false
  UnnecessaryTemporaryInstantiation: #类型转化的性能检查
    active: true

potential-bugs:
  active: true
  ExplicitGarbageCollectionCall: # 垃圾回收的代码检查
    active: true
  InvalidRange: #非法的range
    active: false
  IteratorHasNextCallsNextMethod: #迭代器代码检查
    active: true
  IteratorNotThrowingNoSuchElementException: # 迭代器实现规范
    active: false
  MissingWhenCase: #when的case条件不全
      active: true
  LateinitUsage:
    active: true
    excludeAnnotatedProperties: ""
    ignoreOnClassesPattern: ""
  UnsafeCallOnNullableType: # 不安全的空指针判断
    active: true
  UnsafeCast: #不安全的类型转换
    active: true
  UselessPostfixExpression: # 无用的后缀表达式
    active: true


以上这些定制规则,只是截取的一部分,大家使用detekt时,可以根据自己的需要来选择。
规则文档

输出结果

当你在命令行运行gradlew detekt之后,代码检查的结果会出现在你之前配置的路径下,比如build目录下:


image

输出的代码检查报告有不同的格式:xml,html,txt等,本质上是对代码检查结果的不同组织形式。

一下时html格式在浏览器中打开的样子如下:

image

这样,你就很方便的看到项目中所有kotlin代码的问题了,根据它的提示一一改正之后,重新运行代码分析插件直到不再有错误即可。

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

推荐阅读更多精彩内容