Listener: Fehler bei getHealth()

[MrPyro13]

Hallo Community,

ich habe das Problem, das in meinem Code das getHealth() rot unterstrichen wird.

Spoiler: SPOILER

package de.mrpyro13.ImpComs;

import org.bukkit.entity.Entity;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.EntityDamageByEntityEvent;

public class Listener_DamageMessage implements Listener {
    
    public Listener_DamageMessage(ImpComs_main plugin) {
        plugin.getServer().getPluginManager().registerEvents(this, plugin);
    }
    
    @EventHandler
    public void onDamage(EntityDamageByEntityEvent ev) {
        Entity en0 = ev.getDamager();
        if (en0 instanceof Player) {
            Entity en1 = ev.getEntity();
            if (en1 instanceof Player) {
                Player p0 = (Player) en0; //Fügt Schaden hinzu
                Player p1 = (Player) en1; //Bekommt Schaden
                String msg = "§6" + p1.getName() + "§7's Lebensanzeige: §4";
                for(int i = 0; i < p1.[COLOR=#ff0000][U]getHealth()[/U][/COLOR]; i++) {
                    msg = msg + "¦";
                }
                p0.sendMessage(msg);
            }
        }
    }

}

Fehlermeldung:

The method getHealth() is ambiguous for the type Player

 

[Elrontur]

Ich habe aus dem Thread zwar nur die Hälfte rauslesen können, den ich dir hier auch mal verlinke, aber es scheint schon lange eine Änderung vorgenommen zu sein:
Seit 1.6 geht getHealth() nicht mehr mit int sondern mit double. -> http://forums.bukkit.org/threads/oops-i-didnt-break-your-plugins-ambiguous-gethealth.156975/

 

[MiCrJonas]

Caste mal p1 zu Damageable. Also:

((Damageable) p1).getHealth()

Dann funktioniert es.

 

[MrPyro13]

Super, das von MiCrJonas1997 funktioniert!
Um nun diese Lebensanzeige zu halbieren muss ich doch einfach /2 machen oder?

for(int i = 0; i < (((Damageable) p1).getHealth())/2; i++) {

 

[MiCrJonas]

Ja, du solltest aber vor der Schleife 1x ausrechnen, wann die Schleife enden soll, anstatt das jedes mal zu machen, wenn die Schleife durchlaufen wird, das dies Resourcenschonender ist. Außerdem solltest du einen StringBuilder benutzen, weil dies auch Resourcen spart, wenn du so oft etwas zum String hinzufügst. Das ganze macht aber nur einen minimalen unmerklichen Unterschied zu deiner Variante.