使用OClint進行iOS專案的靜態程式碼掃描

iTech發表於2016-03-03

使用OClint進行iOS專案的靜態程式碼掃描 

原文連結:http://blog.yourtion.com/static-code-analysis-ios-using-oclint.html


最近需要一個靜態分析程式碼工具,幫助我們釋出執行應用前找到程式碼潛在的問題。

其實對於iOS開發,我們的日常開發上已經用到了這樣一個靜態分析的工具,那就是 Clang, Clang 是支援CC++Objective-C 和 Swift 的一個前端編譯工具,他將 OC 或者 Swift 的程式碼輸出抽象語法樹(Abstract Syntax Tree),然後編譯成 LLVM 的 bitcode,最後由 LLVM 編譯成 machine code。這個工具支撐著我們日常的開發和除錯。

OCLint 就是一個建立在 Clang 上的工具,能夠發現程式碼中潛在的問題。具體的功能請見官方文件,這裡主要介紹的是其安裝與使用。

安裝軟體

XCtool

brew install xctools

OClint

brew tap oclint/formulae
brew install oclint

測試

進入專案目錄(以 SuperLogger 為例),

下載專案切換到有問題的位置(),並進入 SuperLoggerDemo 目錄:

git clone https://github.com/yourtion/SuperLogger.git
cd SuperLogger
git checkout 0e64637459996ed91e0dd15718efb5d7200a9971
cd SuperLoggerDemo

測試執行:

# Cleanup before building
rm -f compile_commands.json
xctool -project SuperLoggerDemo.xcodeproj -scheme SuperLoggerDemo clean

# Build Project
xctool build \
	-project SuperLoggerDemo.xcodeproj -scheme SuperLoggerDemo \
	-reporter json-compilation-database:compile_commands.json

# Analyze Project
oclint-json-compilation-database -e Pods -- \
	-max-priority-1=100000 \
	-max-priority-2=100000 -max-priority-3=100000 \
   	-disable-rule=InvertedLogic \
   	-disable-rule=CollapsibleIfStatements \
 	-disable-rule=UnusedMethodParameter \
	-disable-rule=LongLine \
	-disable-rule=LongVariableName \
	-disable-rule=ShortVariableName \
	-disable-rule=UselessParentheses \
	-disable-rule=IvarAssignmentOutsideAccessorsOrInit | sed 's/\(.*\.\m\{1,2\}:[0-9]*:[0-9]*:\)/\1 warning:/'

# Final cleanup
rm -f compile_commands.json

輸出

OCLint Report

Summary: TotalFiles=14 FilesWithViolations=4 P1=0 P2=2 P3=6

SuperLoggerPreviewView.m:77:37: warning: replace with container literal [migration|P3]
SuperLogerListView.m:206:37: warning: empty catch statement [empty|P2]
SuperLogerListView.m:25:15: warning: empty if statement [empty|P2]
SuperLogerListView.m:119:1: warning: long method [size|P3] Method with 92 lines exceeds limit of 50
SuperLogerListView.m:171:41: warning: replace with container literal [migration|P3]
SuperLogerListView.m:110:21: warning: replace with object subscripting [migration|P3]
SuperLogger.m:60:30: warning: replace with object subscripting [migration|P3]
SuperLogger.m:108:31: warning: replace with object subscripting [migration|P3]

[OCLint (http://oclint.org) v0.10.2]

具體問題可以參考:http://docs.oclint.org/en/stable/rules/index.html

修復後

OCLint Report

Summary: TotalFiles=14 FilesWithViolations=0 P1=0 P2=0 P3=0


[OCLint (http://oclint.org) v0.10.2]

大概的功能就是這樣,使用過程有什麼問題歡迎大家一起交流

原文連結:http://blog.yourtion.com/static-code-analysis-ios-using-oclint.html

相關文章