189 8069 5689

OCLint+Xcode实现CodeReview-创新互联

Code Review是开发过程中保证代码质量不可或缺的一部分,但是呢,要么是懒,要么是真没时间,在我们公司code review已从原来的流于形式到彻底废弃了。最近看了看项目代码,咋写的都有,看着确实难受,于是就动了“code review自动化”的念头。xcode里有内置的Analyser,但由于默认的规则太少导致功能实在很有限。找了半天,就盯上OCLint了。OCLint是啥我就不多说啦,具体可以查看OCLint官网。本篇博客记录了我如何使用oclint进行静态代码分析,只针对Mac。

10年的台江网站建设经验,针对设计、前端、开发、售后、文案、推广等六对一服务,响应快,48小时及时工作处理。营销型网站建设的优势是能够根据用户设备显示端的尺寸不同,自动调整台江建站的显示方式,使网站能够适用不同显示终端,在浏览器中调整网站的宽度,无论在任何一种浏览器上浏览网站,都能展现优雅布局与设计,从而大程度地提升浏览体验。成都创新互联从事“台江网站设计”,“台江网站推广”以来,每个客户项目都认真落实执行。

一、环境配置

需要安装oclintxcpretty

1、安装oclint

方法一:brew安装

命令行执行:

$brew tap oclint/formulae   
$brew install oclint
方法二:安装包安装

(1)进入到github上,下载最新(当前为oclint-0.13-x86_64-darwin-16.7.0.tar.gz)安装包,解压出来为oclint-0.13,放到如下目录:/Users/layne/OCLint,即路径为/Users/layne/OCLint/oclint-0.13
(2)将oclint添加到环境变量。vim打开~/.bash_profile(若没有则创建),添加如下代码:

 OCLINT_HOME=/Users/layne/OCLint/oclint-0.13
 export PATH=$OCLINT_HOME/bin:$PATH

终端执行:

 $oclint

出现如下内容证明没问题:

 oclint: Not enough positional command line arguments specified!
 Must specify at least 1 positional argument: See: oclint -help
方法三:源码安装 (若要自定义规则,则必须使用源码方式安装,后边会说到具体方法。)

1、安装CMake和Ninja

brew install cmake ninja

CMake和Ninja是代码编译工具,因此必须要先安装。
2、从github上下载oclint源码,解压之后重命名为oclint-0.13,然后放到如下目录(随意):/Users/layne/OCLint,最终为/Users/layne/OCLint/oclint-0.13
3、打开终端进入到/Users/layne/OCLint/oclint-0.13/oclint-scripts

cd /Users/layne/OCLint/oclint-0.13/oclint-scripts

然后执行:

./make

之后就开始下载和编译,不过时间会比较长(40min左右),且还必须能够爬出去才可以。成功之后会有如下路径:/Users/layne/OCLint/oclint-0.13/build/oclint-release,这个就是oclint的路径。
4、添加oclint到环境变量。执行:

vim ~/.bash_profile

将如下内容写入:

OCLINT_HOME=/Users/layne/OCLint/oclint-0.13/build/oclint-release
export PATH=$OCLINT_HOME/bin:$PATH

保存退出。重启终端之后在终端执行:oclint --version,出现如下内容:

