Add SubCommand.description

SpigotCore_Main/src/de/steamwar/command/SWCommand.java

@@ -92,16 +92,7 @@ public abstract class SWCommand {
                     Bukkit.getLogger().log(Level.WARNING, "The method '" + method.toString() + "' is lacking the varArgs parameters of type '" + String.class.getTypeName() + "' as last Argument");
                     return;
                 }
-                if (method.getName().equals("internalHelp")) {
+                commandHelpList.add(new SubCommand(this, method, anno.description(), anno.value(), new HashMap<>()));
-                    System.out.println(method + " " + hasHelp);
-                    if (hasHelp) {
-                        return;
-                    }
-                } else if (!hasHelp) {
-                    commandHelpList.clear();
-                    hasHelp = true;
-                }
-                commandHelpList.add(new SubCommand(this, method, anno.value(), new HashMap<>()));
             });
         }
         for (Method method : methods) {
@@ -123,7 +114,7 @@ public abstract class SWCommand {
                         return;
                     }
                 }
-                commandList.add(new SubCommand(this, method, anno.value(), localTypeMapper));
+                commandList.add(new SubCommand(this, method, anno.description(), anno.value(), localTypeMapper));
             });
 
             this.commandList.sort((o1, o2) -> {
@@ -171,15 +162,8 @@ public abstract class SWCommand {
     }
 
     private List<Method> methods() {
-        Class<?> current = getClass();
+        List<Method> methods = Arrays.asList(getClass().getDeclaredMethods());
-        List<Method> methods = new ArrayList<>();
+        methods.addAll(Arrays.asList(SWCommand.class.getDeclaredMethods()));
-        int count = 0;
-        while (current != null && count < 4) {
-            methods.addAll(Arrays.asList(current.getDeclaredMethods()));
-            current = current.getSuperclass();
-            count++;
-        }
-        System.out.println(methods);
         return methods;
     }
 
@@ -209,24 +193,26 @@ public abstract class SWCommand {
             commandList.forEach(subCommand -> {
                 StringBuilder st = new StringBuilder();
                 st.append("§8/§7").append(command.getName()).append(" ");
-                st.append("§7").append(String.join(" ", subCommand.subCommand)).append(" ");
+                st.append("§7").append(String.join(" ", subCommand.subCommand));
-                int i = 1;
+                String cmd = Arrays.stream(subCommand.parameters)
-                while (i < subCommand.parameters.length) {
+                        .skip(1)
-                    Parameter parameter = subCommand.parameters[i];
+                        .map(parameter -> {
                             Name name = parameter.getAnnotation(Name.class);
-                    st.append("§8[§7");
                             if (name != null) {
-                        st.append(name.name());
+                                return name.name();
                             } else {
-                        st.append(parameter.getName());
+                                return parameter.getName();
-                    }
-                    st.append("§8]");
-                    i++;
                             }
+                        })
+                        .map(param -> "§8[§e" + param + "§8]")
+                        .collect(Collectors.joining(" "));
+                st.append(cmd);
                 if (subCommand.varArgType != null) {
                     st.append("§7...");
                 }
-                st.append("§8 - §7Description Here");
+                if (!subCommand.description.isEmpty()) {
+                    st.append("§8 - §7").append(subCommand.description);
+                }
                 help.add(st.toString());
             });
         }
@@ -240,6 +226,8 @@ public abstract class SWCommand {
     protected @interface Register {
         String[] value() default {};
 
+        String description() default "";
+
         boolean help() default false;
 
         @Retention(RetentionPolicy.RUNTIME)

SpigotCore_Main/src/de/steamwar/command/SubCommand.java

@@ -34,6 +34,7 @@ class SubCommand {
 
     private SWCommand swCommand;
     private Method method;
+    String description;
     Parameter[] parameters;
     String[] subCommand;
     TypeMapper<?>[] arguments;
@@ -41,9 +42,10 @@ class SubCommand {
     private Function<CommandSender, ?> commandSenderFunction;
     Class<?> varArgType = null;
 
-    SubCommand(SWCommand swCommand, Method method, String[] subCommand, Map<String, TypeMapper<?>> localTypeMapper) {
+    SubCommand(SWCommand swCommand, Method method, String description, String[] subCommand, Map<String, TypeMapper<?>> localTypeMapper) {
         this.swCommand = swCommand;
         this.method = method;
+        this.description = description;
 
         parameters = method.getParameters();
         commandSenderPredicate = sender -> parameters[0].getType().isAssignableFrom(sender.getClass());