sonarqube中添加p3c-pmd整合阿里java开发规范

sonarqube: 7.4
sonar-pmd: 3.0.1
p3c-pmd: 1.3.6
mvn: 3.6
jdk:1.8

以上版本组合的原因:

  1. sonar-pmd的编译需要mvn3.6+
  2. 使用阿里p3c的规则进行代码分析时,用到了类com.google.gson.Gson,但对应的jar包gson在sonar-pmd3.2.1中并没有依赖到,所以报错找不到类而分析失败。
    sonar-pmd3.1+版本中都没有,但3.0.1中有
  3. 截至目前最新版本p3c-pmd(1.3.6)依赖的pmd版本为5.5.2,但最新版sonar-pmd(3.2.1)依赖的pmd版本为6.10.0,版本冲突,在使用阿里p3c规则进行代码分析时,因为pmd-java版本为6.10.0,导致缺少类net.sourceforge.pmd.lang.java.rule.comments.AbstractCommentRule而分析失败
  4. sonar-pmd3.0.1依赖的是pmd5.4.2,包含类net.sourceforge.pmd.lang.java.rule.comments.AbstractCommentRule,可通过调整p3c-pmd使两者依赖的版本保持一致。
  5. sonar-pmd3.2.0以下版本不支持sonar7.6以上版本,sonar3.0.1支持了sonar7.4
  6. 虽然gitbub上已有集成了p3c的sonar-pmd,但版本是2.6,不支持7.x版本的sonar,且代码规则只有48条,目前阿里p3c-pmd中java规则53条。

ps: IDEA和Eclipse中阿里代码规约插件中共58条。
又ps: Eclipse中阿里代码规约插件:https://p3c.alibaba.com/plugin/eclipse/update
  IDEA中则直接搜alibaba即可。
双ps: gitbub上已有的sonar-p3c-pmd地址:https://github.com/mrprince/sonar-p3c-pmd
叒ps: 如果可以接受以上版本组合,那么可以直接使用笔者本次整理的jar包。jar包地址:【sonar-pmd-plugin-3.0.1.jar】。
叕ps: 如下是笔者本次基于上述内容的jar包制作过程。制作结果的源码地址:https://github.com/95MISTAKE/sonar/tree/master/sonar-p3c-pmd

进入正题

参考文章://www.greatytc.com/p/b849175dd38b

一、sonar-pmd中添加p3c规则
1、pom文件中添加p3c-pmd的引用
<dependency>
    <groupId>com.alibaba.p3c</groupId>
    <artifactId>p3c-pmd</artifactId>
    <version>1.3.6</version>
</dependency>
2、添加p3c规则文件

路径sonar-pmd-plugin\src\main\resources\org\sonar\l10n\pmd\rules下存放规则的描述文件html,新建文件夹pmd-p3c,本次制作基于sonar-p3c-pmd,从其中拷贝并修改,对比eclipse,修改描述文件内容为中文,并补充description,由于sonar-p3c-pmd只有48条,手动新建html文件,增加其余5条规则。
内容示例:

<p>单个方法的总行数不超过80行。</p>
<p>Description:</p>
<p>   说明:包括方法签名、结束右大括号、方法内代码、注释、空行、回车及任何不可见字符的总行数不超过80行。</p>
<p>Examples:</p>
<pre>
</pre>

然后在路径sonar-pmd-plugin\src\main\resources\org\sonar\plugins\pmd下新建rules-p3c.xml,本次制作该文件也是从sonar-p3c-pmd中拷贝,并补充缺少的规则:

<rule key="MethodTooLongRule">
    <priority>MAJOR</priority>
    <configKey><![CDATA[rulesets/java/ali-other.xml/MethodTooLongRule]]></configKey>
</rule>

其中key值以及configKey中对应规则名,priority则对比eclipse中该规则的priority进行配置,至于xml文件则对比p3c-pmd的p3c-master\p3c-pmd\src\main\resources\rulesets\java文件夹下所有xml,以确认当前规则属于哪个。
emmm这种方法实在笨拙,或者说本次制作过程都很笨拙。

3、修改pmd中配置,添加p3c规则配置

涉及到的文件有两个:
sonar-pmd-plugin\src\main\resources\com\sonar\sqale\pmd-model.xml
sonar-pmd-plugin\src\main\resources\org\sonar\l10n\pmd.properties
从sonar-pmd的gitbub pull下来代码后,里面默认是pmd自身的规则配置,由于本次目的是使用阿里的java开发规范进行代码规则检查,所以可注释掉,当然不注释也行。
\color{red}{注意:} 注释时pmd.properties文件中所有以rule.pmd.*.param.*=开头的不能注释掉,否则sonar启动加载pmd插件时会报错。
首先向pmd.properties中添加阿里规则,以ClassNamingShouldBeCamelRule为例,

rule.pmd.ClassNamingShouldBeCamelRule.name=[p3c]Class names should be nouns in UpperCamelCase except domain models: DO, BO, DTO, VO, etc.

其中[p3c]是为了在sonarqube中激活代码规则时方便而做的标识,其后是规则概述。可能由于sonarqube对properties文件的读取方式问题,文件内容不能使用中文,否则乱码。
概述信息可参考p3c-pmd中的p3c-master\p3c-pmd\src\main\resources\messages_en.xml文件,也可对比eclipse和idea进行配置。本次制作过程是基于sonar-p3c-pmd并对比eclipse和idea进行配置,过程中也发现了sonar-p3c-pmd中pmd.properties文件部分错误。
然后向pmd-model.xml中添加阿里规则,首先区分定义阿里规则,在文件末尾</sqale>标签前添加如下代码:

<chc>
    <key>P3C-PMD</key>
    <name>p3c-pmd</name>
    <chc>
        <key>P3C_PMD</key>
        <name>p3c_pmd</name>
        <!--在此处添加rule的chc-->
    </chc>
</chc>

然后再上面那部分代码的注释处添加规则,依旧以ClassNamingShouldBeCamelRule规则为例,其它规则在它之后追加,内容基本一致,只需要改动rule-key标签中的值即可。

<chc>
    <rule-repo>pmd</rule-repo>
    <rule-key>ClassNamingShouldBeCamelRule</rule-key>
    <prop>
        <key>remediationFunction</key>
        <txt>CONSTANT_ISSUE</txt>
    </prop>
    <prop>
        <!--此处为该技术债务所需要的时间,单位使用min,此处设为2min-->
        <key>offset</key>
        <val>2</val>
        <txt>min</txt>
    </prop>
</chc>
4、添加p3c规则的引入

涉及到的文件:
sonar-pmd-plugin\src\main\java\org\sonar\plugins\pmd\rule\PmdRulesDefinition.java

5、打包

使用mvn package -Dmaven.test.skip=true打包即可,如果规则的描述文件html中包含中文内容,那么注意文件编码需要是utf-8

二、sonarqube中启用p3c规则
1、将插件jar包添加到sonar

路径sonarqube-7.7/extensions/plugins,jar包放里面,重启sonarqube即可。

2、sonarqube中配置p3c规则

在Quality Profiles【质量配置】页面点击Create【创建】创建质量配置,Name【名称】自己填,Language【语言】选Java,点击Create【创建】后跳转到该质量配置的详情界面,然后点击Activate more【更多激活规则】,在Search for rules...【搜索规则...】中搜索p3c,然后点击Bulk Change【批量修改】,确认Apply【应用】到刚新建的质量配置即可。



然后回到Quality Profiles【质量配置】页面,将该质量配置Set as Default【设为默认】即可。

咩。
完事儿。
分析看下。


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