Add lots of synchronization.

[Sone.git] / src / main / java / net / pterodactylus / sone / data / Post.java

diff --git a/src/main/java/net/pterodactylus/sone/data/Post.java b/src/main/java/net/pterodactylus/sone/data/Post.java
index 15992a3..c30665a 100644 (file)
--- a/src/main/java/net/pterodactylus/sone/data/Post.java
+++ b/src/main/java/net/pterodactylus/sone/data/Post.java
@@ -17,9 +17,6 @@
 
 package net.pterodactylus.sone.data;
 
-import java.util.Collections;
-import java.util.HashSet;
-import java.util.Set;
 import java.util.UUID;
 
 /**
@@ -34,16 +31,23 @@ public class Post {
        private final UUID id;
 
        /** The Sone this post belongs to. */
-       private final Sone sone;
+       private volatile Sone sone;
 
        /** The time of the post (in milliseconds since Jan 1, 1970 UTC). */
-       private final long time;
+       private volatile long time;
 
        /** The text of the post. */
-       private final String text;
+       private volatile String text;
 
-       /** The replies that have been loaded for this post. */
-       private final Set<Reply> replies = new HashSet<Reply>();
+       /**
+        * Creates a new post.
+        *
+        * @param id
+        *            The ID of the post
+        */
+       public Post(String id) {
+               this(id, null, 0, null);
+       }
 
        /**
         * Creates a new post.
@@ -68,7 +72,7 @@ public class Post {
         *            The text of the post
         */
        public Post(Sone sone, long time, String text) {
-               this(UUID.randomUUID(), sone, time, text);
+               this(UUID.randomUUID().toString(), sone, time, text);
        }
 
        /**
@@ -83,8 +87,8 @@ public class Post {
         * @param text
         *            The text of the post
         */
-       public Post(UUID id, Sone sone, long time, String text) {
-               this.id = id;
+       public Post(String id, Sone sone, long time, String text) {
+               this.id = UUID.fromString(id);
                this.sone = sone;
                this.time = time;
                this.text = text;
@@ -113,55 +117,57 @@ public class Post {
        }
 
        /**
-        * Returns the time of the post.
+        * Sets the Sone of this post.
         *
-        * @return The time of the post (in milliseconds since Jan 1, 1970 UTC)
+        * @param sone
+        *            The Sone of this post
+        * @return This post (for method chaining)
         */
-       public long getTime() {
-               return time;
+       public Post setSone(Sone sone) {
+               this.sone = sone;
+               return this;
        }
 
        /**
-        * Returns the text of the post.
+        * Returns the time of the post.
         *
-        * @return The text of the post
+        * @return The time of the post (in milliseconds since Jan 1, 1970 UTC)
         */
-       public String getText() {
-               return text;
+       public long getTime() {
+               return time;
        }
 
        /**
-        * Returns all replies to this post in unspecified order.
+        * Sets the time of this post.
         *
-        * @return All replies to this post
+        * @param time
+        *            The time of this post (in milliseconds since Jan 1, 1970 UTC)
+        * @return This post (for method chaining)
         */
-       public Set<Reply> getReplies() {
-               return Collections.unmodifiableSet(replies);
+       public Post setTime(long time) {
+               this.time = time;
+               return this;
        }
 
        /**
-        * Adds a reply to this post. The reply will not be added if its
-        * {@link Reply#getPost() post} is not equal to this post.
+        * Returns the text of the post.
         *
-        * @param reply
-        *            The reply to add
+        * @return The text of the post
         */
-       public void addReply(Reply reply) {
-               if (reply.getPost().equals(this)) {
-                       replies.add(reply);
-               }
+       public String getText() {
+               return text;
        }
 
        /**
-        * Removes a reply from this post.
+        * Sets the text of this post.
         *
-        * @param reply
-        *            The reply to remove
+        * @param text
+        *            The text of this post
+        * @return This post (for method chaining)
         */
-       public void removeReply(Reply reply) {
-               if (reply.getPost().equals(this)) {
-                       replies.remove(reply);
-               }
+       public Post setText(String text) {
+               this.text = text;
+               return this;
        }
 
        //
@@ -173,7 +179,7 @@ public class Post {
         */
        @Override
        public int hashCode() {
-               return id.hashCode() ^ sone.hashCode() ^ (int) (time >> 32) ^ (int) (time & 0xffffffff) ^ text.hashCode();
+               return id.hashCode();
        }
 
        /**
@@ -185,7 +191,15 @@ public class Post {
                        return false;
                }
                Post post = (Post) object;
-               return post.id.equals(id) && post.sone.equals(sone) && (post.time == time) && post.text.equals(text);
+               return post.id.equals(id);
+       }
+
+       /**
+        * {@inheritDoc}
+        */
+       @Override
+       public String toString() {
+               return getClass().getName() + "[id=" + id + ",sone=" + sone + ",time=" + time + ",text=" + text + "]";
        }
 
 }