[Chat] Kill counter prefix

I would only save them in the onDisable() and load them in the onEnable()

 

autoit4you Okay and is there any chance that can you give me some tips on how do save this?

 

This should work, if you put it into the main class:

Code:java

public void saveKills(HashMap<String, Integer> kills) {
  for(String name : kills.keySet()) {
    getConfig().set("kills." + name, kills.get(name));
  }
  saveConfig();
}
 
public HashMap<String, Integer> getKills() {
  HashMap<String, Integer> kills = new HashMap<String, Integer>();
  ConfigurationSection players = getConfig().getConfigurationSection("kills");
  if(players == null)
    return kills;
  for(String name : players.getKeys()) {
    kills.put(name, (Integer)getConfig().getInt("kills." + name));
  }
  return kills;
}

 

autoit4you What do I do for the OnDisable and OnEnable?

autoit4you If a player has { 0 } then if gets killed it looks like {-1} how would I change this so if the player has {0} it cant go any lower?

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

 

Something like this:

Code:java

private ChatListener cl = null;
 
public void onEnable() {
  cl = new ChatListener();
  cl.kills = getKills();
  getServer().getPluginManager().registerEvents(cl, this);
}
 
public void onDisable() {
  saveKills(cl.kills);
}

post the code of the onPlayerDeath function

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

 

autoit4you

Code:java

@EventHandler(priority = EventPriority.HIGHEST)
    public void onPlayerDeath(PlayerDeathEvent event) {
        if (event.getEntity().getKiller() instanceof Player) {
            Player killer = event.getEntity().getKiller();
            Player victim = event.getEntity();
            if (!kills.containsKey(killer.getName()))
                kills.put(killer.getName(), 0);
            if (!kills.containsKey(victim.getName()))
                kills.put(victim.getName(), 0);
 
            kills.put(killer.getName(), kills.get(killer.getName()) + 2);
            kills.put(victim.getName(), - 4);

and now the prefix shows up like this

 

Are you sure you do not set a displayname?

 

You can also use "p.setPrefix()" instead of "p.setDisplayName()"

 

Code:java

 
import java.util.HashMap;
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.EventPriority;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.AsyncPlayerChatEvent;
 
public class ChatListener implements Listener {
 
    public HashMap<String, Integer> kills = new HashMap<String, Integer>();
 
    @EventHandler(priority = EventPriority.HIGHEST)
    public void OnPlayerChat(AsyncPlayerChatEvent e) {
        int killAmount = 0;
        if (kills.containsKey(e.getPlayer().getName()))
            killAmount = kills.get(e.getPlayer().getName());
        String format = e.getFormat();
        e.setFormat(ChatColor.GRAY + "{ " + ChatColor.GOLD + killAmount
                + ChatColor.GRAY + " } " + ChatColor.RESET + format);
    }
 
    @EventHandler(priority = EventPriority.HIGHEST)
    public void onPlayerDeath(PlayerDeathEvent event) {
        if (event.getEntity().getKiller() instanceof Player) {
            Player killer = event.getEntity().getKiller();
            Player victim = event.getEntity();
            if (!kills.containsKey(killer.getName()))
                kills.put(killer.getName(), 0);
            if (!kills.containsKey(victim.getName()))
                kills.put(victim.getName(), 0);
 
            kills.put(killer.getName(), kills.get(killer.getName()) + 2);
            kills.put(victim.getName(), - 4);
        }
    }
}
 

 

add a System.out.println("AsyncChatEvent"); to the onPlayerChat function
And then chat ingame and copy the console output.

 

autoit4you

[INFO] AsyncChatEvent
[INFO] AsyncChatEvent
[INFO] { 0 } { 2 } <historio13> t

 

looks like the AsyncPlayerChatEvent gets fired two times

Code:

if(!format.startsWith(ChatColor.GRAY + "{") {
  //the e.setFormat stuff
}

 

autoit4you Fixed and how do I stop {0} from going like {-4}

 

You currently set it for everyone who is killed to -4.
Do you want to set it for everyone who is killed by a player to 0?
Or do you want to substract 4 from the current kills the player has?

 

autoit4you substract 4 from the current kills the player has

Code:java

kills.put(killer.getName(), kills.get(killer.getName()) + 2);
            kills.put(victim.getName()kills.get(victim.getName()) - 4);
        }

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

 

If you want that the kills number does not go under 0 use this:

Code:java

if(kills.get(victim.getName()) -4 < 0) {
  kills.put(victim.getName(), 0);
} else {
  kills.put(victim.getName(), kills.get(victim.getName()) - 4);
}

 

autoit4you Thanks

autoit4you How would I do this a player reaches Gray{ Gold 100 Gray} then his/her prefix changes colour
to like Green{ Dark_Green100 Green} ?

Hope this makes sense.

P.s sorry autoit for asking you more questions D:

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

 

get the kills and if he has 100 or more kills it uses format a else it uses format b

 

autoit4you Like this? :

Code:java

@EventHandler(priority = EventPriority.HIGHEST)
public void OnPlayerChat(AsyncPlayerChatEvent e) {
int killAmount = 0;
if (kills.containsKey(e.getPlayer().getName()))
killAmount = kills.get(e.getPlayer().getName());
String format = e.getFormat();
if (!format.startsWith(ChatColor.GRAY + "{")) {
e.setFormat(ChatColor.GRAY + "{ " + ChatColor.GOLD + killAmount
+ ChatColor.GRAY + " } " + ChatColor.RESET + format);
} else {
 
if(killAmount = int 100);
e.setFormat(ChatColor.BLACK + "{ " + ChatColor.GREEN + killAmount
+ ChatColor.BLACK + " } " + ChatColor.RESET + format);
}
}

But (Line 28) " if(killAmount = int 100);" gives me this error "Type mismatch: cannot convert from int to boolean"

 

historio nearly.

Code:java

if (!format.startsWith(ChatColor.GRAY + "{")) {
if(killAmount >= 100) {
e.setFormat(ChatColor.BLACK + "{ " + ChatColor.GREEN + killAmount + ChatColor.BLACK + " } " + ChatColor.RESET + format);
} else {
e.setFormat(ChatColor.GRAY + "{ " + ChatColor.GOLD + killAmount + ChatColor.GRAY + " } " + ChatColor.RESET + format);
}

killAmount >= 100 means killAmount equals 100 or is higher than 100

 

autoit4you Thanks I'll test it tomorrow because I have to go now Bye

autoit4you How would I make a command that would add kills to a player.

I have done this so far:

Code:java

if (commandLabel.equalsIgnoreCase("Addkills")) {
Player p = (Player) sender;
if(p.hasPermission("AdvancedKitPvp.addkills"))
kills.put(p.getName(), kills.get(p.getName()));

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

 

check if the arguments list is > 0
then parse argument0 to an Integer
than add the parsed number to the kills the player already has