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.

FirstPrevNext

Re: Its always good to check for nothing..... Bernhard Hiller27-Jul-14 21:14 Bernhard Hiller 27-Jul-14 21:14  Did the guy face some problems with lazy loading? The optimizer might remove an empty foreach, hence he accesses a property of every item. Why not create a ForceLoad function in the list, and call it? - Well, obfuscating the intent of some code is such a great idea. Re: Its always good to check for nothing..... TheGreatAndPowerfulOz30-Jul-14 6:19 TheGreatAndPowerfulOz 30-Jul-14 6:19  It's invisible NSA code If your actions inspire others to dream more, learn more, do more and become more, you are a leader.-John Q. Adams You must accept one of two basic premises: Either we are alone in the universe, or we are not alone in the universe. And either way, the implications are staggering.-Wernher von Braun Only two things are infinite, the universe and human stupidity, and I'm not sure about the former.-Albert Einstein Sometimes, I'm too clever for my own good... OriginalGriff24-Jul-14 0:00 OriginalGriff 24-Jul-14 0:00  It's official - I have too many eBooks to cope with without help, so I decided to knock up a quick SQL DB app to organise them (and play with creating a new DataGridView based UserControl with automatic fuzzy filtering). So I've just spent the last hour or so wondering where my data is / was / should be: C# string[] files = Directory.GetFiles(path, "*.*", SearchOption.AllDirectories); IEnumerable<Book> books = files.Select(f => new Book(f, path)); Author.SaveAllNew(); Series.SaveAllNew(); Book.SaveAllNew(); The Book constructor determines the Series and Author and creates them as needed. So where is everybody? My DB is still empty... ...later. ...much later. ...until I remembered Linq methods use deferred execution, that is... C# string[] files = Directory.GetFiles(path, "*.*", SearchOption.AllDirectories); IEnumerable<Book> books = files.Select(f => new Book(f, path)).ToList(); Author.SaveAllNew(); Series.SaveAllNew(); Book.SaveAllNew(); Those who fail to learn history are doomed to repeat it. --- George Santayana (December 16, 1863 – September 26, 1952) Those who fail to clear history are doomed to explain it. --- OriginalGriff (February 24, 1959 – ∞) Re: Sometimes, I'm too clever for my own good... Nagy Vilmos24-Jul-14 0:30 Nagy Vilmos 24-Jul-14 0:30  :confusedFaceAtMisleadingTitle: Re: Sometimes, I'm too clever for my own good... Richard Deeming24-Jul-14 2:11 Richard Deeming 24-Jul-14 2:11  Assuming you're using .NET 4.0 or higher, Directory.EnumerateFiles[^] is more efficient than Directory.GetFiles[^], as it doesn't have to allocate an array to contain every file path. Also, unless you specifically want to filter out files without extensions, use "*" instead of "*.*" as your search pattern. And since you don't seem to be using the result of the ToList call, it would probably be better to avoid creating a list. Unfortunately, there's no built-in Consume method, but it's not hard to write one: C# public static class NomNomNom { public static void Consume<T>(this IEnumerable<T> source) { if (source == null) throw new ArgumentNullException("source"); foreach (T item in source); } } "These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer Re: Sometimes, I'm too clever for my own good... OriginalGriff24-Jul-14 2:19 OriginalGriff 24-Jul-14 2:19  I use the list later on for UI updates - I just didn't show that part! Those who fail to learn history are doomed to repeat it. --- George Santayana (December 16, 1863 – September 26, 1952) Those who fail to clear history are doomed to explain it. --- OriginalGriff (February 24, 1959 – ∞) Re: Sometimes, I'm too clever for my own good... Argonia24-Jul-14 2:29 Argonia 24-Jul-14 2:29  One does not simply lecture OG on programming . Microsoft ... the only place where VARIANT_TRUE != true Re: Sometimes, I'm too clever for my own good... Richard Deeming24-Jul-14 3:00 Richard Deeming 24-Jul-14 3:00  Oh, come on! He describes himself as the CEO of Wales. When was the last time you met a CEO who knew the first thing about writing code? "These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer Re: Sometimes, I'm too clever for my own good... Argonia24-Jul-14 3:20 Argonia 24-Jul-14 3:20  Richard Deeming wrote: When was the last time you met a CEO who knew the first thing about writing code? I couldn't know after all I work as a code monkey. I don't meet such people like CEOs. But your post was funny Microsoft ... the only place where VARIANT_TRUE != true Re: Sometimes, I'm too clever for my own good... PIEBALDconsult24-Jul-14 5:18 PIEBALDconsult 24-Jul-14 5:18  Argonia wrote: I don't meet such people like CEOs Undo another button or two. You'll never get very far if all you do is follow instructions. Re: Sometimes, I'm too clever for my own good... OriginalGriff24-Jul-14 5:45 OriginalGriff 24-Jul-14 5:45  He can if he wants - I make mistakes like everyone else, and am both very aware and very thankful that I don't know everything. Mind you, I did know everything when I was 18... Those who fail to learn history are doomed to repeat it. --- George Santayana (December 16, 1863 – September 26, 1952) Those who fail to clear history are doomed to explain it. --- OriginalGriff (February 24, 1959 – ∞) Re: Sometimes, I'm too clever for my own good... V.24-Jul-14 19:36 V. 24-Jul-14 19:36  OriginalGriff wrote: when I was 18... That was surely before the dawn of men (and thus before the age of the personal computer) V. (MQOTD rules and previous solutions) Re: Sometimes, I'm too clever for my own good... OriginalGriff24-Jul-14 22:19 OriginalGriff 24-Jul-14 22:19  Pre-PC indeed, but not Pre-DoM: I'm not 'Enry Minute! Those who fail to learn history are doomed to repeat it. --- George Santayana (December 16, 1863 – September 26, 1952) Those who fail to clear history are doomed to explain it. --- OriginalGriff (February 24, 1959 – ∞) Re: Sometimes, I'm too clever for my own good... CBadger27-Jul-14 21:50 CBadger 27-Jul-14 21:50  OriginalGriff wrote: Pre-PC technology indeed, FTFY »»» <small>Loading Signature</small> ««« · · · <small>Please Wait</small> · · ·             System.Data.IDataParameterCollection.Item PIEBALDconsult15-Jul-14 7:15 PIEBALDconsult 15-Jul-14 7:15  This continues to irk me no end. Due to the lack of generics in .net V1, IDataParameterCollection.Item is of type object rather than IDataParameter, so I have to keep casting. You'll never get very far if all you do is follow instructions. Re: System.Data.IDataParameterCollection.Item dan!sh 19-Jul-14 12:06 dan!sh 19-Jul-14 12:06  Many still create their own collections the same way. I have no clue why. My CP workspace: Incredibly trivial and probably useless code samples[^] Re: System.Data.IDataParameterCollection.Item johannesnestler24-Jul-14 4:30 johannesnestler 24-Jul-14 4:30  I know... I had to add a "Don't use the obsolete collection pattern" chapter to the coding style guide for my team... Re: System.Data.IDataParameterCollection.Item ZurdoDev24-Jul-14 4:54 ZurdoDev 24-Jul-14 4:54  PIEBALDconsult wrote: This continues to irk me no end. Due to the lack of generics in .net V1 It did for me too. That is until 2005 when .Net 2.0 came out. Just saying. There are only 10 types of people in the world, those who understand binary and those who don't. Re: System.Data.IDataParameterCollection.Item PIEBALDconsult24-Jul-14 5:02 PIEBALDconsult 24-Jul-14 5:02  Yes, but they didn't add a generic alternative to the collection so I still have to cast even today. You'll never get very far if all you do is follow instructions. Expression containment field... Duncan Edwards Jones14-Jul-14 5:43 Duncan Edwards Jones 14-Jul-14 5:43  C# if (((cmd != null))) Not so much "belt and braces" and "braces and braces and braces" code. Re: Expression containment field... PIEBALDconsult14-Jul-14 5:47 PIEBALDconsult 14-Jul-14 5:47  Macro replacement? You'll never get very far if all you do is follow instructions. Re: Expression containment field... Duncan Edwards Jones14-Jul-14 6:07 Duncan Edwards Jones 14-Jul-14 6:07  Probably - there's a flock of seagulls in the same file. Re: Expression containment field... Nagy Vilmos16-Jul-14 22:43 Nagy Vilmos 16-Jul-14 22:43  I wondered what had happened to them. Re: Expression containment field... Rob Grainger20-Jul-14 22:40 Rob Grainger 20-Jul-14 22:40  I think they got arrested by the bad hair squad.. Bad Hair Day[^] "If you don't fail at least 90 percent of the time, you're not aiming high enough." Alan Kay. Re: Expression containment field... Dave Kreskowiak14-Jul-14 6:41 Dave Kreskowiak 14-Jul-14 6:41  That containment field seems a little weak. Tack on a few more pairs to be sure it safely contains that massive expression. A guide to posting questions on CodeProject Click this: Asking questions is a skill. Seriously, do it. Dave Kreskowiak

Last Visit: 31-Dec-99 18:00     Last Update: 2-May-24 2:49 Refresh ᐊ Prev1...218219220221222223224225226227 Next ᐅ

General    News    Suggestion    Question    Bug    Answer    Joke    Praise    Rant    Admin   

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