Skip to main content

Is there a better way to combine two string sets in java?



I need to combine two string sets while filtering out redundant information, this is the solution I came up with, is there a better way that anyone can suggest? Perhaps something built in that I overlooked? Didn't have any luck with google.







Set<String> oldStringSet = getOldStringSet();

Set<String> newStringSet = getNewStringSet();



for(String currentString : oldStringSet)

{

if (!newStringSet.contains(currentString))

{

newStringSet.add(currentString);

}

}




Comments

  1. Since a Set only contains one instance of a element. You can therefor combind the two by:

    newStringSet.addAll(oldStringSet);


    It does not matter if you add things twice, the set will only contain the element once... e.g it's no need to check using contains method.

    ReplyDelete
  2. Just use newStringSet.addAll(oldStringSet). No need to check for duplicates as the Set implementation does this already.

    ReplyDelete
  3. Use boolean addAll(Collection<? extends E> c)
    Adds all of the elements in the specified collection to this set if they're not already present (optional operation). If the specified collection is also a set, the addAll operation effectively modifies this set so that its value is the union of the two sets. The behavior of this operation is undefined if the specified collection is modified while the operation is in progress.

    newStringSet.addAll(oldStringSet)

    ReplyDelete
  4. From the definition Set contain only unique elements.

    Set<String> distinct = new HashSet<String>();
    distinct.addAll(oldStringSet);
    distinct.addAll(newStringSet);


    To enhance your code you may create a generic method for that

    public static <T> Set<T> distinct(Collection<T>... lists) {
    Set<T> distinct = new HashSet<T>();

    for(Collection<T> list : lists) {
    distinct.addAll(list);
    }
    return distinct;
    }

    ReplyDelete
  5. http://docs.oracle.com/javase/7/docs/api/java/util/Set.html#addAll(java.util.Collection)

    Since sets can't have duplicates, just adding all the elements of one to the other generates the correct union of the two.

    ReplyDelete
  6. Set.addAll()

    Adds all of the elements in the specified collection to this set if they're not already present (optional operation). If the specified collection is also a set, the addAll operation effectively modifies this set so that its value is the union of the two sets

    newStringSet.addAll(oldStringSet)

    ReplyDelete
  7. newStringSet.addAll(oldStringSet);


    This will produce Union of s1 and s2

    ReplyDelete

Post a Comment

Popular posts from this blog

[韓日関係] 首相含む大幅な内閣改造の可能性…早ければ来月10日ごろ=韓国

div not scrolling properly with slimScroll plugin

I am using the slimScroll plugin for jQuery by Piotr Rochala Which is a great plugin for nice scrollbars on most browsers but I am stuck because I am using it for a chat box and whenever the user appends new text to the boxit does scroll using the .scrollTop() method however the plugin's scrollbar doesnt scroll with it and when the user wants to look though the chat history it will start scrolling from near the top. I have made a quick demo of my situation http://jsfiddle.net/DY9CT/2/ Does anyone know how to solve this problem?

Why does this javascript based printing cause Safari to refresh the page?

The page I am working on has a javascript function executed to print parts of the page. For some reason, printing in Safari, causes the window to somehow update. I say somehow, because it does not really refresh as in reload the page, but rather it starts the "rendering" of the page from start, i.e. scroll to top, flash animations start from 0, and so forth. The effect is reproduced by this fiddle: http://jsfiddle.net/fYmnB/ Clicking the print button and finishing or cancelling a print in Safari causes the screen to "go white" for a sec, which in my real website manifests itself as something "like" a reload. While running print button with, let's say, Firefox, just opens and closes the print dialogue without affecting the fiddle page in any way. Is there something with my way of calling the browsers print method that causes this, or how can it be explained - and preferably, avoided? P.S.: On my real site the same occurs with Chrome. In the ex