This list summarizes the top 10 mistakes that Java developers frequently make.
#1. Convert Array to ArrayList
To convert an array to an ArrayList
, developers often do this:
Arrays.asList()
will return an ArrayList
which is a private static class inside Arrays
, it is not the java.util.ArrayList
class. The java.util.Arrays.ArrayList
class has set()
,get()
, contains()
methods, but does not have any methods for adding elements, so its size is fixed. To create a real ArrayList
, you should do:
The constructor of ArrayList
can accept a Collection type, which is also a super type forjava.util.Arrays.ArrayList
.
#2. Check If an Array Contains a Value
Developers often do:
The problem is there is no need to convert a list to set. It can as simple as:
or
The first one is more readable than the second one.
#3. Remove an Element from a List Inside a Loop
Consider the following code which removes elements during iteration:
The output is:
[b, d]
There is a serious problem in this method. When an element is removed, the size of the list shrinks and the index changes. So if you want to delete multiple elements inside a loop by using the index, that will not work properly.
You may know that using iterator is the right way to delete elements inside loops, and you know foreach loop in Java works like an iterator, but actually it is not. Consider the following code:
It will throw out ConcurrentModificationException.
Instead the following is OK:
.next()
must be called before .remove()
. In the foreach loop, compiler will make the .next()
called after the operation of removing element, which caused theConcurrentModificationException
.
#4. Hashtable vs HashMap
By conventions in algorithm, Hashtable is the name of the data structure. But in Java, the data structure's name is HashMapHashtable
and HashMap
is that Hashtable
is synchronized. So very often you don't need Hashtable
, instead HashMap
should be used.
HashMap vs. TreeMap vs. Hashtable vs. LinkedHashMap
Top 10 questions about Map
#5. Use Raw Type of Collection
In Java, raw type and unbounded wildcard type are easy to mixed together. Take Set for example, Set
is raw type, while Set<?>
is unbounded wildcard type.
Consider the following code which uses a raw type List
as a parameter:
This code will throw an exception:
Exception in thread "main" java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.String at ...
Using raw type collection is dangerous as the raw type collections skip the generic type checking and not safe. There are huge differences between Set
, Set<?>
, and Set<Object>
. Check out
Raw type vs. Unbounded wildcard and Type Erasure
#6. Access Level
Very often developers use public for class field. It is easy to get the field value by directly referencing, but this is a very bad design. The rule of thumb is giving access level for members as low as possible.
public, default, protected, and private
#7. ArrayList vs. LinkedList
When developers do not know the difference between ArrayList
and LinkedList
, they often useArrayList
, because it looks familiar. However, there is a huge performance difference between them. In brief, LinkedList
should be preferred if there are a large number of add/remove operations and there are not a lot of random access operations. Check out ArrayList
vs. LinkedList
to get more information about their performance if this is new to you.
#8. Mutable vs. Immutable
Immutable objects have many advantages such simplicity, safety, etc. But it requires a separate object for each distinct value, and too many objects might cause high cost of garbage collection. There should be a balance when choosing between mutable and immutable.
In general, mutable objects are used to avoid producing too many intermediate objects. One classic example is concatenating a large number of strings. If you use an immutable string, you would produce a lot of objects that are eligible for garbage collection immediately. This wastes time and energy on the CPU, using a mutable object the right solution (e.g. StringBuilder
).
There are other situations when mutable objects are desirable. For example passing mutable objects into methods lets you collect multiple results without jumping through too many syntactic hoops. Another example is sorting and filtering: of course, you could make a method that takes the original collection, and returns a sorted one, but that would become extremely wasteful for larger collections. (From dasblinkenlight's answer on Stack Overflow)
#9. Constructor of Super and Sub
This compilation error occurs because the default super constructor is undefined. In Java, if a class does not define a constructor, compiler will insert a default argument-less constructor for the class by default. If a constructor is defined in Super class, in this case Super(String s), compiler will not insert the default argument-less constructor. This is the situation for the Super class above.
The constructors of the Sub class, either with-argument or argument-less, will call the argument-less Super constructor. Since compiler tries to insert super() to the 2 constructors in the Sub class, but the Super's default constructor is not defined, compiler reports the error message.
To fix this problem, simply 1) add a Super() constructor to the Super class like
, or 2) remove the self-defined Super constructor.
#10. "" or Constructor?
String can be created by two ways:
What is the difference?
The following examples can provide a quick answer:
For more details about how they are allocated in memory, check out Create Java String Using ” ” or Constructor?.
Future Work
The list is based on my analysis of a large number of open source projects on GitHub, Stack Overflow questions, and popular Google queries. There is no evaluation to prove that they are precisely the top 10, but definitely they are very common. Please leave your comment, if you don’t agree with any part. I would really appreciate it if you could point out some other mistakes that are more common.
'security_downloads' 카테고리의 다른 글
5월 3주 프로그래밍 소식 (0) | 2014.05.17 |
---|---|
C++Now 2014 Presentations (0) | 2014.05.17 |
OSX 커맨드 라인 유저가 알아야할 터미널 유틸리티 정리 (0) | 2014.05.16 |
Arcus is the NAVER memcached cloud with list, set and b+tree collections. (0) | 2014.05.16 |
5 BASIC STEPS FOR SECURITY ENGINEERING & RISK ANALYSIS (0) | 2014.05.16 |