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) |