TotalFreedomMC/TF-ProjectKorra
1
0
Fork 0
mirror of https://github.com/TotalFreedomMC/TF-ProjectKorra.git synced 2025-02-18 22:24:35 +00:00
Code Issues Projects Releases Wiki Activity

Update BaseAbility

Browse source 
* Add 3 new fields:
* UniqueId
* Player
* StockAbility
* Add Final modifier to putInstance(), removeInstance()
getInstance(StockAbilities)
* Add private modifier to removeInstance()
* Implement remove() calling removeInstance()
* Created Convenience method getInstance() to be called from abilities
themselves
* Add abstract method getStockAbility()
* Add getters for Player. UniqueId
This commit is contained in:
Jack Lin 2015-07-15 17:42:34 +12:00
parent 3a7242e2d2
commit 50cbaade71
1 changed files with 70 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

81
src/com/projectkorra/ProjectKorra/Ability/BaseAbility.java
View file

@ -3,6 +3,8 @@ package com.projectkorra.ProjectKorra.Ability;
import java.util.UUID; import java.util.UUID;
import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.ConcurrentHashMap;
import org.bukkit.entity.Player;
import com.projectkorra.ProjectKorra.configuration.ConfigLoadable; import com.projectkorra.ProjectKorra.configuration.ConfigLoadable;
/** /**
@ -10,28 +12,49 @@ import com.projectkorra.ProjectKorra.configuration.ConfigLoadable;
*/ */
public abstract class BaseAbility implements Ability { public abstract class BaseAbility implements Ability {
/**
* ConcurrentHashMap that stores all Ability instances under UUID key.
* To access this hashmap use either {@link #getInstance()} from the
* ability instance or {@link #getInstance(StockAbilities)} from the
* outside.
*/
private static ConcurrentHashMap<StockAbilities, ConcurrentHashMap<UUID, BaseAbility>> instances = new ConcurrentHashMap<>(); private static ConcurrentHashMap<StockAbilities, ConcurrentHashMap<UUID, BaseAbility>> instances = new ConcurrentHashMap<>();
//protected static AbilityMap<Ability> instances = new AbilityMap<>(); //protected static AbilityMap<Ability> instances = new AbilityMap<>();
protected void putInstance(StockAbilities sb, UUID uuid, BaseAbility ability) { private final StockAbilities stockAbility = getStockAbility();
if (instances.containsKey(sb)) { private Player player;
instances.get(sb).put(uuid, ability); private UUID uniqueId;
protected final void putInstance(Player player, BaseAbility ability) {
this.uniqueId = player.getUniqueId();
this.player = player;
if (instances.containsKey(stockAbility)) {
instances.get(stockAbility).put(uniqueId, ability);
} else { } else {
ConcurrentHashMap<UUID, BaseAbility> map = new ConcurrentHashMap<>(); ConcurrentHashMap<UUID, BaseAbility> map = new ConcurrentHashMap<>();
map.put(uuid, ability); map.put(uniqueId, ability);
instances.put(sb, map); instances.put(stockAbility, map);
} }
} }
protected void removeInstance(StockAbilities sb, UUID uuid) { /**
if (instances.containsKey(sb)) { * Removes the UUID from the instances map
if (instances.get(sb) != null) { */
instances.get(sb).remove(uuid); private final void removeInstance() {
if (instances.containsKey(stockAbility)) {
if (instances.get(stockAbility) != null) {
instances.get(getStockAbility()).remove(uniqueId);
} }
} }
} }
public static ConcurrentHashMap<UUID, BaseAbility> getInstance(StockAbilities abilty) { /**
* An access method to get an the instances of a {@link StockAbilities StockAbility}.
*
* @param abilty The instances map to get
* @return a map of instances from the specified {@link StockAbilities StockAbility}
*/
public final static ConcurrentHashMap<UUID, BaseAbility> getInstance(StockAbilities abilty) {
if (instances.containsKey(abilty)) { if (instances.containsKey(abilty)) {
return instances.get(abilty); return instances.get(abilty);
} }
@ -42,8 +65,9 @@ public abstract class BaseAbility implements Ability {
* Convenience method that calls {@link #progress()} for all instances. * Convenience method that calls {@link #progress()} for all instances.
*/ */
public static void progressAll(StockAbilities ability) { public static void progressAll(StockAbilities ability) {
for (UUID uuid : getInstance(ability).keySet()) for (UUID uuid : getInstance(ability).keySet()) {
instances.get(ability).get(uuid).progress(); instances.get(ability).get(uuid).progress();
}
} }
/** /**
@ -54,5 +78,40 @@ public abstract class BaseAbility implements Ability {
instances.get(ability).get(uuid).remove(); instances.get(ability).get(uuid).remove();
} }
} }
/**
* Calls {@link #removeInstance()}, Developers can override this
* method to do other things when remove is called but they
* <strong>MUST</strong> remember to call {@code super.remove()}
* for the UUID to be properly removed from the {@link #instances}.
*/
@Override
public void remove() {
removeInstance();
}
/**
* Gets the {@link StockAbilities StockAbility} that created this instance.
*
* @return stockabilities enum or null
*/
public abstract StockAbilities getStockAbility();
/**
* Convenience method to get instance map for current ability class.
*
* @return {@link #getInstance(StockAbilities)} for the current ability
*/
public ConcurrentHashMap<UUID, BaseAbility> getInstance() {
return getInstance(stockAbility);
}
public Player getPlayer() {
return player;
}
public UUID getUniqueId() {
return uniqueId;
}
} }