*cppcheck
cppcheck是一個靜態代碼檢查工具。基本上編譯器不檢查的問題他都檢查,效果還是不錯的。
cppcheck的最終目標是沒有誤報,任重道遠啊。
先貼上我使用的命令
cppcheck.exe -v -s -f -a -I bo2 -I pg ^
項目若干... ^
2>check.log
經過若干小時后,生成了26.8k的check.log。
屬于計算密集型任務,耗時漫長,下次再檢查的時候,采用多進程的方式。
1.過濾C-style pointer casting,得到nocasting.log-13k。
cppcheck不建議使用c類型的指針轉換,而是用static_cast之流。不過要打字太多,基本上沒人使用。
2.過濾The class 'xxxx' has no constructor,得到noconstructor.log-12.7k。
有些類是不必寫構造函數的啦。默認的就可以了。
3.提取Member variable not initialized in the constructor,得到init.log-8k;不屬于該條目的其他內容保存到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條指出某些父類沒有虛析構函數,不過那些父類沒有析構函數也無關緊要。
內存泄露有幾條,不過只有1條正確,其余的屬于誤報。
崩潰錯誤一條,在循環內erase(iterator)而沒有break。
其他違反最佳實踐的有幾條,可改可不改。
總的來說,cppcheck還是不錯的,是一個不花什么力氣進行簡單代碼審核的工具。
posted on 2009-08-24 13:35
LOGOS 閱讀(17004)
評論(4) 編輯 收藏 引用