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. Comments?

Comments?

Scheduled Pinned Locked Moved The Lounge
question
32 Posts 15 Posters 3 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.
  • T Tomasz Sowinski

    Comments - yes, they're essential. Hungarian Notation - definitely not essential. Tomasz Sowinski -- http://www.shooltz.com.pl

    S Offline
    S Offline
    Simon Brown
    wrote on last edited by
    #12

    $ set mode /flame=on :mad: Oh, I do not agree! I have been looking at the source for HTMLToPDF - a fine free utility which converts HTML to PDF. This has been [1] written and [2] without a coding standard. I find this a big shame - time and effort was invested, with a little more care it would have been twice as maintainable. Professional programmers are proud of their code - how is this possible without Hungarian Notation? If it's worth writing, it's worth writing properly :confused: or is it? $ set mode /flame=simmer :rolleyes: Old Simon

    T 1 Reply Last reply
    0
    • S Simon Brown

      $ set mode /flame=on :mad: Oh, I do not agree! I have been looking at the source for HTMLToPDF - a fine free utility which converts HTML to PDF. This has been [1] written and [2] without a coding standard. I find this a big shame - time and effort was invested, with a little more care it would have been twice as maintainable. Professional programmers are proud of their code - how is this possible without Hungarian Notation? If it's worth writing, it's worth writing properly :confused: or is it? $ set mode /flame=simmer :rolleyes: Old Simon

      T Offline
      T Offline
      Tomasz Sowinski
      wrote on last edited by
      #13

      <box owner="Pandora"> : Professional programmers are proud of their code - : how is this possible without Hungarian Notation? Are you suggesting that Hungarian is *good* coding standard? Even MS dumped it away in .NET </box> ;P Tomasz Sowinski -- http://www.shooltz.com.pl

      S 1 Reply Last reply
      0
      • T Tomasz Sowinski

        <box owner="Pandora"> : Professional programmers are proud of their code - : how is this possible without Hungarian Notation? Are you suggesting that Hungarian is *good* coding standard? Even MS dumped it away in .NET </box> ;P Tomasz Sowinski -- http://www.shooltz.com.pl

        S Offline
        S Offline
        Simon Brown
        wrote on last edited by
        #14

        Well, I would say that for C++ and C it's good, in that it is obvious what a variable's type and usage is. I have no experience of .NET, so can't comment. What gets me annoyed is a variable such as a buried in 10,000 lines of code. I know what type nRange is, but for a I have to dig all over the code to find the type definition. :( Old Simon

        T 1 Reply Last reply
        0
        • S Simon Brown

          Well, I would say that for C++ and C it's good, in that it is obvious what a variable's type and usage is. I have no experience of .NET, so can't comment. What gets me annoyed is a variable such as a buried in 10,000 lines of code. I know what type nRange is, but for a I have to dig all over the code to find the type definition. :( Old Simon

          T Offline
          T Offline
          Tomasz Sowinski
          wrote on last edited by
          #15

          Nobody claims that a is good variable name. Hungarian is about prefixes and suffixes. It's of course matter of personal taste, but for me range sounds *much* better than nRange. Let's stop the flame here :-D Tomasz Sowinski -- http://www.shooltz.com.pl

          S 1 Reply Last reply
          0
          • T Tomasz Sowinski

            Nobody claims that a is good variable name. Hungarian is about prefixes and suffixes. It's of course matter of personal taste, but for me range sounds *much* better than nRange. Let's stop the flame here :-D Tomasz Sowinski -- http://www.shooltz.com.pl

            S Offline
            S Offline
            Simon Brown
            wrote on last edited by
            #16

            Nope, Flame is still burning ;P Why not pszRange or nRange ? As was mentioned by someone else, code must be maintainable, code that is hard to maintain costs money - my money :mad: ! I believe all programmers should strive to raise their coding standard to the highest level possible. I've been coding professionally for 20 years, and always find ways to make my code more readable. Now then, let's start on the topic: Programmers Should Learn To Write Documentation :-D Old Simon

            T 1 Reply Last reply
            0
            • S Simon Brown

              Yes, I agree :rose: - comments and Hungarian Notation (see MSDN) are essential, otherwise code is unmaintainable. :suss: Old Simon

              C Offline
              C Offline
              Chris Maunder
              wrote on last edited by
              #17

              Hungarian Notation are essential Say it like it is brother! cheers, Chris Maunder

              S 1 Reply Last reply
              0
              • C Chris Maunder

                Hungarian Notation are essential Say it like it is brother! cheers, Chris Maunder

                S Offline
                S Offline
                Simon Brown
                wrote on last edited by
                #18

                Code must also be user friendly: http://www.userfriendly.org/ :-O Old Simon

                1 Reply Last reply
                0
                • S Simon Brown

                  Nope, Flame is still burning ;P Why not pszRange or nRange ? As was mentioned by someone else, code must be maintainable, code that is hard to maintain costs money - my money :mad: ! I believe all programmers should strive to raise their coding standard to the highest level possible. I've been coding professionally for 20 years, and always find ways to make my code more readable. Now then, let's start on the topic: Programmers Should Learn To Write Documentation :-D Old Simon

                  T Offline
                  T Offline
                  Tomasz Sowinski
                  wrote on last edited by
                  #19

                  : As was mentioned by someone else, code must be maintainable, : code that is hard to maintain costs money - my money No disagreement here - coding standards are a must. But Hungarian is only one of *many* coding standards. It has its pros and cons, which any project manager enforcing standards should be aware of. For quick overview, visit: http://ootips.org/hungarian-notation.html PS: Chris - maybe the next CodeProject poll should be about Hungarian Notation? Tomasz Sowinski -- http://www.shooltz.com.pl

                  S 1 Reply Last reply
                  0
                  • S Simon Brown

                    Yes, I agree :rose: - comments and Hungarian Notation (see MSDN) are essential, otherwise code is unmaintainable. :suss: Old Simon

                    S Offline
                    S Offline
                    Stephen Kellett
                    wrote on last edited by
                    #20

                    Comments are essential. Hungarian notation is absolutely NOT essential. Disk storage is cheap and compilers are not limited to 8 character variable names. Do yourself a favour and do not use hungarian notation. Stephen Kellett -- C++/Java/Win NT/Unix variants Memory leaks/corruptions/performance/system problems. UK based. Problems with RSI/WRULD? Contact me for advice.

                    S 1 Reply Last reply
                    0
                    • J Jim A Johnson

                      The mark of a bad, or at least naive, programmer, for sure. In a nutshell: coding is about communcaition. Many people think that the target is the compiler, but it's not: it's other programmers, or yourself, sometime in the future. I used to think I could make my code self-documenting. It's not practical. You need comments to make it understandable. If your comments are out of date, keep them up to date.

                      S Offline
                      S Offline
                      Stephen Kellett
                      wrote on last edited by
                      #21

                      There is another problem with 'self documenting' code, which no one has pointed out. If the code is correct and self documenting then the programmer has to read the code and work out what it is doing. This takes up more time than reading some well written comments. This is true for a correct algorithm and also true for an incorrect algorithm. The difference is that with comments for the correct algorithm, the comments match the code. For the incorrect algorithm, at some point you'll realise that the comments and the code don't match and you can then fix the code. For the case of an incorrect algorithm that is 'self documenting' code all you are doing is reading the code but not knowing that the the intent of the code is incorrect. So you've spent 30 minutes looking at the code, worked out what it does. And then you think 'is this meant to be doing this?'. BEcause there are no comments you can't tell if it is correct or not. Stephen Kellett -- C++/Java/Win NT/Unix variants Memory leaks/corruptions/performance/system problems. UK based. Problems with RSI/WRULD? Contact me for advice.

                      1 Reply Last reply
                      0
                      • T Tomasz Sowinski

                        : As was mentioned by someone else, code must be maintainable, : code that is hard to maintain costs money - my money No disagreement here - coding standards are a must. But Hungarian is only one of *many* coding standards. It has its pros and cons, which any project manager enforcing standards should be aware of. For quick overview, visit: http://ootips.org/hungarian-notation.html PS: Chris - maybe the next CodeProject poll should be about Hungarian Notation? Tomasz Sowinski -- http://www.shooltz.com.pl

                        S Offline
                        S Offline
                        Simon Brown
                        wrote on last edited by
                        #22

                        Hi, Thanks for the link - I hadn't seen it before. Old Simon

                        1 Reply Last reply
                        0
                        • S Stephen Kellett

                          Comments are essential. Hungarian notation is absolutely NOT essential. Disk storage is cheap and compilers are not limited to 8 character variable names. Do yourself a favour and do not use hungarian notation. Stephen Kellett -- C++/Java/Win NT/Unix variants Memory leaks/corruptions/performance/system problems. UK based. Problems with RSI/WRULD? Contact me for advice.

                          S Offline
                          S Offline
                          Simon Brown
                          wrote on last edited by
                          #23

                          Hi, Hungarian has nowt to do with 8 character names - it's about the ability to look at code and see immediately what it does, and also how it does it. Generalization: 10% of the cost of a product is the coding, 30% to 50% maintenance. Good programmers write easily maintainable code. Old Simon

                          S 1 Reply Last reply
                          0
                          • C coder8472

                            Comments? What are comments? :confused:

                            maXallion

                            "Is there any Tea on this Spaceship?"
                            - Arthur Dent

                            Home of The Code Devil

                            S Offline
                            S Offline
                            Slavo Furman
                            wrote on last edited by
                            #24

                            >> Comments? What are comments? :confused: << Hmmm... you never need know this, I think. ;P SlavoF "I hear and I forget. I see and I remember. I do and I understand." --Confucius

                            1 Reply Last reply
                            0
                            • C coder8472

                              Comments? What are comments? :confused:

                              maXallion

                              "Is there any Tea on this Spaceship?"
                              - Arthur Dent

                              Home of The Code Devil

                              S Offline
                              S Offline
                              Slavo Furman
                              wrote on last edited by
                              #25

                              >> Comments? What are comments? :confused: << Hmmm... you will never need to know this, I think. ;P SlavoF "I hear and I forget. I see and I remember. I do and I understand." --Confucius

                              C 1 Reply Last reply
                              0
                              • J James Pullicino

                                // Hello fellow programmers, // I was just wondering how programmers today use comments in their source code. // I try to minimize the ammount of comments I use since they clutter my code and // get outdated, which can be misleading. // I believe that it is better to write code which is easy to understand than // to write comments which explain what the code is doing. James

                                D Offline
                                D Offline
                                Dark Angel
                                wrote on last edited by
                                #26

                                You want your comments to state the intention of the code rather than what it says. i.e. A useless comment: i++; // increment i A useful comment: i++; // move to the next record in the SchoolBus table If you always describe what's supposed to happen rather than the code itself, you'll find that the comments will not get outdated so fast. The only exceptions to this rule that I follow is to should comment any areas that use language trickery, math, or places where I have hard coded "magic numbers". Dropping through cases in a switch() statement is a good example of language trickery. Font.Position -= charglyph.aWidth; // Why am I doing this? Font.Position += 1; // magic number - Why are we adding 1? // I won't remember what the 1 is for in a week from now. just my 2c "Harland Pepper, would you stop naming nuts" - Harland Pepper

                                1 Reply Last reply
                                0
                                • J James Pullicino

                                  // Hello fellow programmers, // I was just wondering how programmers today use comments in their source code. // I try to minimize the ammount of comments I use since they clutter my code and // get outdated, which can be misleading. // I believe that it is better to write code which is easy to understand than // to write comments which explain what the code is doing. James

                                  F Offline
                                  F Offline
                                  Felix Gartsman
                                  wrote on last edited by
                                  #27

                                  Correctly written comments never clutter code. Comments not always describe code statements, they can describe data flow, not trivial algorithms, optimizations, etc... I highly recommend for you to read chapter 19 of the book Code Complete from MS Press, it has great tips for writing comments.

                                  1 Reply Last reply
                                  0
                                  • S Slavo Furman

                                    >> Comments? What are comments? :confused: << Hmmm... you will never need to know this, I think. ;P SlavoF "I hear and I forget. I see and I remember. I do and I understand." --Confucius

                                    C Offline
                                    C Offline
                                    coder8472
                                    wrote on last edited by
                                    #28

                                    Good! This C++ stuff is complicated enough without any comments! maXallion
                                      "I code, therefore I am!" - The Code Devil
                                      www.maxallion.de - coded evil & more

                                    1 Reply Last reply
                                    0
                                    • J James Pullicino

                                      // Hello fellow programmers, // I was just wondering how programmers today use comments in their source code. // I try to minimize the ammount of comments I use since they clutter my code and // get outdated, which can be misleading. // I believe that it is better to write code which is easy to understand than // to write comments which explain what the code is doing. James

                                      RaviBeeR Offline
                                      RaviBeeR Offline
                                      RaviBee
                                      wrote on last edited by
                                      #29

                                      We have a nice set of coding style guidelines where I work. One of the requirements to pass code review is that the code should be easy to understand. An oft used rule of thumb is: "Remove all text except comments and block boundaries. The resulting text should form a reasonable low-level design doc." This seems to work amazingly well and helps decide when a comment is pertinent, simplistic or superflous. I won't directly address the "comments are for weenies" sentiments in this thread, save to say that support and maintenance is the most expensive part of engineering, and properly commented code directly helps keep these costs down. We're an XP shop - check out http://www-106.ibm.com/developerworks/java/library/j-xp/. It's worth a read. /ravi "There is always one more bug..." http://www.ravib.com ravib@ravib.com

                                      1 Reply Last reply
                                      0
                                      • S Simon Brown

                                        Hi, Hungarian has nowt to do with 8 character names - it's about the ability to look at code and see immediately what it does, and also how it does it. Generalization: 10% of the cost of a product is the coding, 30% to 50% maintenance. Good programmers write easily maintainable code. Old Simon

                                        S Offline
                                        S Offline
                                        Stephen Kellett
                                        wrote on last edited by
                                        #30

                                        I replied to this by email, but suffice to say that I agree with the points Simon makes except that I think Hungarian notation sucks. Despite having made a successful career start with computer games in 1983 and finding that the majority of people (that I know or have worked with) whose ability I respect do not use hungarian notation, I was rather surpsied to read the Hungarian notation thread here where most of the posters state they use it, although I was pleased to read that most only use it where they felt needed, rather than the impression I got from Simon, which (I may be incorrect) was "must be used all the time". I'm much more in favour of proper descriptive variable names than a cryptic lpcszName, I mean the hungarian part is longer than 'Name'! I think this is on the verge of descending into a religious war of the form of where whitespace should go. So I'll not comment on this any further. Stephen Kellett -- C++/Java/Win NT/Unix variants Memory leaks/corruptions/performance/system problems. UK based. Problems with RSI/WRULD? Contact me for advice.

                                        S 1 Reply Last reply
                                        0
                                        • S Stephen Kellett

                                          I replied to this by email, but suffice to say that I agree with the points Simon makes except that I think Hungarian notation sucks. Despite having made a successful career start with computer games in 1983 and finding that the majority of people (that I know or have worked with) whose ability I respect do not use hungarian notation, I was rather surpsied to read the Hungarian notation thread here where most of the posters state they use it, although I was pleased to read that most only use it where they felt needed, rather than the impression I got from Simon, which (I may be incorrect) was "must be used all the time". I'm much more in favour of proper descriptive variable names than a cryptic lpcszName, I mean the hungarian part is longer than 'Name'! I think this is on the verge of descending into a religious war of the form of where whitespace should go. So I'll not comment on this any further. Stephen Kellett -- C++/Java/Win NT/Unix variants Memory leaks/corruptions/performance/system problems. UK based. Problems with RSI/WRULD? Contact me for advice.

                                          S Offline
                                          S Offline
                                          Simon Brown
                                          wrote on last edited by
                                          #31

                                          I believe in standards - and use Hungarian everywhere, and will until something better comes along. Of course, I'm only writing in C and C++ with Hungarian. I farm out my Java and HTML work. ;) What is a tad absurd is that we call ourselves 'professional', yet don't have a standard for the work we do. Old Simon

                                          J 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