Remove the shell stuff, make objects mutable.

[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 a42cfaa..5124b49 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,11 @@
 
 package net.pterodactylus.sone.data;
 
-import java.util.Collection;
+import java.util.ArrayList;
 import java.util.Collections;
+import java.util.Comparator;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Set;
 import java.util.UUID;
 
@@ -35,13 +37,13 @@ public class Post {
        private final UUID id;
 
        /** The Sone this post belongs to. */
-       private final Sone sone;
+       private Sone sone;
 
        /** The time of the post (in milliseconds since Jan 1, 1970 UTC). */
-       private final long time;
+       private long time;
 
        /** The text of the post. */
-       private final String text;
+       private String text;
 
        /** The replies that have been loaded for this post. */
        private final Set<Reply> replies = new HashSet<Reply>();
@@ -69,7 +71,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);
        }
 
        /**
@@ -84,8 +86,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;
@@ -114,6 +116,18 @@ public class Post {
        }
 
        /**
+        * Sets the Sone of this post.
+        *
+        * @param sone
+        *            The Sone of this post
+        * @return This post (for method chaining)
+        */
+       public Post setSone(Sone sone) {
+               this.sone = sone;
+               return this;
+       }
+
+       /**
         * Returns the time of the post.
         *
         * @return The time of the post (in milliseconds since Jan 1, 1970 UTC)
@@ -123,6 +137,18 @@ public class Post {
        }
 
        /**
+        * Sets the time of this post.
+        *
+        * @param time
+        *            The time of this post (in milliseconds since Jan 1, 1970 UTC)
+        * @return This post (for method chaining)
+        */
+       public Post setTime(long time) {
+               this.time = time;
+               return this;
+       }
+
+       /**
         * Returns the text of the post.
         *
         * @return The text of the post
@@ -132,12 +158,33 @@ public class Post {
        }
 
        /**
+        * Sets the text of this post.
+        *
+        * @param text
+        *            The text of this post
+        * @return This post (for method chaining)
+        */
+       public Post setText(String text) {
+               this.text = text;
+               return this;
+       }
+
+       /**
         * Returns all replies to this post in unspecified order.
         *
         * @return All replies to this post
         */
-       public Collection<Reply> getReplies() {
-               return Collections.unmodifiableSet(replies);
+       public List<Reply> getReplies() {
+               List<Reply> sortedReplies = new ArrayList<Reply>(replies);
+               Collections.sort(sortedReplies, new Comparator<Reply>() {
+
+                       @Override
+                       public int compare(Reply leftReply, Reply rightReply) {
+                               return (int) Math.max(Integer.MIN_VALUE, Math.min(Integer.MAX_VALUE, leftReply.getTime() - rightReply.getTime()));
+                       }
+
+               });
+               return sortedReplies;
        }
 
        /**
@@ -174,7 +221,7 @@ public class Post {
         */
        @Override
        public int hashCode() {
-               return id.hashCode();
+               return id.hashCode() ^ sone.hashCode() ^ (int) (time >> 32) ^ (int) (time & 0xffffffff) ^ text.hashCode();
        }
 
        /**
@@ -185,7 +232,16 @@ public class Post {
                if (!(object instanceof Post)) {
                        return false;
                }
-               return ((Post) object).id.equals(id);
+               Post post = (Post) object;
+               return post.id.equals(id) && post.sone.equals(sone) && (post.time == time) && post.text.equals(text);
+       }
+
+       /**
+        * {@inheritDoc}
+        */
+       @Override
+       public String toString() {
+               return getClass().getName() + "[id=" + id + ",sone=" + sone + ",time=" + time + ",text=" + text + ",replies(" + replies.size() + ")]";
        }
 
 }