Solved Need help making a player bleed 1/2 heart

Hello, I have been learning Java for the past week or 2 and have come into a bump on the plugin i am working on. I want the plugin to make you "Bleed" 1/2 Hearts every 20 seconds or so, I tried using the wither effect but that sadly didn't work because i cannot alter the strength to a decimal or percent. This is the code I have so far:

Code:java

package com.aidan.TeamHeal;
 
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;
import org.bukkit.potion.PotionEffect;
import org.bukkit.potion.PotionEffectType;
 
 
public class BukkitListener implements Listener {
 
    public static Bukkit plugin;
 
    public BukkitListener(Bukkit instance) {
        plugin = instance;
    }
 
    @EventHandler
    public void onEntityDamageByEntity(EntityDamageByEntityEvent event) {
        if(Math.random() <0.10) {
            Entity e = event.getEntity();
//            Entity z = event.getDamager();
 
            if(e instanceof Player) {
                Player player = (Player) e;
 
                player.addPotionEffect(new PotionEffect(PotionEffectType.BLINDNESS, 100, 1));
 
            }
 
 
        }
        if(Math.random()<0.30) {
            Entity e = event.getEntity();
 
            if(e instanceof Player) {
                Player player = (Player) e;
 
                player.addPotionEffect(new PotionEffect(PotionEffectType.WITHER, 100000, 0));
            }
 
 
        }
    }
 

 

Aiddamaid Can't you just use a scheduler and Player.damage(1.0D); ?

 

Aiddamaid as Aengo said, use a sheduler i sugest a repeating task and every X seconds loop thru all players in the server and do the damage thing
PS: just a tim you may alreaby be aware of but... ok, in bukkit when u do sethealth(gethealth-1) the -1 is a/2 if a heart

 

Aengo mine-care , Ok, I will try your methods of doing this, thanks for the help, Just one question, i can use this on just specific players that get damaged by entity's, right?

 

Aiddamaid wait wait wait, you want the players to be damaged when they are hit by a entity or randomly damage them?

 

mine-care I want the players to have a %10 chance that they will bleed for __ amount of time until they heal themselves, and i need them to bleed 1/2 heart every 20 seconds.

 

Aiddamaid

Create a repeating task that damages all players in a list using a random number to determine whether they will be damaged. Add and remove players from this list when the are "poisoned" or "damaged."

 

mythbusterma got it first

 

Ok, So this is the new code I have for the bleeding effect but it doesn't make you bleed, am i doing something wrong with it?

Code:java

@EventHandler
    public void onEntityDamageByEntityEvent(EntityDamageByEntityEvent event) {
        if(Math.random()<0.10) {
        Entity entity = event.getEntity();
 
        if(entity instanceof Player) {
            Player player = (Player) entity;
 
                getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
                    public void run() {
                        EntityDamageEvent.DamageCause.STARVATION;
                    }
                }, 20, 300);
 
            }
        }
    }

 

Aiddamaid Change

Code:java

EntityDamageEvent.DamageCause.STARVATION;

to

Code:java

player.damage(1.0D);

 

I changed the damageEvent to player.damage but it is not repeatedly damaging me every 15 seconds, Do you know a fix for that?

 

Aiddamaid You're only factoring in the randomness when scheduling the task.
Make it factor randomness on every task run, like so:

Code:java

@EventHandler
public void onEntityDamageByEntityEvent(EntityDamageByEntityEvent event) {
    Entity entity = event.getEntity();
    if(entity instanceof Player) {
        Player player = (Player) entity;
        getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable() {
            public void run() {
                if(Math.random()<0.10) {
                    EntityDamageEvent.DamageCause.STARVATION;
                }
            }
        }, 20, 300);
    }
}

 

Everything you helped me with worked out, is there any way to make it so that when you are bleeding, and you get damaged, you won't get another bleeding effect, because currently the bleeding effects can stack so every 15 seconds you lose 5 hearts instead of 1 half heart, this is my updated code:

Code:java

@EventHandler
public void onEntityDamageByEntityEvent(EntityDamageEvent event) {
Entity e = event.getEntity();
 
 
 
plugin.getServer().getScheduler().scheduleSyncRepeatingTask(plugin, new Runnable() {
public void run() {
if(Math.random()<=1);
 
if(e instanceof Player) {
Player player = (Player) e;
 
player.damage(1.0D);
player.sendMessage(ChatColor.RED + "You are bleeding, find a bandage and heal yourself");
}
 
}
}, 20, 300);
}
 
}

 

Aiddamaid You could make a List<String> for the names of players that are bleeding. If the list contains the player, don't let them bleed again. Once they bandage up, remove them from the list.

 

Ok, i will try that,

kps1796 So I made it so the players only get damaged once, but how would you make a piece of paper on right click cancel the event?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.