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. The Lounge
  3. No comment

No comment

Scheduled Pinned Locked Moved The Lounge
businesscollaboration
77 Posts 58 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.
  • R Offline
    R Offline
    Rob Philpott
    wrote on last edited by
    #1

    Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

    Regards, Rob Philpott.

    R J 0 Richard Andrew x64R L 47 Replies Last reply
    0
    • R Rob Philpott

      Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

      Regards, Rob Philpott.

      R Offline
      R Offline
      realJSOP
      wrote on last edited by
      #2

      I would be quite verbal in my protestations regarding the "no comments allowed" policy. Agile is fine (I guess), but stupidity and ignorance should be avoided at all costs.

      .45 ACP - because shooting twice is just silly
      -----
      "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
      -----
      "The staggering layers of obscenity in your statement make it a work of art on so many levels." - J. Jystad, 2001

      I W M 3 Replies Last reply
      0
      • R Rob Philpott

        Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

        Regards, Rob Philpott.

        J Offline
        J Offline
        jeron1
        wrote on last edited by
        #3

        While code should have the goal of being self commenting, there are times I find that some comments are appropriate. For instance, in functions where complex computations are involved, I'll often add comments so that in X years from now, after I've totally forgotten what I did, I don't have to spend an appreciable amount of time coming back up to speed...Or maybe it's that I came from an assembler background :rolleyes: .

        1 Reply Last reply
        0
        • R realJSOP

          I would be quite verbal in my protestations regarding the "no comments allowed" policy. Agile is fine (I guess), but stupidity and ignorance should be avoided at all costs.

          .45 ACP - because shooting twice is just silly
          -----
          "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
          -----
          "The staggering layers of obscenity in your statement make it a work of art on so many levels." - J. Jystad, 2001

          I Offline
          I Offline
          Ian Shlasko
          wrote on last edited by
          #4

          John Simmons / outlaw programmer wrote:

          stupidity and ignorance should be avoided at all costs.

          Been a while since you were in the business world? :)

          Proud to have finally moved to the A-Ark. Which one are you in? Author of Guardians of Xen (Sci-Fi/Fantasy novel)

          R 1 Reply Last reply
          0
          • R Rob Philpott

            Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

            Regards, Rob Philpott.

            0 Offline
            0 Offline
            0x3c0
            wrote on last edited by
            #5

            I would follow their rules to the letter. No comments, at all. #regions, on the other hand...

            OSDev :)

            R 1 Reply Last reply
            0
            • R Rob Philpott

              Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

              Regards, Rob Philpott.

              Richard Andrew x64R Offline
              Richard Andrew x64R Offline
              Richard Andrew x64
              wrote on last edited by
              #6

              Sounds like the rules were made by a zealot who doesn't have to write or read code himself/herself.

              J 1 Reply Last reply
              0
              • R realJSOP

                I would be quite verbal in my protestations regarding the "no comments allowed" policy. Agile is fine (I guess), but stupidity and ignorance should be avoided at all costs.

                .45 ACP - because shooting twice is just silly
                -----
                "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
                -----
                "The staggering layers of obscenity in your statement make it a work of art on so many levels." - J. Jystad, 2001

                W Offline
                W Offline
                wolfbinary
                wrote on last edited by
                #7

                At a previous job a consultant mentioned and then explained what agile was and then he declared that's what we'd been doing all along. He missed the part about defining what you're doing and then not changing that until sometime later as another cycle. Agile, as I've seen it, mostly keeps documentation of the system from happening and foresight of the product or system from taking place. When customers ask for documentation we didn't have much to say other than to sputter and then commit to making some later. It was never part of the project. Poorly written comments are just as bad as no comments. What the code is doing doesn't necessarily explain the why. Just my opinion though.

                P A 2 Replies Last reply
                0
                • I Ian Shlasko

                  John Simmons / outlaw programmer wrote:

                  stupidity and ignorance should be avoided at all costs.

                  Been a while since you were in the business world? :)

                  Proud to have finally moved to the A-Ark. Which one are you in? Author of Guardians of Xen (Sci-Fi/Fantasy novel)

                  R Offline
                  R Offline
                  realJSOP
                  wrote on last edited by
                  #8

                  Nope, I just have a mindset about using comments.

                  .45 ACP - because shooting twice is just silly
                  -----
                  "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
                  -----
                  "The staggering layers of obscenity in your statement make it a work of art on so many levels." - J. Jystad, 2001

                  I 1 Reply Last reply
                  0
                  • R Rob Philpott

                    Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                    Regards, Rob Philpott.

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

                    I worked at a place a few years ago where they though agile would be good because it meant you didn't have to plan things. :sigh:

                    Join the cool kids - Come fold with us[^]

                    1 Reply Last reply
                    0
                    • R Rob Philpott

                      Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                      Regards, Rob Philpott.

                      P Offline
                      P Offline
                      Pete OHanlon
                      wrote on last edited by
                      #10

                      Ahhhhh. Dogmatic stupidity enshrined as process.

                      "WPF has many lovers. It's a veritable porn star!" - Josh Smith

                      As Braveheart once said, "You can take our freedom but you'll never take our Hobnobs!" - Martin Hughes.

                      My blog | My articles | MoXAML PowerToys | Onyx

                      1 Reply Last reply
                      0
                      • R realJSOP

                        Nope, I just have a mindset about using comments.

                        .45 ACP - because shooting twice is just silly
                        -----
                        "Why don't you tie a kerosene-soaked rag around your ankles so the ants won't climb up and eat your candy ass..." - Dale Earnhardt, 1997
                        -----
                        "The staggering layers of obscenity in your statement make it a work of art on so many levels." - J. Jystad, 2001

                        I Offline
                        I Offline
                        Ian Shlasko
                        wrote on last edited by
                        #11

                        Yes, comments are good... But do you really think you can avoid stupidity and ignorance in the business world?

                        Proud to have finally moved to the A-Ark. Which one are you in? Author of Guardians of Xen (Sci-Fi/Fantasy novel)

                        R R P 3 Replies Last reply
                        0
                        • R Rob Philpott

                          Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                          Regards, Rob Philpott.

                          K Offline
                          K Offline
                          Keith Barrow
                          wrote on last edited by
                          #12

                          Sounds fine to me, except:

                          Rob Philpott wrote:

                          ). Code should be self describing, and if you need to comment something, you're better really need to consider taking it and putting in its own method with a meaningful name.

                          In general, with refactoring and sensible names, few methods need to be more than 10 lines long and the code does become largely self describing. Comments are needed occaisionally, especially if something difficult to understand is happening (e.g. a bit of mathematical trickery is being used) Heavy-commenting was drilled into me at university and was a difficult habit to break. It was a real epiphany for me when I went through a refactoring exercise with someone much more experienced than me, and the code came out pretty much self describing and clear.

                          CCC solved so far: 2 (including a Hard One!) 37!?!! - Randall, Clerks

                          1 Reply Last reply
                          0
                          • 0 0x3c0

                            I would follow their rules to the letter. No comments, at all. #regions, on the other hand...

                            OSDev :)

                            R Offline
                            R Offline
                            Rob Philpott
                            wrote on last edited by
                            #13

                            I don't think they're allowed either, which is fine by me as I HATE them. First thing I always do is CTRL+M+P to expand them away, you know so I can see the code.

                            Regards, Rob Philpott.

                            K 0 U 3 Replies Last reply
                            0
                            • R Rob Philpott

                              Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                              Regards, Rob Philpott.

                              N Offline
                              N Offline
                              Nagy Vilmos
                              wrote on last edited by
                              #14

                              Use a gun on the moron who came up with the no comment policy.


                              Panic, Chaos, Destruction. My work here is done.

                              J 1 Reply Last reply
                              0
                              • R Rob Philpott

                                Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                                Regards, Rob Philpott.

                                I Offline
                                I Offline
                                Ian Shlasko
                                wrote on last edited by
                                #15

                                Geeky Proof that Agile is Bad, from a former Warcraft Addict: * The "Agile" process is all about... Wait for it... Agility! * Agility is a stat favored by rogues and hunters. * Therefore, "Agile" development is for woodsmen and brigands, which shouldn't be allowed anywhere near code.

                                Proud to have finally moved to the A-Ark. Which one are you in? Author of Guardians of Xen (Sci-Fi/Fantasy novel)

                                S 1 Reply Last reply
                                0
                                • R Rob Philpott

                                  Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                                  Regards, Rob Philpott.

                                  F Offline
                                  F Offline
                                  fred_
                                  wrote on last edited by
                                  #16

                                  Comments should be allowed for special situation like when it's not clear why a process order is required to work around an issue. Having worked in Share Point alot this year, there are workarounds I had to do that w/o comments would make future programmers looking at the code go wtf.

                                  1 Reply Last reply
                                  0
                                  • R Rob Philpott

                                    Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                                    Regards, Rob Philpott.

                                    R Offline
                                    R Offline
                                    R Giskard Reventlov
                                    wrote on last edited by
                                    #17

                                    Well we do agile (been doing 'agile' for years and years and years) but anyone that didn't take a coupe of seconds to comment what they are up to would be hung, drawn and eighthed by everyone else. Utter arrogance to believe that your code is self describing to anyone else 3 months or so from now however simple you believe it to be.

                                    me, me, me

                                    1 Reply Last reply
                                    0
                                    • R Rob Philpott

                                      I don't think they're allowed either, which is fine by me as I HATE them. First thing I always do is CTRL+M+P to expand them away, you know so I can see the code.

                                      Regards, Rob Philpott.

                                      K Offline
                                      K Offline
                                      Keith Barrow
                                      wrote on last edited by
                                      #18

                                      I second that, at best they hide code, at worst (and I have seen this) they surround stuff that should be encapsulated in a new class type.

                                      Rob Philpott wrote:

                                      First thing I always do is CTRL+M+P to expand them away, you know so I can see the code.

                                      And then delete them.

                                      CCC solved so far: 2 (including a Hard One!) 37!?!! - Randall, Clerks

                                      1 Reply Last reply
                                      0
                                      • R Rob Philpott

                                        Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                                        Regards, Rob Philpott.

                                        C Offline
                                        C Offline
                                        CalvinHobbies
                                        wrote on last edited by
                                        #19

                                        The whole " no comment" thing sounds like someone(s) trying to save their jobs. Think we had a posting on that the other day. I'm on the mind frame any comments are better then no comments at all. Personally I use comments as "bookmarks" of sorts. That way after an hour of lunch or time after work, once I come back it's not " what was I doing again?" type of deal. Even if comments get... sloppy, least there is a path. Now if say there was planning and design before hand of whats to be done, than fine. The planning becomes a form of commenting. There is a way to be "agile" and efficient with out killing off the quality. it's finding that happy medium. edit; found the article " how to write unmaintainable code and keep your job for life." :D I love that article :)

                                        ///////////////// Groucho Marx Those are my principals, if you don't like them… I have others.

                                        modified on Wednesday, December 23, 2009 11:27 AM

                                        1 Reply Last reply
                                        0
                                        • R Rob Philpott

                                          Where I work presently, we are 'agile' if you will. Very disciplined agile. One rule the team has is that comments in code are not allowed (no, really). Code should be self describing, and if you need to comment something, you're better taking it and putting in its own method with a meaningful name. I'd be interested to hear what people's opionions on this are.

                                          Regards, Rob Philpott.

                                          D Offline
                                          D Offline
                                          Dan Neely
                                          wrote on last edited by
                                          #20

                                          Is there any limitation on method names?

                                          public class SambaWatcher_ThisClassWrapsAnOlderWin32ApiToMonitorFileChangesOnASambaShareThatDoesntSupportTheModernApiCalledByFileChangeNotification

                                          Without an explanation of why the "proper" way to do this won't work there's nothing to keep an agidiot from helpfully refactoring the redundant class out of existence. :rolleyes:

                                          3x12=36 2x12=24 1x12=12 0x12=18

                                          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