link1107 / DonationExecutor

Плагин для ру-стримеров, генерирующий события на сервере Minecraft (Spigot/Paper) на основании донатов с "Donation Alerts"
GNU Affero General Public License v3.0
217 stars 22 forks source link

Немного поправил код, добавил некоторые полезные методы и переделал конфиг #4

Open monikon22 opened 2 years ago

monikon22 commented 2 years ago

Сразу предупреждаю комрадов, я специально кардинально не менял код Игоря и не до всех моментов cмог дойти. Это первый пул. Просто мой взгляд на то, что надо бы поправить в плагине.

Мои изменения:

RedCrabVb commented 2 years ago

Стиль кода значительно улучшился!

mrypdm commented 2 years ago

не шарю за Java, но вряд ли коллекции в ней thread-safe. и никакой защиты от этого нет

Kartex7260 commented 2 years ago

Ещё можно в Utils.isBlackListed(String) вывести HashMap в статическое поле и заполнить в статическом конструкторе.

link1107 commented 2 years ago

Работа, прямо скажем, титаническая. Спасибо огромное! Я мержить, наверное, не буду, если ты не против, ибо я лучше буду этот коммит постепенно для себя разбирать, что и как там делается, и добавлять вручную с какими-то своими правками, чтобы ясно понимать суть изменений, в том числе и на будущее.

P1NG2WIN commented 2 years ago

не шарю за Java, но вряд ли коллекции в ней thread-safe. и никакой защиты от этого нет

Есть пакет java.util.concurrent.* в котором предоставляются обертки для thread safe коллекций. и многое другое что важно для нормального многопотока

serega6531 commented 2 years ago

Minecraft сервер по большей части однопоточный. В той части, где это важно для плагинов.