forge EntityPlayer setHealth(), komme nicht weiter

[neocromicon]

Hi,

ich bin seit Wochen am verzweifeln.
Ich möchte dem Spieler bei zuviel Genuss von Bier sterben lassen, klingt erstmal einfach aber es geht einfach nicht.
Ich kann ihm zwar Schaden zufügen und auch Sterben lassen aber das ist alles nur Fake, nach 2-3 Sek hat der Spieler wieder volle Herzen und beidem Tot kann man nicht Respawn drücken.

Ich hatte vor einer ganzen weile erfahren das ich das dem Server mitteilen muss, also habe ich mir das Packet Tut auf minecraftforge.net durchgelesen aber es bringt genau die gleichen Ergebnisse.

Hier noch die Codes die ich bereits verwendet hatte:

entityplayer.setEntityHealth((entityplayer.getHealth()-20));
entityplayer.setEntityHealth(0);
FMLClientHandler.instance().getClient().thePlayer.attackEntityFrom(DamageSource.causePlayerDamage(entityplayer), 10);
FMLClientHandler.instance().getClient().thePlayer.setEntityHealth((entityplayer.getHealth()-20));

mfg

[Gnoccy]

mit

player.setEntityHealth(player.getHealth() - 1);

funktioniert es auf jeden Fall. Ich verwende das selbe in meinem aktuellem Projekt. Ich vermute den Fehler deshalb eher in der Datenübermittlung zum Server. Kannst du mal den entsprechenden Code posten?

[neocromicon]

@Gnoccy

package net.neocromicon.src.Bier;

import cpw.mods.fml.client.FMLClientHandler;
import cpw.mods.fml.common.Side;
import cpw.mods.fml.common.asm.SideOnly;
import net.minecraft.src.EntityPlayer;
import net.minecraft.src.EnumAction;
import net.minecraft.src.EnumRarity;
import net.minecraft.src.ItemFood;
import net.minecraft.src.ItemStack;
import net.minecraft.src.Potion;
import net.minecraft.src.PotionEffect;
import net.minecraft.src.World;
import net.neocromicon.src.BierModBase.BierMod;

public class SchwarzBierItem extends ItemFood{
	
	public final int field_35430_a;
    private final int healAmount;
    private final float saturationModifier;
    private final boolean isWolfsFavoriteMeat;
    private boolean alwaysEdible;
    private int potionId;
    private int potionDuration;
    private int potionAmplifier;
    private int DrunkLevelBlack;
    private float potionEffectProbability; 
	
public SchwarzBierItem(int i, int var1, int var2, float var3, boolean var4) 
 { 
	super(i, var1, var2, var4);
    this.field_35430_a = 32;
    this.healAmount = var2;
    this.isWolfsFavoriteMeat = var4;
    this.saturationModifier = var3;
    this.maxStackSize = 8;   
 }

public ItemStack onFoodEaten(ItemStack var1, World var2, EntityPlayer entityplayer)
{   
	--var1.stackSize;
	entityplayer.getFoodStats().addStats(this);
	++DrunkLevelBlack;	
    
    
    if (DrunkLevelBlack == 2)
    {   
    	System.out.println("1");
    	entityplayer.addPotionEffect(new PotionEffect(Potion.moveSpeed.id, 700, 2));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.resistance.id, 900, 2));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.damageBoost.id, 1000, 2));
    }
	if (DrunkLevelBlack == 4)
    {
		System.out.println("2");
    	entityplayer.addPotionEffect(new PotionEffect(Potion.moveSpeed.id, 1200, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.resistance.id, 1500, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.damageBoost.id, 1400, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.confusion.id, 2800, 5));
    }
	if (DrunkLevelBlack == 6)
    {
		System.out.println("3");
    	entityplayer.addPotionEffect(new PotionEffect(Potion.moveSpeed.id, 1400, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.resistance.id, 1700, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.damageBoost.id, 1600, 4));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.confusion.id, 3200, 30));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.blindness.id, 300 * 20, 6));
    	entityplayer.addPotionEffect(new PotionEffect(Potion.hunger.id, 1700, 3));
    }
	if (DrunkLevelBlack == 8)
    { 	
		System.out.println("4");
        entityplayer.setEntityHealth((entityplayer.getHealth()-20)); <---------------------
		entityplayer.addChatMessage("Stay safe drinking water!");
    	DrunkLevelBlack = 0;
    }

    if (var1.stackSize <= 0)
    {
        return new ItemStack(BierMod.LeeresBierGlas);
    }    
    else
    {
    	entityplayer.inventory.addItemStackToInventory(new ItemStack(BierMod.LeeresBierGlas));
        return var1;
    }
}

