added private constructor to util classes and typos fixed

core/src/main/java/me/koply/kcommando/boot/AnnotationChecks.java

@@ -49,9 +49,9 @@ public AnnotationBox check(Method method, Class<? extends Annotation> annotation
      */
     private BoxType commandoCheck(Method method) {
         Class<?> returnType = method.getReturnType();
-        boolean isboolean;
-        boolean isok = (isboolean = returnType == Boolean.TYPE) || returnType == Void.TYPE;
-        if (!isok) {
+        boolean isBoolean = returnType.equals(Boolean.TYPE);
+        boolean isOk = isBoolean || returnType == Void.TYPE;
+        if (!isOk) {
             if (KCommando.verbose) {
                 Kogger.info("The return type of " + method.getName() + " neither Boolean nor Void.");
             }
@@ -74,7 +74,7 @@ private BoxType commandoCheck(Method method) {
             }
         }
 
-        return isboolean && type != BoxType.UNKNOWN ? BoxType.fromValue(type.value+3) : type;
+        return isBoolean && type != BoxType.UNKNOWN ? BoxType.fromValue(type.value+3) : type;
     }
 
     /**
@@ -108,13 +108,13 @@ private BoxType similarCallbackCheck(Method method) {
 
         String typename = params[1].getParameterizedType().getTypeName();
 
-        boolean islist = typename.equals("java.util.List<java.lang.String>");
-        boolean isset = typename.equals("java.util.Set<java.lang.String>");
+        boolean isList = typename.equals("java.util.List<java.lang.String>");
+        boolean isSet = typename.equals("java.util.Set<java.lang.String>");
 
-        if (!(islist || isset))
+        if (!(isList || isSet))
             return null;
 
-        int value = islist ? 9 : 10;
+        int value = isList ? 9 : 10;
 
         if (params.length == 3 && params[2].getType() == String.class)
             value += 2;

core/src/main/java/me/koply/kcommando/boot/ClassLooter.java

@@ -10,6 +10,8 @@
 
 public class ClassLooter {
 
+    private ClassLooter() {}
+
     public static Set<Class<?>> getClasses(List<String> paths) {
         Set<Class<?>> classes = new HashSet<>();
         paths.forEach(path -> classes.addAll(getClasses(path)));

core/src/main/java/me/koply/kcommando/boot/KInitializer.java

@@ -83,10 +83,10 @@ private void registerCommandBox(Object instance, AnnotationBox box) {
         HandleCommand ann = (HandleCommand) box.annotation;
 
         int type = box.type.value;
-        boolean isboolean = type > 3;
+        boolean isBoolean = type > 3;
         CommandBox commandBox = new CommandBox(instance, box.method, box.clazz,
                 CommandBox.CommandType.fromBoxType(type),
-                isboolean ? CommandBox.ReturnType.BOOLEAN : CommandBox.ReturnType.VOID, ann);
+                isBoolean ? CommandBox.ReturnType.BOOLEAN : CommandBox.ReturnType.VOID, ann);
 
         for (String alias : ann.aliases()) {
             commandManager.commands.put(alias, commandBox);

core/src/main/java/me/koply/kcommando/handler/CommandHandler.java

@@ -165,7 +165,6 @@ public boolean process(Parameters p) {
                 if (methodName == null) return true;
 
                 FalseBox fbox = options.falseBoxMap.get(methodName);
-                System.out.println(methodName);
                 if (fbox != null) {
                     if (KCommando.verbose) Kogger.info("Calling false method: " + methodName);
                     invoke(fbox.type.value, fbox.instance, fbox.method, p.event, cmdArgs, prefix);
@@ -197,7 +196,7 @@ private Object invoke(int invokeValue, Object obj, Method method, Object...param
                 Kogger.warn("An impossible situation happened.");
                 result = null;
         }} catch (InvocationTargetException | IllegalAccessException e) {
-            Kogger.warn("An error occured while handling the command. Stacktrace:");
+            Kogger.warn("An error occurred while handling the command. Stacktrace:");
             e.printStackTrace();
             result = null;
         }

core/src/main/java/me/koply/kcommando/handler/SlashCommandHandler.java

@@ -30,7 +30,7 @@ public void process(Parameters p) {
         try {
             box.method.invoke(box.instance, p.event);
         } catch (InvocationTargetException | IllegalAccessException e) {
-            Kogger.warn("An error occured while handling a slash command. Stacktrace:");
+            Kogger.warn("An error occurred while handling a slash command. Stacktrace:");
             e.printStackTrace();
         }
     }

core/src/main/java/me/koply/kcommando/integration/Integration.java

@@ -40,7 +40,7 @@ public void removeCustomPrefix(long guildID, String prefix) {
             temp.remove(prefix);
             return temp;
         });
-        if (customGuildPrefixes.get(guildID).size() == 0) customGuildPrefixes.remove(guildID);
+        if (customGuildPrefixes.get(guildID).isEmpty()) customGuildPrefixes.remove(guildID);
     }
     /**
      * Removes guild from the custom prefix map

internal/src/main/java/me/koply/kcommando/internal/Kogger.java

@@ -10,15 +10,17 @@
 
 public class Kogger {
 
+    private Kogger() {}
+
     private static final Logger LOGGER;
     static {
         LOGGER = Logger.getLogger("KCommando");
         LOGGER.setUseParentHandlers(false);
         ConsoleHandler consoleHandler = new ConsoleHandler();
         consoleHandler.setFormatter(new Formatter() {
             private final DateFormat formatter = new SimpleDateFormat("HH:mm:ss.SSS");
-            public String format(LogRecord record) {
-                return String.format("[%s %s] %s -> %s\n", formatter.format(new Date(record.getMillis())), record.getLevel(), record.getLoggerName(), record.getMessage());
+            public String format(LogRecord logRecord) {
+                return String.format("[%s %s] %s -> %s%n", formatter.format(new Date(logRecord.getMillis())), logRecord.getLevel(), logRecord.getLoggerName(), logRecord.getMessage());
             }
         });
         LOGGER.addHandler(consoleHandler);

internal/src/main/java/me/koply/kcommando/internal/boxes/Box.java

@@ -8,7 +8,7 @@ public abstract class Box {
     public final Method method;
     public final Class<?> clazz;
 
-    public Box(Object instance, Method method, Class<?> clazz) {
+    protected Box(Object instance, Method method, Class<?> clazz) {
         this.instance = instance;
         this.method = method;
         this.clazz = clazz;

internal/src/main/java/me/koply/kcommando/internal/util/PackageReader.java

@@ -9,6 +9,8 @@
 
 public class PackageReader {
 
+    private PackageReader() {}
+
     // currently unavailable
     // experimental package reader method (to be developed)
     // inspired from MaeveS2/SnowballNebula

internal/src/main/java/me/koply/kcommando/internal/util/StringUtil.java

@@ -4,6 +4,8 @@
 
 public class StringUtil {
 
+    private StringUtil() {}
+
     /**
      * JaroWinklerDistance
      * Copied from https://commons.apache.org/sandbox/commons-text/jacoco/org.apache.commons.text.similarity/JaroWinklerDistance.java.html

javacord-integration/src/main/java/me/koply/kcommando/integration/impl/javacord/JavacordIntegration.java

@@ -30,6 +30,7 @@
 public class JavacordIntegration extends Integration {
 
     public final DiscordApi api;
+
     public JavacordIntegration(DiscordApi api) {
         super(api.getClientId());
         this.api = api;
@@ -56,7 +57,6 @@ public void registerSlashCommand(SlashBox box) {
 
         String name = info.name();
         String desc = info.desc();
-        boolean isglobal = info.global();
 
         Option[] options = info.options();
         List<SlashCommandOption> optionList = new ArrayList<>();
@@ -79,7 +79,7 @@ public void registerSlashCommand(SlashBox box) {
 
         box.getPerm().ifPresent(perm -> builder.setDefaultEnabledForPermissions(Util.getPermissions(perm.value())));
 
-        if (isglobal) {
+        if (info.global()) {
             builder.createGlobal(api).join();
             return;
         }

javacord-integration/src/main/java/me/koply/kcommando/integration/impl/javacord/Util.java

@@ -4,6 +4,8 @@
 
 public class Util {
 
+    private Util() {}
+
      public static PermissionType[] getPermissions(String[] values) {
         PermissionType[] ret = new PermissionType[values.length];
         PermissionType[] perms = PermissionType.values();

javacord-integration/src/main/java/me/koply/kcommando/integration/impl/javacord/listeners/CommandListener.java

@@ -40,7 +40,7 @@ public void onMessageCreate(MessageCreateEvent e) {
                if (result) cooldownList.put(authorID, ms);
             });
         } catch (Exception ex) {
-            Kogger.warn("An error occured while processing a MessageCreateEvent. Stacktrace:");
+            Kogger.warn("An error occurred while processing a MessageCreateEvent. Stacktrace:");
             ex.printStackTrace();
         }
     }

javacord-integration/src/main/java/me/koply/kcommando/integration/impl/javacord/listeners/SlashListener.java

@@ -18,7 +18,7 @@ public void onSlashCommandCreate(SlashCommandCreateEvent event) {
         try {
             executorService.submit(() -> handler.process(new SlashCommandHandler.Parameters(event, event.getSlashCommandInteraction().getCommandName())));
         } catch (Exception ex) {
-            Kogger.warn("An error occured while processing a SlashCommandCreateEvent. Stacktrace:");
+            Kogger.warn("An error occurred while processing a SlashCommandCreateEvent. Stacktrace:");
             ex.printStackTrace();
         }
     }

jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/JDAIntegration.java

@@ -52,30 +52,30 @@ public void registerSlashCommand(SlashBox box) {
 
         String name = info.name();
         String desc = info.desc();
-        boolean isglobal = info.global();
+        boolean isGlobal = info.global();
 
         Option[] options = info.options();
-        OptionData[] optionDatas = new OptionData[options.length];
-        int filledDatas = 0;
+        OptionData[] optionData = new OptionData[options.length];
+        int filledData = 0;
         for (Option option : options) {
-            if (options[filledDatas].type() == me.koply.kcommando.internal.OptionType.UNKNOWN) continue;
+            if (options[filledData].type() == me.koply.kcommando.internal.OptionType.UNKNOWN) continue;
             OptionType type = OptionType.fromKey(option.type().value);
-            optionDatas[filledDatas] = new OptionData(type, option.name(), option.desc(), option.required());
-            filledDatas++;
+            optionData[filledData] = new OptionData(type, option.name(), option.desc(), option.required());
+            filledData++;
         }
 
-        OptionData[] rolledOptionDatas = new OptionData[filledDatas];
-        System.arraycopy(optionDatas, 0, rolledOptionDatas, 0, filledDatas);
+        OptionData[] rolledOptionData = new OptionData[filledData];
+        System.arraycopy(optionData, 0, rolledOptionData, 0, filledData);
 
         boolean guildOnly = !info.enabledInDms();
 
         CommandData data = new CommandDataImpl(name, desc)
                 .setGuildOnly(guildOnly)
-                .addOptions(rolledOptionDatas);
+                .addOptions(rolledOptionData);
 
         box.getPerm().ifPresent(perm -> data.setDefaultPermissions(DefaultMemberPermissions.enabledFor(Util.getPermissions(perm.value()))));
 
-        if (isglobal) {
+        if (isGlobal) {
             api.upsertCommand(data).queue();
             return;
         }
@@ -89,7 +89,7 @@ public void registerSlashCommand(SlashBox box) {
             Guild guild = api.getGuildById(guildId);
             if (guild != null) {
                 guild.upsertCommand(data).queue();
-            } if (KCommando.verbose) {
+            } else if (KCommando.verbose) {
                 Kogger.warn("Guild not found for Slash Command named as " + name);
             }
         }

jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/Util.java

@@ -4,6 +4,8 @@
 
 public class Util {
 
+    private Util() {}
+
     public static Permission[] getPermissions(String[] values) {
         Permission[] ret = new Permission[values.length];
         Permission[] perms = Permission.values();

jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/listeners/CommandListener.java

@@ -38,7 +38,7 @@ public void onMessageReceived(@NotNull MessageReceivedEvent e) {
                 )) cooldownList.put(authorID, ms);
             });
         } catch (Exception ex) {
-            Kogger.warn("An error occured while processing a MessageReceivedEvent. Stacktrace:");
+            Kogger.warn("An error occurred while processing a MessageReceivedEvent. Stacktrace:");
             ex.printStackTrace();
         }
     }

jda-integration/src/main/java/me/koply/kcommando/integration/impl/jda/listeners/SlashListener.java

@@ -19,7 +19,7 @@ public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent even
         try {
             executorService.submit(() -> handler.process(new SlashCommandHandler.Parameters(event, event.getName())));
         } catch (Exception ex) {
-            Kogger.warn("An error occured while processing a SlashCommandEvent. Stacktrace:");
+            Kogger.warn("An error occurred while processing a SlashCommandEvent. Stacktrace:");
             ex.printStackTrace();
         }
     }

plugin/src/main/java/me/koply/kcommando/plugin/LightYML.java

@@ -7,25 +7,25 @@
 import java.util.Map;
 
 public class LightYML {
-    private boolean Ok = true;
+    private boolean ok = true;
 
     private final Map<String, String> attributes = new HashMap<>();
     public LightYML(final InputStream fileInputStream) {
         try {
-            final InputStreamReader isreader = new InputStreamReader(fileInputStream);
-            final BufferedReader reader = new BufferedReader(isreader);
+            final InputStreamReader isReader = new InputStreamReader(fileInputStream);
+            final BufferedReader reader = new BufferedReader(isReader);
 
             String line;
             while ((line = reader.readLine()) != null) {
                 String[] sided = line.split(":");
                 attributes.put(sided[0].trim(), sided[1].trim());
             }
 
-            isreader.close();
+            isReader.close();
             reader.close();
         } catch (Exception e) {
             e.printStackTrace();
-            Ok = false;
+            ok = false;
         }
     }
 
@@ -34,7 +34,7 @@ public final Map<String, String> getAttributes() {
     }
 
     public final boolean isOk() {
-        return Ok && attributes.containsKey("main");
+        return ok && attributes.containsKey("main");
     }
 
     /* Example plugin.yml

plugin/src/main/java/me/koply/kcommando/plugin/PluginManager.java

@@ -77,17 +77,17 @@ public void detectPlugins() {
     }
 
     public void enablePlugins() {
-        final ArrayList<PluginFile<E>> toremove = new ArrayList<>();
+        final ArrayList<PluginFile<E>> toRemove = new ArrayList<>();
         for (PluginFile<E> plugin : plugins) {
             final String pluginName = plugin.getYml().getAttributes().get("name");
 
             Class<?> firstSuperClass = plugin.getMainClass().getSuperclass(); // like JDAPlugin and JavacordPlugin
             Class<?> secondSuperClass = firstSuperClass.getSuperclass(); // for JavaPlugin from JDAPlugin
-            boolean isOkey = firstSuperClass == JavaPlugin.class || secondSuperClass == JavaPlugin.class;
+            boolean isOk = firstSuperClass == JavaPlugin.class || secondSuperClass == JavaPlugin.class;
 
-            if (!isOkey) {
+            if (!isOk) {
                 logger.warning(pluginName + " could not be enabled. Main class is not extends JavaPlugin.");
-                toremove.add(plugin);
+                toRemove.add(plugin);
                 continue;
             }
 
@@ -106,13 +106,13 @@ public void enablePlugins() {
                 plugin.setInstance(instance);
             } catch (Exception e) {
                 logger.warning(pluginName + " could not be enabled.");
-                toremove.add(plugin);
+                toRemove.add(plugin);
             }
         }
 
         PluginCargo.setDelivery(null);
 
-        for (PluginFile<E> b : toremove) {
+        for (PluginFile<E> b : toRemove) {
             plugins.remove(b);
         }
     }