A quick list of the things I've seen: ON ERROR RESUME starting every routine Using 70 textboxes to simulate a spreadsheet Everytime the database was opened it was never closed Everytime data was retrieved it was using 'Select * from', even when getting 1 field Variables declared and never used, and Variables used that were never Declared, both in the same routines. Creating temporary records in the database and never deleting them. A whole section dedicated to an internal email system, in a program where a majority of the clientele were mom & pop used car dealers with 1 computer. All of these were in the same code I was hired to maintain. I eventually gave up trying to wade through fixing it and re-wrote the entire program in my spare time.

I have a 32" Visio lcd TV I use as my programming monitor, with Visual Studio maximized, it makes it easier to see everything in the development environment. I also have 2 19" CRT monitors. One is used for testing programs in various resolutions, and the other has the email and IM clients running, so I don't have to keep swapping out windows.