Basic Code Review CheckList in C++

[itkid]

Hi Friend, I am creating very basic C++ checklist for developer. I have include lots of comments however I am looking for more..... Can you please let me know some of your valuable comments. Thanks ;)

[Maxwell Chen]

itkid wrote:

very basic C++ checklist for developer.

Such as?! Would you mind listing one or two samples for us to get clearer what the check list is about? Then I can add some of mine.

---

Maxwell Chen

[ThatsAlok]

itkid wrote:

I am creating very basic C++ checklist for developer

1. coding guidelines (formatting, naming, constructs to avoid, etc.)OK 2. comments (redundant, missing, consistency with code)OKNOK(function comments) 3. function/procedure/method sizeOK 4. defensive programming (memory management, exception handling, variable initialization, floating point calculations)OKNOK(exception handling ) 5 correctness (program logic)OK. 6. maintainability OK 7. efficiency OK

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers, Alok Gupta VC Forum Q&A :- I/ IV

[itkid]

Thank Alok, You have given few very good point to think. Regards :)

[markkuk]

Have a look at the Mozilla Code Reviewer's Guide[^].