How do i add an amount of time to a command like /mute?

I am developing my first plugin and I need help on how to add a time field to a /mute command or any command like that such as freeze. Any help would be greatly appreciated! Thanks!

 

Use bukkit scheduler. Take a look at http://wiki.bukkit.org/Scheduler_Programming#Delayed_Tasks, that is probably what you need.

You probably want to set the player as muted (storing his NAME in a hashmap, storing his actual player instance will cause memory leaks, in a hashmap something like: Map<String, boolean> mutelist = new HashMap<String, boolean>(); )

Then on the command, store the player.getName() string as the string, and the boolean as true.
Using bukkit scheduler, after the time specified, have it remove the player from that list.

In a PlayerChatEvent handler, have it check if the player is in the hashmap, and if so, cancel his chat event.

 

You can accomplish the same effect without scheduling, you just need to note when they can talk again, then compare against the current time.

 

its more memory eficent to use an Set<String> instaead of an Map<String,Boolean>, but the others aspects, good tutorial

 

im currently using public ArrayList<String> mute = new ArrayList<String>(); to store the list of muted players.

 

chance it to an set for more speed and less memory, you know, the small details matter

 

Actually according to some IBM presentation a HashSet uses more memory and does less than a HashMap.
http://t.co/oQIhYWRY

Officialjake
If you need to keep a collection of players but don't care about their order a set is the correct data structure.

 

so, most maps use sets as back ground, how can maps be smaller than sets if they use sets?

 

Your argument is invalid Sagacious_Zed posted a very long pdf which probably explains it in which I did not read (I actually agree with you hash sets use less than hash maps)

 

The short version is that a HashSet is implemented with a HashMap. Therefore a HashSet is a HashMap with more data.
Not all Sets are bigger than Maps. This is a unique situation where a HashSet is bigger than a HashMap.

EDIT: here is another article this one is shorter http://www.ibm.com/developerworks/java/library/j-codetoheap/index.html the other one came to mind first.

 

my test shows something else

Code:java

public static void main(String[] args)
	{
		long memoryFirst = 0L;
 
		System.gc();
 
		memoryFirst = Runtime.getRuntime().freeMemory();
		System.out.println("Testing HashMap first: free mem now: " + Runtime.getRuntime().freeMemory());
		HashMap<String, Boolean> map = new HashMap<String, Boolean>(100);
 
		System.out.println("Testing HashMap after creation: free mem now: " + Runtime.getRuntime().freeMemory());
		for (int i = 0; i < 100; i++)
		{
			map.put(Integer.toString(i), true);
		}
 
		System.out.println("Testing HashMap after putting: free mem now: " + Runtime.getRuntime().freeMemory());
		System.out.println("usedmem: " + (memoryFirst-Runtime.getRuntime().freeMemory()));
 
		map.clear();
		map = null;
 
		System.gc();
 
		memoryFirst = Runtime.getRuntime().freeMemory();
		System.out.println("Testing HashSet now: free mem now: " + Runtime.getRuntime().freeMemory());
		HashSet<String> set = new HashSet<String>(100);
		System.out.println("Testing HashSet after creation: free mem now: " + Runtime.getRuntime().freeMemory());
		for (int i = 0; i < 100; i++)
		{
			set.add(Integer.toString(i));
		}
 
		System.out.println("Testing HashSet after putting: free mem now: " + Runtime.getRuntime().freeMemory());
		System.out.println("usedmem: " + (memoryFirst-Runtime.getRuntime().freeMemory()));
 
		set.clear();
		set = null;
	}

Code:

Testing HashMap first: free mem now: 16101464
Testing HashMap after creation: free mem now: 16009608
Testing HashMap after putting: free mem now: 16009608
usedmem: 91856
Testing HashSet now: free mem now: 16101288
Testing HashSet after creation: free mem now: 16101288
Testing HashSet after putting: free mem now: 16101288
usedmem: 91816

Sets use 40 bytes less for 100 elements

 

I need to have the command be like /mute <player> <time> with the time being in d,h,m,s format.

I need to have the command be like /mute <player> <time> with the time being in d,h,m,s format.

can anyone help?

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

 

Well, it would kindof be hard to do it in days in the way I know how to code. I am not experienced at all but you could use a time int

Code:

int time = Integer.parseInt(args[1])*20;

That would make it so that the mute time would be in seconds. Every 20 ticks is 1 second. That is why we have the *20.

 

Thanks yea for a mute command hours and days are impractical.

 

Create a HashMap<String, Long> the string being the name and the long being (system.currentTimeMillis + the time to be muted)
Then create repeatedsheduleevent and go through the list to see when the currentTimeMillis is larger than the stored value of the player, if so, you can remove that player from the list.

And the improve performance you should sort the collection and use binary search to lookup names when they try to talk.

You really shouldn't be worried about memory usage. It would take a lot (10000+) before you'd being use ~1MB of heap space. (based on 2M entries using 200MB heapspace).