Your mixing database logic with UI logic.
Not usually a good idea.

Additionally as the guy said above, you're not disposing.
Use a using(){} block for the objects that implement IDisposable.

Wouldn't you want to dipose of your objects, or use using blocks, instead of waiting around for GC to pick em up?