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. Programmers should not be allowed to, well, program.

Programmers should not be allowed to, well, program.

Scheduled Pinned Locked Moved The Lounge
helprubycomtestingbeta-testing
25 Posts 15 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.
  • M Offline
    M Offline
    Marc Clifton
    wrote on last edited by
    #1

    For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

    Unit Testing Succinctly

    M L Mike HankeyM B G 9 Replies Last reply
    0
    • M Marc Clifton

      For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

      Unit Testing Succinctly

      M Offline
      M Offline
      Mycroft Holmes
      wrote on last edited by
      #2

      But we all know that if you decide to publish and therefore start commenting and documenting your code it turns into a major effort. You find bugs, you suddenly wonder why you did something this way instead of that way. It is much easier just to dump your code into something like github and ignore it. You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

      Never underestimate the power of human stupidity RAH

      M G B N 4 Replies Last reply
      0
      • M Mycroft Holmes

        But we all know that if you decide to publish and therefore start commenting and documenting your code it turns into a major effort. You find bugs, you suddenly wonder why you did something this way instead of that way. It is much easier just to dump your code into something like github and ignore it. You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

        Never underestimate the power of human stupidity RAH

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

        Mycroft Holmes wrote:

        You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

        Good point! Marc

        Unit Testing Succinctly

        1 Reply Last reply
        0
        • M Marc Clifton

          For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

          Unit Testing Succinctly

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

          Marc Clifton wrote:

          There are no examples.

          From that site's Wiki

          "Here is an example which speaks for itself:"

          I actually agree with you in many cases - but in this case there seems to be quite a lot of documentation and (brief) usage examples - much more than I've seen on many a page. And no code comments? Judging by the younger programmers I work with, we need to get used to it, because they honestly believe code comments aren't worth it as their code is self documenting... which leads neatly back to the subject of your post :)

          MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

          L M 2 Replies Last reply
          0
          • M Marc Clifton

            For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

            Unit Testing Succinctly

            Mike HankeyM Offline
            Mike HankeyM Offline
            Mike Hankey
            wrote on last edited by
            #5

            Marc Clifton wrote:

            Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work.

            The problem is that is probably the same quality they put out at work?

            VS2010/Atmel Studio 6.1 ToDo Manager Extension The problem with the gene pool is that there is no lifeguard. -Steven Wright

            M 1 Reply Last reply
            0
            • L Lost User

              Marc Clifton wrote:

              There are no examples.

              From that site's Wiki

              "Here is an example which speaks for itself:"

              I actually agree with you in many cases - but in this case there seems to be quite a lot of documentation and (brief) usage examples - much more than I've seen on many a page. And no code comments? Judging by the younger programmers I work with, we need to get used to it, because they honestly believe code comments aren't worth it as their code is self documenting... which leads neatly back to the subject of your post :)

              MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

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

              _Maxxx_ wrote:

              because they honestly believe code comments aren't worth it as their code is self documenting...

              Did you test that claim yet? :-D

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

              L C 2 Replies Last reply
              0
              • M Marc Clifton

                For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                Unit Testing Succinctly

                B Offline
                B Offline
                bryce
                wrote on last edited by
                #7

                It sounds to me old chap like 1. You answered your own question ;) 2. The code comments /understanding were left as an exercise for the reader 3. Its a perfect example of a situation where you can add to the knowledge base by submitting your code comments/knowledge additions 4. A great chance to write an article for junior programmers! (remember to send a signed copy to chris *chortle* ) one of the apps I maintain makes about 1.2 billion dollars worth of decisions per year. around 400-600 million dollars gets actually spent based on this app. Code comments? None (i mean NONE!) till I got my hands on it. Bryce

                MCAD ---

                1 Reply Last reply
                0
                • L Lost User

                  _Maxxx_ wrote:

                  because they honestly believe code comments aren't worth it as their code is self documenting...

                  Did you test that claim yet? :-D

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

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

                  On more than one occasion I have been flummoxed Here's an example of class naming. There were some view model classes called QuickFindThenAssignToxxxxxxViewModel Which performed a search for the user then sent a message with the results to a particular entity (the xxxxxxx) Now, forget the fact that this is more complex than it should be and less flexible than it needs to be, at some point the dev decided that a class name of QuickFindThenAssignToCustomerViewModel would be too verbose, so started calling the class QuickFindThenAssignToCViewModel instead (I hope he never wants to look up Clients!) But finally laziness took hold entirely the resource class I needed to check out was called QFTA2SAVResource I still don't know what SAV is...

                  MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

                  G S 2 Replies Last reply
                  0
                  • L Lost User

                    _Maxxx_ wrote:

                    because they honestly believe code comments aren't worth it as their code is self documenting...

                    Did you test that claim yet? :-D

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

                    C Offline
                    C Offline
                    CBadger
                    wrote on last edited by
                    #9

                    Eddy Vluggen wrote:

                    code comments aren't worth it as their code is self documenting

                    Well I can vouch for that. Not that it is true but that it seems to be going in that direction. I was looking at Uncle Bob's clean code videos the other day and he also proclaims commenting to be no need. BUT, if one is not going to comment then yes, your classes and methods should be named as such to explain what it does in merely its name. It seems code is the one of a very few trades where it is apparent that change is the only constant :-)

                    1 Reply Last reply
                    0
                    • L Lost User

                      On more than one occasion I have been flummoxed Here's an example of class naming. There were some view model classes called QuickFindThenAssignToxxxxxxViewModel Which performed a search for the user then sent a message with the results to a particular entity (the xxxxxxx) Now, forget the fact that this is more complex than it should be and less flexible than it needs to be, at some point the dev decided that a class name of QuickFindThenAssignToCustomerViewModel would be too verbose, so started calling the class QuickFindThenAssignToCViewModel instead (I hope he never wants to look up Clients!) But finally laziness took hold entirely the resource class I needed to check out was called QFTA2SAVResource I still don't know what SAV is...

                      MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

                      G Offline
                      G Offline
                      grralph1
                      wrote on last edited by
                      #10

                      A SAV is a red coloured sausage made from unknown indiscriminate matter which is usually served on an elongated bread role with sauce or simply battered and deep fried.

                      "Rock journalism is people who can't write interviewing people who can't talk for people who can't read." Frank Zappa 1980

                      1 Reply Last reply
                      0
                      • M Marc Clifton

                        For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                        Unit Testing Succinctly

                        G Offline
                        G Offline
                        Gary Wheeler
                        wrote on last edited by
                        #11

                        I think I've figured out the problem, Marc. You think open source should be developed with the same care professional software is developed, where the word 'professional' means 'paid'. Open source must be free! Making a profit from your labor is abhorent, at least according to Richard Stallman(*) :rolleyes:. (*) At least I didn't call him The Great Satan, like I was tempted to.

                        Software Zen: delete this;

                        1 Reply Last reply
                        0
                        • M Mycroft Holmes

                          But we all know that if you decide to publish and therefore start commenting and documenting your code it turns into a major effort. You find bugs, you suddenly wonder why you did something this way instead of that way. It is much easier just to dump your code into something like github and ignore it. You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

                          Never underestimate the power of human stupidity RAH

                          G Offline
                          G Offline
                          Gary Wheeler
                          wrote on last edited by
                          #12

                          Excellent point. CP does create a sort of peer review mechanism.

                          Software Zen: delete this;

                          1 Reply Last reply
                          0
                          • M Mycroft Holmes

                            But we all know that if you decide to publish and therefore start commenting and documenting your code it turns into a major effort. You find bugs, you suddenly wonder why you did something this way instead of that way. It is much easier just to dump your code into something like github and ignore it. You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

                            Never underestimate the power of human stupidity RAH

                            B Offline
                            B Offline
                            BobJanova
                            wrote on last edited by
                            #13

                            Interesting point. I'd never considered CP in that way before but you are right – if you post rubbish here you'll know about it and either fix it or take it down, so the standard of what ends up being available is quite high. My own articles have had corrections to both article and code thanks to people pointing out mistakes.

                            1 Reply Last reply
                            0
                            • L Lost User

                              Marc Clifton wrote:

                              There are no examples.

                              From that site's Wiki

                              "Here is an example which speaks for itself:"

                              I actually agree with you in many cases - but in this case there seems to be quite a lot of documentation and (brief) usage examples - much more than I've seen on many a page. And no code comments? Judging by the younger programmers I work with, we need to get used to it, because they honestly believe code comments aren't worth it as their code is self documenting... which leads neatly back to the subject of your post :)

                              MVVM # - I did it My Way ___________________________________________ Man, you're a god. - walterhevedeich 26/05/2011 .\\axxx (That's an 'M')

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

                              _Maxxx_ wrote:

                              From that site's Wiki

                              "Here is an example which speaks for itself:"

                              The fact that I cannot find this on the site's wiki (nor does it come up when I google for it) means something. Not sure what. :~ Marc

                              Unit Testing Succinctly

                              L 1 Reply Last reply
                              0
                              • Mike HankeyM Mike Hankey

                                Marc Clifton wrote:

                                Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work.

                                The problem is that is probably the same quality they put out at work?

                                VS2010/Atmel Studio 6.1 ToDo Manager Extension The problem with the gene pool is that there is no lifeguard. -Steven Wright

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

                                Mike Hankey wrote:

                                The problem is that is probably the same quality they put out at work?

                                Probably! :) Marc

                                Unit Testing Succinctly

                                1 Reply Last reply
                                0
                                • M Marc Clifton

                                  For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                                  Unit Testing Succinctly

                                  A Offline
                                  A Offline
                                  agolddog
                                  wrote on last edited by
                                  #16

                                  I think some of it is experience. People how haven't been coding that long don't have the experience of looking at their own (much less someone else's) code they wrote six months ago and thinking "WTF was I doing again?" Once you've done that a few times, you learn (one hopes) to put comments in spots which were tough to get going, or where you chose a design/implementation out of several pretty equal ideas. The other facet of experience is realizing when what you're building is functional, but just doesn't feel great. Maybe the light bulb doesn't go on for you about how to refactor, or you simply don't have the requisite knowledge of the "righter" approach, but putting comments in that "there's got to be a better way, but I can't think of it right now, here's what I'm doing" should help. All in all, I believe we've got to learn to create the philosophical documentation as much than the technical. By that I mean, explaining what you wree thinking when you chose a particular implementation. Most people can read the code and tell you what it is doing; it's not always obvious what you intended for it to do.

                                  1 Reply Last reply
                                  0
                                  • M Marc Clifton

                                    For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                                    Unit Testing Succinctly

                                    R Offline
                                    R Offline
                                    RafagaX
                                    wrote on last edited by
                                    #17

                                    Well, most open source projects are "I made this that find helpful, if you find it helpful great", so i never expect high quality documentation, although I expect at least an example or two of usage so I can figure out how to use it without reading the code. But when I have to modify the source of a project for whatever reason, then I expect either to the code the be self documenting (or pretty obvious) or that it include comments, an example of the former is Mono.Cecil, their code is not documented at all, no docs, no comments, nothing, yet it's easy to follows and in most cases obvious what it does.

                                    CEO at: - Rafaga Systems - Para Facturas - Modern Components for the moment...

                                    1 Reply Last reply
                                    0
                                    • M Marc Clifton

                                      For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                                      Unit Testing Succinctly

                                      M Offline
                                      M Offline
                                      mrchief_2000
                                      wrote on last edited by
                                      #18

                                      Most of the code is actually self documenting (I'm excluding code /vendor assuming that's not his). For example

                                        def copy\_locale
                                          if Gmaps4rails.pipeline\_enabled?
                                            directory assets\_source\_path, assets\_destination\_path
                                            copy\_file "../../../public/stylesheets/gmaps4rails.css", "vendor/assets/stylesheets/gmaps4rails.css"
                                          else
                                            %w( base google openlayers bing ).each do |filename|
                                              copy\_file js\_source\_path(filename), js\_destination\_path(filename)
                                            end
                                            copy\_file "../../../public/stylesheets/gmaps4rails.css", "public/stylesheets/gmaps4rails.css"
                                          end
                                        end
                                      

                                      or this one:

                                        def show\_readme
                                          readme 'README' if behavior == :invoke
                                        end
                                      

                                      This reads like plain english and is quite evident what the code is doing, and even though I'm not a ruby programmer, I can understand most of it. This is true for almost all of his codebase. I know Marc's reputation, so obviously he's seeing something that I don't. So what am I missing?

                                      B 1 Reply Last reply
                                      0
                                      • M Mycroft Holmes

                                        But we all know that if you decide to publish and therefore start commenting and documenting your code it turns into a major effort. You find bugs, you suddenly wonder why you did something this way instead of that way. It is much easier just to dump your code into something like github and ignore it. You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

                                        Never underestimate the power of human stupidity RAH

                                        N Offline
                                        N Offline
                                        Nathan Nowak
                                        wrote on last edited by
                                        #19

                                        Mycroft Holmes wrote:

                                        You are describing the difference between open source and CP, here your crap is reviewed and the therefore the quality is somewhat higher and that is the reason CP is so valuable.

                                        I may be willing to concede that the quality of code in an average CP article is higher than that of a randomly selected open source project but I don't think that's what makes CP so valuable. Honestly, with the 10 million member celebration I've been thinking about whether CodeProject is still relevant after all these years and what exactly is its purpose or mission. I've been having trouble coming up with positive answers to these questions. Judging by some of the other comments CP might be valuable mostly to authors. It seems to serve as a one stop place to have your code thoughtfully peer reviewed. However, as a place to find code to use in your own projects or as a place to learn about coding in general, I'm somewhat skeptical. I frequently make use of code from Stack Overflow and open source projects but I can not think of a single instance where I have used CP code in any of my software. In all fairness, the lack of utility I've found may have more to do with the fact that CP is C# focused, I haven't programmed in C# in years, rather than because it lacks in any objective measure of value. However, my point is that quality does not equal value. In this case, CP failed to even attempt to address the problem Marc had. We all hate software managers who push quantity over quality but next to infinite loops the biggest bottleneck in any piece of software is the code that isn't written. As long as it takes more time to write high quality code than low quality code and the demand for code outstrips the supply we are doomed to be awash in a sea of poor code. This seems completely unrelated to whether or not we can see those errors because the code is open source. Perhaps I misread, but I detected a note of animosity towards open source in your reply. Maybe like many you are afraid that open source software will steal your lunch. Maybe you have every right to be angry because open source software ate your baby. Regardless of whether your reaction is justified I think you are making an unfair comparison in this case. You are comparing a single open source project in its fledgling stages to CP, a mature community which has been in existence for twelve years and now has ten million members. If you look at the code of an open source project of a simila

                                        M 1 Reply Last reply
                                        0
                                        • M Marc Clifton

                                          For example.[^] OK, it works - and it seems to work quite well. And I'm not wanting to criticize the hard work this fellow put into this package, far from it actually, it simply makes for a good example of why I groan, sigh, and generally feel like we have a lot of room to grow. There isn't a single comment that I've found in the code. The documentation on the home page is, well, sort of sufficient but not quite. There are no examples. Found examples elsewhere, and also poorly written with at least one glaring bug. The best example I found was simply another github application, again with no documentation -- the fellow couldn't even take the time to change the boilerplate text that github generates when you create a project -- but it was as basic an example as necessary. Good enough. This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption. Now, don't get me wrong, I'm very appreciative of the fact that I can take advantage of other people's hard work for free. And I suppose that's the point, it's free, so don't look a gift horse in the mouth, right? And gee Marc, it's open source, so if you want comments, contribute back to the project! Riiight. I think the issue for me is I'd like programmers to realizing that, if you're going to post something for the community at large to use, I for one would really appreciate it if the job was done with same professionalism as if you'd gotten paid to do the work. Oh wait, I see the error in my ways... :sigh: You might ask, why do I want documentation and code comments? Well, documentation so I don't spend 3 hours doing something that should take 30 minutes. Code comments so that I can learn from others - most of this stuff is foreign to me and I was hoping to learn some cool things without having to dissect the code. P.S - and the reason I use the term "programmer" is that this is an example of the kind of code and documentation I guess I expect from a "programmer", whereas I have a higher standard for "developer", and even higher for "software engineer". Marc

                                          Unit Testing Succinctly

                                          P Offline
                                          P Offline
                                          patbob
                                          wrote on last edited by
                                          #20

                                          Marc Clifton wrote:

                                          This isn't really a rant, it's more about the overall lack of quality in what people put up there for general consumption

                                          You have too high of expectations for open source. In my experience, what you observe is typical.

                                          We can program with only 1's, but if all you've got are zeros, you've got nothing.

                                          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