Question

Got to know that we can initialize a collection in java by using double brace initialization. And did some search on that and found that it is not advised to use it because of its performance issue.

private static final Set<String> VALID_CODES = new HashSet<String>() {{
    add("XZ13s");
    add("AB21/X");
    add("YYLEX");
    add("AR2D");
 }};

Just want to know, is there any positive side or advantage of DOUBLE BRACE INITIALIZATION??

Answer 1

not advised to use it because of its performance issue.

I don’t see any performance issue. Whenever you see someone say I did/didn’t do something for performance reason you should expect to see detailed analysis comparing the alternatives explaining how one met the specific required performance and the other did not. If you don’t see all this you might assume the author is just guessing.

EDIT: While I will concede each class takes a small amount of time to load, the running performance is exactly the same. I have demonstrated here https://stackoverflow.com/a/14627268/57695

I would use double brace notation if you believe it’s simpler and clearer.

A disadvantage is that your are changing the type of the collection which can confuse functions where that is not expected. e.g. equals.

Note: As Lukas Eder points out, you want to be careful if you do this in a non-static context. The anonymous sub-class collection will implicitly have a reference to the outer instance and if it lives longer than the collection, this would be a memory leak. Have you ever thought of the possibility of a memory leak?