技术头条 - 一个快速在微博传播文章的方式     搜索本站
您现在的位置首页 --> 其他 --> Clojure世界:静态代码分析

Clojure世界:静态代码分析

浏览:1610次  出处信息
    Java世界里有findbugs这样的神器,可以让你避免很多“简单愚蠢”的bug。同样,Clojure世界里也有相应的替代品,这就是今天要介绍的kibit。不过kibit现在还比较年轻,判断的规则较少,但是已经可以使用起来做clojure代码的静态检查。

    

     项目主页:https://github.com/jonase/kibit

     使用:

     1.安装lein插件:

    

lein plugin install jonase/kibit 0.0.2

     2.在项目的根目录运行

    

lein kibit

     kibit会分析项目里所有clojure源码,每个namespace分别分析,例如我分析clojure-control的输出:

    

    

== control.commands ==

     == control.core ==

     [186] Consider (zero? (:status (ssh host user cluster (str "test -e " file)))) instead of (= (:status (ssh host user cluster (str "test -e " file))) 0)

     == control.main ==

     == leiningen.control ==

     [null] Consider Integer/parseInt instead of (fn* [p1__61444#] (Integer/parseInt p1__61444#))

     [null] Consider Integer/parseInt instead of (fn* [p1__65254#] (Integer/parseInt p1__65254#))

       显然,kibit一个一个namespace分析过去,并且按照规则对它认为有问题的地方打印出来,并提出建议。例如这里它建议我用

    

(zero? (:status (ssh host user cluster (str "test -e " file))))
    替换control.core里186行的:

    

 (= (:status (ssh host user cluster (str "test -e " file))) 0)

       目前kibit大多数是这类代码风格上的检查,还没有做到类似findbugs那样更丰富的检查,例如NPE异常检查等。此外kibit还提供反射检查,任何有反射调用的地方都给出警告。

       kibit是基于core.logic实现的,它的规则都放在了这里,通过defrules宏来定义检查规则,源码中对算术运算的规则定义:

    

(defrules rules

    [(+ ?x 1) (inc ?x)]

    [(+ 1 ?x) (inc ?x)]

    [(- ?x 1) (dec ?x)]

    [(* ?x (* . ?xs)) (* ?x . ?xs)]

    [(+ ?x (+ . ?xs)) (+ ?x . ?xs)])

   

       第一个规则,任何对类似(+ 1 x)的代码,都建议替换成(inc x),后面的与此类似。理论上你也可以自定义规则,并提交给官方。总体上说kibit仍然是比不上findbugs的,期待未来发展的更好。

    

dennis 2012-03-23 21:28 发表评论

建议继续学习:

  1. Clojure世界:XML处理    (阅读:1965)
  2. Clojure世界:Http Client    (阅读:1950)
  3. Clojure世界:文件IO    (阅读:1849)
  4. Clojure世界:如何做性能测试    (阅读:1729)
  5. Clojure世界:API文档生成    (阅读:1622)
  6. Clojure世界:使用rlwrap增强REPL    (阅读:1552)
  7. Clojure世界:单元测试    (阅读:1443)
QQ技术交流群:445447336,欢迎加入!
扫一扫订阅我的微信号:IT技术博客大学习
© 2009 - 2024 by blogread.cn 微博:@IT技术博客大学习

京ICP备15002552号-1