*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 閱讀(17026)
評論(4) 編輯 收藏 引用