TheKarma-Fridge

TheKarma-Fridge 1.1.4

Нет прав для скачивания
Обзор История Обсуждение
TheKarma2021

TheKarma2021

Пользователь
Заблокирован
Сообщения
31
TheKarma2021 добавил(а) новый ресурс:

TheKarma-Fridge - Автоматы с едой.

Russian:
Автоматы с едой, SAMP
English:
Vending machine with food, SAMP​


YAML: 
settings:
  prefix: '&f[&eАвтомат&f]'
  location:
  - 229 65 13
messages:
  full: '%prefix% У вас полный запас еды!'
  notmoney: '&f[&cАвтомат&f] Товар стоит &a%price%$&f. Вам не хватает &a%money%$ &fдля совершения покупки!'
  buy: '&f[&aАвтомат&f] Покупка прошла успешно.'
inventory:
  name: '&lВыберите еду'
  items:
    '1':
      name: '&eКартофель-Фри'...
Нажмите для раскрытия...

Узнать больше об этом ресурсе...
 
Что могу сказать по коду....

You must be registered for see links
:
Проверку на наличие Vault стоит сделать при запуске плагина, а не при каждой попытке взаимодействовать с балансом.

You must be registered for see links
:
SpigotPlugin лучше переименовать в название плагина - KarmaFridge или как он у тебя зовётся. По стандартам шкодеров названия классов должны быть понятны читателям.
У тебя нет в SpigotPlugin никаких слушателей событий, убери implements Listener.
Тебе не нужно делать проверку, создан ли конфиг, т.к. метод saveDefaultConfig() делает то же самое.
Избавиться от public static.

You must be registered for see links
:
Нет проверок на мир, в котором кликают игроки. Если хотят сделать холодильник (или что это у тебя там, я так и не понял, я из холодильника бесплатно достаю свою еду), то он будет кликабельным во всех мирах на этих координатах.
Почему только каменная кнопка? Почему не убрать эту проверку? Пусть игроки тыкают на то, что там и так стоит.
Велосипед при клике на еду.
event.getSlot() сделать через свич, дальнейшую работу вынести в другой метод. А вообще, нафига вот эти всякие заморочки, если у тебя всё по айди сделано? event.getSlot()+1 не катит вместо того, чтобы везде цифры ставить?
 
BuseSo написал(а):
Что могу сказать по коду....

You must be registered for see links
:
Проверку на наличие Vault стоит сделать при запуске плагина, а не при каждой попытке взаимодействовать с балансом.

You must be registered for see links
:
SpigotPlugin лучше переименовать в название плагина - KarmaFridge или как он у тебя зовётся. По стандартам шкодеров названия классов должны быть понятны читателям.
У тебя нет в SpigotPlugin никаких слушателей событий, убери implements Listener.
Тебе не нужно делать проверку, создан ли конфиг, т.к. метод saveDefaultConfig() делает то же самое.
Избавиться от public static.

You must be registered for see links
:
Нет проверок на мир, в котором кликают игроки. Если хотят сделать холодильник (или что это у тебя там, я так и не понял, я из холодильника бесплатно достаю свою еду), то он будет кликабельным во всех мирах на этих координатах.
Почему только каменная кнопка? Почему не убрать эту проверку? Пусть игроки тыкают на то, что там и так стоит.
Велосипед при клике на еду.
event.getSlot() сделать через свич, дальнейшую работу вынести в другой метод. А вообще, нафига вот эти всякие заморочки, если у тебя всё по айди сделано? event.getSlot()+1 не катит вместо того, чтобы везде цифры ставить?
Нажмите для раскрытия...
SpigotPlugin я всегда так классы называю, если плагин пойдет на Spigot сайт.
Про Vault ты прав. Player Listener попробую реализовать.
Авто объединение сообщений:

TheKarma2021 написал(а):
SpigotPlugin я всегда так классы называю, если плагин пойдет на Spigot сайт.
Про Vault ты прав. Player Listener попробую реализовать.
Нажмите для раскрытия...
Спасибо за ваш комментарий, я очень вам благодарен.
 
Написал просто /fridge
Ядро Paper 1.12.2
Вам необходимо зарегистрироваться для просмотра изображений-вложений
 
GhostSetuper написал(а):
Написал просто /fridge
Ядро Paper 1.12.2
Вам необходимо зарегистрироваться для просмотра изображений-вложений
Нажмите для раскрытия...
А, ой, команд нету.
Авто объединение сообщений:

GhostSetuper написал(а):
Написал просто /fridge
Ядро Paper 1.12.2
Вам необходимо зарегистрироваться для просмотра изображений-вложений
Нажмите для раскрытия...
Я забыл из plugin.yml убрать кмд.
Авто объединение сообщений:

Открой jar как как rar, открой plugin.yml и сделай так:
Тебе просто нужно убрать строчку commands: и все кмд. Их в коде если что нету, я тупанул. Обновлю ресурс позже
Авто объединение сообщений:

А, все, извиняюсь. Я понял ошибку, сейчас исправлю и обновлю ресурс
 

Вложения

  • 1641040839191.png
    1641040839191.png
    11.6 KB · Просмотры: 4
Последнее редактирование:
TheKarma2021 обновил(а) ресурс TheKarma-Fridge новой записью:

1.1.2 Фикс.

1. Фикс багов.
Нажмите для раскрытия...

Узнать больше об этом обновлении...
Авто объединение сообщений:

GhostSetuper написал(а):
Написал просто /fridge
Ядро Paper 1.12.2
Вам необходимо зарегистрироваться для просмотра изображений-вложений
Нажмите для раскрытия...
Скачайте новую версию плагина.
 
Блин, все равно плагин не работает, команду криво подключил. Поэтому переписал и заработало
Авто объединение сообщений:

Вам необходимо зарегистрироваться для просмотра изображений-вложений


+ Исправил проблему с плейсхолдером для строки buy
Вам необходимо зарегистрироваться для просмотра изображений-вложений

Вам необходимо зарегистрироваться для просмотра изображений-вложений
 
Последнее редактирование:
GhostSetuper написал(а):
Блин, все равно плагин не работает, команду криво подключил. Поэтому переписал и заработало
Авто объединение сообщений:

Вам необходимо зарегистрироваться для просмотра изображений-вложений


+ Исправил проблему с плейсхолдером для строки buy
Вам необходимо зарегистрироваться для просмотра изображений-вложений

Вам необходимо зарегистрироваться для просмотра изображений-вложений
Нажмите для раскрытия...
o-o, крутые! Исправлю сам потом
 
Могу форкнуть твой репозиторий, там будет легче. Исправлял в программе Intellij IDEA, для eclipse могу помочь)
 
GhostSetuper написал(а):
Могу форкнуть твой репозиторий, там будет легче. Исправлял в программе Intellij IDEA, для eclipse могу помочь)
Нажмите для раскрытия...
Да не, спасиб. Я там ток replace забыл прописать. А команды там вообще по идеи не должно быть. Либо я её добавлю но уже в след. обновлениях)
Авто объединение сообщений:

Все.
 

Вложения

  • 1641295509961.png
    1641295509961.png
    26.3 KB · Просмотры: 4
А я добавил строку
commands:
fridge:

И новый класс, с помощью которого будет открываться меню
 
Войдите или зарегистрируйтесь для ответа.
Назад
Сверху Снизу