Spring Boot聊天应用消息异常：新消息替换首条而非追加至末尾

问题 #

使用Spring Boot开发聊天应用时，遇到消息容量管理异常：当聊天消息数量达到上限时，预期移除最旧消息并将新消息追加至列表末尾，但实际表现为新消息替换了列表第一条消息。

相关代码如下：

ChatMessageServiceImpl 核心代码 #

@Service
@RequiredArgsConstructor
public class ChatMessageServiceImpl implements ChatMessageService {

    private final ChatRepository chatRepository;
    private final UserService userService;

    @Value("${monopoly.app.chat.max-size}")
    private Integer maxSize;

    @Override
    public ChatMessage save(Chat chat, ChatMessageDto chatMessageDto) {
        ChatMessage chatMessage = ChatMessage.builder()
                .sender(userService.findByUsername(chatMessageDto.getSender()))
                .content(chatMessageDto.getContent())
                .timestamp(LocalDateTime.now())
                .receiver(chatMessageDto.getReceiver() != null ?
                        userService.findByUsername(chatMessageDto.getReceiver()) : null)
                .chat(chat)
                .build();

        // 检查聊天消息是否达到最大容量
        if (chat.getMessages().size() >= maxSize) {
            chat.getMessages().remove(0); // 移除第一条消息
        }

        chat.getMessages().add(chatMessage); // 将新消息添加至末尾

        chatRepository.save(chat);
        return chatMessage;
    }
}

Chat 实体类代码 #

@NoArgsConstructor
@AllArgsConstructor
@Getter
@Setter
@Builder
@Entity
@Table(name = "chats")
public class Chat {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Long id;

    private String name;

    private Integer unreadMessages = 0;

    @OneToMany(mappedBy = "chat", fetch = FetchType.EAGER, cascade = CascadeType.ALL, orphanRemoval = true)
    private List<ChatMessage> messages = new ArrayList<>();
}

已确认：消息列表为标准List，支持remove和add操作；尝试过显式创建新列表、复制消息后更新Chat实体的消息列表；数据库已正确更新Chat实体的消息列表。

原因分析与解决方案 #

核心原因 #

1. 集合加载顺序无约束：JPA从数据库加载messages集合时，默认没有指定排序规则，导致列表顺序依赖数据库返回的原生顺序（可能是主键ID顺序而非消息创建时间顺序）。此时调用remove(0)移除的并非预期的最旧消息，后续添加新消息后，前端展示时容易产生“替换第一条”的视觉误解。
2. EAGER加载的顺序不稳定：使用FetchType.EAGER加载集合时，JPA不会自动按消息时间戳排序，若消息主键ID与创建时间未严格同步（如数据插入异常），会进一步加剧列表顺序的混乱。

解决步骤 #

1. 给集合指定排序规则
   在Chat实体的@OneToMany注解中添加@OrderBy，强制按消息时间戳升序排序，确保加载的列表始终是最旧消息在前：

   @OneToMany(mappedBy = "chat", fetch = FetchType.EAGER, cascade = CascadeType.ALL, orphanRemoval = true)
   @OrderBy("timestamp ASC") // 按时间戳升序，最旧消息固定在列表首位
   private List<ChatMessage> messages = new ArrayList<>();
   

2. 添加日志验证逻辑执行
   在操作前后打印关键信息，确认移除和添加的消息符合预期：

   // 检查聊天消息是否达到最大容量
   if (chat.getMessages().size() >= maxSize) {
       ChatMessage oldestMsg = chat.getMessages().get(0);
       log.info("移除最旧消息ID：{}，时间：{}", oldestMsg.getId(), oldestMsg.getTimestamp());
       chat.getMessages().remove(0);
   }
   log.info("添加新消息ID：{}，时间：{}", chatMessage.getId(), chatMessage.getTimestamp());
   chat.getMessages().add(chatMessage);
   

3. 排查前端展示逻辑
   确认前端是否按消息时间戳正确排序渲染，避免前端倒序或错误排序导致的视觉假象。

内容的提问来源于stack exchange，提问作者dimitri08