1. which management decisions lead to no reconstruction?
Relying on the consciousness of programmers, the results must not be reconstructed.
If there is no check or punishment, the result is certainly not reconstructed.
No assessment, no management.
---- Peter * Drucker
2. Quality is available only when importance is attached.
How Can I scan the world without scanning a house.
As a result, there are no accumulated steps, no more than a thousand miles, no small streams, no rivers and seas.
To achieve a major event, you must start from a trivial matter.
3. Code Review
Cross-Review: Before submitting to CVS, let two people review it at the same time, reconstruct the bad taste, and sign the reviewer!
Group review: In each iteration cycle, the Group members hold a 1.5-hour meeting, randomly select the code for review, and refactor the places with bad taste.
Expert review: This is also called spot check review. Some codes may be extracted from the experts for review within a certain period of time, and the restructures of the places with bad taste will be implemented.
Quality Department review
4. Review content
Based on the principle, we only need to review the following content:
Encoding specifications: format, name, comment
Duplicate code
Category
Too long Function
Too long parameter
Divergent changes
Conditional expressions: Policy mode and factory Mode
Code reviews are not responsible for checking functions and logic. Unit Tests and QA work are required to solve these problems!
5. Code review tools
Fxcop
Microsoft internal code check tool Microsoft Source Analysis for C #
6. How to manage 70 yards !!!
Each developer has his own opinions on the meaning of code quality, and most teams also have their own ideas on how to write poorly-developed code. It is subjective. I believe that the code that smells terrible may be the best code that others have ever written. Do the following phrases sound familiar?
Yes, it looks a little messy at first, but you need to see how scalable it is !!!
It confuses you, but you obviously do not understand its pattern.