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

Why is this Javascript much *slower* than its jQuery equivalent?

I have a HTML list of about 500 items and a "filter" box above it. I started by using jQuery to filter the list when I typed a letter (timing code added later): $('#filter').keyup( function() { var jqStart = (new Date).getTime(); var search = $(this).val().toLowerCase(); var $list = $('ul.ablist > li'); $list.each( function() { if ( $(this).text().toLowerCase().indexOf(search) === -1 ) $(this).hide(); else $(this).show(); } ); console.log('Time: ' + ((new Date).getTime() - jqStart)); } ); However, there was a couple of seconds delay after typing each letter (particularly the first letter). So I thought it may be slightly quicker if I used plain Javascript (I read recently that jQuery's each function is particularly slow). Here's my JS equivalent: document.getElementById('filter').addEventListener( 'keyup', function () { var jsStart = (new Date).getTime()...

Is it possible to have IF statement in an Echo statement in PHP

Thanks in advance. I did look at the other questions/answers that were similar and didn't find exactly what I was looking for. I'm trying to do this, am I on the right path? echo " <div id='tabs-".$match."'> <textarea id='".$match."' name='".$match."'>". if ($COLUMN_NAME === $match) { echo $FIELD_WITH_COLUMN_NAME; } else { } ."</textarea> <script type='text/javascript'> CKEDITOR.replace( '".$match."' ); </script> </div>"; I am getting the following error message in the browser: Parse error: syntax error, unexpected T_IF Please let me know if this is the right way to go about nesting an IF statement inside an echo. Thank you.