NewInstance

It was a late evening when I fall in buggy a code like this: Destination newDestination = null; if (! originalDestination.getId().equals(getDestinationId())); { newDestination = context.loadDestination(getDestinationId()); prefix.setDestination(newDestination); } context.save(prefix); if (newDestination != null) { newDestination.addPrefix(prefix); originalDestination.removePrefix(prefix); context.save(originalDestination); context.save(newDestination); } When the first if block is true, the second one will also be true, because the line [...]

That building Strings using StringBuffer (and in Java 5 StringBuilder) is much faster and less memory expensive than using String concatenation, is not a secret. But porting this concept to extremes, you can fall in coding conventions that mandate to never use String concatenation. And you can face hibernate queries like this: 1String alias = [...]