| Performance - Could
be refactored into a named static inner class |
| Performance - Could be refactored into a
static inner class |
| Performance - Explicit garbage
collection; extremely dubious except in benchmarking code |
| Performance - Huge string constants is
duplicated across multiple class files |
| Performance - Inefficient use of keySet
iterator instead of entrySet iterator |
| Performance - Maps and sets of URLs can
be performance hogs |
| Performance - Method allocates a boxed
primitive just to call toString |
Performance - Method allocates an object,
only to get the class object
|
| Performance - Method calls static Math
class method on a constant value |
| Performance - Method concatenates strings
using + in a loop |
| Performance - Method invokes inefficient
Boolean constructor; use Boolean.valueOf(...) instead |
| Performance - Method invokes inefficient
floating-point Number constructor; use static valueOf instead |
| Performance - Method invokes inefficient
new String() constructor |
| Performance - Method invokes inefficient
new String(String) constructor |
| Performance - Method invokes inefficient
Number constructor; use static valueOf instead |
| Performance - Method invokes toString()
method on a String |
| Performance - Method uses toArray() with
zero-length array argument |
| Performance - Primitive value is boxed
and then immediately unboxed |
| Performance - Primitive value is boxed
then unboxed to perform primitive coercion |
| Performance - Private method is never
called |
| Performance - Should be a static inner
class |
| Performance - The equals and hashCode
methods of URL are blocking |
| Performance - Unread field |
| Performance - Unread field: should this
field be static? |
| Performance - Unused field |
| Performance - Use the nextInt method of
Random rather than nextDouble to generate a random integer |