In addition to what has been said (e.g. nolock, nesting views and procs, poor formatting etc)

I don't like it when stored procedures are called "sprocs"

I don't like calling a server by some nickname (e.g. prod2 when the box name is prod4) - talk about confusion there.

Here's one more - Devs with SA access when the DBA doesn't