Chatty icon indicating copy to clipboard operation
Chatty copied to clipboard

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

Open DrupalDoesNotExists opened this issue 2 years ago • 1 comments

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

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

DrupalDoesNotExists avatar Aug 19 '23 18:08 DrupalDoesNotExists

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

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

DrupalDoesNotExists avatar Aug 19 '23 19:08 DrupalDoesNotExists