Ehh... I'm not really all that surprised. That's how half my code starts if I'm doing something where I'm not familiar with the services I'm using. Once I get

Ehh... I'm not really all that surprised. That's how half my code starts if I'm doing something where I'm not familiar with the services I'm using. Once I get it where I'm like, "Oh yeah, that's how this will come together" -- then I start refactoring. The fact that the user kept TextBox1, Button1_Click, Label1 -- makes me think the assignment included something like, "just make it work -- don't worry about cleaning it up". And in the clean up phase you do the renaming, parameterized queries, moving the logic out of the event handler, doing the SQL async w/ visual feedback, etc... Everything here tells me this person didn't spend much time on it. Not that they're not a good developer. And I personally love when my devs show me early code. I don't ever want them to be afraid to show me something for feedback because it isn't cleaned up.