>Again, the code has stuff that smell bad.
>
>don't return a message if there is an error. Throw a new exception and handle it.
>don't return bool. Return a enum or better, return the thing that you're interested in (the actual datarow or datatable)
>don't pass by ref. It adds lots of overhead and slows down your app
>don't use names like databaseParam. use database
>

I don't recall asking for a sermon ;-)