From 9a90018a8ba8f85bcfd4c48b05a2aab0f8a642a3 Mon Sep 17 00:00:00 2001 From: TheMode Date: Tue, 30 Nov 2021 19:20:51 +0100 Subject: [PATCH] Move warning to constructor Signed-off-by: TheMode --- .../packet/client/play/ClientChatMessagePacket.java | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/src/main/java/net/minestom/server/network/packet/client/play/ClientChatMessagePacket.java b/src/main/java/net/minestom/server/network/packet/client/play/ClientChatMessagePacket.java index 6e5c80d10..900c68caf 100644 --- a/src/main/java/net/minestom/server/network/packet/client/play/ClientChatMessagePacket.java +++ b/src/main/java/net/minestom/server/network/packet/client/play/ClientChatMessagePacket.java @@ -6,15 +6,18 @@ import net.minestom.server.utils.binary.BinaryWriter; import org.jetbrains.annotations.NotNull; public record ClientChatMessagePacket(@NotNull String message) implements ClientPacket { + public ClientChatMessagePacket { + if (message.length() > 256) { + throw new IllegalArgumentException("Message cannot be more than 256 characters long."); + } + } + public ClientChatMessagePacket(BinaryReader reader) { this(reader.readSizedString(256)); } @Override public void write(@NotNull BinaryWriter writer) { - if (message.length() > 256) { - throw new IllegalArgumentException("Message cannot be more than 256 characters long."); - } writer.writeSizedString(message); } }