Good Practise?
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
The code should reflect the most recent functionality, and comments should describe what the code is intended to do. Use version control (e.g., Subversion and TortoiseSVN) to keep track of changes to code. Do not leave old code commented out in your new code. Delete it.
-
The code should reflect the most recent functionality, and comments should describe what the code is intended to do. Use version control (e.g., Subversion and TortoiseSVN) to keep track of changes to code. Do not leave old code commented out in your new code. Delete it.
Thank you. As many know I am an accountant by trade, not a developer, but learning quickly and it is little things like this that help. Thanks for your time.
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
We always remove old code, since everything is in a source control system. If need be, we can look back over changes, compare contents, etc.etc.etc.
:..::. Douglas H. Troy ::..
Bad Astronomy |VCF|wxWidgets|WTL -
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
Delete it. You shouldn't leave old code stinking up the woodpile.
"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.
-
We always remove old code, since everything is in a source control system. If need be, we can look back over changes, compare contents, etc.etc.etc.
:..::. Douglas H. Troy ::..
Bad Astronomy |VCF|wxWidgets|WTLDouglas Troy wrote:
everything is in a source control system
Versioning is for the weak of memory.
"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.
-
Douglas Troy wrote:
everything is in a source control system
Versioning is for the weak of memory.
"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.
Better solution?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
Delete! To see the evolution, look in source repository!
Fight Big Government:
http://obamacareclassaction.com/
http://obamacaretruth.org/ -
Delete! To see the evolution, look in source repository!
Fight Big Government:
http://obamacareclassaction.com/
http://obamacaretruth.org/seems to be the concensus.
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
Better solution?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
I'm fairly certain he's joking. :)
-
I'm fairly certain he's joking. :)
They don't have sarcasm on my planet.
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
They don't have sarcasm on my planet.
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
Really? What planet are you from?
-
Really? What planet are you from?
Fintlewoodlewix
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
The code should reflect the most recent functionality, and comments should describe what the code is intended to do. Use version control (e.g., Subversion and TortoiseSVN) to keep track of changes to code. Do not leave old code commented out in your new code. Delete it.
right. keeping old code, even in comment, confuses the reader, results in false search hits, and in general has no use at all. Less is definitely more. and that also applies to old comments! :)
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
Prolific encyclopedia fixture proof-reader browser patron addict?
We all depend on the beast below.
-
Really? What planet are you from?
I guess DD is from Dark Drivelus, an obscure nebula beyond Orion; some call it Skaro. :)
Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]
Prolific encyclopedia fixture proof-reader browser patron addict?
We all depend on the beast below.
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
Delete it and preserve the history in your revision control system. Lots of commented out code looks extremely unprofessional in a production code base IMO.
And above all things, never think that you're not good enough yourself. A man should never think that. My belief is that in life people will take you at your own reckoning. --Isaac Asimov Avoid the crowd. Do your own thinking independently. Be the chess player, not the chess piece. --Ralph Charell
-
Douglas Troy wrote:
everything is in a source control system
Versioning is for the weak of memory.
"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.
Pete O'Hanlon wrote:
Versioning is for the weak of memory.
That would be me. :) As my wife likes to teas me the most common phrase I say to her is "What are we taking about?"
And above all things, never think that you're not good enough yourself. A man should never think that. My belief is that in life people will take you at your own reckoning. --Isaac Asimov Avoid the crowd. Do your own thinking independently. Be the chess player, not the chess piece. --Ralph Charell
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
As many have already said, delete it and use your source control system to be able to get back to it if needed. I would also add, decent log messages for commits are a big plus - makes it a lot easier to find that old code! Dybs
The shout of progress is not "Eureka!" it's "Strange... that's not what i expected". - peterchen
-
Thank you. As many know I am an accountant by trade, not a developer, but learning quickly and it is little things like this that help. Thanks for your time.
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
-
Pete O'Hanlon wrote:
Versioning is for the weak of memory.
That would be me. :) As my wife likes to teas me the most common phrase I say to her is "What are we taking about?"
And above all things, never think that you're not good enough yourself. A man should never think that. My belief is that in life people will take you at your own reckoning. --Isaac Asimov Avoid the crowd. Do your own thinking independently. Be the chess player, not the chess piece. --Ralph Charell
-
When you alter/edit/revise code, which is better, to delete the old code or to comment it out so you can see the evolution when you go back some time later?
------------------------------------ I will never again mention that I was the poster of the One Millionth Lounge Post, nor that it was complete drivel. Dalek Dave
First time around I leave the code in using comments pending the code review so what has been done can be compared to what has previously been done. After review and stable build, release, I routinely remove the old code because, as others have said, that is what source control is for. Also, keep in mind, this is why comments in source control check-ins are CRUCIAL, yet so many are way too lazy to bother.
Need custom software developed? I do custom programming based primarily on MS tools with an emphasis on C# development and consulting. A man said to the universe: "Sir I exist!" "However," replied the universe, "The fact has not created in me A sense of obligation." --Stephen Crane