Sonar

sonar 扫描代码并不是一件很容易的事。
为什么这么说呢,出结果出报告,相对来说很容易,但是谁去follow,谁是owner,优先级等等,背后是一个组织,而且是一个高效、认同、执行力强的组织。所以还是慎入。
整理下sonar相关的信息。

sonar key point

code viewer

核心是代码审查,展示代码源文件和高层次的数据。
-> 行数
-> 问题数
-> 单元覆盖率
-> 重复度
-> SCM信息(近期提交代码某行代码的人和时间)
-> 给定测试文件的测试结果,执行时间和状态

coverage

coverage中有三种颜色标记:红色表示没有被覆盖;橙色表示部分覆盖;绿色表示完全覆盖。Jacoco还可以显示覆盖特定一行不同测试的数量,同时也能列出这些测试,并定位出这些测试的文件。

duplications

重复代码的行数并定位。重复密度=重复行数/总行数*100

tests

可以查询哪个文件被单元测试覆盖,以及有多少行代码被测试覆盖。

issues

issues有5个等级:

  1. confirm 确认一个问题,从open变成confirm。
  2. false positive 在全文中查看发现不是一个真正的问题,标记false positive。
  3. won‘t fix 可以被接受的技术债。
  4. change severity 根据自己的经验来调整严重等级。
  5. resolve 修复,修改正确,下次状态就会关闭,否则状态仍然开放。

dispositioning

讨论决定由谁来解决问题,默认为最后一个提交代码的人。

general

在issue生命周期中,都可以备注注释,在运行日志中显示细节注释。

bulk change

所有的这些变更都可以一次性做成多个问题,通过使用问题搜索结果面板的Bulk Change。

complexity

基于代码路径数量计算复杂度。功能控制流分裂,复杂度依次增加。

  1. class 类的平均复杂度
  2. file 文件的平均复杂度
  3. method 函数的平均复杂度

documentation

  1. line 包含注释或注释代码的行数。不包括仅有注释符号的行数。
  2. comment(%) 注释行数密度=注释行数/(代码行数+注释行数)*100。
  3. public documented API(%) 公有记录API密度=(公有的API-公有未记录的API)/公有API*100。
  4. public undocumented API 不带注释头的公有API
  5. commented-out LOC(%) 代码注释行数

severity

  1. blocker 致命的
  2. critical 关键的
  3. major 主要的
  4. minor 微小的
  5. info 未知的

maintainability

code smells 既不算bug,也不在脆弱性里。
new code smells
maintainability Rating:A=0-0.05, B=0.06-0.1, C=0.11-0.20, D=0.21-0.5, E=0.51-1
technical Debt:技术债,修复所有维护问题的成本。
technical Debt on new code:新代码上的技术债
technical Debt Ratio:技术债比例=修复成本/开发成本
technical Debt Ratio on new code:开发者变更代码的花费与相关问题的花费比
例如下图

image.png

bugs

A = 0 Bug
B = at least 1 Minor Bug
C = at least 1 Major Bug
D = at least 1 Critical Bug
E = at least 1 Blocker Bug

上图中coverage的百分比
Coverage = (CT + CF + LC)/(2*B + EL),其中
CT = 条件判断中至少一次true
CF = 条件判断中至少一次false
LC = 代码覆盖的行数
B = 条件总数
EL = 执行的总数

在实践过程中有时会遇到统计不出覆盖率,需要配置jacoco。

            <plugin>
                <groupId>org.jacoco</groupId>
                <artifactId>jacoco-maven-plugin</artifactId>
                <version>${plugin.jacoco.version}</version>
                <configuration>
                    <destFile>${sonar.jacoco.reportPath}</destFile>
                </configuration>
                <executions>
                    <execution>
                        <id>agent</id>
                        <goals>
                            <goal>prepare-agent</goal>
                        </goals>
                    </execution>
                    <execution>
                        <id>post-test</id>
                        <phase>test</phase>
                        <goals>
                            <goal>report</goal>
                        </goals>
                    </execution>
                </executions>
            </plugin>

执行跑测试的代码逻辑。

mvn clean jacoco:prepare-agent install -Dmaven.test.failure.ignore=true -Dcontent.validation.skip=true -Dsettings.security=${MAVEN_SECURITY_PATH} jacoco:report -e -B -nsu

下面是扫描的mvn 命令。

-Dsonar.branch=${ArtifactionVersion}  sonar:sonar -Psonar-dev -e -nsu

当然扫描完之后,正如开头说的,需要发给相关关联人去看,当然也包括boss了。然后我们可以把sonar分析出来的结果放在邮件中,可以采用selenium截屏的方式,截出指定位置的图片。亲测效果不错。

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