Замечание автору.
Нашёл одну неточность в действии HEAL.
В файле ActionHeal.java есть такой кусок кода:
double health = BukkitCompatibilityFix.getEntityHealth(player);
double healthMax = BukkitCompatibilityFix.getEntityMaxHealth(player);
if ((hp > 0) && (health < healthMax))
BukkitCompatibilityFix.setEntityHealth(player, Math.max(hp + health, healthMax));
Если hp = 0, то игрок не исцелится, т.к. условие не будет выполнено, хотя в описании этого действия сказано, что если параметр hp не указывать, то будет полное выздоровление (по умолчанию hp = 0).
Это первая ситуация, но есть и не очень хорошая ситуация, когда, например, health=19, а hp =2. В этом случае функция Math.max(hp + health, healthMax) возвратит максимум hp + health=21, т.к. healthMax=20.
Но в функцию setEntityHealth нельзя устанавливать значение превышающее healthMax, иначе выскакивает предупреждение типа:
Caused by: java.lang.IllegalArgumentException: Health must be between 0 and 20.0
Есть ещё и 3-я ситуация, когда, например, health=5, а hp =2. В этом случае функция Math.max(hp + health, healthMax) возвратит опять же максимум, который равен healthMax=20. Т.е. нельзя поднять хп на одно сердечко, вам сразу установиться полное здоровье.
Поэтому предлагаю изменить код на следующий:
double health = BukkitCompatibilityFix.getEntityHealth(player);
double healthMax = BukkitCompatibilityFix.getEntityMaxHealth(player);
if ((hp >= 0) && (health < healthMax))
{
double hp_new = hp + health;
if (hp_new > healthMax || hp == 0) hp_new = healthMax;
BukkitCompatibilityFix.setEntityHealth(player, hp_new);
}
При этом значение устанавливаемого ХП не будет превышать healthMax, можно будет поднимать здоровье на необходимое значение, а также в действии не нужно будет указывать параметр hp, чтобы полностью исцелиться.
Может быть, предлагаемый мной код не ахти, но, думаю, заслуживает внимания.
Спасибо.
