Skip to content

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

Open
monikon22 wants to merge 1 commit intolink1107:masterfrom
monikon22:master
Open

Немного поправил код, добавил некоторые полезные методы и переделал конфиг#4
monikon22 wants to merge 1 commit intolink1107:masterfrom
monikon22:master

Conversation

@monikon22
Copy link
Copy Markdown

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

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

  • поменял конфиг (добавил маркер версии и конвертацию, что достаточно полезно, если структура конфига будет меняться).
  • в некоторых местах оптимизировал код (также упростил компиляцию предметов в один метод)
  • оптимизировал логику Сталина (перенёс всё в листы, чтобы код не мазолил глаза)

@RedCrabVb
Copy link
Copy Markdown

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

@mrypdm
Copy link
Copy Markdown

mrypdm commented Jan 23, 2022

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

@Kartex7260
Copy link
Copy Markdown

Kartex7260 commented Jan 23, 2022

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

@link1107
Copy link
Copy Markdown
Owner

link1107 commented Jan 23, 2022

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

@link1107 link1107 force-pushed the master branch 4 times, most recently from b8b25a4 to b5313d1 Compare January 31, 2022 18:57
@link1107 link1107 force-pushed the master branch 4 times, most recently from 5da4933 to 77060ca Compare February 9, 2022 13:03
@ZiraiMode
Copy link
Copy Markdown

ZiraiMode commented Feb 10, 2022

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

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

@serega6531
Copy link
Copy Markdown

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

@link1107 link1107 force-pushed the master branch 10 times, most recently from e4079e4 to 8e4a78e Compare February 12, 2022 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants