Re: Here's what I've changed.
On Aug 4, 2:06 pm, Michael Doubez <michael.dou...@free.fr> wrote:
On 4 ao=FBt, 11:17, p...@informatimago.com (Pascal J. Bourguignon)
wrote:
[...]
As a junior, I've had to write documentation before code; it
was reviewed and signed by a peer, the manager and the QA
manager. It was not a perfect process and I can't say it
caught a lot of design issue but but I got a lot of value
from it from the design perspective.
How was the review process carried out? I've found good reviews
catch a lot of errors (but the manager and the QA manager are
never part of the review team---that's not their role).
Another thing which helps junior employees a lot is having them
act as part of the review team. They may not catch many errors
(although one never knows---one of the most difficult errors
I've ever seen was found by a "stagiaire" (not sure of the
English word)), but they sure do learn a lot that way. (A good
review process helps at several different levels: it catches
errors, of course, but it also ensures that the code is easily
readable, it teaches less experienced programmers what is
expected, and it helps build a consensus with regards to what is
needed, and creates a community feeling.)
--
James Kanze (GABI Software) email:james.kanze@gmail.com
Conseils en informatique orient=E9e objet/
Beratung in objektorientierter Datenverarbeitung
9 place S=E9mard, 78210 St.-Cyr-l'=C9cole, France, +33 (0)1 30 23 00 34