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. Don't waste time on code reviews

Don't waste time on code reviews

Scheduled Pinned Locked Moved The Insider News
javacsssharepointcom
16 Posts 9 Posters 1 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.
  • R Ron Anders

    Code: If it works, it works.

    P Offline
    P Offline
    PIEBALDconsult
    wrote on last edited by
    #5

    All code is flawless until proven otherwise.

    N 1 Reply Last reply
    0
    • P PIEBALDconsult

      Easy. No time spent on code reviews is ever wasted.

      R Offline
      R Offline
      Ravi Bhavnani
      wrote on last edited by
      #6

      :thumbsup: /ravi

      My new year resolution: 2048 x 1536 Home | Articles | My .NET bits | Freeware ravib(at)ravib(dot)com

      1 Reply Last reply
      0
      • P PIEBALDconsult

        All code is flawless until proven otherwise.

        N Offline
        N Offline
        Nguyen H H Dang
        wrote on last edited by
        #7

        That awkward moment, when my code works and i don't know why :doh:

        In code we trust !

        T 1 Reply Last reply
        0
        • K Kent Sharkey

          DZone[^]:

          Less than half of development teams do code reviews and the other half are probably not getting as much out of code reviews as they should.

          4 Stars: I laughed, I cried.

          R Offline
          R Offline
          Ranjan D
          wrote on last edited by
          #8

          This is what happens when the authors article is not reviewed by his editor :laugh:

          1 Reply Last reply
          0
          • P PIEBALDconsult

            Easy. No time spent on code reviews is ever wasted.

            L Offline
            L Offline
            Lost User
            wrote on last edited by
            #9

            Amen.

            Bastard Programmer from Hell :suss: If you can't read my code, try converting it here[^]

            1 Reply Last reply
            0
            • P PIEBALDconsult

              Easy. No time spent on code reviews is ever wasted.

              T Offline
              T Offline
              thrakazog
              wrote on last edited by
              #10

              Strongly disagree. I've sat through hours and hours of them with previous employers. In many cases I'd argue that the outcome was worse code. But as the article pointed out, these fell in the: "But you don’t have to do code reviews this way – and you shouldn’t." category.

              Play my game Gravity: Android[^], Windows Phone 7[^]

              P L 2 Replies Last reply
              0
              • N Nguyen H H Dang

                That awkward moment, when my code works and i don't know why :doh:

                In code we trust !

                T Offline
                T Offline
                thrakazog
                wrote on last edited by
                #11

                You ever open up some old code.... and wonder how it even compiled? :rolleyes:

                Play my game Gravity: Android[^], Windows Phone 7[^]

                1 Reply Last reply
                0
                • T thrakazog

                  Strongly disagree. I've sat through hours and hours of them with previous employers. In many cases I'd argue that the outcome was worse code. But as the article pointed out, these fell in the: "But you don’t have to do code reviews this way – and you shouldn’t." category.

                  Play my game Gravity: Android[^], Windows Phone 7[^]

                  P Offline
                  P Offline
                  PIEBALDconsult
                  wrote on last edited by
                  #12

                  Still not wasted. You learned something.

                  T 1 Reply Last reply
                  0
                  • P PIEBALDconsult

                    Still not wasted. You learned something.

                    T Offline
                    T Offline
                    thrakazog
                    wrote on last edited by
                    #13

                    I learned not to do code reviews. So yeah, that's something. :-D

                    Play my game Gravity: Android[^], Windows Phone 7[^]

                    P 1 Reply Last reply
                    0
                    • T thrakazog

                      I learned not to do code reviews. So yeah, that's something. :-D

                      Play my game Gravity: Android[^], Windows Phone 7[^]

                      P Offline
                      P Offline
                      PIEBALDconsult
                      wrote on last edited by
                      #14

                      I thought you learned to find a job at a company with a better process.

                      1 Reply Last reply
                      0
                      • T thrakazog

                        Strongly disagree. I've sat through hours and hours of them with previous employers. In many cases I'd argue that the outcome was worse code. But as the article pointed out, these fell in the: "But you don’t have to do code reviews this way – and you shouldn’t." category.

                        Play my game Gravity: Android[^], Windows Phone 7[^]

                        L Offline
                        L Offline
                        Lost User
                        wrote on last edited by
                        #15

                        thrakazog wrote:

                        Strongly disagree. I've sat through hours and hours of them with previous employers

                        Into the same category as the smoking uncle of 120 years old.

                        Bastard Programmer from Hell :suss: If you can't read my code, try converting it here[^]

                        T 1 Reply Last reply
                        0
                        • L Lost User

                          thrakazog wrote:

                          Strongly disagree. I've sat through hours and hours of them with previous employers

                          Into the same category as the smoking uncle of 120 years old.

                          Bastard Programmer from Hell :suss: If you can't read my code, try converting it here[^]

                          T Offline
                          T Offline
                          thrakazog
                          wrote on last edited by
                          #16

                          I'd say it belongs more in the category of the uncle once bit by a dog who was forever more a cat person. There might be a right way to do code reviews. But I've yet to see it.

                          Play my game Gravity: Android[^], Windows Phone 7[^]

                          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