aboutsummaryrefslogtreecommitdiff
path: root/juick-core/src/main/java/com/juick
diff options
context:
space:
mode:
authorGravatar Alexander Alexeev2016-12-07 18:56:43 +0700
committerGravatar Vitaly Takmazov2016-12-07 22:37:17 +0300
commit69a5bdba32cdf0a48e51baa8c9fb28dca013f515 (patch)
tree678ca66ebd5e3c6e0e46f179542ff35fcf52c767 /juick-core/src/main/java/com/juick
parentbad2718b1b85d806221fce52c76fa1d388993396 (diff)
message improvenments, checking for tag's duplicates
Diffstat (limited to 'juick-core/src/main/java/com/juick')
-rw-r--r--juick-core/src/main/java/com/juick/Message.java150
-rw-r--r--juick-core/src/main/java/com/juick/Tag.java14
2 files changed, 95 insertions, 69 deletions
diff --git a/juick-core/src/main/java/com/juick/Message.java b/juick-core/src/main/java/com/juick/Message.java
index 702b74e5..ae13d7e9 100644
--- a/juick-core/src/main/java/com/juick/Message.java
+++ b/juick-core/src/main/java/com/juick/Message.java
@@ -21,7 +21,9 @@ import com.fasterxml.jackson.annotation.JsonFormat;
import com.fasterxml.jackson.annotation.JsonIgnore;
import com.fasterxml.jackson.annotation.JsonProperty;
import com.juick.xml.adapters.SimpleDateAdapter;
+import org.apache.commons.collections4.CollectionUtils;
import org.apache.commons.lang3.StringUtils;
+import org.apache.commons.lang3.builder.ToStringBuilder;
import javax.xml.bind.annotation.*;
import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
@@ -29,7 +31,6 @@ import java.util.*;
import java.util.stream.Collectors;
/**
- *
* @author Ugnich Anton
*/
@XmlRootElement(name = "juick", namespace = "http://juick.com/message")
@@ -40,7 +41,7 @@ public class Message implements Comparable {
private int replyto = 0;
private String text = null;
private User user = null;
- private List<Tag> Tags = new ArrayList<>();
+ private final List<Tag> tags;
private Date date;
@XmlTransient
@JsonIgnore
@@ -67,12 +68,14 @@ public class Message implements Comparable {
public String Video = null;
public Place Place = null;
private int likes;
- public List<Message> childs = new ArrayList<>();
+ public final List<Message> childs;
private PM PM;
private Recommendation Recommendation;
private String replyQuote;
public Message() {
+ tags = new ArrayList<>();
+ childs = new ArrayList<>();
}
public Message(Message msg) {
@@ -81,7 +84,7 @@ public class Message implements Comparable {
replyto = msg.replyto;
setText(msg.getText());
setUser(msg.getUser());
- Tags = msg.Tags;
+ tags = msg.tags;
setDate(msg.getDate());
TimeAgo = msg.TimeAgo;
privacy = msg.privacy;
@@ -100,46 +103,65 @@ public class Message implements Comparable {
}
public void parseTags(String strTags) {
- if (strTags != null) {
- List<String> tags = Arrays.asList(strTags.split(" "));
- Tags.addAll(tags.stream().map(Tag::new).collect(Collectors.toList()));
+ if (StringUtils.isNotEmpty(strTags)) {
+ Set<Tag> tagSet = new TreeSet<>(tags);
+ for (String str : strTags.split(" ")) {
+ Tag tag = new Tag(str);
+ if (!tagSet.contains(tag)) {
+ tags.add(tag);
+ tagSet.add(tag);
+ }
+ }
}
}
@Override
+ public String toString() {
+ return new ToStringBuilder(this)
+ .append("mid", mid)
+ .append("rid", rid)
+ .append("replyto", replyto)
+ .append("TimeAgo", TimeAgo)
+ .append("privacy", privacy)
+ .append("FriendsOnly", FriendsOnly)
+ .append("ReadOnly", ReadOnly)
+ .append("Hidden", Hidden)
+ .append("VisitorCanComment", VisitorCanComment)
+ .append("replies", replies)
+ .append("likes", likes)
+ .toString();
+ }
+
+ @Override
public boolean equals(Object obj) {
- if (!(obj instanceof Message)) {
+ if (obj == this)
+ return true;
+
+ if (!(obj instanceof Message))
return false;
- }
+
Message jmsg = (Message) obj;
return (this.getMid() == jmsg.getMid() && this.getRid() == jmsg.getRid());
}
@Override
public int compareTo(Object obj) throws ClassCastException {
- if (!(obj instanceof Message)) {
+ if (obj == this)
+ return 0;
+
+ if (!(obj instanceof Message))
throw new ClassCastException();
- }
+
Message jmsg = (Message) obj;
- if (this.getMid() != jmsg.getMid()) {
- if (this.getMid() > jmsg.getMid()) {
- return -1;
- } else {
- return 1;
- }
- }
+ int cmp = Integer.compare(jmsg.getMid(), getMid());
- if (this.getRid() != jmsg.getRid()) {
- if (this.getRid() < jmsg.getRid()) {
- return -1;
- } else {
- return 1;
- }
- }
+ if (cmp == 0)
+ cmp = Integer.compare(getRid(), jmsg.getRid());
- return 0;
+ return cmp;
}
+
@JsonIgnore
public int getChildsCount() {
int cnt = childs.size();
@@ -161,42 +183,43 @@ public class Message implements Comparable {
@JsonIgnore
public String getAttachmentURL() {
if (attachmentType != null) {
- String url = "http://i.juick.com/";
- url += attachmentType.equals("mp4") ? "video" : "photos-1024";
- url += "/" + getMid();
- if (getRid() > 0) {
- url += "-" + getRid();
- }
- url += "." + attachmentType;
- return url;
- } else {
- return null;
+ StringBuilder builder = new StringBuilder();
+
+ builder.append("http://i.juick.com/");
+ builder.append(attachmentType.equals("mp4") ? "video" : "photos-1024");
+ builder.append("/").append(getMid());
+
+ if (getRid() > 0)
+ builder.append("-").append(getRid());
+
+ builder.append(".").append(attachmentType);
+
+ return builder.toString();
}
+ return null;
}
+
@JsonIgnore
public String getTagsString() {
- String ret = "";
- if (!Tags.isEmpty()) {
- for (Tag Tag : Tags) {
- ret += " *" + Tag.getName();
- }
- if (FriendsOnly) {
- ret += " *friends";
- }
- if (privacy == -2) {
- ret += " *private";
- }
- if (privacy == -1) {
- ret += " *friends";
- }
- if (privacy == 2) {
- ret += " *public";
- }
- if (ReadOnly) {
- ret += " *readonly";
- }
+ StringBuilder builder = new StringBuilder();
+ if (!tags.isEmpty()) {
+ for (Tag Tag : tags)
+ builder.append(" *").append(Tag.getName());
+
+ if (FriendsOnly)
+ builder.append(" *friends");
+
+ if (privacy == -2)
+ builder.append(" *private");
+ else if (privacy == -1)
+ builder.append(" *friends");
+ else if (privacy == 2)
+ builder.append(" *public");
+
+ if (ReadOnly)
+ builder.append(" *readonly");
}
- return ret;
+ return builder.toString();
}
@JsonProperty("mid")
@@ -239,7 +262,7 @@ public class Message implements Comparable {
}
@JsonProperty("timestamp")
- @JsonFormat(shape= JsonFormat.Shape.STRING, pattern="yyyy-MM-dd HH:mm:ss", timezone="UTC")
+ @JsonFormat(shape = JsonFormat.Shape.STRING, pattern = "yyyy-MM-dd HH:mm:ss", timezone = "UTC")
@XmlAttribute(name = "ts")
@XmlJavaTypeAdapter(SimpleDateAdapter.class)
public Date getDate() {
@@ -247,7 +270,6 @@ public class Message implements Comparable {
}
public void setDate(Date date) {
-
this.date = date;
}
@@ -262,6 +284,7 @@ public class Message implements Comparable {
public Recommendation getRecommendation() {
return Recommendation;
}
+
public void setRecommendation(Recommendation recommendation) {
this.Recommendation = recommendation;
}
@@ -270,6 +293,7 @@ public class Message implements Comparable {
public String getReplyQuote() {
return replyQuote;
}
+
public void setReplyQuote(String quote) {
replyQuote = quote;
}
@@ -287,11 +311,13 @@ public class Message implements Comparable {
@JsonProperty("tags")
@XmlElement(name = "tag")
public List<Tag> getTags() {
- return Tags;
+ return tags;
}
public void setTags(List<Tag> tags) {
- Tags = tags;
+ this.tags.clear();
+ if (CollectionUtils.isNotEmpty(tags))
+ this.tags.addAll(tags);
}
@XmlAttribute
@@ -329,6 +355,7 @@ public class Message implements Comparable {
public void setReplies(int replies) {
this.replies = replies;
}
+
@XmlTransient
public int getLikes() {
return likes;
@@ -346,6 +373,7 @@ public class Message implements Comparable {
public void setRepliesBy(String repliesBy) {
this.repliesBy = repliesBy;
}
+
@JsonProperty("photo")
@XmlTransient
public Photo getPhotoURLs() {
diff --git a/juick-core/src/main/java/com/juick/Tag.java b/juick-core/src/main/java/com/juick/Tag.java
index 98fe60ff..9675390b 100644
--- a/juick-core/src/main/java/com/juick/Tag.java
+++ b/juick-core/src/main/java/com/juick/Tag.java
@@ -23,6 +23,7 @@ import javax.xml.bind.annotation.XmlAccessType;
import javax.xml.bind.annotation.XmlAccessorType;
import javax.xml.bind.annotation.XmlRootElement;
import javax.xml.bind.annotation.XmlValue;
+import java.util.Comparator;
import java.util.Objects;
/**
@@ -30,16 +31,12 @@ import java.util.Objects;
*/
@XmlRootElement(name = "tag", namespace = "http://juick.com/message")
@XmlAccessorType(XmlAccessType.PROPERTY)
-public class Tag {
- private String name;
+public class Tag implements Comparable<Tag> {
+ private final String name;
public int TID = 0;
public int SynonymID = 0;
- public Tag() {
-
- }
-
public Tag(String name) {
this.name = name;
}
@@ -61,7 +58,8 @@ public class Tag {
return name;
}
- public void setName(String name) {
- this.name = name;
+ @Override
+ public int compareTo(Tag o) {
+ return Objects.compare(name, o.getName(), Comparator.naturalOrder());
}
}