*cppcheck
cppcheck是一個靜態(tài)代碼檢查工具。基本上編譯器不檢查的問題他都檢查,效果還是不錯的。
cppcheck的最終目標是沒有誤報,任重道遠啊。
先貼上我使用的命令
cppcheck.exe -v -s -f -a -I bo2 -I pg ^
項目若干... ^
2>check.log
經(jīng)過若干小時后,生成了26.8k的check.log。
屬于計算密集型任務(wù),耗時漫長,下次再檢查的時候,采用多進程的方式。
1.過濾C-style pointer casting,得到nocasting.log-13k。
cppcheck不建議使用c類型的指針轉(zhuǎn)換,而是用static_cast之流。不過要打字太多,基本上沒人使用。
2.過濾The class 'xxxx' has no constructor,得到noconstructor.log-12.7k。
有些類是不必寫構(gòu)造函數(shù)的啦。默認的就可以了。
3.提取Member variable not initialized in the constructor,得到init.log-8k;不屬于該條目的其他內(nèi)容保存到noinit.log-4.6k。
成員變量未初始化偶爾是一個debug/release不一致的禍根,老大建議以后再改,不過這個量有點大啊。大家太疏忽了。
4.過濾The scope of the variable de can be limited和Redundant condition. It is safe to deallocate a NULL pointer,最后得到final.log-2k。
The scope of the variable de can be limited,cppcheck建議某個臨時變量的作用域可以進一步縮小。
Redundant condition. It is safe to deallocate a NULL pointer,cppcheck認為空指針可以直接delete。也不知哪里遺傳的壞毛病,非要if一下。不過我不是這樣的。
5.剩下的2k里面包含了19個錯誤條目。
有5條指出某些父類沒有虛析構(gòu)函數(shù),不過那些父類沒有析構(gòu)函數(shù)也無關(guān)緊要。
內(nèi)存泄露有幾條,不過只有1條正確,其余的屬于誤報。
崩潰錯誤一條,在循環(huán)內(nèi)erase(iterator)而沒有break。
其他違反最佳實踐的有幾條,可改可不改。
總的來說,cppcheck還是不錯的,是一個不花什么力氣進行簡單代碼審核的工具。
posted on 2009-08-24 13:35
LOGOS 閱讀(17004)
評論(4) 編輯 收藏 引用