Greetings fellow Monks

This is a dillema about codereview.

I work a at company that creates applications for transport. Basically we make the software that controls the flow of the traffic. It is not one application but a whole bunch of applications that all communicate with each other.

These applications grew and evolved over many years and are written in many languages (C, C++, Java, Pascal, Perl). So we have lots of source code. A few milion lines of code. Some parts are of good quality and comply to our coding standard, other parts suffer from software rot or from too tight deadlines.

A lot of the code is constantly modified as the requirements keep on changing. In an effort to improve the quality of the development process we are thinking of having code reviews. The idea is to review every change to code. It would have been better if all code was reviewed and then fixed, but there is no money for this (nor are there any unittest to make sure nothing breaks when things get fixed).

So management is not entirely happy with code review idea. They worry about what the returns will be for the time invested. They also claim that the reviewers will be overwhelmed by all the faults in the old code each time they have to review changed code.

I proposed to have the reviewer add all remarks in header of a sourcefile so the next time it is reviewed these points can be skipped. Also with a little perl program all the points can be extracted from the code and one can create a nice summary of what needs to be done. But that was rejected since the client then sees how good or bad our code really is.

Does anyone has any experience with this? How can one best review changes to heaps of old code?

Have Fun

In reply to Howto best review code that has not been reviewed before? by gumpu

Title:
Use:  <p> text here (a paragraph) </p>
and:  <code> code here </code>
to format your post, it's "PerlMonks-approved HTML":


  • Posts are HTML formatted. Put <p> </p> tags around your paragraphs. Put <code> </code> tags around your code and data!
  • Titles consisting of a single word are discouraged, and in most cases are disallowed outright.
  • Read Where should I post X? if you're not absolutely sure you're posting in the right place.
  • Please read these before you post! —
  • Posts may use any of the Perl Monks Approved HTML tags:
    a, abbr, b, big, blockquote, br, caption, center, col, colgroup, dd, del, details, div, dl, dt, em, font, h1, h2, h3, h4, h5, h6, hr, i, ins, li, ol, p, pre, readmore, small, span, spoiler, strike, strong, sub, summary, sup, table, tbody, td, tfoot, th, thead, tr, tt, u, ul, wbr
  • You may need to use entities for some characters, as follows. (Exception: Within code tags, you can put the characters literally.)
            For:     Use:
    & &amp;
    < &lt;
    > &gt;
    [ &#91;
    ] &#93;
  • Link using PerlMonks shortcuts! What shortcuts can I use for linking?
  • See Writeup Formatting Tips and other pages linked from there for more info.