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. You've got to be kidding me

You've got to be kidding me

Scheduled Pinned Locked Moved The Lounge
comquestion
25 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.
  • J Joe Woodbury

    If the code is clear, well formatted and well written, who cares? I've found modifying third party libraries generally a bad idea anyway since it makes merging patches and new versions a nightmare.

    Anyone who thinks he has a better idea of what's good for people than people do is a swine. - P.J. O'Rourke

    J Offline
    J Offline
    James R Twine
    wrote on last edited by
    #15

    Joe Woodbury wrote:

    If the code is clear, well formatted and well written, who cares?

    Clear and well formatted according to whose coding standards and experience?  For example, I have seen tons of older code from older Unix side, "Maddog-looking" developers with math degrees that looks completely clear and well written to them, but looks like pages of complex mathematical formulas to me due to the formatting and terse identifier names...!    Also, you could have clear, well formatted, and "well written" code that is absolutely incorrect and buggy as hell.  Without comments, how do you know what the code is supposed to be doing?  Simply looking at raw code just tells you what the code is doing and how it is doing it, but not what it is supposed to be doing.    Also, not everyone understands that iter may be a prefix for an STL iterator, and it may not be clear if the increment operator being applied in a particular situation is supposed to be prefix, or if it is supposed to be postfix.  Comments, when properly used to explain and/or describe the intent of the code, allow things like this to be spotted quickly and easily.    Of course, this assumes that the comments are correct and written in accordance with whatever language is common for the target audience, but that is another problem in itself! :) :) Note: John "Maddog" Hall is a well-known Linux evangelist who used to work near me at DEC.  I mean no offense to him, I have lots of respect for him, but if you have ever met him, you know what I mean by "Maddog-looking".    Peace!

    -=- James


    If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
    Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
    DeleteFXPFiles & CheckFavorites (Please rate this post!)

    1 Reply Last reply
    0
    • P peterchen

      Without comments, it compiles faster. Have you ever looked at STL sources? :rolleyes:


      Some of us walk the memory lane, others plummet into a rabbit hole
      Tree in C# || Fold With Us! || sighist

      J Offline
      J Offline
      James R Twine
      wrote on last edited by
      #16

      No, it pre-processes faster! :) :laugh: :-D    (Yes, I know that the two normally go hand-in-hand.)    Peace!

      -=- James


      If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
      Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
      DeleteFXPFiles & CheckFavorites (Please rate this post!)

      W P 2 Replies Last reply
      0
      • J James R Twine

        No, it pre-processes faster! :) :laugh: :-D    (Yes, I know that the two normally go hand-in-hand.)    Peace!

        -=- James


        If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
        Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
        DeleteFXPFiles & CheckFavorites (Please rate this post!)

        W Offline
        W Offline
        Warren Stevens
        wrote on last edited by
        #17

        James R. Twine wrote:

        No, it pre-processes faster!

        There is no pre-processor in C# But not having comments will make code (any language) compile or pre-process faster (i.e. less characters to look at, smaller files) Not that that is a good reason to not add comments...

        J 1 Reply Last reply
        0
        • M Marc Clifton

          So, my client purchased the DevExpress "experience" thing with source. So for giggles, I pick the TreeList control to look at. 189K in one .cs file. That's the first warning sign. And in that 189K, there is one single comment. The copyright notice/warning/threat. You're kidding me, right? I mean, this isn't the same code that they maintain inhouse, right? Please tell me it ain't so!!! They strip out all the comments when they ship the code, right? Christ. And people say open source is bad. I'll tell you something, open source is WAY BETTER than 189K in one .cs file that is TOTALLY UNDOCUMENTED. :omg: :wtf: X| :sigh: Marc

          XPressTier

          Some people believe what the bible says. Literally. At least [with Wikipedia] you have the chance to correct the wiki -- Jörgen Sigvardsson

          People are just notoriously impossible. --DavidCrow

          V Offline
          V Offline
          Varindir Rajesh Mahdihar
          wrote on last edited by
          #18

          switch to opensource and linux then.

          M 1 Reply Last reply
          0
          • W Warren Stevens

            James R. Twine wrote:

            No, it pre-processes faster!

            There is no pre-processor in C# But not having comments will make code (any language) compile or pre-process faster (i.e. less characters to look at, smaller files) Not that that is a good reason to not add comments...

            J Offline
            J Offline
            James R Twine
            wrote on last edited by
            #19

            Yes, I was referring to C/C++-kind of setups where a seperate preprocessor step is used.    And yes, I know that the lexing of the contents of a file is faster without ignored content (like comments or whitespace for that matter).  (Actually, that is an interesting thought - comments are used by Intellisense/Codesense-type features...)    Peace!

            -=- James


            If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
            Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
            DeleteFXPFiles & CheckFavorites (Please rate this post!)

            1 Reply Last reply
            0
            • V Varindir Rajesh Mahdihar

              switch to opensource and linux then.

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

              Varindir Rajesh Mahdihar wrote:

              switch to opensource and linux then.

              The point is, I expected better source code quality from a toolkit one spends over a thousand of dollars on, per developer. I expected to learn neato things. As usual, my expectations lead to disappointments. Marc

              XPressTier

              Some people believe what the bible says. Literally. At least [with Wikipedia] you have the chance to correct the wiki -- Jörgen Sigvardsson
              People are just notoriously impossible. --DavidCrow
              There's NO excuse for not commenting your code. -- John Simmons / outlaw programmer

              R 1 Reply Last reply
              0
              • J James R Twine

                No, it pre-processes faster! :) :laugh: :-D    (Yes, I know that the two normally go hand-in-hand.)    Peace!

                -=- James


                If you think it costs a lot to do it right, just wait until you find out how much it costs to do it wrong!
                Avoid driving a vehicle taller than you and remember that Professional Driver on Closed Course does not mean your Dumb Ass on a Public Road!
                DeleteFXPFiles & CheckFavorites (Please rate this post!)

                P Offline
                P Offline
                peterchen
                wrote on last edited by
                #21

                :doh: you are right. 5 :)


                Some of us walk the memory lane, others plummet into a rabbit hole
                Tree in C# || Fold With Us! || sighist

                1 Reply Last reply
                0
                • M Marc Clifton

                  Varindir Rajesh Mahdihar wrote:

                  switch to opensource and linux then.

                  The point is, I expected better source code quality from a toolkit one spends over a thousand of dollars on, per developer. I expected to learn neato things. As usual, my expectations lead to disappointments. Marc

                  XPressTier

                  Some people believe what the bible says. Literally. At least [with Wikipedia] you have the chance to correct the wiki -- Jörgen Sigvardsson
                  People are just notoriously impossible. --DavidCrow
                  There's NO excuse for not commenting your code. -- John Simmons / outlaw programmer

                  R Offline
                  R Offline
                  Roger Alsing 0
                  wrote on last edited by
                  #22

                  Now come one it cant be that bad, show the code to us ;) :P

                  M 1 Reply Last reply
                  0
                  • R Roger Alsing 0

                    Now come one it cant be that bad, show the code to us ;) :P

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

                    Here you go:

                    #region Copyright (c) 2000-2006 Developer Express Inc.
                    /*
                    {*******************************************************************}
                    { }
                    { Developer Express .NET Component Library }
                    { XtraTreeList }
                    { }
                    { Copyright (c) 2000-2006 Developer Express Inc. }
                    { ALL RIGHTS RESERVED }
                    { }
                    { The entire contents of this file is protected by U.S. and }
                    { International Copyright Laws. Unauthorized reproduction, }
                    { reverse-engineering, and distribution of all or any portion of }
                    { the code contained in this file is strictly prohibited and may }
                    { result in severe civil and criminal penalties and will be }
                    { prosecuted to the maximum extent possible under the law. }
                    { }
                    { RESTRICTIONS }
                    { }
                    { THIS SOURCE CODE AND ALL RESULTING INTERMEDIATE FILES }
                    { ARE CONFIDENTIAL AND PROPRIETARY TRADE }
                    { SECRETS OF DEVELOPER EXPRESS INC. THE REGISTERED DEVELOPER IS }
                    { LICENSED TO DISTRIBUTE THE PRODUCT AND ALL ACCOMPANYING .NET }
                    { CONTROLS AS PART OF AN EXECUTABLE PROGRAM ONLY. }
                    { }
                    { THE SOURCE CODE CONTAINED WITHIN THIS FILE AND ALL RELATED }
                    { FILES OR ANY PORTION OF ITS CONTENTS SHALL AT NO TIME BE }
                    { COPIED, TRANSFERRED, SOLD, DISTRIBUTED, OR OTHERWISE MADE }
                    { AVAILABLE TO OTHER INDIVIDUALS WITHOUT EXPRESS WRITTEN CONSENT }
                    { AND PERMISSION FROM DEVELOPER EXPRESS INC. }
                    { }
                    { CONSULT THE END USER LICENSE AGREEMENT FOR INFORMATION ON }
                    { ADDITIONAL RESTRICTIONS. }
                    { }
                    {*******************************************************************}
                    */
                    #endregion Copyright (c) 2000-2006 Developer Express Inc.

                    ;P Marc

                    1 Reply Last reply
                    0
                    • S Super Lloyd

                      Well.... I guess that depends ;-) There is a nice, fully managed, TreeControl which appeared recently on CodeProject! Otherwise I could email you mine too if you want. The one on code project has nice already build cell and a columned mode, whereas mine handle gracefully variable size cell.

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

                      Super Lloyd wrote:

                      The one on code project has nice already build cell and a columned mode

                      Yeah, I'm actually using it on a project. It's very cool.

                      Super Lloyd wrote:

                      whereas mine handle gracefully variable size cell.

                      I'd love to take a look. email away! :-D Marc

                      XPressTier

                      Some people believe what the bible says. Literally. At least [with Wikipedia] you have the chance to correct the wiki -- Jörgen Sigvardsson
                      People are just notoriously impossible. --DavidCrow
                      There's NO excuse for not commenting your code. -- John Simmons / outlaw programmer

                      S 1 Reply Last reply
                      0
                      • M Marc Clifton

                        Super Lloyd wrote:

                        The one on code project has nice already build cell and a columned mode

                        Yeah, I'm actually using it on a project. It's very cool.

                        Super Lloyd wrote:

                        whereas mine handle gracefully variable size cell.

                        I'd love to take a look. email away! :-D Marc

                        XPressTier

                        Some people believe what the bible says. Literally. At least [with Wikipedia] you have the chance to correct the wiki -- Jörgen Sigvardsson
                        People are just notoriously impossible. --DavidCrow
                        There's NO excuse for not commenting your code. -- John Simmons / outlaw programmer

                        S Offline
                        S Offline
                        Super Lloyd
                        wrote on last edited by
                        #25

                        Fired to your GMail address!! The project might some clean up.... ;) It was ... planned!

                        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