Killstreak Help.

Hello! I am developing a Killstreak plugin, at 5 kills, I want there to be a nuke, which would basically kill all players and count as a kill for the player that got the killstreak, I believe I'll have problems. So if there is any advice or help anyone could give that be great!

 

Can you show us your code that you already tried??

 

MCraftGamer35
When the player dies set add them to a hashmap with a value of 1
Check the value of the hashmap
If the value is 5 then kill everyone

When they die remove them from the hashmap

 

here's the code.

Code:java

package me.Alex.ElectricityKillstreak;
 
import java.util.HashMap;
 
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Killstreaks extends JavaPlugin {
 
    private HashMap<String, Integer> killstreak = new HashMap<String, Integer>(); {
    }
        Player p = (Player) sender;
        if(killstreak.containsKey(player.getName()))
            killstreak.put(player.getName(), killstreak.get(player.getName()) + 1);
        else
            killstreak.put(player.getName(), 1);
        if(killstreak.get(player.getName()) >= 5)
        {
            Player player = (Player) sender;
            player.sendMessage(ChatColor.GREEN + "A nuke has been called in!");
            {
 
 
}
 
        }
 
        }
 
}

 

MCraftGamer35
Ummm..... Do you know what you're doing?

 

This should help:
http://docs.oracle.com/javase/tutorial/
http://wiki.bukkit.org/Plugin_Tutorial

 

I know what I'm doing, i've coded multiple plugins.

 

MCraftGamer35
Your code doesn't make sense O_O at all...

 

Exactly. That's why I posted this. xD

 

Not to be rude, but are you sure? You have random brackets all over the place, you're missing both onEnable() and onDisable() methods, and the code that should to add/check for kills isn't even inside of a method. It seems to me like you need a lot more experience than you have right now before you start making plugins.

 

I have made a couple plugins. With a little help from tutorials. Also I haven't coded in 2 weeks, so I'm a little rusty.

 

MCraftGamer35
The amount of plugins you made doesn't really determine how well you are good with Java.
Quantity != Quality

krazytraynz
You don't need an onEnable or onDisable method in your main class. You only need one or both of them when you need to execute code when the plugin starts up is enabled (onEnable) or when the plugin is going to be disabled (onDisable).

Edit: Silly onLoad is where the plugin is loaded.

 

MCraftGamer35

Video tutorials?

They clearly haven't been of much use in helping you understand Java or the Bukkit API. Check out this Plugin Tutorial on BukkitWiki, and the Bukkit JavaDocs. Knowledge of Java is essential.

 

Here, this is my less sloppy version, and more working on version. Sorry for the confusion. I am tired and rusty.

Code:java

package me.Alex.ElectricityKillstreak;
 
import java.util.HashMap;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Killstreaks extends JavaPlugin {
 
    public void onEnable() {
        Bukkit.getServer().getLogger().info("Plugin enabled!");
    }
 
    public void onDisable() {
        Bukkit.getServer().getLogger().info("Plugin enabled!");
    }
 
    private HashMap<String, Integer> killstreak = new HashMap<String, Integer>(); {
 
    }
 
        if(killstreak.containsKey(player.getName()))
            killstreak.put(player.getName(), killstreak.get(player.getName()) + 1);
        else
            killstreak.put(player.getName(), 1);
        if(killstreak.get(player.getName()) >= 5);
        }
    }
 

 

Gater12
Huh, I've always assumed those were needed. My bad.

Spoiler (open) Spoiler (close)

 

Why do you have brackets when you use your HashMap? Weird.

[CLUE]: Try use onDeathEvent
Get the person who killed them? Then get there name? Use it as the key for the HashMap? Add a point to there name?
That is all I'm going to say.

 

MCraftGamer35
I'd recommend you watch some more video tutorials. I don't think you fully understand the Syntax of Java.

Edit:
You should do some independent research on Java and Object Oriented Programming.

 

JustinsCool15

You're kidding, right?

 

Dragonphase
Nope.avi

 

JustinsCool15

You should be telling him to do some independent research on Java and Object Oriented Programming...

 

Its always a good idea to plan before coding so you know what your going to do.
Let me help you out.

> Listen for player death.
> Get the killer and add that player to a hash map and give them a kill streak.
> If the player has 5 kills on the hash map then kill all players.
> Get how many people that were killed and add that number to the number of kill streak player has.

> If player dies then reset his kill streak to 0

 

Okk, my sugestion is:
1. first learn java! Cough cough Player casting, random brackets,ect.
2. Dont copy paste code or see the "tutorial" and type the code shown. You dont learn a thing from it.
3. read java docs when needed,
No offence, how would you rate yourself in a scale of 1-10 in terms of java knowledge?