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.
  • 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.

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

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

    R L T 3 Replies 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.

      P Offline
      P Offline
      Paul M Watt
      wrote on last edited by
      #3

      I liked that. Got more out of it than I expected. Two thumbs up!

      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
        Ron Anders
        wrote on last edited by
        #4

        Code: If it works, it works.

        P 1 Reply Last reply
        0
        • 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