Jump to content
Search In
  • More options...
Find results that contain...
Find results in...

Damage player every second for 10sec


Niernen
 Share

Recommended Posts

For my mod I want a player receive drowning damage for 10sec, every second 1 heart of damage.

The damage part is no problem using

player.attackEntityFrom(DamageSource.drown, 2);

but how can I make this repeat every second for 10 seconds?

Link to comment
Share on other sites

I think you could try subscribing to onPlayerTick() using:

 

 @SubscribeEvent
public void onPlayerTick(PlayerTickEvent evt)

 

There are 20 ticks per second. Adding a tick counter is what I would do.

 

More reading on ticks subject can be found here.

 

Update - something like this could work:

private int tickCounter = 0;

@SubscribeEvent
public void onPlayerTick(PlayerTickEvent evt)
{
  
  if (tickCounter % 20 == 0 && tickCounter < 201)
  {
      player.attackEntityFrom(DamageSource.drown, 2);
  }

  tickCounter++;
}

Link to comment
Share on other sites

PlayerTickEvent is fired for each player on both sides, you need to:

 

1. Perform damaging on server - if (!event.player.worldObj.isRemote)

 

2. TickEvents have 2 phases - START and END - pick one (event.phase).

 

3.

private int tickCounter = 0;

@SubscribeEvent
public void onPlayerTick(PlayerTickEvent evt)
{
  
  if (tickCounter % 20 == 0 && tickCounter < 201)
  {
      player.attackEntityFrom(DamageSource.drown, 2);
  }

  tickCounter++;
}

 

While code itself if not bad in mean of ticks/timing, it is bad regarding design.

Held "tickCounter" will be shared between all players (on server side/thread), you can't do it this way.

 

What you need to do is:

* Use IEEP (1.8.9-) or @Capability (1.8.9+) to store counter per-player.

* Use stored counter in event.

* Capability CAN be server-only - unless you need other stuff, or need client to display counter, damaging happens server only, so counter can be too server sided.

 

Links:

http://mcforge.readthedocs.io/en/latest/datastorage/capabilities/

http://mcforge.readthedocs.io/en/latest/concepts/sides/

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

Ernio:

Thank you for your input, I will have a look into materials you provided.

 

Just one more question to clarify exactly what you meant:

 

'While code itself if not bad in mean of ticks/timing, it is bad regarding design.'

Is there a typo in 'itself if not bad' or not? Like if you meant to say 'is not bad' implying that it is correct to subscribe to PlayerTickEvent, or you meant to say that I am questionably wrong with subscribing to that event?

 

Thanks in Advance. :-)

Link to comment
Share on other sites

I ment that this is the proper way (one of them as d7 suggested using potions) to perform tick operations on player.

BUT

It has flaw in design where you need to have player-specific counters.

@SubscribeEvent
public void onPlayerTick(PlayerTickEvent evt)
{
  // if server
  // if phase
  int counter = player#capability#counter
  if (counter > 0 && counter % 20 == 0)
  {
      player.attackEntityFrom(DamageSource.drown, 2);
  }

  --player#cpability#counter;
}

Where this is pseudocode assuming you are holding integer counter in player's capability. Whenever capability counter is bigger than 0, decrement it from tick event.

 

How to make such per-player data storage has been linked.

 

And yeah - this is good advice:

May I suggest using a PotionEffect? :D

1.7.10 is no longer supported by forge, you are on your own.

Link to comment
Share on other sites

Couldn't you also make it player-specific by checking the player's ticksExisted in your tick counter? Like this:

 

if (event.player.ticksExisted % 20 == 0)

 

Only if you knew when to stop applying the damage.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share



  • Recently Browsing

    No registered users viewing this page.

  • Posts

    • Thanks everybody! I managed to fix it. I just had some missing attributes in my OilFluid class
    • W10 Pro. Android 11 on my S20+(or is it 12 already?). My two other computers are running on W10 home, all behind G-Data security suite, and a Fritz. I'm supposed to use Linux, no? I hate making things compatible, and Linux has limited access to some games I love, so...Mac is even worth I guess? I'm buying servers btw. It's not I couldn't set one up for Linux, but I lack the skills in online security, and I'm paranoid. So not a good combo I guess?
    • Yeah, the whole registry process can be quite confusing and daunting for how MC is working nowadays, but once you get the grasp of it, it actually saves a whole lot of effort, and quantity of code. I'd spend 3 days+ to actually figure I had to apply the getter(.get())[<-- is this even named getter?], to any of the .class references, because stupid Eclipse refused to de-obfuscate the source of problems. It was just making stupid a** auto fix suggestions, that didn't help at all. Google found the answer quite often though. And don't even get me started when I was trying to implement the Oil to world generation. I was literally tearing my hair out for days. But don't give up. We're not the only ones dealing with the reinvention of MC code wheel. I mean I get it. There's always room for improvements. But a wheel is just a wheel, right?
    • Thank you everybody, I have figured out the cause. I was accidentally passing in the constructor for the Fluids instead of the RegistryObject. I changed: super(new ForgeFlowingFluid.Properties( OilFluid.Source::new, OilFluid.Flowing::new, to: super(new ForgeFlowingFluid.Properties( ModFluids.OIL, ModFluids.FLOWING_OIL, And now the fluid actually works! But it doesn't spread though, I think I'd be able to figured it out from now. Thanks a lot Luis_ST and Cratthorax!
    • Because I was used to doing it like that in the past. Also, a block is very sparse in code quantity. Also, also, I would use the blocks to check against blocks of same type at lower levels, to mark as an indicator to what can be found digging deeper into the ground. Doing that with anything but the same object type is just unnecessarily bloating code. However, I had to use creative ways to get something working. What I did was rendering my block and its voxel shape invisible, make it nonsolid and notBlockMovement, then apply the randomTick() method, and using this: worldIn.destroyBlock(pos, true);...which basically drops the item after the blocks gets randomly destroyed. The good thing about that is, with the new DataPack function, I can do all that stuff with a single block object, and then just use instances of other block registers and their properties. I can also customize the loot tables for any of them, but don't need an actual block.class for any of them. Given how I have more then 60 ore blocks in my old mod, that is a really great thing about newer MC versions. The final solution was actually found in vanilla Block.AIR, after messing around with various methods, none of them doing what was easily done in the past MC versions with onBlockAdded(), and dropBlockAsItem(). I'll just leave the block code, and register code here for reference, which should give you a picture of the many things I was trying without success:
  • Topics

  • Who's Online (See full list)

×
×
  • Create New...

Important Information

By using this site, you agree to our Privacy Policy.