Click here to Skip to main content
15,445,997 members

The Weird and The Wonderful

   

The Weird and The Wonderful forum is a place to post Coding Horrors, Worst Practices, and the occasional flash of brilliance.

We all come across code that simply boggles the mind. Lazy kludges, embarrassing mistakes, horrid workarounds and developers just not quite getting it. And then somedays we come across - or write - the truly sublime.

Post your Best, your worst, and your most interesting. But please - no programming questions . This forum is purely for amusement and discussions on code snippets. All actual programming questions will be removed.

 
GeneralRe: Taking responsibility Pin
PIEBALDconsult27-Apr-22 11:45
professionalPIEBALDconsult27-Apr-22 11:45 
GeneralRe: Taking responsibility Pin
Andreas Mertens27-Apr-22 12:08
professionalAndreas Mertens27-Apr-22 12:08 
GeneralRe: Taking responsibility Pin
PIEBALDconsult27-Apr-22 13:11
professionalPIEBALDconsult27-Apr-22 13:11 
GeneralRe: Taking responsibility Pin
charlieg26-Jul-22 10:37
Membercharlieg26-Jul-22 10:37 
GeneralRe: Taking responsibility Pin
honey the codewitch30-Apr-22 2:15
mvahoney the codewitch30-Apr-22 2:15 
GeneralRe: Taking responsibility Pin
Andreas Mertens30-Apr-22 3:46
professionalAndreas Mertens30-Apr-22 3:46 
GeneralRe: Taking responsibility Pin
Richard Deeming27-Apr-22 21:20
mveRichard Deeming27-Apr-22 21:20 
GeneralRe: Taking responsibility Pin
Andreas Mertens28-Apr-22 4:05
professionalAndreas Mertens28-Apr-22 4:05 
Exactly. Mind you, some of this code is really old, and I think they started with Visual SourceSafe which had a lot of problems.

There are literally hundreds of these repos, and we use a number of automated processes to scan and report on these projects. Having the else extraneous projects in here causes all sorts of false positives as to the state of the code. It's important to have accurate reporting on all of these, as it impacts budgets and allocation time and developer resources to address these.

But as I said earlier, there should be some level of common sense on cleaning this up. As senior-level developers, there should be a trust that we can address these things without requiring convening a committee, schedule meetings, and getting senior management to make a decision for us.

This is just another form of YAGNI, but instead of adding code that isn't going to be used, it is removing code that is no longer used. And as developers we should address technical debt to clean up up any project we are working on. I liken this to the Boy Scouts, where we should leave the campsite/project cleaner and in better shape than we found it.

It is very common here for multiple teams to work on a single project. Add to that employee churn. So making sure that the code is a clean, concise and understandable is important for whoever has to look at the project the next time around.

GeneralRe: Taking responsibility Pin
charlieg26-Jul-22 10:39
Membercharlieg26-Jul-22 10:39 
GeneralRe: Taking responsibility Pin
Andreas Mertens26-Jul-22 12:57
professionalAndreas Mertens26-Jul-22 12:57 
GeneralRe: Taking responsibility Pin
charlieg27-Jul-22 10:32
Membercharlieg27-Jul-22 10:32 
GeneralRe: Taking responsibility Pin
Bernhard Hiller28-Apr-22 20:39
MemberBernhard Hiller28-Apr-22 20:39 
GeneralRe: Taking responsibility Pin
abmv3-May-22 7:15
professionalabmv3-May-22 7:15 
GeneralRe: Taking responsibility Pin
kmoorevs3-May-22 11:26
Memberkmoorevs3-May-22 11:26 
GeneralRe: Taking responsibility Pin
Andreas Mertens3-May-22 13:53
professionalAndreas Mertens3-May-22 13:53 
GeneralRe: Taking responsibility Pin
englebart1-Sep-22 14:54
professionalenglebart1-Sep-22 14:54 
GeneralRe: Taking responsibility Pin
jschell8-May-22 9:08
Memberjschell8-May-22 9:08 
GeneralPathYetAnotherMakeUniqueName Pin
Brisingr Aerowing4-Apr-22 7:56
professionalBrisingr Aerowing4-Apr-22 7:56 
GeneralRe: PathYetAnotherMakeUniqueName Pin
raddevus4-Apr-22 10:14
mvaraddevus4-Apr-22 10:14 
GeneralRe: PathYetAnotherMakeUniqueName Pin
Mircea Neacsu4-Apr-22 13:20
mvaMircea Neacsu4-Apr-22 13:20 
GeneralRe: PathYetAnotherMakeUniqueName Pin
Cpichols6-Apr-22 1:21
MemberCpichols6-Apr-22 1:21 
GeneralRe: PathYetAnotherMakeUniqueName Pin
deepok14-Apr-22 19:35
Memberdeepok14-Apr-22 19:35 
GeneralRe: PathYetAnotherMakeUniqueName Pin
ShawnVN6-Apr-22 7:16
MemberShawnVN6-Apr-22 7:16 
GeneralNull Strikes Again! (even with C# special operators) Pin
raddevus22-Feb-22 11:25
mvaraddevus22-Feb-22 11:25 
GeneralRe: Null Strikes Again! (even with C# special operators) Pin
Jon McKee22-Feb-22 11:44
professionalJon McKee22-Feb-22 11:44 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.