Storing commands done, then getting them back as an int?

Ok, the title may been a little fuzzy, but what I was wondering was if it is possible, or more, how to store commands in a hashmap, hashset, arraylist or something familiar.
Then if you for an example do "/check /lol" to check how many times the command /lol have been done in the server, you'd get something like "the command /lol have been done __ times".

So I've done something like

Code:java

public Set<String> command = new HashSet<String>();

Then in the PlayerCommandPreprocessEvent I've done something like

Code:java

Player player = event.getPlayer();
String command = event.getMessage();
 
plugin.command.add(command);
player.sendMessage(ChatColor.GOLD + event.getMessage() + " have been typed "+ plugin.command.size() + " times.");

This is just some test code, but the first problem I get is that it'll display how much "/" have been typed, not how many times "/lol" have been typed or, "/smile".. (It does not register the command separately, only when the slash is typed.)
The second problem is that it does not register every command either, it just suddenly stops rising the size.
The third known problem is that when the plugin is reloaded or unloaded someway, it will reset the amount.

Yes, my knowledge isn’t on the top right now, and I've not tested to much either, but I'm kind of lost.
I've tried to explain my best here, hope you can help me someway

 

Create a HashMap<String, int>, where string is the command and int is the number of times used. Simply increment the value every time the command is handled.

 

Code:java

public Map<String, int> command = new HashMap<String, int>();

This returns the error:

Code:

unexpected type
required: reference
found: int

I do not seem to find any sulution on how to make a HashMap using int, because of that error, any fix?

EDIT: Sorry, found the problem, You have to use "Integer" instead of "int".

 

It's called auto-boxing. Generics don't allow primitive types (like int, byte, float, double, short, long, char), only classes. So every primitive type has auto-boxed equivalents, which are Objects so they can be used as generics.
The one for int is called "Integer" (exactly like that).

Reference: http://docs.oracle.com/javase/1.5.0/docs/guide/language/autoboxing.html

 

Sorry if I'm wrong, but

Code:java

@EventHandler
public static void main(PlayerChatEvent event) {
String[] args = null;
Map<String, Integer> m = new TreeMap<String, Integer>();
Player player = event.getPlayer();
if (event.getMessage().startsWith("/")) {
for (String word : args) {
Integer freq = m.get(word);
m.put(word, (freq == null ? 1 : freq + 1));
}
System.out.println(m);
player.sendMessage(ChatColor.GOLD + event.getMessage() + "have been done "+ m + "times.");
}
}

Doesn't return the player or the console any message, when a command is done :/

The event has been registered too.

 

You are not supposed to just copy and paste the example code from the linked page, it is not at all related to bukkit, just an informational page about Java auto-boxing ...
Stick to PlayerCommandPreprocessEvent, you don't want to capture all chat messages, only commands.

The HashMap (which shouldn't be a TreeMap) may not be a local variable in your event handler, it's supposed to be an attribute of your class. "public static void main(String[] args)" is the method head for the entry point of a Java program, why would you use that as an event listener ...

 

What about placing the incrementing code in the onCommand function?

 

I suppose his goal is to get statistics for all commands, not just the ones of his plugin. onCommand is only called for commands that you registered in your plugin.yml.

 

Thanks for the info