My first thoughts are too many lines of code in a method. In most cases it's easy to break something apart into multiple methods at least.
I have many more, but that seems to be one of the simplest to avoid. It really bugs me to see a method with 1000 lines of code, this also seems to be a common problem. Anything where you need to scroll up/down is IMO should be begging you break it apart.
But having a list issues could help a lot of new developers learn to identify problems and provide some good practices.