Adding Parameters

[ZacDoesStuff]

Alright, the method I have is public void onArmorTickUpdate(World world, EntityPlayer player, ItemStack itemStack), but I want it to also check for a block that the player is on, so I'm using world.getBlockID(par2, par3, par4), but how do I correctly add the par2, par3, par4 into the method?

[coolboy4531]

par2, par3, and par4 are the X, Y, and Z coords.

 

You can get those coords by getting the players position - use casting if required.

player.posX, player.posY, player.posZ

[ZacDoesStuff]

  On 4/20/2014 at 4:47 PM, coolboy4531 said:

You can get those coords by getting the players position - use casting if required.

player.posX, player.posY, player.posZ

 

I tried that, it didn't do anything

[ZacDoesStuff]

public void onArmorTickUpdate(World world, EntityPlayer player, ItemStack itemStack)

    {

super.onArmorTickUpdate(world, player, itemStack);

ItemStack boots = player.getCurrentArmor(0);

ItemStack legs = player.getCurrentArmor(1);

ItemStack chest = player.getCurrentArmor(2);

ItemStack helm = player.getCurrentArmor(3);

// legs = 1, chest = 2, helm = 3

 

 

if (boots != null && boots.getItem() == CanadianMod.skates && legs != null && legs.getItem() == CanadianMod.hockeyPants

&& chest != null && chest.getItem() == CanadianMod.hockeyChest &&  helm != null && helm.getItem() == CanadianMod.hockeyHelmet && world.getBlockId((int)player.posX, (int)player.posX, (int)player.posX) == Block.ice.blockID){

 

 

player.motionX *= 1.1D;

player.motionZ *= 1.1D;

 

}

 

}

[coolboy4531]

Why did you input posX for all fields?

[ZacDoesStuff]

Woops, forgot to change that. Put I already had tried that before asking on the forums When I was asked for the code I just quickly put it back in

[ZacDoesStuff]

Changed it, still doesn't work...

[coolboy4531]

Can I see your code again as of now?

[ZacDoesStuff]

Here you go, keep in mind I'm in 1.6.4

 

public void onArmorTickUpdate(World world, EntityPlayer player, ItemStack itemStack)

    {

super.onArmorTickUpdate(world, player, itemStack);

ItemStack boots = player.getCurrentArmor(0);

ItemStack legs = player.getCurrentArmor(1);

ItemStack chest = player.getCurrentArmor(2);

ItemStack helm = player.getCurrentArmor(3);

// legs = 1, chest = 2, helm = 3

 

 

if (boots != null && boots.getItem() == CanadianMod.skates && legs != null && legs.getItem() == CanadianMod.hockeyPants

&& chest != null && chest.getItem() == CanadianMod.hockeyChest &&  helm != null && helm.getItem() == CanadianMod.hockeyHelmet && world.getBlockId((int)player.posX, (int)player.posY, (int)player.posZ) == Block.ice.blockID){

 

 

player.motionX *= 1.1D;

player.motionZ *= 1.1D;

 

}}

 

 

The player.motionX and Z will change to probably a speed potion effect when on ice, since currently that has some weird effects when on ice

[Mecblader]

Check the block below you by doing player.posY-1

[ZacDoesStuff]

I've done that too

[Mecblader]

Do a System.out.println("Working")  after the check to test it because I think I remember trying motion.x +=1 or something like that, but it didn't work.

[ZacDoesStuff]

  On 4/20/2014 at 9:13 PM, Mecblader said:

Do a System.out.println("Working")  after the check to test it because I think I remember trying motion.x +=1 or something like that, but it didn't work.

 

The problem isn't in the motion.x, that works fine without looking for a block. The problem itself is looking for a block

[Mecblader]

I think I figured it out, for some reason if you get the player y coord from the server it outputs a different coord than if it is client. Just add a check before looking for the block to see if the world is not remote because you want to do the checking on the server. (!Remote = Server, Remote = Client).

[ZacDoesStuff]

It's still not working:

 

public void onArmorTickUpdate(World world, EntityPlayer player, ItemStack itemStack)

    {

 

ItemStack boots = player.getCurrentArmor(0);

ItemStack legs = player.getCurrentArmor(1);

ItemStack chest = player.getCurrentArmor(2);

ItemStack helm = player.getCurrentArmor(3);

// legs = 1, chest = 2, helm = 3

 

if (!world.isRemote){

 

if (boots != null && boots.getItem() == CanadianMod.skates && legs != null && legs.getItem() == CanadianMod.hockeyPants

&& chest != null && chest.getItem() == CanadianMod.hockeyChest &&  helm != null && helm.getItem() == CanadianMod.hockeyHelmet && world.getBlockId((int)player.posX, (int)player.posY-1, (int)player.posZ) == Block.ice.blockID){

 

 

player.motionX *= 1.1D;

player.motionZ *= 1.1D;

 

}

}

    }

[delpi]

I'm not sure about your experience, but I've had Bette luck with .equals instead if == for forge.

 

Also on your coords, I'd use a floor before the int cast.  I've had issues with neg coords before.