LLVM (http://llvm.org/):
LLVM version 5.0.0svn-r.320669
Optimized build.
Default target: x86_64-apple-darwin17.3.0
Host CPU: broadwell

OCLint (http://oclint.org/):
OCLint version 0.13.
Built Dec 14 2017 (16:03:48).

至此oclint安装成功。

2、安装xcpretty

 gem install xcpretty

说明:这里安装的xcpretty是最新版,github上的oclint源码应该是针对最新版的xcpretty进行了兼容。为什么这么说呢?因为我一开始是采用的方法二安装的oclint,运行oclint现成的规则没有问题。之后想要自定义规则,但是方法三又太麻烦了,于是我就偷懒从网上下载了别人事先编译好的oclint-0.12(这里说的"编译好的oclint"保留了当初编译的“现场”,可以进行自定义规则,而方法二中的是“干净“的oclint),然后进行自定义规则,可是跑起来一直报错(形如"/usr/sh fail with exit code 1")。于是乎我不得不用oclint源码重新编译一遍,再运行的时候就没有错误了。

二、xcode配置

以项目LayneStudy为例。

1、创建Aggregate类型target

打开LayneStudy项目,new一个新的target,类型选择Aggregate,命名为OCLint,确定。说明:在xcode9中,Aggregate类型在Cross-platform等目录下(而非iOS、watchOS、macOS等目录下)。

2、编写shell脚本

(1)选择target OCLint,在build phases里添加New Run Script Phase。在框里输入如下脚本代码:

 chmod -R 777 $SRCROOT/oclint
 $SRCROOT/oclint/oclint.sh

(2)编写脚本oclint.sh,内容如下:

 source ~/.bash_profile
 #获取项目路径
 PROJECT_DIR=$(cd `dirname $0`;cd ..;pwd)
 cd ${PROJECT_DIR}

 buildPath="${PROJECT_DIR}/oclint/build"
 compilecommandsJsonFolderPath="${PROJECT_DIR}/oclint"
 compilecommandsJsonFilePath="${PROJECT_DIR}/oclint/compile_commands.json"

 rm -rf "$compilecommandsJsonFolderPath/build"

 xcodebuild SYMROOT=$buildPath | xcpretty -r json-compilation-database -o $compilecommandsJsonFilePath

 cd $compilecommandsJsonFolderPath

 oclint-json-compilation-database -- -report-type xcode \
 -rc CYCLOMATIC_COMPLEXITY=10 \
 -rc LONG_CLASS=1000 \
 -rc LONG_METHOD=50 \
 -rc LONG_LINE=140 \
 -rc LONG_VARIABLE_NAME=30 \
 -rc SHORT_VARIABLE_NAME=1 \
 -rc MAXIMUM_IF_LENGTH=5 \
 -rc MINIMUM_CASES_IN_SWITCH=2 \
 -rc NCSS_METHOD=30 \
 -rc NESTED_BLOCK_DEPTH=5 \
 -rc TOO_MANY_METHOD=30 \
 -rc TOO_MANY_PARAMETERS=5 \
 -max-priority-1 0 \
 -max-priority-2 5 \
 -max-priority-3 10

将oclint.sh放到项目根目录下的oclint文件夹中(要先创建oclint文件夹)。最终目录结构如下:

 ../LayneStudy.xcodeproj
 ../LayneSutdy
 ../oclint
 ../oclint/oclint.sh
3、执行

回到xcode,scheme选择OCLint,command+B,编译完成之后xcode则出现各种警告,证明你成功了。

补充:

①若出现python错误,则通过设置环境变量使alias python=python3,即使用最新的python。
②若出现/Library/Ruby/Gems/2.3.0/gems/xcpretty-0.3.0/lib/xcpretty/parser.rb:434:in `===': invalid byte sequence in US-ASCII (ArgumentError) ”这种错误,则是编码问题:在~/.bash_profile中设置编码:

export LANGUAGE=en_US.UTF-8
export LANG=en_US.UTF-8
export LC_ALL=en_US.UTF-8

自定义规则

1、自定义阈值

像脚本oclint.sh中写的,使用-rc设置各种阈值,详见threshold command line,也可以通过写配置文件的形式配置阈值,参照threshold configuration file。有一点需要注意,配置文件要放到oclint-json-compilation-database命令执行的目录下,例如oclint.sh中有一段:

 ......
 cd $compilecommandsJsonFolderPath
 oclint-json-compilation-database -- -report-type xcode \
 ......

oclint-json-compilation-database是在$compilecommandsJsonFolderPath 中执行的,那么配置文件也要放到$compilecommandsJsonFolderPath下,且命名为.oclint(会被隐藏)。如果配置文件无法命名为.oclint,则需要vim创建一个名为.oclint的文件,并将配置的内容写入,最后移动它到$compilecommandsJsonFolderPath目录下。

2、自定义规则

通过自定义规则类,才是真正的自定义

默认规则

在下载的oclint包已经有默认的一些规则,路径如下:
/Users/layne/OCLint/oclint-0.13/lib/oclint/rules。你没看错,规则都是些以dylib结尾的动态库文件。若在oclint-json-compilation-database执行时没有指定规则加载的路径,那么将会加载这些默认的规则。若要自定义规则路径,则要改写oclint.sh 如下:

 ......
 oclint-json-compilation-database -- -report-type xcode \
 ......

改为

 ......
 oclint-json-compilation-database -- -R $compilecommandsJsonFolderPath/oclint_rules -report-type xcode \
 ......

(要事先在$compilecommandsJsonFolderPath目录下创建oclint_rules文件夹)这样oclint就会加载$compilecommandsJsonFolderPath/oclint_rules中的规则。你可以将一些默认规则文件放到这个目录下,也可以将自定义的规则文件放到这个目录下,进行统一管理。

自定义规则

再次强调: 若要自定义规则,则必须要使用oclint源码
OCLint提供了scaffoldRule工具进行自定义规则。
1、创建规则cpp文件。在如下路径处Users/layne/OCLint/oclint-0.13打开终端窗口,输入如下命令(可用的选项为:SourceCodeReader、ASTVisitor和ASTMatcher):

oclint-scripts/scaffoldRule ObjCNoSynthesize -t SourceCodeReader

这样就在以下路径生成了对应的cpp文件和MakeFile:

oclint-rules/rules/custom
oclint-rules/test/custom

2、 为了方便开发,生成xcodeproj文件,使用xcode开发规则。
(1)在oclint-0.13目录下创建文件夹oclint-xcoderules
(2)进入到oclint-xcoderules目录(Users/layne/OCLint/oclint-0.13/oclint-xcoderules),创建shell文件create-xcode-rules.sh,内容如下:

#! /bin/sh -e

cmake -G Xcode \
      -D CMAKE_CXX_COMPILER=../build/llvm-install/bin/clang++  \
      -D CMAKE_C_COMPILER=../build/llvm-install/bin/clang \
      -D OCLINT_BUILD_DIR=../build/oclint-core \
      -D OCLINT_SOURCE_DIR=../oclint-core \
      -D OCLINT_METRICS_SOURCE_DIR=../oclint-metrics \
      -D OCLINT_METRICS_BUILD_DIR=../build/oclint-metrics \
      -D LLVM_ROOT=../build/llvm-install/ ../oclint-rules

保存退出之后,更改create-xcode-rules.sh的访问权限:

 chmod 777 create-xcode-rules.sh

然后执行:

 ./create-xcode-rules.sh

最后如果出现如下内容证明xcodeproj生成成功了:

 ......
 -- Configuration done
 -- Generating done
 -- Build files have been written to:/Users/layne/OCLint/oclint-0.13/oclint-xcoderules

并且在oclint-xcoderules文件夹下生成了工程目录。如下图:
OCLint+Xcode实现Code Review

3、双击打开OCLINT_RULES.xcodeproj,左侧显示所有的规则源文件,
OCLint+Xcode实现Code Review
拉到最底下就可以看到自己创建的规则ObjCNoSynthesize了。在ObjCNoSynthesizeRule.cpp中编写规则逻辑,之后在Scheme中选择对应的scheme,然后Command+B进行build,成功之后会在目录/Users/layne/OCLint/oclint-0.13/oclint-xcoderules/rules.dl/DEBUG中找到libObjCNoSynthesizeRule.dylib.至此我们就完成了规则的自定义。

4、创建新规则,即往OCLINT_RULES.xcodeproj项目中添加新规则。
(1)使用命令行创建新规则:
回到如下路径Users/layne/OCLint/oclint-0.13打开终端窗口,输入如下命令:

oclint-scripts/scaffoldRule ObjCPointerStarShouldBeNearerToVariable -t ASTVisitor

(2)在OCLINT_RULES项目的scheme中随便选一个target,然后Commnand+B,这样新创建规则就会被加入到项目中来。

5、删除规则。
(1)从左边选择要删除的规则的cpp文件->delete->move to trash;
(2)打开对应的CMakeLists.txt,删除掉对应的配置。
(3)Command+B.
DONE!

补充说明:

1、规则的制定主要基于三个类:AbstractASTVisitorRule、AbstractASTMatcherRule和AbstractSourceCodeReaderRule,它们的关系如下:
RuleBase
   |
   |-AbstractASTRuleBase
   |      |_ AbstractASTVisitorRule
   |             |_AbstractASTMatcherRule
   |
   |-AbstractSourceCodeReaderRule
AbstractSourceCodeReaderRule:

提供eachLine方法,每行的读取源码。对对于想要从每行的内容编写规则的rule,可以使用继承自AbstractSourceCodeReaderRule。

AbstractASTVisitorRule:(一般继承这个类)

继承自AbstractASTVisitorRule的rule,可以实现访问AST上特定类型的所有节点,可以检查特定类型的所有节点是递归实现的,在AbstractASTVisitorRule的apply方法中可以看到代码实现,开发者只需要通过重写bool Visit*方法来访问特定类型的节点,在该函数中实现检查操作,其返回值往往是返回true,返回值表示是否继续递归检查。

AbstractASTMatcherRule: (目前没用过)

继承自AbstractASTMatcherRule的rule,实现setUpMatcher方法,在setUpMatcher()方法中实现添加matcher,当检查发现匹配的结果时,会调用callback()方法,故重新callback方法来对匹配的结果进行处理操作。

以上就是使用oclint的详细教程,至于自定义的规则如何去写,个人建议还是先看看现有规则的源码,找找感觉,然后再比着写。
下面是3个参考链接,我获益良多,建议深入阅读,尤其是里面的例子...

http://oriochan.com/codeReview01.html
https://juejin.im/post/595370986fb9a06bcb7f7d56
http://blog.csdn.net/hdwhappy/article/details/61924772

另外有需要云服务器可以了解下创新互联scvps.cn,海内外云服务器15元起步,三天无理由+7*72小时售后在线,公司持有idc许可证,提供“云服务器、裸金属服务器、高防服务器、香港服务器、美国服务器、虚拟主机、免备案服务器”等云主机租用服务以及企业上云的综合解决方案,具有“安全稳定、简单易用、服务可用性高、性价比高”等特点与优势,专为企业上云打造定制,能够满足用户丰富、多元化的应用场景需求。


当前题目:OCLint+Xcode实现CodeReview-创新互联
本文链接:http://jkwzsj.com/article/dsjoeg.html

其他资讯