피드백 반영

Author: bs8841

data/channels/3126edda-9b93-496b-8ed5-9cb219c303d5.ser

@@ -4,7 +4,7 @@
 import java.util.UUID;
 
 public abstract class BaseEntity implements Serializable {
-    private static final long serialVersionUID = 1L;
+    private static final long serialVersionUID = -571597392321654385L;
 
     private final UUID id;
     private final long createdAt;

data/messages/60dbe397-bcff-4dfc-9733-0f2bb18afb19.ser

@@ -4,7 +4,7 @@
 import java.util.*;
 
 public class Channel extends BaseEntity implements Serializable {
-    private static final long serialVersionUID = 1L;
+    private static final long serialVersionUID = 3253334103732539416L;
     private String channelName;
     private final Set<UUID> userIds = new HashSet<>();
     private final List<UUID> messageIds = new ArrayList<>();

data/users/437981ca-9bbd-4982-bb5b-644023a9e270.ser

@@ -5,7 +5,7 @@
 import java.util.UUID;
 
 public class Message extends BaseEntity implements Serializable {
-    private static final long serialVersionUID = 1L;
+    private static final long serialVersionUID = 5140283631663474458L;
     private String msgContent;
     private final UUID senderId;
     private final UUID channelId;

src/main/java/com/sprint/mission/discodeit/entity/BaseEntity.java

@@ -7,31 +7,21 @@
 import java.util.UUID;
 
 public class User extends BaseEntity implements Serializable {
-    private static final long serialVersionUID = 1L;
+    private static final long serialVersionUID = -1421022282607757997L;
     private String userName;
-    private final Set<UUID> channelIds = new HashSet<>();
 
     public User(String userName) {
         this.userName = userName;
     }
 
-    public Set<UUID> getChannelIds() {
-        return channelIds;
-    }
-
     public void updateUserName(String userName) {
         this.userName = userName;
         updateTime();
     }
 
-    public void addChannel(UUID channelId) {
-        channelIds.add(channelId);
-    }
-
     @Override
     public String toString() {
         return "User{name='" + userName + '\'' +
-                ", channelIds=" + channelIds +
                 ", createdAt=" + new Date(getCreatedAt()) +
                 ", updatedAt=" + new Date(getUpdatedAt()) +
                 '}';

src/main/java/com/sprint/mission/discodeit/entity/Channel.java

@@ -8,8 +8,19 @@
 public class JCFChannelRepository implements ChannelRepository {
     private final Map<UUID, Channel> data = new HashMap<>();
 
-    public Channel save(Channel channel) { return data.put(channel.getId(), channel); }
-    public Optional<Channel> findById(UUID channelId) { return Optional.ofNullable(data.get(channelId)); }
-    public List<Channel> findAll() { return new ArrayList<>(data.values()); }
-    public void deleteById(UUID channelId) { data.remove(channelId); }
+    public Channel save(Channel channel) {
+        return data.put(channel.getId(), channel);
+    }
+
+    public Optional<Channel> findById(UUID channelId) {
+        return Optional.ofNullable(data.get(channelId));
+    }
+
+    public List<Channel> findAll() {
+        return new ArrayList<>(data.values());
+    }
+
+    public void deleteById(UUID channelId) {
+        data.remove(channelId);
+    }
 }

src/main/java/com/sprint/mission/discodeit/entity/Message.java

@@ -8,8 +8,19 @@
 public class JCFMessageRepository implements MessageRepository {
     private final Map<UUID, Message> data = new HashMap<>();
 
-    public Message save(Message message) { return data.put(message.getId(), message); }
-    public Optional<Message> findById(UUID messageId) { return Optional.ofNullable(data.get(messageId)); }
-    public List<Message> findAll() { return new ArrayList<>(data.values()); }
-    public void deleteById(UUID messageId) { data.remove(messageId); }
+    public Message save(Message message) {
+        return data.put(message.getId(), message);
+    }
+
+    public Optional<Message> findById(UUID messageId) {
+        return Optional.ofNullable(data.get(messageId));
+    }
+
+    public List<Message> findAll() {
+        return new ArrayList<>(data.values());
+    }
+
+    public void deleteById(UUID messageId) {
+        data.remove(messageId);
+    }
 }

src/main/java/com/sprint/mission/discodeit/entity/User.java

@@ -8,8 +8,19 @@
 public class JCFUserRepository implements UserRepository {
     private final Map<UUID, User> data = new HashMap<>();
 
-    public User save(User user) { return data.put(user.getId(), user); }
-    public Optional<User> findById(UUID userId) { return Optional.ofNullable(data.get(userId)); }
-    public List<User> findAll() { return new ArrayList<>(data.values()); }
-    public void deleteById(UUID userId) { data.remove(userId); }
+    public User save(User user) {
+        return data.put(user.getId(), user);
+    }
+
+    public Optional<User> findById(UUID userId) {
+        return Optional.ofNullable(data.get(userId));
+    }
+
+    public List<User> findAll() {
+        return new ArrayList<>(data.values());
+    }
+
+    public void deleteById(UUID userId) {
+        data.remove(userId);
+    }
 }