Плагин [SEC/FIX] DupeFixer v3.2 - фикс дюпов, багов, X-Ray [>=1.9]

Лучше плагин переписать. Говнокода ооочень много.

 

Где он тебе там мерещится? Подробнее давай.

 

1. Называй классы с большой буквы, а переменные с маленькой, чтобы было понятно в коде, где у тебя класс, а где переменная.
2. Не используй Minecraft Logger. В Bukkit давно есть нормальный PluginLogger. Получить его можно с помощью getLogger() в главном классе плагина. Все сообщения, которые ты в него будешь отправлять, будут автоматом получать префикс с названием твоего плагина.
3. Узнай что такое коллекции. А то сделал какую-то херню с названием "memory".
4. Ну кто так работает с enum?
   Код:

   if(event.getEntityType().equalsIgnoreCase("MINECART_CHEST")) { ...

   Они сделаны для того чтобы было удобно сравнивать.
   Код:

   if(event.getEntityType() == EntityType.MINECART_CHEST) { ...

5. Ванильные блоки и предметы определяй не по ID, а по Material. Так тебе будет самому будет понятно, с чем ты проводишь сравнение, а ID можно забыть.
   Код:

   if(block.getTypeId() == 56) { ... // алмазный блок
   if(block.getType() == Material.DIAMOND_BLOCK) { ... // вот это выглядит куда лучше и сразу понятно, что тут проверяют

6. Это вообще убило. Про Block.getRelative() не слышал???
7. У @EventHandler есть ещё атрибут ignoreCancelled для пропуска отменённых эвентов - не нужен никакой вызов Cancellable.isCancelled().
8. Получить путь к папке плагина (plugins/MyPlugin) можно с помощью getDataFolder() в главном классе плагина. И не нужно всяких костылей для форточек. Если уж пишешь пути к файлам, то в качестве разделителя директорий используй / будет работать и на форточках, и на *nix, либо вместо него писать File.separator.

Это были лишь нарекания по поводу кода. Есть ещё косяки в логике плагина и ненужные функции о которых я ранее писал.

 

Это все кусок из AllFixer, который я писал вместо HelloWorld когда учил джаву. Потом исправлю всю эту хрень.

Бесят эти тупые коллекции, с memory как-то удобнее работать, он прост как топор. Да и быстрее будет, чем тяжелый класс ворочать.

Не совсем понял, как это? Сначала и был слэш, нифига оно не работает в винде

Ну, оно мне как-то пофигу, мне и так понятно.

Какие косяки? Антидюп цветов не буду убирать, кому-то это нужно. На последнем спиготе не проверял, а на 1.7 он хорошо работал.
.

 

Слеш работает ещё как, у меня все плагины с ним, а сервак на Win Server 2008 + проверено на Win 7. Если уж так не хочется, делай вместо слеша " + File.separator + "

 

Вот фиг его знает почему, как у меня сейчас путь прописывается для линя в этом плагине - не работало. Буду сепаратор использовать, так правда проще.

Вот бы еще воронки зафиксить, магию ковыряния ядра на ходу не осилю пока что

 

На 1.6.4 что то есть?

 

А я хз, протестируй. Но пилить поддержку для старья не буду.

 

Не работает, проверил! Хотя папку создал!
Мне бы хотя бы ивент какой то, на фикс дюпа рельс

 

Сейчас BlockPistonExtendEvent используется, а какой там в 1.6.4 был не знаю, чето не могу отыскать джавадок.

 

Я буду очень благодарен если вы что то сделаете)(164)

 

ModFix разве не фиксит это всё?

 

Шевчик говорил что фиксит, а в конфиге и описании ни слова. Не понятно что и как фиксится, как настраивать это. Вот почему и взялся, не хочу себе кота в мешке.

 

@Shevchik обычно не врёт. Раз сказал, что фиксит, значит так и есть.

 

Дюп рельс ModFixNG никогда не фиксил, это было зафикшено краном с 1.8.3. А также фикс долгое время был в TickOptimizer.

 

Ну а кроме рельс?

 

А я как раз таки помню, что давным-давно был фикс рельс

 

Тупо запрещалось двигать рельсы поршнями. Даже фиксом назвать нельзя.

 

Можно))0)00

 

Так что делать?