I am working with a codebase that is just…. braindamaged.
352 classes and NOT A SINGLE LINE OF COMMENTS!!!
Added to this loverly fact is code like this:
Iterator it = set.iterator(); for (readfileindex = 0; it.hasNext(); readfileindex++) { String filename = (String) it.next(); //do something with it }
Anybody with a bit of programming experience will realize that, though this block is valid, it makes NO LOGICAL SENSE. Especially when you realize that the readfileindex is used somewhere else for UI purposes. It violates about every common sense rule for effective programming.
joy.