Brikster / Chatty

Bukkit-compatible chat management system
MIT License
90 stars 50 forks source link

Закэшировать строку очистки чата в ClearChatCommand #249

Open DrupalDoesNotExists opened 1 year ago

DrupalDoesNotExists commented 1 year ago

На данный момент там используется цикл на 100 итераций, который в каждой итерации отправляет по сообщению. Это потенциально может спровоцировать 100 пакетов (которые так же будут занесены в очередь, так что ещё и 100 объектов). Эти 100 итераций выполняются на каждом вызове команды по новой.

Их можно эффективно упразднить, если закэшировать строку на 100 "\n" заранее в какой нибудь приватной статической переменной.

DrupalDoesNotExists commented 1 year ago

Ссылка на строку кода в файле:

https://github.com/Brikster/Chatty/blob/master/spigot/src/main/java/ru/mrbrikster/chatty/commands/ClearChatCommand.java#L22