generic task creation for NumberTopHolder

Author: HSGamer

template/top-player-number/src/main/java/me/hsgamer/topper/template/topplayernumber/TopPlayerNumberTemplate.java

@@ -35,7 +35,9 @@ protected TopPlayerNumberTemplate(Settings settings) {
 
     public abstract Optional<ValueProvider<UUID, Double>> createValueProvider(Map<String, Object> settings);
 
-    public abstract Agent createTaskAgent(Runnable runnable, boolean async, long delay);
+    public abstract Agent createTask(Runnable runnable, NumberTopHolder.TaskType taskType);
+
+    public abstract Agent createUpdateTask(Runnable runnable, boolean async);
 
     public abstract void logWarning(String message, @Nullable Throwable throwable);
 
@@ -90,16 +92,10 @@ public NameProviderManager getNameProviderManager() {
     public interface Settings {
         Map<String, NumberTopHolder.Settings> holders();
 
-        int taskSaveDelay();
-
         int taskSaveEntryPerTick();
 
         int taskUpdateEntryPerTick();
 
-        int taskUpdateDelay();
-
-        int taskUpdateSetDelay();
-
         int taskUpdateMaxSkips();
     }
 }

template/top-player-number/src/main/java/me/hsgamer/topper/template/topplayernumber/holder/NumberTopHolder.java

@@ -39,7 +39,7 @@ public NumberTopHolder(TopPlayerNumberTemplate template, String name, Settings s
         storageAgent.setMaxEntryPerCall(template.getSettings().taskSaveEntryPerTick());
         agents.add(storageAgent);
         agents.add(storageAgent.getLoadAgent(this));
-        agents.add(template.createTaskAgent(storageAgent, true, template.getSettings().taskSaveDelay()));
+        agents.add(template.createTask(storageAgent, TaskType.STORAGE));
         entryAgents.add(storageAgent);
 
         ValueProvider<UUID, Double> valueProvider = template.createValueProvider(settings.valueProvider()).orElseGet(() -> {
@@ -67,15 +67,15 @@ public NumberTopHolder(TopPlayerNumberTemplate template, String name, Settings s
         }
         updateAgent.setMaxSkips(template.getSettings().taskUpdateMaxSkips());
         entryAgents.add(updateAgent);
-        agents.add(template.createTaskAgent(updateAgent.getUpdateRunnable(template.getSettings().taskUpdateEntryPerTick()), isAsync, template.getSettings().taskUpdateDelay()));
-        agents.add(template.createTaskAgent(updateAgent.getSetRunnable(), true, template.getSettings().taskUpdateSetDelay()));
+        agents.add(template.createUpdateTask(updateAgent.getUpdateRunnable(template.getSettings().taskUpdateEntryPerTick()), isAsync));
+        agents.add(template.createTask(updateAgent.getSetRunnable(), TaskType.SET));
 
         this.snapshotAgent = SnapshotAgent.create(this);
         boolean reverseOrder = settings.reverse();
         snapshotAgent.setComparator(reverseOrder ? Comparator.naturalOrder() : Comparator.reverseOrder());
         snapshotAgent.setFilter(entry -> entry.getValue() != null);
         agents.add(snapshotAgent);
-        agents.add(template.createTaskAgent(snapshotAgent, true, 20L));
+        agents.add(template.createTask(snapshotAgent, TaskType.SNAPSHOT));
 
         entryAgents.add(new DataEntryAgent<UUID, Double>() {
             @Override
@@ -126,6 +126,12 @@ public ValueDisplay getValueDisplay() {
         return valueDisplay;
     }
 
+    public enum TaskType {
+        STORAGE,
+        SET,
+        SNAPSHOT,
+    }
+
     public interface Settings {
         Double defaultValue();