| Bad practice -
Abstract class defines covariant compareTo() method |
| Bad practice - Abstract class defines
covariant equals() method |
| Bad practice - Certain swing methods
needs to be invoked in Swing thread |
| Bad practice - Check for sign of bitwise
operation |
| Bad practice - Class defines clone() but
doesn't implement Cloneable |
| Bad practice - Class defines
compareTo(...) and uses Object.equals() |
| Bad practice - Class defines equals() and
uses Object.hashCode() |
| Bad practice - Class defines equals() but
not hashCode() |
| Bad practice - Class defines hashCode()
and uses Object.equals() |
| Bad practice - Class defines hashCode()
but not equals() |
| Bad practice - Class implements Cloneable
but does not define or use clone method |
| Bad practice - Class inherits equals()
and uses Object.hashCode() |
| Bad practice - Class is Externalizable
but doesn't define a void constructor |
| Bad practice - Class is not derived from
an Exception, even though it is named as such |
| Bad practice - Class is Serializable but
its superclass doesn't define a void constructor |
| Bad practice - Class is Serializable, but
doesn't define serialVersionUID |
| Bad practice - Class names shouldn't
shadow simple name of implemented interface |
| Bad practice - Class names shouldn't
shadow simple name of superclass |
| Bad practice - Classloaders should only
be created inside doPrivileged block |
| Bad practice - clone method does not call
super.clone() |
| Bad practice - Clone method may return
null |
| Bad practice - Comparator doesn't
implement Serializable |
| Bad practice - Comparison of String
objects using == or != |
| Bad practice - Comparison of String
parameter using == or != |
| Bad practice - Confusing method names |
| Bad practice - Covariant compareTo()
method defined |
| Bad practice - Covariant equals() method
defined |
| Bad practice - Creates an empty jar file
entry |
| Bad practice - Creates an empty zip file
entry |
| Bad practice - Dubious catching of
IllegalMonitorStateException |
| Bad practice - Empty finalizer should be
deleted |
| Bad practice - Equals checks for
noncompatible operand |
| Bad practice - equals method fails for
subtypes |
| Bad practice - Equals method should not
assume anything about the type of its argument |
| Bad practice - equals() method does not
check for null argument |
| Bad practice - Explicit invocation of
finalizer |
| Bad practice - Fields of immutable
classes should be final |
| Bad practice - Finalizer does not call
superclass finalizer |
| Bad practice - Finalizer does nothing but
call superclass finalizer |
| Bad practice - Finalizer nullifies
superclass finalizer |
| Bad practice - Finalizer nulls fields |
| Bad practice - Finalizer only nulls
fields |
| Bad practice - Iterator next() method
can't throw NoSuchElementException |
| Bad practice - Method doesn't override
method in superclass due to wrong package for parameter |
| Bad practice - Method ignores exceptional
return value |
| Bad practice - Method ignores results of
InputStream.read() |
| Bad practice - Method ignores results of
InputStream.skip() |
| Bad practice - Method invoked that should
be only be invoked inside a doPrivileged block |
| Bad practice - Method invokes dangerous
method runFinalizersOnExit |
| Bad practice - Method invokes
System.exit(...) |
| Bad practice - Method may fail to close
database resource |
| Bad practice - Method may fail to close
database resource on exception |
| Bad practice - Method may fail to close
stream |
| Bad practice - Method may fail to close
stream on exception |
| Bad practice - Method might drop
exception |
| Bad practice - Method might ignore
exception |
| Bad practice - Method with Boolean return
type returns explicit null |
| Bad practice - Needless instantiation of
class that only supplies static methods |
| Bad practice - Non-serializable class has
a serializable inner class |
| Bad practice - Non-serializable value
stored into instance field of a serializable class |
| Bad practice - Random object created and
used only once |
| Bad practice - Serializable inner class |
| Bad practice - serialVersionUID isn't
final |
| Bad practice - serialVersionUID isn't
long |
| Bad practice - serialVersionUID isn't
static |
| Bad practice - Static initializer creates
instance before all static final fields assigned |
| Bad practice - Store of non serializable
object into HttpSession |
| Bad practice - Superclass uses subclass
during initialization |
| Bad practice - Suspicious reference
comparison |
| Bad practice - The readResolve method
must be declared with a return type of Object. |
| Bad practice - toString method may return
null |
| Bad practice - Transient field that isn't
set by deserialization. |
| Bad practice - Unchecked type in generic
call |
| Bad practice - Usage of GetResource may
be unsafe if class is extended |
| Bad practice - Use of identifier that is
a keyword in later versions of Java |
| Bad practice - Use of identifier that is
a keyword in later versions of Java |
| Bad practice - Very confusing method
names (but perhaps intentional) |