public EnumAction getItemUseAction(ItemStack var1)
{
    return EnumAction.drink;
}

public int getHealAmount()
{
    return this.healAmount;
}

@Side Only(Side.CLIENT) 
public String getTextureFile()
  { 
    return "/BierModTextures/Bier.png";
  }

@Side Only(Side.CLIENT)
public int getIconFromDamage(int i)
  { 
    return 2;
  }

@Side Only(Side.CLIENT)
public EnumRarity getRarity(ItemStack par1)
{
   return EnumRarity.uncommon;
}

@Side Only(Side.CLIENT)
public boolean hasEffect(ItemStack par1ItemStack)
{
    return par1ItemStack.getItemDamage() > 0;
}
}

Der "DrunkLevel" Code ist noch lange nicht final, aber ich komme halt an der besagten stelle nicht weiter

[Gnoccy]

Also wenn ich das richtig sehe ist das Problem, dass der Code nur Clientseitig ausgefürht wird. Du musst ein Packet zum Server senden, dass dem Server sagt, dass es den Spieler verletzen soll.
In wie weit hast du dich mit dem Thema schon auseinander gesetzt? Wenn man es erst einmal verstanden hat ist es nicht wirklich schwer.

[neocromicon]

@Gnoccy
Ich habe mich an dieses:
http://www.minecraftforge.net/wiki/T...acket_Handling Tutorial gehalten.

Wie gesagt gebracht hat es reichlich wenig . So richtig auseinander gesetzt habe ich mich damit nie. Wenn du also weist wie es funktioniert wäre ich dir echt dankbar.

[Gnoccy]

Also an der stelle

entityplayer.setEntityHealth((entityplayer.getHealth()-20));

musst du prüfen, oder der Code gerade auf dem Client, oder auf dem Server ausgefürht wird. Das machst du zum Beispiel mit

if(world.isRemote){}

Liefert dies Abfrage true, bist du auf dem Client und musst ein Packet an dern Server schicken. Liefert sie false bist du auf dem Server und kannst deinen Code wie gehabt ausführen.
Wie man ein Packet baut, verschickt und auswertet weist du? Falls nicht meld dich nochmal.

Ich werde gleich erstmal ein wenig lol zocken. Wenn du also noch Fragen hast, dann stell sie jetzt, ich denke ich werde in etwa eine viertel Stunde weg sein.

[neocromicon]

@Gnoccy
So habe gerade isRemote ausprobiert, da ich das mit deiner Abfrage nicht ganz verstehe gebe ich dir nochmal den Code, jedenfalls spuckt er mir bei if(var2....) die Consolen Nachricht aus, ich nehme mal an das meinst du mit true.

if (DrunkLevelBlack == 8)
    { 	
		if(var2.isRemote)
	    {
	    	System.out.println("if");
	    	entityplayer.setEntityHealth((entityplayer.getHealth()-20));
	    }
	    else
	    {
	    	System.out.println("else");
	    	//entityplayer.setEntityHealth((entityplayer.getHealth()-20));
	    }
		System.out.println("4");
        //entityplayer.setEntityHealth((entityplayer.getHealth()-20));
		entityplayer.addChatMessage("Stay safe drinking water!");
    	DrunkLevelBlack = 0;
    }

Console:

2012-12-09 01:36:52 [INFO] [STDOUT] if

Und nein wie gesagt habe nur dieses Tut von minecraforge.net befolgt, aber verstanden habe ich es ehrlich gesagt nicht . Bin halt ein mensch der von Theorie nicht viel lernt, deshalb kann ich auch nix mit den ganzen "Fachbegriffen" anfangen. (Das läuft in meinem Kopf vollkommen anders ab als bei anderen)

[Gnoccy]

Hast du Skype? Dann erkläre ich es dir eben schnell.

[pitman-87]

Wie Gnoccy schon meinte wird die Funktion im Server nicht aufgerufen und da sowas wie Gesundheit und Tot sich im Server abspielen, wird der neue Zustand jedes vom alten überschrieben (beim synchronisieren zwischen Server und Client).

Also alle Klassen (Modklasse, entities, items, blöcke) solltest du in den common-Ordner packen (server + cient).
Alles was nur clientseitig gebraucht wird in den minecraft-ordner (Renderer, GUI, Modelle usw)

[neocromicon]

@pitman-87

Danke noch für den letzten hinweis, aber Gnoccy hat mir sehr gut in Skype weiter geholfen. Mittlerweile funktioniert alles super und ich habe auch verstanden was da passiert .

Kann also im Prinzip geschlossen werden.

mfg