Skip to content
  • Categories
  • Recent
  • Tags
  • Popular
  • World
  • Users
  • Groups
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse
Code Project
  1. Home
  2. Other Discussions
  3. The Insider News
  4. How peer review leads to quality code

How peer review leads to quality code

Scheduled Pinned Locked Moved The Insider News
htmlcsscomquestioncode-review
8 Posts 6 Posters 0 Views 1 Watching
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • K Offline
    K Offline
    Kent Sharkey
    wrote on last edited by
    #1

    CIO[^]:

    What if you could deploy resources you already have at your disposal to improve quality, get projects done earlier – with fewer errors and with less budget impact – by simply implementing a software quality control method that most likely already exists in other parts of your organization?

    "Many hands make light work."

    R M M 3 Replies Last reply
    0
    • K Kent Sharkey

      CIO[^]:

      What if you could deploy resources you already have at your disposal to improve quality, get projects done earlier – with fewer errors and with less budget impact – by simply implementing a software quality control method that most likely already exists in other parts of your organization?

      "Many hands make light work."

      R Offline
      R Offline
      Ron Anders
      wrote on last edited by
      #2

      Yeah? Well you're not gettin a look at mine as you'll make me rework this and that and this and that. :-D

      1 Reply Last reply
      0
      • K Kent Sharkey

        CIO[^]:

        What if you could deploy resources you already have at your disposal to improve quality, get projects done earlier – with fewer errors and with less budget impact – by simply implementing a software quality control method that most likely already exists in other parts of your organization?

        "Many hands make light work."

        M Offline
        M Offline
        Matthew Dennis
        wrote on last edited by
        #3

        Earlier in my career, I was a ASQ Certified Software Quality Engineer. One of the things I learned in my studies was that Code Reviews was the most effective means of removing defects from project deliverable. Defect removal is usually greater than 75% and as it can occur earlier in the project, finds the defects when it is least expensive to repair. Also, Reviews are not only applicable to code, but can be applied to specifications, design, prototypes, almost anything, and the earlier the better. One important reason why it is good to have other eyes look at your work is that you will not see the kinds of mistakes that you make, or else you would have fixed it already. It requires someone who thinks, works, and programs slightly, or extremely, differently than you to real savage your deliverables, looking for problems. Just remember to practice egoless coding. You are not your code, be proud that you can make it better. Also remember that you write code for people, not computers, otherwise we would just be inputting ones and zeroes. I personally consider it a great day of coding when I end up with less code that I started with, it does more, and reads like a book so almost anyone can understand it.

        V 1 Reply Last reply
        0
        • K Kent Sharkey

          CIO[^]:

          What if you could deploy resources you already have at your disposal to improve quality, get projects done earlier – with fewer errors and with less budget impact – by simply implementing a software quality control method that most likely already exists in other parts of your organization?

          "Many hands make light work."

          M Offline
          M Offline
          Marc Clifton
          wrote on last edited by
          #4

          Assert.That(NumBadProgrammers * 2 == NumBadProgrammers) Marc

          Imperative to Functional Programming Succinctly Contributors Wanted for Higher Order Programming Project!

          K B 2 Replies Last reply
          0
          • M Marc Clifton

            Assert.That(NumBadProgrammers * 2 == NumBadProgrammers) Marc

            Imperative to Functional Programming Succinctly Contributors Wanted for Higher Order Programming Project!

            K Offline
            K Offline
            Kent Sharkey
            wrote on last edited by
            #5

            We'll have to get someone to review that ;)

            TTFN - Kent

            1 Reply Last reply
            0
            • M Marc Clifton

              Assert.That(NumBadProgrammers * 2 == NumBadProgrammers) Marc

              Imperative to Functional Programming Succinctly Contributors Wanted for Higher Order Programming Project!

              B Offline
              B Offline
              Bernhard Hiller
              wrote on last edited by
              #6

              Assertion failed due to Arithmetic Overflow Error

              1 Reply Last reply
              0
              • M Matthew Dennis

                Earlier in my career, I was a ASQ Certified Software Quality Engineer. One of the things I learned in my studies was that Code Reviews was the most effective means of removing defects from project deliverable. Defect removal is usually greater than 75% and as it can occur earlier in the project, finds the defects when it is least expensive to repair. Also, Reviews are not only applicable to code, but can be applied to specifications, design, prototypes, almost anything, and the earlier the better. One important reason why it is good to have other eyes look at your work is that you will not see the kinds of mistakes that you make, or else you would have fixed it already. It requires someone who thinks, works, and programs slightly, or extremely, differently than you to real savage your deliverables, looking for problems. Just remember to practice egoless coding. You are not your code, be proud that you can make it better. Also remember that you write code for people, not computers, otherwise we would just be inputting ones and zeroes. I personally consider it a great day of coding when I end up with less code that I started with, it does more, and reads like a book so almost anyone can understand it.

                V Offline
                V Offline
                Valery Possoz
                wrote on last edited by
                #7

                I could not agree more. I would just add that code reviews are a great way to learn. I enjoy being shown where my code could be improved! Valery.

                M 1 Reply Last reply
                0
                • V Valery Possoz

                  I could not agree more. I would just add that code reviews are a great way to learn. I enjoy being shown where my code could be improved! Valery.

                  M Offline
                  M Offline
                  Matthew Dennis
                  wrote on last edited by
                  #8

                  Absolutely. Reading other peoples code is a great way to learn new patterns and language features. I've been writing .NET code since the original beta, and still find new ideas and techniques from code reviews and reading code from articles and GitHub. You also see some great examples of how NOT to do things as well, typically over-architected, 40 layers of abstraction, theoretically correct horrors.

                  1 Reply Last reply
                  0
                  Reply
                  • Reply as topic
                  Log in to reply
                  • Oldest to Newest
                  • Newest to Oldest
                  • Most Votes


                  • Login

                  • Don't have an account? Register

                  • Login or register to search.
                  • First post
                    Last post
                  0
                  • Categories
                  • Recent
                  • Tags
                  • Popular
                  • World
                  • Users
                  • Groups