X-Git-Url: https://git.pterodactylus.net/?a=blobdiff_plain;f=src%2Fmain%2Fjava%2Fnet%2Fpterodactylus%2Fsone%2Fdata%2Fimpl%2FSoneImpl.java;h=5d8f1c4d5f419d54160793ec5ce8b39b11f8f1e3;hb=ae7ed7856de14eb339e77ae86e6d4a8b9284162d;hp=551e2298b8f62c512e2e20c534e86b693c3b0d15;hpb=56a2875b53db60fc26dd7ee089d721227508703c;p=Sone.git

diff --git a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
index 551e229..5d8f1c4 100644
--- a/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
+++ b/src/main/java/net/pterodactylus/sone/data/impl/SoneImpl.java
@@ -25,6 +25,7 @@ import java.net.MalformedURLException;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import java.util.concurrent.CopyOnWriteArraySet;
@@ -47,6 +48,7 @@ import net.pterodactylus.sone.freenet.wot.OwnIdentity;
 
 import freenet.keys.FreenetURI;
 
+import com.google.common.collect.FluentIterable;
 import com.google.common.hash.Hasher;
 import com.google.common.hash.Hashing;
 
@@ -78,7 +80,7 @@ public class SoneImpl implements LocalSone {
 	private volatile long latestEdition;
 
 	/** The time of the last inserted update. */
-	private volatile long time;
+	private final long time;
 
 	/** The status of this Sone. */
 	private volatile SoneStatus status = SoneStatus.unknown;
@@ -87,13 +89,13 @@ public class SoneImpl implements LocalSone {
 	private volatile Profile profile = new Profile(this);
 
 	/** The client used by the Sone. */
-	private volatile Client client;
+	private final Client client;
 
 	/** Whether this Sone is known. */
 	private volatile boolean known;
 
 	/** All posts. */
-	private final Set<Post> posts = new CopyOnWriteArraySet<Post>();
+	private final Collection<Post> posts = new HashSet<Post>();
 
 	/** All replies. */
 	private final Set<PostReply> replies = new CopyOnWriteArraySet<PostReply>();
@@ -119,11 +121,14 @@ public class SoneImpl implements LocalSone {
 	 * @param local
 	 * 		{@code true} if the Sone is a local Sone, {@code false} otherwise
 	 */
-	public SoneImpl(Database database, Identity identity, boolean local) {
+	public SoneImpl(Database database, Identity identity, boolean local, long time, Client client, Collection<Post> posts) {
 		this.database = database;
 		this.id = identity.getId();
 		this.identity = identity;
 		this.local = local;
+		this.time = time;
+		this.client = client;
+		this.posts.addAll(posts);
 	}
 
 	//
@@ -239,18 +244,6 @@ public class SoneImpl implements LocalSone {
 	}
 
 	/**
-	 * Sets the time of the last inserted update of this Sone.
-	 *
-	 * @param time
-	 * 		The time of the update (in milliseconds since Jan 1, 1970 UTC)
-	 * @return This Sone (for method chaining)
-	 */
-	public Sone setTime(long time) {
-		this.time = time;
-		return this;
-	}
-
-	/**
 	 * Returns the status of this Sone.
 	 *
 	 * @return The status of this Sone
@@ -313,7 +306,6 @@ public class SoneImpl implements LocalSone {
 	 * @return This Sone (for method chaining)
 	 */
 	public Sone setClient(Client client) {
-		this.client = client;
 		return this;
 	}
 
@@ -365,51 +357,8 @@ public class SoneImpl implements LocalSone {
 	 * @return All posts of this Sone
 	 */
 	public List<Post> getPosts() {
-		List<Post> sortedPosts;
-		synchronized (this) {
-			sortedPosts = new ArrayList<Post>(posts);
-		}
-		Collections.sort(sortedPosts, Post.TIME_COMPARATOR);
-		return sortedPosts;
-	}
-
-	/**
-	 * Sets all posts of this Sone at once.
-	 *
-	 * @param posts
-	 * 		The new (and only) posts of this Sone
-	 * @return This Sone (for method chaining)
-	 */
-	public Sone setPosts(Collection<Post> posts) {
 		synchronized (this) {
-			this.posts.clear();
-			this.posts.addAll(posts);
-		}
-		return this;
-	}
-
-	/**
-	 * Adds the given post to this Sone. The post will not be added if its {@link
-	 * Post#getSone() Sone} is not this Sone.
-	 *
-	 * @param post
-	 * 		The post to add
-	 */
-	public void addPost(Post post) {
-		if (post.getSone().equals(this) && posts.add(post)) {
-			logger.log(Level.FINEST, String.format("Adding %s to “%s”.", post, getName()));
-		}
-	}
-
-	/**
-	 * Removes the given post from this Sone.
-	 *
-	 * @param post
-	 * 		The post to remove
-	 */
-	public void removePost(Post post) {
-		if (post.getSone().equals(this)) {
-			posts.remove(post);
+			return FluentIterable.from(posts).toSortedList(Post.TIME_COMPARATOR);
 		}
 	}