import net.pterodactylus.util.config.ConfigurationException;
import com.google.common.base.Optional;
+import com.google.common.collect.HashMultimap;
+import com.google.common.collect.Multimap;
import com.google.common.collect.SortedSetMultimap;
import com.google.common.collect.TreeMultimap;
import com.google.common.util.concurrent.AbstractService;
private final Map<String, Post> allPosts = new HashMap<String, Post>();
/** All posts by their Sones. */
- private final Map<String, Collection<Post>> sonePosts = new HashMap<String, Collection<Post>>();
+ private final Multimap<String, Post> sonePosts = HashMultimap.create();
/** All posts by their recipient. */
- private final Map<String, Collection<Post>> recipientPosts = new HashMap<String, Collection<Post>>();
+ private final Multimap<String, Post> recipientPosts = HashMultimap.create();
/** Whether posts are known. */
private final Set<String> knownPosts = new HashSet<String>();
lock.writeLock().lock();
try {
/* remove all posts by the Sone. */
- getPostsFrom(sone.getId()).clear();
- for (Post post : posts) {
+ Collection<Post> oldPosts = getPostsFrom(sone.getId());
+ for (Post post : oldPosts) {
allPosts.remove(post.getId());
if (post.getRecipientId().isPresent()) {
getPostsTo(post.getRecipientId().get()).remove(post);
* @return All posts
*/
private Collection<Post> getPostsFrom(String soneId) {
- Collection<Post> posts = null;
lock.readLock().lock();
try {
- posts = sonePosts.get(soneId);
+ return sonePosts.get(soneId);
} finally {
lock.readLock().unlock();
}
- if (posts != null) {
- return posts;
- }
-
- posts = new HashSet<Post>();
- lock.writeLock().lock();
- try {
- sonePosts.put(soneId, posts);
- } finally {
- lock.writeLock().unlock();
- }
-
- return posts;
}
/**
* @return All posts
*/
private Collection<Post> getPostsTo(String recipientId) {
- Collection<Post> posts = null;
lock.readLock().lock();
try {
- posts = recipientPosts.get(recipientId);
+ return recipientPosts.get(recipientId);
} finally {
lock.readLock().unlock();
}
- if (posts != null) {
- return posts;
- }
-
- posts = new HashSet<Post>();
- lock.writeLock().lock();
- try {
- recipientPosts.put(recipientId, posts);
- } finally {
- lock.writeLock().unlock();
- }
-
- return posts;
}
/** Loads the known posts. */