Doing things the right way considered harmful
So, let's suppose you're a C# developer who's built a pretty extensive class library in support of a pretty complicated application, and you finally have to take a whack at giving it a UI. And the natural idiom for presenting one of your collections of objects is in a DataGridView. It's simplicity itself to hook up your KeyedCollection to a BindingSource, attach the BindingSource to the DataGridView, create some columns, and in minutes - really, minutes, especially if you've done this once or twice before - you have something that resembles a UI.
But there are a lot of objects in your collection. The UI's really only going to be usable if you can sort them. No problem, right? Just click on the column header and...nothing happens.
And now, friend, you are in a world of pain. Because while anything that implements IList can be iterated by a BindingSource, if you want to sort, or filter, your collection needs to implement IBindingList.
And my god, is IBindingList elaborate. Because it has to provide all of the services whose implementation details BindingSource is hiding from you: adding new items, sorting, filtering, raising events when items change...
And when you start browsing around the interwebs, you find things like open-source projects to produce a BindingList<T>, which is kind of a cool thing if you a) want to incorporate a lot of other people's code in your project and b) figure out how BindingList<T> implements IBindingList and c) implement IComparers and d) well, spend a whole lot of time debugging and testing.
And there's a book you can buy that has a whole chapter devoted to the exciting ways that generic templates and interfaces can eat a week of your life so that your DataGridView can, as the Ruby on Rails folks like to say, just work.
Although, man, it's gonna be sweet that you can create an instance of BindingList<T> and hook up its PropertyDescriptors (once you figure out how PropertyDescriptor works) up with IComparers (once you've written them) to make your collection sortable alphabetically any way you want!
No. Stop. Step away from the architecture.
Do not get seduced by the appeal of spending a week making sure your widget implements all the right methods the right way, raises the right events, works with any type T, in short, getting every last detail of the IBindingList implementation right and then some. You'll feel very smart that you've built a piece of software that Microsoft should have in the first place, but you'll be left with this nagging feeling:
Why didn't they?
Why, in the enormous thicket of code that is the .Net framework, isn't there a class that you can put your objects into and bind to a DataGridView and have it just work?
If you ask the question the right way, the answer's obvious: there is such a class. It's the one that the BindingSource (and DataGridView) has clearly been designed from the ground up to interoperate with: the DataTable.
Create an unbound DataSource, add a DataTable to it, write a little method to read through your collection and create a row for each object and populate its columns with properties.
Look how you now have something that implements IBindingList! You can sort! You can filter! You can insert and delete rows! Why, it's almost like a table that contains data!
Yes, it's true that if your collection has ten thousand objects in it, copying their properties into a DataTable is going to be less than instantaneous, and use up twice as much memory (or more) than your tidy little collection did. And while you don't have to implement any methods to deal with insertion and deletion, since the DataTable already knows a thing or two about that, you're going to have to write event handlers on the BindingSource that update your collection (since the BindingSource isn't bound to it).
On the other hand, that just took you thirty minutes, not forty hours.
("But wait," you say, "what if I derive a class from DataTable and overload its NewRow method?" Step away from the keyboard.)
0 TrackBacks
Listed below are links to blogs that reference this entry: Doing things the right way considered harmful.
TrackBack URL for this entry: http://www.koaxkoaxkoax.com/cgi-sys/cgiwrap/uhhhclem/managed-mt/mt-tb.cgi/12

Leave a comment