Click here to Skip to main content
15,867,453 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: You know it's time to go home... Pin
StM0n10-Jul-12 19:40
StM0n10-Jul-12 19:40 
GeneralRe: You know it's time to go home... Pin
SoMad10-Jul-12 11:51
professionalSoMad10-Jul-12 11:51 
GeneralRe: You know it's time to go home... Pin
henrikqwerty32125-Jul-12 14:52
henrikqwerty32125-Jul-12 14:52 
GeneralDictionary Misuse PinPopular
Martin Thwaites9-Jul-12 4:22
Martin Thwaites9-Jul-12 4:22 
GeneralRe: Dictionary Misuse Pin
Henning Dieterichs9-Jul-12 8:58
Henning Dieterichs9-Jul-12 8:58 
GeneralRe: Dictionary Misuse Pin
PIEBALDconsult9-Jul-12 10:55
mvePIEBALDconsult9-Jul-12 10:55 
GeneralRe: Dictionary Misuse Pin
Harley L. Pebley10-Jul-12 5:16
Harley L. Pebley10-Jul-12 5:16 
GeneralRe: Dictionary Misuse Pin
Martin Thwaites10-Jul-12 5:24
Martin Thwaites10-Jul-12 5:24 
I wasn't saying it was a hotspot, just that it's poor use of code. It's actually nowhere near to being a hotspot (they have 1 particular click in the application that takes 15 minutes to load the page, and therefore only works when there is a decent internet connection and a high timeout set!).

The object thing is FAR worse, however, in the context of how they've wrote the application it does make sense. That is actually one of the things I'm tackling at the moment.

This is one of many issues I've found that they've put in (before I started reviewing the code), and I suspect that they will be doing less now.

I also appreciate it can't be replaced by exactly what I've said based on the code sample, and in the application itself it would actually need to be different than you've suggested.
GeneralRe: Dictionary Misuse Pin
Brisingr Aerowing10-Jul-12 6:16
professionalBrisingr Aerowing10-Jul-12 6:16 
AnswerRe: Dictionary Misuse Pin
Harley L. Pebley10-Jul-12 6:21
Harley L. Pebley10-Jul-12 6:21 
GeneralRe: Dictionary Misuse Pin
James Lonero11-Jul-12 4:57
James Lonero11-Jul-12 4:57 
GeneralRe: Dictionary Misuse Pin
Florin Jurcovici12-Jul-12 22:02
Florin Jurcovici12-Jul-12 22:02 
GeneralRe: Dictionary Misuse Pin
ely_bob10-Jul-12 9:55
professionalely_bob10-Jul-12 9:55 
GeneralGreat implementation Pin
cansino6-Jul-12 2:53
cansino6-Jul-12 2:53 
GeneralRe: Great implementation Pin
Bernhard Hiller6-Jul-12 4:27
Bernhard Hiller6-Jul-12 4:27 
GeneralRe: Great implementation Pin
PIEBALDconsult6-Jul-12 5:40
mvePIEBALDconsult6-Jul-12 5:40 
GeneralRe: Great implementation Pin
cansino8-Jul-12 21:19
cansino8-Jul-12 21:19 
GeneralRe: Great implementation Pin
OriginalGriff8-Jul-12 21:46
mveOriginalGriff8-Jul-12 21:46 
GeneralRe: Great implementation Pin
Henning Dieterichs9-Jul-12 5:45
Henning Dieterichs9-Jul-12 5:45 
JokeToString() and ToUpper() are like, so cool! Pin
Nathan D Cook5-Jul-12 9:27
Nathan D Cook5-Jul-12 9:27 
GeneralRe: ToString() and ToUpper() are like, so cool! Pin
CDP18025-Jul-12 10:17
CDP18025-Jul-12 10:17 
GeneralRe: ToString() and ToUpper() are like, so cool! Pin
OriginalGriff5-Jul-12 21:03
mveOriginalGriff5-Jul-12 21:03 
GeneralRe: ToString() and ToUpper() are like, so cool! Pin
Gary R. Wheeler6-Jul-12 1:47
Gary R. Wheeler6-Jul-12 1:47 
GeneralRe: ToString() and ToUpper() are like, so cool! Pin
OriginalGriff6-Jul-12 2:47
mveOriginalGriff6-Jul-12 2:47 
GeneralRe: ToString() and ToUpper() are like, so cool! Pin
Gary R. Wheeler6-Jul-12 3:16
Gary R. Wheeler6-Jul-12 3:16 

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.