Update instructions order when registering listeners

This vulnerability is very unlikely to ever happen, since the only way to modify the YAML file is to edit it yourself, and it would be useless for a bot owner to RCE their own bot. No other person can edit the configuration file remotely (eg. with bot commands), so realistically, this could not happen.

.drone.yml

@@ -1,14 +1,14 @@
-kind: template
+kind: pipeline
-load: java-build-deploy.yaml
+name: default
-data:
+
-  arch: arm64
+trigger:
-  os: linux
+  branch:
-  build_branches:
     - main
-    - develop
+
-  build_events:
+steps:
-    - push
+  - name: build
-    - pull_request
+    image: maven:3-eclipse-temurin-16
-  sonar_project_key: HidekoBot
+    commands:
-  deploy_targets:
+      - mvn install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
-    - production
+      - mvn javadoc:javadoc
+      - mvn test -B

.gitignore

@@ -1,4 +1,3 @@
 target/
 .idea/
-scripts/
 *.sqlite

.pipelines/java-build-deploy.yaml

@@ -1,88 +0,0 @@
----
-kind: pipeline
-type: docker
-name: build
-platform:
-  os: {{ .input.os }}
-  arch: {{ .input.arch }}
-
-trigger:
-  branch:
-    {{- range .input.build_branches }}
-    - {{ . }}
-    {{- end }}
-  event:
-    {{- range .input.build_events }}
-    - {{ . }}
-    {{- end }}
-
-# Global project-specific environment variables
-environment:
-  {{- range .input.envs }}
-    {{ .name }}: {{ .value }}
-    {{- end }}
-
-steps:
-  # Test if it compiles correctly
-  - name: build
-    image: maven:3-eclipse-temurin-21
-    commands:
-      - mvn verify --no-transfer-progress -DskipTests=true -Dmaven.javadoc.skip=true -B -V
-
-  # Run unit tests
-  - name: test
-    image: maven:3-eclipse-temurin-21
-    commands:
-      - mvn test --no-transfer-progress -B -V
-
-  # Check maven dependencies
-  - name: dependency-check
-    image: owasp/dependency-check:latest
-    commands:
-      - dependency-check --scan /src --format ALL --out /src/target --nvdApiKey $NVD_API_KEY
-    environment:
-      NVD_API_KEY:
-        from_secret: nvd_api_key
-
-  # Run code analysis
-  - name: code-analysis
-    when:
-      event:
-        - push
-    image: maven:3-eclipse-temurin-21
-    commands:
-      - mvn sonar:sonar --no-transfer-progress -Dsonar.projectKey={{ .input.sonar_project_key }} -Dsonar.host.url=$SONAR_INSTANCE_URL -Dsonar.token=$SONAR_LOGIN_KEY -B -V
-    environment:
-      SONAR_INSTANCE_URL:
-        from_secret: sonar_instance_url
-      SONAR_LOGIN_KEY:
-        from_secret: sonar_login_key
----
-kind: pipeline
-type: kubernetes
-name: deploy
-trigger:
-  event:
-    - promote
-  target:
-    {{- range .input.deploy_targets }}
-    - {{ . }}
-    {{- end }}
-
-# Global project-specific environment variables
-environment:
-  {{- range .input.envs }}
-    {{ .name }}: {{ .value }}
-    {{- end }}
-
-steps:
-  # Upload to Maven repository
-  - name: maven-deploy
-    image: maven:3-eclipse-temurin-21
-    commands:
-      - mvn deploy --no-transfer-progress -DskipTests=true -Dmaven.javadoc.skip=true -B -V -gs settings.xml -Dmaven.repo.username=$MAVEN_REPO_USERNAME -Dmaven.repo.password=$MAVEN_REPO_PASSWORD
-    environment:
-      MAVEN_REPO_USERNAME:
-        from_secret: maven_repo_username
-      MAVEN_REPO_PASSWORD:
-        from_secret: maven_repo_password

README.MD

@@ -1,16 +1,7 @@
 # HidekoBot  
-[![Reliability Rating](https://sonar.beatrice.wtf/api/project_badges/measure?project=HidekoBot&metric=reliability_rating&token=0a63c149148555d6d2ee40665af1afae8f67cc3f)](https://sonar.beatrice.wtf/dashboard?id=HidekoBot)
-[![Maintainability Rating](https://sonar.beatrice.wtf/api/project_badges/measure?project=HidekoBot&metric=sqale_rating&token=0a63c149148555d6d2ee40665af1afae8f67cc3f)](https://sonar.beatrice.wtf/dashboard?id=HidekoBot) 
-[![Security Rating](https://sonar.beatrice.wtf/api/project_badges/measure?project=HidekoBot&metric=security_rating&token=0a63c149148555d6d2ee40665af1afae8f67cc3f)](https://sonar.beatrice.wtf/dashboard?id=HidekoBot)
-[![Build Status](https://drone.beatrice.wtf/api/badges/bea/HidekoBot/status.svg)](https://drone.beatrice.wtf/bea/HidekoBot)
-[![Lines of Code](https://sonar.beatrice.wtf/api/project_badges/measure?project=HidekoBot&metric=ncloc&token=0a63c149148555d6d2ee40665af1afae8f67cc3f)](https://sonar.beatrice.wtf/dashboard?id=HidekoBot)
   
 Hideko is a general-purpose Discord bot.  
   
-## Download  
-The latest stable version is always uploaded automatically to the [Maven repository](https://nexus.beatrice.wtf/#browse/browse:maven-releases:wtf%2Fbeatrice%2Fhidekobot%2FHidekoBot).  
-You can download the JAR directly by clicking [here](https://nexus.beatrice.wtf/service/rest/v1/search/assets/download?sort=version&repository=maven-releases&maven.groupId=wtf.beatrice.hidekobot&maven.artifactId=HidekoBot&maven.extension=jar).    
-  
 ## Startup
 Download a prebuilt JAR file or build it from source, then run it with:
 ```bash
@@ -23,11 +14,11 @@ Additionally available parameters are:
   - **verbose**: log every message that the bot receives, plus additional debugging messages. Very spammy and performance heavy.
   - **refresh**: force refresh the slash commands. This is useful in case there was a simple update to a command that did not drastically change it, so no changes are found at bootup (eg: fixing a typo in the command description).
 
-*Note: Java 21 or later is required.*
+*Note: Java 16 or later is required.*
   
 ## Initial setup  
   
-Run the startup command once. The bot will generate a `config.yml` file in your current directory (`$PWD` on GNU/Linux).  
+Run the startup command once. The bot will generate a `config.yml` file in the directory you were when you ran it.  
   
 Edit the configuration file and set all values according to your needs.  
   
@@ -37,12 +28,5 @@ already set-up. The bot supports both slash commands and message commands, with
 commands support both systems, but some of them are limited in one way or another.  
   
 The bot currently supports SQLite as a database backend. A database file will be created after the first boot
-in your current directory. Do not delete the database file to avoid corruption and unpredictable
+in the same directory that you ran it. Do not delete the database file to avoid corruption and unpredictable
 behavior.
-
-# Development
-## Versioning
-This project uses the `x.y.z-releaseType` schema for releases.  
-Development builds are tagged as `x.y.z-SNAPSHOT` and sometimes pushed to the snapshots Maven repository.  
-Stable builds are tagged as `x.y.z` and always pushed to the releases Maven repository, by promoting the build on
-[Drone](https://drone.beatrice.wtf/). Currently, promoting stable builds is a manual process.

pom.xml

@@ -6,107 +6,58 @@
 
     <groupId>wtf.beatrice.hidekobot</groupId>
     <artifactId>HidekoBot</artifactId>
-    <version>0.6.3-SNAPSHOT</version>
+    <version>0.5.16</version>
 
     <properties>
-        <maven.compiler.source>21</maven.compiler.source>
+        <maven.compiler.source>16</maven.compiler.source>
-        <maven.compiler.target>21</maven.compiler.target>
+        <maven.compiler.target>16</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-        <sonar.dependencyCheck.htmlReportPath>./target/dependency-check-report.html</sonar.dependencyCheck.htmlReportPath>
-        <sonar.dependencyCheck.jsonReportPath>./target/dependency-check-report.json</sonar.dependencyCheck.jsonReportPath>
-        <sonar.dependencyCheck.summarize>true</sonar.dependencyCheck.summarize>
     </properties>
 
     <dependencies>
-        <!-- Basic JDA dependency for Discord API -->
         <dependency>
             <groupId>net.dv8tion</groupId>
             <artifactId>JDA</artifactId>
-            <version>5.6.1</version>
+            <version>5.0.0-beta.2</version>
         </dependency>
-        <!-- JDA depends on SLF4J for logging -->
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
-            <version>2.0.17</version>
+            <version>2.0.4</version>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-simple</artifactId>
-            <version>2.0.17</version>
+            <version>2.0.4</version>
         </dependency>
-
-        <!-- Dependency used for SQLite database connections-->
         <dependency>
             <groupId>org.xerial</groupId>
             <artifactId>sqlite-jdbc</artifactId>
-            <version>3.51.0.0</version>
+            <version>3.39.4.1</version>
         </dependency>
-
-        <!-- Dependency used for YAML configuration files -->
         <dependency>
             <groupId>org.yaml</groupId>
             <artifactId>snakeyaml</artifactId>
-            <version>2.5</version>
+            <version>1.33</version>
         </dependency>
-
-        <!-- JSoup is used to parse HTML into JSON objects for better handling in Java -->
         <dependency>
             <groupId>org.jsoup</groupId>
             <artifactId>jsoup</artifactId>
-            <version>1.21.2</version>
+            <version>1.15.3</version>
         </dependency>
-        <!-- Various String manipulation utils -->
         <dependency>
             <groupId>org.apache.commons</groupId>
             <artifactId>commons-text</artifactId>
-            <version>1.14.0</version>
+            <version>1.10.0</version>
         </dependency>
-        <!-- JSON dependency used for better parsing of JSON files -->
         <dependency>
             <groupId>org.json</groupId>
             <artifactId>json</artifactId>
-            <version>20250517</version>
+            <version>20220924</version>
         </dependency>
 
-        <!-- Start Random.org dependencies -->
-        <dependency>
-            <groupId>com.github.jinahya</groupId>
-            <artifactId>random-org-json-rpc</artifactId>
-            <version>0.0.1</version>
-        </dependency>
-        <dependency>
-            <groupId>com.google.code.gson</groupId>
-            <artifactId>gson</artifactId>
-            <version>2.13.2</version>
-        </dependency>
-        <dependency>
-            <groupId>commons-codec</groupId>
-            <artifactId>commons-codec</artifactId>
-            <version>1.20.0</version>
-        </dependency>
-        <!-- End Random.org dependencies -->
-
-        <!-- Unit Tests Dependencies -->
-        <dependency>
-            <groupId>org.junit.jupiter</groupId>
-            <artifactId>junit-jupiter-api</artifactId>
-            <version>6.0.1</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
-    <!-- override dependencies to use newer versions -->
-    <dependencyManagement>
-        <dependencies>
-            <dependency>
-                <groupId>com.google.protobuf</groupId>
-                <artifactId>protobuf-java</artifactId>
-                <version>4.33.0</version>
-            </dependency>
-        </dependencies>
-    </dependencyManagement>
-
 
     <build>
         <resources>
@@ -143,46 +94,10 @@
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-javadoc-plugin</artifactId>
-                <version>3.12.0</version>
+                <version>3.4.1</version>
-            </plugin>
-
-            <plugin>
-                <groupId>org.sonarsource.scanner.maven</groupId>
-                <artifactId>sonar-maven-plugin</artifactId>
-                <version>5.3.0.6276</version>
-            </plugin>
-
-            <plugin>
-                <groupId>org.owasp</groupId>
-                <artifactId>dependency-check-maven</artifactId>
-                <version>12.1.8</version>
-                <configuration>
-                    <failBuildOnCVSS>8</failBuildOnCVSS>
-                    <!--suppress UnresolvedMavenProperty -->
-                    <nvdApiKey>${nvdApiKey}</nvdApiKey>
-                    <knownExploitedUrl>https://raw.githubusercontent.com/EugenMayer/cisa-known-exploited-mirror/main/known_exploited_vulnerabilities.json</knownExploitedUrl>
-                    <formats>
-                        <format>html</format>
-                        <format>json</format>
-                    </formats>
-                    <suppressionFiles>
-                        <suppressionFile>./suppressions.xml</suppressionFile>
-                    </suppressionFiles>
-                </configuration>
             </plugin>
 
         </plugins>
     </build>
 
-    <distributionManagement>
-        <repository>
-            <id>nexus-releases</id>
-            <url>https://nexus.beatrice.wtf/repository/maven-releases/</url>
-        </repository>
-        <snapshotRepository>
-            <id>nexus-snapshots</id>
-            <url>https://nexus.beatrice.wtf/repository/maven-snapshots/</url>
-        </snapshotRepository>
-    </distributionManagement>
-
 </project>

renovate.json

@@ -1,3 +0,0 @@
-{
-  "$schema": "https://docs.renovatebot.com/renovate-schema.json"
-}

settings.xml

@@ -1,20 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<settings xmlns="http://maven.apache.org/SETTINGS/1.1.0"
-          xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-          xsi:schemaLocation="http://maven.apache.org/SETTINGS/1.1.0 http://maven.apache.org/xsd/settings-1.1.0.xsd">
-
-    <servers>
-        <server>
-            <id>nexus-snapshots</id>
-            <username>${maven.repo.username}</username>
-            <password>${maven.repo.password}</password>
-        </server>
-        <server>
-            <id>nexus-releases</id>
-            <username>${maven.repo.username}</username>
-            <password>${maven.repo.password}</password>
-        </server>
-    </servers>
-
-</settings>
-

src/main/java/wtf/beatrice/hidekobot/Cache.java

@@ -1,8 +1,6 @@
 package wtf.beatrice.hidekobot;
 
 import org.jetbrains.annotations.Nullable;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.datasources.ConfigurationEntry;
 import wtf.beatrice.hidekobot.datasources.ConfigurationSource;
 import wtf.beatrice.hidekobot.datasources.DatabaseSource;
@@ -11,26 +9,27 @@ import wtf.beatrice.hidekobot.listeners.MessageCommandListener;
 import wtf.beatrice.hidekobot.listeners.MessageLogger;
 import wtf.beatrice.hidekobot.listeners.SlashCommandCompletionListener;
 import wtf.beatrice.hidekobot.listeners.SlashCommandListener;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.awt.*;
 import java.lang.reflect.Field;
 import java.time.LocalDateTime;
 import java.util.HashMap;
+import java.util.Random;
 import java.util.concurrent.Executors;
 import java.util.concurrent.ScheduledExecutorService;
 
 public class Cache
 {
 
-    private Cache()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
 
     // todo: make this compatible with the message listener's regex
-    private static final String BOT_PREFIX = "hideko";
+    private static final String botPrefix = "hideko";
-    private static final Logger LOGGER = LoggerFactory.getLogger(Cache.class);
+    private static final Logger logger = new Logger(Cache.class);
+
+    // the Random instance that we should always use when looking for an RNG based thing.
+    // the seed is updated periodically.
+    private static final Random randomInstance = new Random();
 
     // map to store results of "love calculator", to avoid people re-running the same command until
     // they get what they wanted.
@@ -43,11 +42,11 @@ public class Cache
     private static DatabaseSource databaseSource = null;
     private static boolean verbose = false;
     private static MessageLogger verbosityLogger = null;
-    private static final long BOT_MAINTAINER_ID = 979809420714332260L;
+    private static final long botMaintainerId = 979809420714332260L;
-    private static final String EXPIRY_TIMESTAMP_FORMAT = "yy/MM/dd HH:mm:ss";
+    private final static String expiryTimestampFormat = "yy/MM/dd HH:mm:ss";
 
     // note: discord sets interactions' expiry time to 15 minutes by default, so we can't go higher than that.
-    private static final long EXPIRY_TIME_SECONDS = 30L;
+    private final static long expiryTimeSeconds = 30L;
 
     // used to count e.g. uptime
     private static LocalDateTime startupTime = null;
@@ -56,22 +55,22 @@ public class Cache
     private static final LocalDateTime botBirthDate = LocalDateTime.of(2022, 8, 25, 21, 50);
 
     // the scheduler that should always be used when running a scheduled task.
-    private static final ScheduledExecutorService taskScheduler = Executors.newSingleThreadScheduledExecutor(); // todo: try-with-resources
+    private final static ScheduledExecutorService taskScheduler = Executors.newSingleThreadScheduledExecutor(); // todo: try-with-resources
 
-    private static final String EXEC_PATH = System.getProperty("user.dir");
+    private final static String execPath = System.getProperty("user.dir");
-    private static final String BOT_NAME = "Hideko";
+    private static final String botName = "Hideko";
 
     private static SlashCommandListener slashCommandListener = null;
     private static SlashCommandCompletionListener slashCommandCompletionListener = null;
     private static MessageCommandListener messageCommandListener = null;
 
-    private static final String DEFAULT_INVITE_LINK =
+    private final static String defaultInviteLink =
             "https://discord.com/api/oauth2/authorize?client_id=%userid%&scope=bot+applications.commands&permissions=8";
 
     private static String botApplicationId = "";
 
     // discord api returns a broken image if you don't use specific sizes (powers of 2), so we limit it to these
-    private static final int[] supportedAvatarResolutions = {16, 32, 64, 128, 256, 512, 1024};
+    private static final int[] supportedAvatarResolutions = { 16, 32, 64, 128, 256, 512, 1024 };
 
     /**
      * Get an array of all the Discord-supported avatar resolutions.
@@ -79,21 +78,22 @@ public class Cache
      *
      * @return array of supported resolutions.
      */
-    public static int[] getSupportedAvatarResolutions()
+    public static int[] getSupportedAvatarResolutions() { return supportedAvatarResolutions; }
-    {
+
-        return supportedAvatarResolutions;
+    public static Random getRandom() {
+        return randomInstance;
     }
 
+    public static void setRandomSeed(long seed) {
+        randomInstance.setSeed(seed);
+    }
 
     /**
      * Checks if the bot has been started with the verbose argument.
      *
      * @return a boolean which is true if the bot is in verbose-mode
      */
-    public static synchronized boolean isVerbose()
+    public static boolean isVerbose() { return verbose; }
-    {
-        return verbose;
-    }
 
     /**
      * Set the bot's verbosity status at runtime.
@@ -101,20 +101,24 @@ public class Cache
      *
      * @param v the verbosity boolean value
      */
-    public static synchronized void setVerbose(boolean v)
+    public static void setVerbose(boolean v)
     {
         verbose = v;
 
-        if (verbosityLogger != null)
+        if(v)
+        {
+            if(verbosityLogger == null)
+            {
+                verbosityLogger = new MessageLogger();
+            }
+
+            HidekoBot.getAPI().addEventListener(verbosityLogger);
+        } else {
+            if(verbosityLogger != null)
             {
                 HidekoBot.getAPI().removeEventListener(verbosityLogger);
                 verbosityLogger = null;
             }
-
-        if (v)
-        {
-            verbosityLogger = new MessageLogger();
-            HidekoBot.getAPI().addEventListener(verbosityLogger);
         }
     }
 
@@ -123,8 +127,7 @@ public class Cache
      *
      * @return a long of the account's id
      */
-    public static long getBotOwnerId()
+    public static long getBotOwnerId() {
-    {
         return configurationSource == null ? 0L : (Long) configurationSource.getConfigValue(ConfigurationEntry.BOT_OWNER_ID);
     }
 
@@ -134,22 +137,17 @@ public class Cache
      *
      * @return a String of the bot's token.
      */
-    public static String getBotToken()
+    public static String getBotToken() {
-    {
         return configurationSource == null ? null : (String) configurationSource.getConfigValue(ConfigurationEntry.BOT_TOKEN);
     }
 
-
     /**
      * Get the bot maintainer's profile id.
      *
      * @return a long of the account's id
      */
     
-    public static long getBotMaintainerId()
+    public static long getBotMaintainerId() { return botMaintainerId; }
-    {
-        return BOT_MAINTAINER_ID;
-    }
 
     /**
      * Set the bot's application id.
@@ -166,19 +164,15 @@ public class Cache
      *
      * @return a string of the bot's application id
      */
-    public static String getBotApplicationId()
+    public static String getBotApplicationId() { return botApplicationId; }
-    {
-        return botApplicationId;
-    }
 
     /**
      * Function to generate an invite link for the bot
      *
      * @return a string containing the invite link
      */
-    public static String getInviteUrl()
+    public static String getInviteUrl() {
-    {
+        return defaultInviteLink.replace("%userid%", botApplicationId);
-        return DEFAULT_INVITE_LINK.replace("%userid%", botApplicationId);
     }
 
     /**
@@ -196,10 +190,7 @@ public class Cache
      *
      * @return the DatabaseSource instance.
      */
-    public static @Nullable DatabaseSource getDatabaseSource()
+    public static @Nullable DatabaseSource getDatabaseSource() { return databaseSource; }
-    {
-        return databaseSource;
-    }
 
     /**
      * Set the properties source instance loaded from the JAR archive.
@@ -216,67 +207,43 @@ public class Cache
      *
      * @return the String of the DateTimeFormatter format.
      */
-    public static String getExpiryTimestampFormat()
+    public static String getExpiryTimestampFormat(){ return  expiryTimestampFormat; }
-    {
-        return EXPIRY_TIMESTAMP_FORMAT;
-    }
 
     /**
      * Get the amount of seconds after which a message expires.
      *
      * @return long value of the expiry seconds.
      */
-    public static long getExpiryTimeSeconds()
+    public static long getExpiryTimeSeconds() { return expiryTimeSeconds; }
-    {
-        return EXPIRY_TIME_SECONDS;
-    }
 
 
-    public static String getBotName()
+    public static String getBotName() { return botName; };
-    {
-        return BOT_NAME;
-    }
 
     /**
      * Get the bot's version.
      *
      * @return a String of the bot version.
      */
-    public static String getBotVersion()
+    public static String getBotVersion() {
-    {
         return propertiesSource.getProperty("bot.version");
     }
 
-    /**
-     * Get the bot's source code URL.
-     *
-     * @return a String containing the base URL of the repository, including a <b>trailing slash</b>.
-     */
-    public static String getRepositoryUrl()
-    {
-        String url = propertiesSource.getProperty("repo.base_url");
-        return url.endsWith("/") ? url : url + "/";
-    }
-
     /**
      * Get the bot's global color.
      *
      * @return the Color object.
      */
-    public static Color getBotColor()
+    public static Color getBotColor() {
-    {
         Color defaultColor = Color.PINK;
-        if (configurationSource == null) return defaultColor;
+        if(configurationSource == null) return defaultColor;
         String colorName = (String) configurationSource.getConfigValue(ConfigurationEntry.BOT_COLOR);
 
         Color color = null;
-        try
+        try {
-        {
             Field field = Color.class.getField(colorName);
-            color = (Color) field.get(null);
+            color = (Color)field.get(null);
-        } catch (RuntimeException | NoSuchFieldException | IllegalAccessException e)
+        } catch (RuntimeException | NoSuchFieldException | IllegalAccessException e) {
-        {
+            logger.log("Unknown color: " + colorName);
-            LOGGER.error("Unknown color: {}", colorName);
         }
         return color == null ? defaultColor : color;
     }
@@ -284,36 +251,21 @@ public class Cache
 
     //todo javadocs
     public static void setSlashCommandListener(SlashCommandListener commandListener)
-    {
+    { slashCommandListener = commandListener; }
-        slashCommandListener = commandListener;
-    }
 
-    public static SlashCommandListener getSlashCommandListener()
+    public static SlashCommandListener getSlashCommandListener() { return slashCommandListener; }
-    {
-        return slashCommandListener;
-    }
 
 
     public static void setSlashCommandCompletionListener(SlashCommandCompletionListener commandCompletionListener)
-    {
+    { slashCommandCompletionListener = commandCompletionListener; }
-        slashCommandCompletionListener = commandCompletionListener;
-    }
 
-    public static SlashCommandCompletionListener getSlashCommandCompletionListener()
+    public static SlashCommandCompletionListener getSlashCommandCompletionListener() { return slashCommandCompletionListener; }
-    {
-        return slashCommandCompletionListener;
-    }
 
 
     public static void setMessageCommandListener(MessageCommandListener commandListener)
-    {
+    { messageCommandListener = commandListener; }
-        messageCommandListener = commandListener;
-    }
 
-    public static MessageCommandListener getMessageCommandListener()
+    public static MessageCommandListener getMessageCommandListener() { return messageCommandListener; }
-    {
-        return messageCommandListener;
-    }
 
     /**
      * Set the bot's startup time. Generally only used at boot time.
@@ -321,9 +273,7 @@ public class Cache
      * @param time a LocalDateTime of the startup moment.
      */
     public static void setStartupTime(LocalDateTime time)
-    {
+    { startupTime = time; }
-        startupTime = time;
-    }
 
 
     /**
@@ -331,54 +281,33 @@ public class Cache
      *
      * @return a LocalDateTime object of the startup instant.
      */
-    public static LocalDateTime getStartupTime()
+    public static LocalDateTime getStartupTime() { return startupTime; }
-    {
-        return startupTime;
-    }
 
     /**
      * Get the time of when the bot was created.
      *
      * @return a LocalDateTime object of the first commit's instant.
      */
-    public static LocalDateTime getBotBirthDate()
+    public static LocalDateTime getBotBirthDate() { return botBirthDate; }
-    {
-        return botBirthDate;
-    }
 
-    public static String getFullHeartBeatLink()
+    public static String getFullHeartBeatLink() {
-    {
         return configurationSource == null ? null : (String) configurationSource.getConfigValue(ConfigurationEntry.HEARTBEAT_LINK);
     }
-
     //todo javadocs
-    public static String getExecPath()
+    public static String getExecPath() { return execPath; }
-    {
-        return EXEC_PATH;
-    }
 
     /*private static ConfigurationSource getConfigurationSource()
     { return configurationSource; }*/
 
-    public static String getRandomOrgApiKey()
-    {
-        return configurationSource == null ? null : (String) configurationSource.getConfigValue(ConfigurationEntry.RANDOM_ORG_API_KEY);
-    }
-
     public static void setConfigurationSource(ConfigurationSource configurationSource)
-    {
+    { Cache.configurationSource = configurationSource; }
-        Cache.configurationSource = configurationSource;
-    }
 
     /**
      * Get the bot's prefix
      *
      * @return a String of the bot's prefix.
      */
-    public static String getBotPrefix()
+    public static String getBotPrefix() { return botPrefix; }
-    {
-        return BOT_PREFIX;
-    }
 
     public static void cacheLoveCalculatorValue(String userId1, String userId2, int value)
     {
@@ -393,7 +322,7 @@ public class Cache
         String merged2 = userId2 + "|" + userId1;
         Integer value = null;
         value = loveCalculatorValues.get(merged1);
-        if (value == null) value = loveCalculatorValues.get(merged2);
+        if(value == null) value = loveCalculatorValues.get(merged2);
         return value;
     }
 
@@ -403,8 +332,7 @@ public class Cache
         loveCalculatorValues.remove(userId2 + "|" + userId1);
     }
 
-    public static ScheduledExecutorService getTaskScheduler()
+    public static ScheduledExecutorService getTaskScheduler() {
-    {
         return taskScheduler;
     }
 

src/main/java/wtf/beatrice/hidekobot/HidekoBot.java

@@ -4,8 +4,7 @@ import net.dv8tion.jda.api.JDA;
 import net.dv8tion.jda.api.JDABuilder;
 import net.dv8tion.jda.api.OnlineStatus;
 import net.dv8tion.jda.api.requests.GatewayIntent;
-import org.slf4j.Logger;
+import sun.misc.Signal;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.commands.completer.ProfileImageCommandCompleter;
 import wtf.beatrice.hidekobot.commands.message.HelloCommand;
 import wtf.beatrice.hidekobot.commands.slash.*;
@@ -15,11 +14,10 @@ import wtf.beatrice.hidekobot.datasources.PropertiesSource;
 import wtf.beatrice.hidekobot.listeners.*;
 import wtf.beatrice.hidekobot.runnables.ExpiredMessageTask;
 import wtf.beatrice.hidekobot.runnables.HeartBeatTask;
-import wtf.beatrice.hidekobot.runnables.RandomOrgSeedTask;
+import wtf.beatrice.hidekobot.runnables.RandomSeedTask;
 import wtf.beatrice.hidekobot.runnables.StatusUpdateTask;
 import wtf.beatrice.hidekobot.util.CommandUtil;
-import wtf.beatrice.hidekobot.util.FormatUtil;
+import wtf.beatrice.hidekobot.util.Logger;
-import wtf.beatrice.hidekobot.util.RandomUtil;
 
 import java.io.File;
 import java.time.LocalDateTime;
@@ -34,31 +32,31 @@ public class HidekoBot
 {
     private static JDA jda;
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(HidekoBot.class);
+    private static final Logger logger = new Logger(HidekoBot.class);
 
     public static void main(String[] args)
     {
 
         // load configuration
-        LOGGER.info("Loading configuration...");
+        logger.log("Loading configuration...");
         String configFilePath = Cache.getExecPath() + File.separator + "config.yml";
         ConfigurationSource configurationSource = new ConfigurationSource(configFilePath);
         configurationSource.initConfig();
         Cache.setConfigurationSource(configurationSource);
-        LOGGER.info("Configuration loaded!");
+        logger.log("Configuration loaded!");
 
         // load properties
-        LOGGER.info("Loading properties...");
+        logger.log("Loading properties...");
         PropertiesSource propertiesSource = new PropertiesSource();
         propertiesSource.load();
         Cache.setPropertiesSourceInstance(propertiesSource);
-        LOGGER.info("Properties loaded!");
+        logger.log("Properties loaded!");
 
         // check loaded bot token
         String botToken = Cache.getBotToken();
-        if (botToken == null || botToken.isEmpty())
+        if(botToken == null || botToken.isEmpty())
         {
-            LOGGER.error("Invalid bot token!");
+            logger.log("Invalid bot token!");
             shutdown();
             return;
         }
@@ -76,14 +74,9 @@ public class HidekoBot
             );
 
             jda = jdaBuilder.build().awaitReady();
-        } catch (InterruptedException e)
-        {
-            LOGGER.error(e.getMessage()); // print the error message, omit the stack trace.
-            Thread.currentThread().interrupt(); // send interrupt to the thread.
-            shutdown(); // if we failed connecting and authenticating, then quit.
         } catch (Exception e)
         {
-            LOGGER.error(e.getMessage()); // print the error message, omit the stack trace.
+            logger.log(e.getMessage()); // print the error message, omit the stack trace.
             shutdown(); // if we failed connecting and authenticating, then quit.
         }
 
@@ -94,39 +87,25 @@ public class HidekoBot
         // store if we have to force refresh commands despite no apparent changes.
         boolean forceUpdateCommands = false;
 
-        // if there is at least one arg, then iterate through them because we have additional things to do.
+        // if there is more than 1 arg, then iterate through them because we have additional things to do.
         // we are doing this at the end because we might need the API to be already initialized for some things.
-        if (args.length > 0)
+        if(args.length > 1) {
-        {
             List<String> argsList = new ArrayList<>(Arrays.asList(args));
 
 
             // NOTE: do not replace with enhanced for, since we might need
             // to know what position we're at or do further elaboration of the string.
             // we were using this for api key parsing in the past.
-            for (int i = 0; i < argsList.size(); i++)
+            for(int i = 0; i < argsList.size(); i++)
             {
                 String arg = argsList.get(i);
 
-                if (arg.equals("verbose")) Cache.setVerbose(true);
+                if(arg.equals("verbose")) Cache.setVerbose(true);
-                if (arg.equals("refresh")) forceUpdateCommands = true;
+                if(arg.equals("refresh")) forceUpdateCommands = true;
             }
 
         }
 
-
-        boolean enableRandomSeedUpdaterTask = false;
-        // initialize random.org object if API key is provided
-        {
-            if (RandomUtil.isRandomOrgKeyValid())
-            {
-                LOGGER.info("Enabling Random.org integration... This might take a while!");
-                RandomUtil.initRandomOrg();
-                enableRandomSeedUpdaterTask = true;
-                LOGGER.info("Random.org integration enabled!");
-            }
-        }
-
         // register slash commands and completers
         SlashCommandListener slashCommandListener = new SlashCommandListener();
         SlashCommandCompletionListener slashCommandCompletionListener = new SlashCommandCompletionListener();
@@ -188,62 +167,54 @@ public class HidekoBot
 
         // update slash commands (delayed)
         final boolean finalForceUpdateCommands = forceUpdateCommands;
-        try (ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor())
+        Executors.newSingleThreadScheduledExecutor().schedule(() -> // todo: try-with-resources
-        {
+                CommandUtil.updateSlashCommands(finalForceUpdateCommands), 1, TimeUnit.SECONDS);
-            executor.schedule(() -> CommandUtil.updateSlashCommands(finalForceUpdateCommands),
-                    1, TimeUnit.SECONDS);
-        }
 
         // set the bot's status
         jda.getPresence().setStatus(OnlineStatus.ONLINE);
 
         // connect to database
-        LOGGER.info("Connecting to database...");
+        logger.log("Connecting to database...");
         String dbFilePath = Cache.getExecPath() + File.separator + "db.sqlite"; // in current directory
         DatabaseSource databaseSource = new DatabaseSource(dbFilePath);
-        if (databaseSource.connect() && databaseSource.initDb())
+        if(databaseSource.connect() && databaseSource.initDb())
         {
-            LOGGER.info("Database connection initialized!");
+            logger.log("Database connection initialized!");
             Cache.setDatabaseSourceInstance(databaseSource);
 
             // load data here...
 
-            LOGGER.info("Database data loaded into memory!");
+            logger.log("Database data loaded into memory!");
-        } else
+        } else {
-        {
+            logger.log("Error initializing database connection!");
-            LOGGER.error("Error initializing database connection!");
         }
 
         // start scheduled runnables
         ScheduledExecutorService scheduler = Cache.getTaskScheduler();
         ExpiredMessageTask expiredMessageTask = new ExpiredMessageTask();
-        scheduler.scheduleAtFixedRate(expiredMessageTask, 5L, 5L, TimeUnit.SECONDS); //every 5 seconds
+        scheduler.scheduleAtFixedRate(expiredMessageTask, 5, 5, TimeUnit.SECONDS); //every 5 seconds
         HeartBeatTask heartBeatTask = new HeartBeatTask();
-        scheduler.scheduleAtFixedRate(heartBeatTask, 10L, 30L, TimeUnit.SECONDS); //every 30 seconds
+        scheduler.scheduleAtFixedRate(heartBeatTask, 10, 30, TimeUnit.SECONDS); //every 30 seconds
         StatusUpdateTask statusUpdateTask = new StatusUpdateTask();
-        scheduler.scheduleAtFixedRate(statusUpdateTask, 0L, 60L * 5L, TimeUnit.SECONDS); // every 5 minutes
+        scheduler.scheduleAtFixedRate(statusUpdateTask, 0, 60 * 5, TimeUnit.SECONDS); // every 5 minutes
-        if (enableRandomSeedUpdaterTask)
+        RandomSeedTask randomSeedTask = new RandomSeedTask();
-        {
+        scheduler.scheduleAtFixedRate(randomSeedTask, 0, 60, TimeUnit.SECONDS); // every minute
-            RandomOrgSeedTask randomSeedTask = new RandomOrgSeedTask();
-            scheduler.scheduleAtFixedRate(randomSeedTask, 15L, 15L, TimeUnit.MINUTES); // every 15 minutes
-        }
 
         // register shutdown interrupt signal listener for proper shutdown.
-        Runtime.getRuntime().addShutdownHook(new Thread(HidekoBot::preShutdown));
+        Signal.handle(new Signal("INT"), signal -> shutdown());
 
         // set startup time.
         Cache.setStartupTime(LocalDateTime.now());
 
         // print the bot logo.
-        LOGGER.info("\n\n{}\nv{} - bot is ready!\n", FormatUtil.getLogo(), Cache.getBotVersion());
+        logger.log("\n\n" + logger.getLogo() + "\nv" + Cache.getBotVersion() + " - bot is ready!\n", 2);
 
 
         // log the invite-link to console so noob users can just click on it.
-        LOGGER.info("Bot User ID: {}", botUserId);
+        logger.log("Bot User ID: " + botUserId, 3);
-        LOGGER.info("Invite Link: {}", Cache.getInviteUrl());
+        logger.log("Invite Link: " + Cache.getInviteUrl(), 4);
 
     }
-
     public static JDA getAPI()
     {
         return jda;
@@ -251,14 +222,9 @@ public class HidekoBot
 
     public static void shutdown()
     {
-        preShutdown();
+        logger.log("WARNING! Shutting down!");
+        if(jda != null) jda.shutdown();
         System.exit(0);
     }
 
-    private static void preShutdown()
-    {
-        LOGGER.warn("WARNING! Shutting down!");
-        if (jda != null) jda.shutdown();
-    }
-
 }

src/main/java/wtf/beatrice/hidekobot/commands/base/Alias.java

@@ -6,20 +6,15 @@ import java.util.LinkedList;
 
 public class Alias
 {
-    private Alias()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static String generateNiceAliases(MessageCommand command)
     {
         LinkedList<String> aliases = command.getCommandLabels();
         StringBuilder aliasesStringBuilder = new StringBuilder();
-        for (int i = 0; i < aliases.size(); i++)
+        for(int i = 0; i < aliases.size(); i++)
         {
             aliasesStringBuilder.append("`").append(aliases.get(i)).append("`");
 
-            if (i + 1 != aliases.size())
+            if(i + 1 != aliases.size())
                 aliasesStringBuilder.append(", "); // separate with comma except on last iteration
         }
 

src/main/java/wtf/beatrice/hidekobot/commands/base/BotInfo.java

@@ -5,7 +5,6 @@ import net.dv8tion.jda.api.entities.MessageEmbed;
 import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.util.FormatUtil;
-import wtf.beatrice.hidekobot.util.RandomUtil;
 
 import java.lang.management.ManagementFactory;
 import java.text.DecimalFormat;
@@ -13,11 +12,6 @@ import java.util.List;
 
 public class BotInfo
 {
-    private BotInfo()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static MessageEmbed generateEmbed(List<String> commandLabels)
     {
         EmbedBuilder embedBuilder = new EmbedBuilder();
@@ -26,7 +20,7 @@ public class BotInfo
 
         // thumbnail
         String botAvatarUrl = HidekoBot.getAPI().getSelfUser().getAvatarUrl();
-        if (botAvatarUrl != null) embedBuilder.setThumbnail(botAvatarUrl);
+        if(botAvatarUrl != null) embedBuilder.setThumbnail(botAvatarUrl);
 
         // help field
         long ownerId = Cache.getBotOwnerId();
@@ -38,12 +32,12 @@ public class BotInfo
 
         // type-specific commands list field
         StringBuilder commandsListBuilder = new StringBuilder();
-        commandsListBuilder.append(commandLabels.size()).append(" total - ");
+        commandsListBuilder.append(commandLabels.size()).append( " total - ");
-        for (int i = 0; i < commandLabels.size(); i++)
+        for(int i = 0; i < commandLabels.size(); i++)
         {
             commandsListBuilder.append("`").append(commandLabels.get(i)).append("`");
 
-            if (i + 1 != commandLabels.size()) // don't add comma in last iteration
+            if(i + 1 != commandLabels.size()) // don't add comma in last iteration
             {
                 commandsListBuilder.append(", ");
             }
@@ -54,38 +48,25 @@ public class BotInfo
         // keep track of how many total commands we have
         int commandsCount = 0;
 
-        // message commands info field
+        // message commands info fields
-        String messageCommandsInfo;
+        StringBuilder messageCommandsInfoBuilder = new StringBuilder();
-        if (Cache.getMessageCommandListener() == null)
+        if(Cache.getMessageCommandListener() == null)
-            messageCommandsInfo = "❌ disabled";
+            messageCommandsInfoBuilder.append("❌ disabled");
-        else
+        else {
-        {
+            messageCommandsInfoBuilder.append("✅ available");
-            messageCommandsInfo = "✅ available";
             commandsCount += Cache.getMessageCommandListener().getRegisteredCommands().size();
         }
-        embedBuilder.addField("Message commands", messageCommandsInfo, true);
+        embedBuilder.addField("Message commands", messageCommandsInfoBuilder.toString(), true);
 
-        // slash commands info field
+        // slash commands info fields
-        String slashCommandsInfo;
+        StringBuilder slashCommandsInfoBuilder = new StringBuilder();
-        if (Cache.getMessageCommandListener() == null)
+        if(Cache.getMessageCommandListener() == null)
-            slashCommandsInfo = "❌ disabled";
+            slashCommandsInfoBuilder.append("❌ disabled");
-        else
+        else {
-        {
+            slashCommandsInfoBuilder.append("✅ available");
-            slashCommandsInfo = "✅ available";
             commandsCount += Cache.getSlashCommandListener().getRegisteredCommands().size();
         }
-        embedBuilder.addField("Slash commands", slashCommandsInfo, true);
+        embedBuilder.addField("Slash commands", slashCommandsInfoBuilder.toString(), true);
-
-        // random.org integration field
-        String randomOrgInfo;
-        if (RandomUtil.isRandomOrgKeyValid())
-        {
-            randomOrgInfo = "✅ connected";
-        } else
-        {
-            randomOrgInfo = "❌ disabled";
-        }
-        embedBuilder.addField("Random.org", randomOrgInfo, true);
 
         // commands count fields
         embedBuilder.addField("Total commands", "Loaded: `" + commandsCount + "`", true);
@@ -113,10 +94,9 @@ public class BotInfo
         embedBuilder.addField("Uptime", FormatUtil.getNiceTimeDiff(Cache.getStartupTime()), true);
 
         // issue tracker field
-
-        String link = "[Issue tracker](" + Cache.getRepositoryUrl() + "issues)";
         embedBuilder.addField("Support",
-                link, true);
+                "[Issue tracker](https://git.beatrice.wtf/mind-overflow/HidekoBot/issues)",
+                true); //todo: we should probably make this a final field in the config class
 
         // bot birthday field
         embedBuilder.addField("Bot age",

src/main/java/wtf/beatrice/hidekobot/commands/base/ClearChat.java

@@ -14,42 +14,31 @@ import java.util.List;
 
 public class ClearChat
 {
-    private ClearChat()
-    {
-        throw new IllegalStateException("Utility class");
-    }
 
-    public static String getLabel()
+    public static String getLabel() {
-    {
         return "clear";
     }
 
-    public static String getDescription()
+    public static String getDescription() {
-    {
         return "Clear the current channel's chat.";
     }
 
-    public static Permission getPermission()
+    public static Permission getPermission() {
-    {
         return Permission.MESSAGE_MANAGE;
     }
 
     public static String checkDMs(Channel channel)
     {
-        if (!(channel instanceof TextChannel))
+        if(!(channel instanceof TextChannel))
-        {
+        { return "\uD83D\uDE22 Sorry! I can't delete messages here."; }
-            return "\uD83D\uDE22 Sorry! I can't delete messages here.";
-        }
 
         return null;
     }
 
     public static String checkDeleteAmount(int toDeleteAmount)
     {
-        if (toDeleteAmount <= 0)
+        if(toDeleteAmount <= 0)
-        {
+        { return  "\uD83D\uDE22 Sorry, I can't delete that amount of messages!"; }
-            return "\uD83D\uDE22 Sorry, I can't delete that amount of messages!";
-        }
 
         return null;
     }
@@ -73,7 +62,7 @@ public class ClearChat
 
         //if there are some messages left, but less than <limit>, we need one more iterations.
         int remainder = toDeleteAmount % limit;
-        if (remainder != 0) iterations++;
+        if(remainder != 0) iterations++;
 
         // set the starting point.
         long messageId = startingMessageId;
@@ -82,62 +71,61 @@ public class ClearChat
         boolean outOfBounds = false;
 
         // do iterate.
-        for (int iteration = 0; iteration < iterations; iteration++)
+        for(int iteration = 0; iteration < iterations; iteration++)
         {
-            if (outOfBounds) break;
+            if(outOfBounds) break;
 
             // set how many messages to delete for this iteration (usually <limit> unless there's a remainder)
             int iterationSize = limit;
 
-            // if we are at the last iteration... check if we have <limit> or fewer messages to delete
+            // if we are at the last iteration...
-            if (iteration + 1 == iterations && remainder != 0)
+            if(iteration+1 == iterations)
             {
-                iterationSize = remainder;
+                // check if we have <limit> or fewer messages to delete
+                if(remainder != 0) iterationSize = remainder;
             }
 
-            if (iterationSize == 1)
+            if(iterationSize == 1)
             {
                 // grab the message
                 Message toDelete = channel.retrieveMessageById(messageId).complete();
                 //only delete one message
-                if (toDelete != null) toDelete.delete().queue();
+                if(toDelete != null) toDelete.delete().queue();
                 else outOfBounds = true;
                 // increase deleted counter by 1
                 deleted++;
-            } else
+            } else {
-            {
                 // get the last <iterationSize - 1> messages.
                 MessageHistory.MessageRetrieveAction action = channel.getHistoryBefore(messageId, iterationSize - 1);
                 // note: first one is the most recent, last one is the oldest message.
                 List<Message> messages = new ArrayList<>();
                 // (we are skipping first iteration since it would return an error, given that the id is the slash command and not a message)
-                if (iteration != 0) messages.add(channel.retrieveMessageById(messageId).complete());
+                if(iteration!=0) messages.add(channel.retrieveMessageById(messageId).complete());
                 messages.addAll(action.complete().getRetrievedHistory());
 
                 // check if we only have one or zero messages left (trying to delete more than possible)
-                if (messages.size() <= 1)
+                if(messages.size() <= 1)
                 {
                     outOfBounds = true;
-                } else
+                } else {
-                {
                     // before deleting, we need to grab the <previous to the oldest> message's id for next iteration.
-                    action = channel.getHistoryBefore(messages.getLast().getIdLong(), 1);
+                    action = channel.getHistoryBefore(messages.get(messages.size() - 1).getIdLong(), 1);
 
                     List<Message> previousMessage = action.complete().getRetrievedHistory();
 
                     // if that message exists (we are not out of bounds)... store it
-                    if (!previousMessage.isEmpty()) messageId = previousMessage.getFirst().getIdLong();
+                    if(!previousMessage.isEmpty()) messageId = previousMessage.get(0).getIdLong();
                     else outOfBounds = true;
                 }
 
                 // queue messages for deletion
-                if (messages.size() == 1)
+                if(messages.size() == 1)
                 {
-                    messages.getFirst().delete().queue();
+                    messages.get(0).delete().queue();
-                } else if (!messages.isEmpty())
+                }
-                {
+                else if(!messages.isEmpty())
-                    try
                 {
+                    try {
                         ((TextChannel) channel).deleteMessages(messages).complete();
                         /* alternatively, we could use purgeMessages, which is smarter...
                         however, it also tries to delete messages older than 2 weeks
@@ -167,22 +155,18 @@ public class ClearChat
     public static String parseAmount(int deleted)
     {
 
-        if (deleted < 1)
+        if(deleted < 1)
         {
             return "\uD83D\uDE22 Couldn't clear any message!";
-        } else if (deleted == 1)
+        } else if(deleted == 1)
         {
             return "✂ Cleared 1 message!";
-        } else
+        } else {
-        {
             return "✂ Cleared " + deleted + " messages!";
         }
     }
 
     // cap the amount to avoid abuse.
-    public static int getMaxAmount()
+    public static int getMaxAmount() { return 1000; }
-    {
-        return 1000;
-    }
 
 }

src/main/java/wtf/beatrice/hidekobot/commands/base/CoinFlip.java

@@ -14,13 +14,8 @@ import java.util.List;
 public class CoinFlip
 {
 
-    private CoinFlip()
-    {
-        throw new IllegalStateException("Utility class");
-    }
 
-    public static Button getReflipButton()
+    public static Button getReflipButton() {
-    {
         return Button.primary("coinflip_reflip", "Flip again")
                 .withEmoji(Emoji.fromUnicode("\uD83E\uDE99"));
     }
@@ -30,11 +25,10 @@ public class CoinFlip
         int rand = RandomUtil.getRandomNumber(0, 1);
         String msg;
 
-        if (rand == 1)
+        if(rand == 1)
         {
             msg = ":coin: It's **Heads**!";
-        } else
+        } else {
-        {
             msg = "It's **Tails**! :coin:";
         }
 
@@ -44,7 +38,7 @@ public class CoinFlip
     public static void buttonReFlip(ButtonInteractionEvent event)
     {
         // check if the user interacting is the same one who ran the command
-        if (!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId())))
+        if(!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId())))
         {
             event.reply("❌ You did not run this command!").setEphemeral(true).queue();
             return;
@@ -62,8 +56,7 @@ public class CoinFlip
                 {
                     // set the command as expiring and restrict it to the user who ran it
                     trackAndRestrict(message, event.getUser());
-                }, (error) -> {
+                }, (error) -> {});
-                });
     }
 
     public static void trackAndRestrict(Message replyMessage, User user)

src/main/java/wtf/beatrice/hidekobot/commands/base/DiceRoll.java

@@ -5,52 +5,43 @@ import net.dv8tion.jda.api.entities.User;
 import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.objects.MessageResponse;
 import wtf.beatrice.hidekobot.objects.fun.Dice;
-import wtf.beatrice.hidekobot.util.RandomUtil;
 
 import java.util.LinkedHashMap;
 import java.util.LinkedList;
-import java.util.Map;
 import java.util.UUID;
 
 public class DiceRoll
 {
-
-    private DiceRoll()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static MessageResponse buildResponse(User author, String[] args)
     {
         LinkedHashMap<Dice, Integer> dicesToRoll = new LinkedHashMap<>();
-        String diceRegex = "d\\d+";
+        String diceRegex = "d[0-9]+";
-        String amountRegex = "\\d+";
+        String amountRegex = "[0-9]+";
 
         Dice currentDice = null;
         int currentAmount;
         UUID lastPushedDice = null;
         int totalRolls = 0;
 
-        for (String arg : args)
+        for(String arg : args)
         {
-            if (totalRolls > 200)
+            if(totalRolls > 200)
             {
                 return new MessageResponse("Too many total rolls!", null);
             }
 
-            if (arg.matches(amountRegex))
+            if(arg.matches(amountRegex))
             {
                 currentAmount = Integer.parseInt(arg);
 
-                if (currentDice == null)
+                if(currentDice == null)
                 {
                     currentDice = new Dice(6);
-                } else
+                } else {
-                {
                     currentDice = new Dice(currentDice);
                 }
 
-                if (currentAmount > 100)
+                if(currentAmount > 100)
                 {
                     return new MessageResponse("Too many rolls (`" + currentAmount + "`)!", null);
                 }
@@ -58,24 +49,25 @@ public class DiceRoll
                 lastPushedDice = currentDice.getUUID();
                 dicesToRoll.put(currentDice, currentAmount);
                 totalRolls += currentAmount;
-            } else if (arg.matches(diceRegex))
+            }
+            else if(arg.matches(diceRegex))
             {
                 int sides = Integer.parseInt(arg.substring(1));
 
-                if (sides > 10000)
+                if(sides > 10000)
                 {
                     return new MessageResponse("Too many sides (`" + sides + "`)!", null);
                 }
 
-                if (args.length == 1)
+                if(args.length == 1)
                 {
                     dicesToRoll.put(new Dice(sides), 1);
                     totalRolls++;
                 } else
                 {
-                    if (currentDice != null)
+                    if(currentDice != null)
                     {
-                        if (lastPushedDice == null || !lastPushedDice.equals(currentDice.getUUID()))
+                        if(lastPushedDice == null || !lastPushedDice.equals(currentDice.getUUID()))
                         {
                             dicesToRoll.put(currentDice, 1);
                             lastPushedDice = currentDice.getUUID();
@@ -88,16 +80,16 @@ public class DiceRoll
             }
         }
 
-        if (lastPushedDice == null)
+        if(lastPushedDice == null)
         {
-            if (currentDice != null)
+            if(currentDice != null)
             {
                 dicesToRoll.put(currentDice, 1);
                 totalRolls++;
             }
         } else
         {
-            if (!lastPushedDice.equals(currentDice.getUUID()))
+            if(!lastPushedDice.equals(currentDice.getUUID()))
             {
                 dicesToRoll.put(new Dice(currentDice), 1);
                 totalRolls++;
@@ -107,19 +99,16 @@ public class DiceRoll
         LinkedList<Dice> rolledDices = new LinkedList<>();
 
         // in case no dice was specified (or invalid), roll a standard 6-sided dice.
-        if (dicesToRoll.isEmpty())
+        if(dicesToRoll.isEmpty())
         {
             Dice standardDice = new Dice(6);
             dicesToRoll.put(standardDice, 1);
             totalRolls = 1;
         }
 
-        for (Map.Entry<Dice, Integer> entry : dicesToRoll.entrySet())
+        for(Dice dice : dicesToRoll.keySet())
         {
-            Dice dice = entry.getKey();
+            for(int roll = 0; roll < dicesToRoll.get(dice); roll++)
-            Integer rollsToMake = entry.getValue();
-
-            for (int roll = 0; roll < rollsToMake; roll++)
             {
                 dice.roll();
                 rolledDices.add(new Dice(dice));
@@ -132,23 +121,17 @@ public class DiceRoll
         embedBuilder.setAuthor(author.getAsTag(), null, author.getAvatarUrl());
         embedBuilder.setTitle("Dice Roll");
 
-        if (RandomUtil.isRandomOrgKeyValid())
-            embedBuilder.setFooter("Seed provided by random.org");
-
         StringBuilder message = new StringBuilder();
         int total = 0;
 
         int previousDiceSides = 0;
-        for (Dice dice : rolledDices)
+        for (Dice dice : rolledDices) {
-        {
             int diceSize = dice.getSides();
 
-            if (previousDiceSides != diceSize)
+            if (previousDiceSides != diceSize) {
-            {
                 message.append("\nd").append(diceSize).append(": ");
                 previousDiceSides = diceSize;
-            } else if (previousDiceSides != 0)
+            } else if (previousDiceSides != 0) {
-            {
                 message.append(", ");
             }
 
@@ -158,7 +141,7 @@ public class DiceRoll
         }
 
         // discord doesn't allow embed fields to be longer than 1024 and errors out
-        if (message.length() > 1024)
+        if(message.length() > 1024)
         {
             return new MessageResponse("Too many rolls!", null);
         }

src/main/java/wtf/beatrice/hidekobot/commands/base/Invite.java

@@ -10,11 +10,6 @@ import wtf.beatrice.hidekobot.HidekoBot;
 public class Invite
 {
 
-    private Invite()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static MessageEmbed generateEmbed()
     {
         EmbedBuilder embedBuilder = new EmbedBuilder();
@@ -23,7 +18,7 @@ public class Invite
         {
             embedBuilder.setColor(Cache.getBotColor());
             String avatarUrl = HidekoBot.getAPI().getSelfUser().getAvatarUrl();
-            if (avatarUrl != null) embedBuilder.setThumbnail(avatarUrl);
+            if(avatarUrl != null) embedBuilder.setThumbnail(avatarUrl);
             embedBuilder.setTitle("Invite");
             embedBuilder.appendDescription("Click on the button below to invite " +
                     Cache.getBotName() +

src/main/java/wtf/beatrice/hidekobot/commands/base/LoveCalculator.java

@@ -10,19 +10,13 @@ import java.util.concurrent.TimeUnit;
 
 public class LoveCalculator
 {
-
-    private LoveCalculator()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static MessageEmbed buildEmbedAndCacheResult(User author, User user1, User user2)
     {
         String userId1 = user1.getId();
         String userId2 = user2.getId();
 
         Integer loveAmount = Cache.getLoveCalculatorValue(userId1, userId2);
-        if (loveAmount == null)
+        if(loveAmount == null)
         {
             loveAmount = RandomUtil.getRandomNumber(0, 100);
             Cache.cacheLoveCalculatorValue(userId1, userId2, loveAmount);
@@ -31,9 +25,9 @@ public class LoveCalculator
         }
 
         String formattedAmount = loveAmount + "%";
-        if (loveAmount <= 30) formattedAmount += "... \uD83D\uDE22";
+        if(loveAmount <= 30) formattedAmount += "... \uD83D\uDE22";
-        else if (loveAmount < 60) formattedAmount += "! \uD83E\uDDD0";
+        else if(loveAmount < 60) formattedAmount += "! \uD83E\uDDD0";
-        else if (loveAmount < 75) formattedAmount += "!!! \uD83E\uDD73";
+        else if(loveAmount < 75) formattedAmount += "!!! \uD83E\uDD73";
         else formattedAmount = "✨ " + formattedAmount + "!!! \uD83D\uDE0D\uD83D\uDCA5";
 
         EmbedBuilder embedBuilder = new EmbedBuilder();

src/main/java/wtf/beatrice/hidekobot/commands/base/MagicBall.java

@@ -14,17 +14,12 @@ import java.util.List;
 public class MagicBall
 {
 
-    private MagicBall()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static LinkedList<String> getLabels()
     {
         return new LinkedList<>(Arrays.asList("8ball", "8b", "eightball", "magicball"));
     }
 
-    private static final List<String> answers = new ArrayList<>(
+    private final static List<String> answers = new ArrayList<>(
             Arrays.asList("It is certain.",
                     "It is decidedly so.",
                     "Without a doubt.",
@@ -56,7 +51,7 @@ public class MagicBall
     {
         // add a question mark at the end, if missing.
         // this might not always apply but it's fun
-        if (!question.endsWith("?")) question += "?";
+        if(!question.endsWith("?")) question += "?";
 
         String answer = getRandomAnswer();
 

src/main/java/wtf/beatrice/hidekobot/commands/base/ProfileImage.java

@@ -8,12 +8,6 @@ import wtf.beatrice.hidekobot.objects.MessageResponse;
 
 public class ProfileImage
 {
-
-    private ProfileImage()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static int parseResolution(int resolution)
     {
         int[] acceptedSizes = Cache.getSupportedAvatarResolutions();
@@ -21,11 +15,9 @@ public class ProfileImage
         // method to find closest value to accepted values
         int distance = Math.abs(acceptedSizes[0] - resolution);
         int idx = 0;
-        for (int c = 1; c < acceptedSizes.length; c++)
+        for(int c = 1; c < acceptedSizes.length; c++){
-        {
             int cdistance = Math.abs(acceptedSizes[c] - resolution);
-            if (cdistance < distance)
+            if(cdistance < distance){
-            {
                 idx = c;
                 distance = cdistance;
             }
@@ -43,15 +35,14 @@ public class ProfileImage
         User.Profile userProfile = user.retrieveProfile().complete();
         ImageProxy bannerProxy = userProfile.getBanner();
 
-        if (imageType == ImageType.AVATAR)
+        if(imageType == ImageType.AVATAR)
         {
             resolutionString = resolution + " × " + resolution;
             imageLink = user.getEffectiveAvatar().getUrl(resolution);
-        } else
+        } else {
-        {
             int verticalRes = 361 * resolution / 1024;
             resolutionString = resolution + " × " + verticalRes;
-            if (bannerProxy != null)
+            if(bannerProxy != null)
                 imageLink = bannerProxy.getUrl(resolution);
         }
 
@@ -68,22 +59,21 @@ public class ProfileImage
 
         // string builder to create a string that links to all available resolutions
         StringBuilder links = new StringBuilder();
-        for (int pos = 0; pos < acceptedSizes.length; pos++)
+        for(int pos = 0; pos < acceptedSizes.length; pos++)
         {
             int currSize = acceptedSizes[pos];
 
             String currLink;
-            if (imageType == ImageType.AVATAR)
+            if(imageType == ImageType.AVATAR)
             {
                 currLink = user.getEffectiveAvatar().getUrl(currSize);
-            } else
+            } else {
-            {
+                if(bannerProxy == null) break;
-                if (bannerProxy == null) break;
                 currLink = bannerProxy.getUrl(currSize);
             }
 
             links.append("**[").append(currSize).append("px](").append(currLink).append(")**");
-            if (pos + 1 != acceptedSizes.length) // don't add a separator on the last iteration
+            if(pos + 1 != acceptedSizes.length) // don't add a separator on the last iteration
             {
                 links.append(" | ");
             }
@@ -92,16 +82,15 @@ public class ProfileImage
         embedBuilder.addField("Available resolutions", links.toString(), false);
 
 
-        if (imageLink != null)
+
+        if(imageLink != null)
             embedBuilder.setImage(imageLink);
 
 
-        if (imageLink == null)
+        if(imageLink == null) {
-        {
             String error = "I couldn't find " + user.getAsMention() + "'s " + imageTypeName + "!";
             return new MessageResponse(error, null);
-        } else
+        } else {
-        {
             return new MessageResponse(null, embedBuilder.build());
         }
     }

src/main/java/wtf/beatrice/hidekobot/commands/base/Say.java

@@ -5,13 +5,7 @@ import net.dv8tion.jda.api.Permission;
 public class Say
 {
 
-    private Say()
+    public static Permission getPermission() {
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
-    public static Permission getPermission()
-    {
         return Permission.MESSAGE_MANAGE;
     }
 }

src/main/java/wtf/beatrice/hidekobot/commands/base/Trivia.java

@@ -11,7 +11,6 @@ import net.dv8tion.jda.api.interactions.components.selections.StringSelectMenu;
 import org.apache.commons.text.StringEscapeUtils;
 import org.json.JSONArray;
 import org.json.JSONObject;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.objects.MessageResponse;
 import wtf.beatrice.hidekobot.objects.comparators.TriviaCategoryComparator;
@@ -35,15 +34,8 @@ import java.util.concurrent.TimeUnit;
 
 public class Trivia
 {
-
+    private final static String triviaLink = "https://opentdb.com/api.php?amount=10&type=multiple&category=";
-    private Trivia()
+    private final static String categoriesLink = "https://opentdb.com/api_category.php";
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
-    private static final org.slf4j.Logger LOGGER = LoggerFactory.getLogger(Trivia.class);
-    private static final String TRIVIA_API_LINK = "https://opentdb.com/api.php?amount=10&type=multiple&category=";
-    private static final String TRIVIA_API_CATEGORIES_LINK = "https://opentdb.com/api_category.php";
 
     public static List<String> channelsRunningTrivia = new ArrayList<>();
 
@@ -53,36 +45,23 @@ public class Trivia
     // first string is the channelId, the list contain all score records for that channel
     public static HashMap<String, LinkedList<TriviaScore>> channelAndScores = new HashMap<>();
 
-    public static String getTriviaLink(int categoryId)
+    public static String getTriviaLink(int categoryId) {return triviaLink + categoryId; }
-    {
+    public static String getCategoriesLink() {return categoriesLink; }
-        return TRIVIA_API_LINK + categoryId;
-    }
 
-    public static String getCategoriesLink()
+    public static String getNoDMsError() {
-    {
-        return TRIVIA_API_CATEGORIES_LINK;
-    }
-
-    public static String getNoDMsError()
-    {
         return "\uD83D\uDE22 Sorry! Trivia doesn't work in DMs.";
     }
 
-    public static String getTriviaAlreadyRunningError()
+    public static String getTriviaAlreadyRunningError() {
-    {
         // todo nicer looking
         return "Trivia is already running here!";
     }
 
     public static MessageResponse generateMainScreen()
     {
+        // todo null checks
         JSONObject categoriesJson = Trivia.fetchJson(Trivia.getCategoriesLink());
-        if (categoriesJson == null)
-            return new MessageResponse("Error fetching trivia!", null); // todo nicer with emojis
         List<TriviaCategory> categories = Trivia.parseCategories(categoriesJson);
-        if (categories.isEmpty())
-            return new MessageResponse("Error parsing trivia categories!", null); // todo nicer with emojis
-
         categories.sort(new TriviaCategoryComparator());
 
         EmbedBuilder embedBuilder = new EmbedBuilder();
@@ -100,7 +79,7 @@ public class Trivia
 
         StringSelectMenu.Builder menuBuilder = StringSelectMenu.create("trivia_categories");
 
-        for (TriviaCategory category : categories)
+        for(TriviaCategory category : categories)
         {
             String name = category.categoryName();
             int id = category.categoryId();
@@ -112,23 +91,21 @@ public class Trivia
 
     public static JSONObject fetchJson(String link)
     {
-        try
+        try {
-        {
             URL url = new URL(link);
             URLConnection connection = url.openConnection();
             BufferedReader bufferedReader = new BufferedReader(new InputStreamReader(connection.getInputStream()));
 
             String currentChar;
             StringBuilder jsonStrBuilder = new StringBuilder();
-            while ((currentChar = bufferedReader.readLine()) != null)
+            while((currentChar = bufferedReader.readLine()) != null)
             {
                 jsonStrBuilder.append(currentChar);
             }
             bufferedReader.close();
             return new JSONObject(jsonStrBuilder.toString());
-        } catch (IOException e)
+        } catch (IOException e) {
-        {
+            e.printStackTrace();
-            LOGGER.error("JSON Parsing Exception", e);
         }
 
         return null;
@@ -140,7 +117,7 @@ public class Trivia
 
         JSONArray results = jsonObject.getJSONArray("results");
 
-        for (Object currentQuestionGeneric : results)
+        for(Object currentQuestionGeneric : results)
         {
             JSONObject questionJson = (JSONObject) currentQuestionGeneric;
             String question = StringEscapeUtils.unescapeHtml4(questionJson.getString("question"));
@@ -149,7 +126,7 @@ public class Trivia
             List<String> incorrectAnswersList = new ArrayList<>();
 
             JSONArray incorrectAnswers = questionJson.getJSONArray("incorrect_answers");
-            for (Object incorrectAnswerGeneric : incorrectAnswers)
+            for(Object incorrectAnswerGeneric : incorrectAnswers)
             {
                 String incorrectAnswer = (String) incorrectAnswerGeneric;
                 incorrectAnswersList.add(StringEscapeUtils.unescapeHtml4(incorrectAnswer));
@@ -166,7 +143,7 @@ public class Trivia
     {
         List<TriviaCategory> categories = new ArrayList<>();
         JSONArray categoriesArray = jsonObject.getJSONArray("trivia_categories");
-        for (Object categoryObject : categoriesArray)
+        for(Object categoryObject : categoriesArray)
         {
             JSONObject categoryJson = (JSONObject) categoryObject;
 
@@ -184,14 +161,14 @@ public class Trivia
         User user = event.getUser();
         String channelId = event.getChannel().getId();
 
-        if (trackResponse(user, event.getChannel()))
+        if(trackResponse(user, event.getChannel()))
         {
             LinkedList<TriviaScore> scores = channelAndScores.get(channelId);
-            if (scores == null) scores = new LinkedList<>();
+            if(scores == null) scores = new LinkedList<>();
             TriviaScore currentUserScore = null;
-            for (TriviaScore score : scores)
+            for(TriviaScore score : scores)
             {
-                if (score.getUser().equals(user))
+                if(score.getUser().equals(user))
                 {
                     currentUserScore = score;
                     scores.remove(score);
@@ -199,27 +176,25 @@ public class Trivia
                 }
             }
 
-            if (currentUserScore == null)
+            if(currentUserScore == null)
             {
                 currentUserScore = new TriviaScore(user);
             }
 
-            if (answerType.equals(AnswerType.CORRECT))
+            if(answerType.equals(AnswerType.CORRECT))
             {
 
                 event.reply(user.getAsMention() + " got it right! \uD83E\uDD73 (**+3**)").queue();
                 currentUserScore.changeScore(3);
 
-            } else
+            } else {
-            {
                 event.reply("❌ " + user.getAsMention() + ", that's not the right answer! (**-1**)").queue();
                 currentUserScore.changeScore(-1);
             }
 
             scores.add(currentUserScore);
             channelAndScores.put(channelId, scores);
-        } else
+        } else {
-        {
             event.reply("☹️ " + user.getAsMention() + ", you can't answer twice!")
                     .queue(interaction ->
                     Cache.getTaskScheduler().schedule(() ->
@@ -234,18 +209,17 @@ public class Trivia
 
         List<String> responders = channelAndWhoResponded.get(channelId);
 
-        if (responders == null)
+        if(responders == null)
         {
             responders = new ArrayList<>();
         }
 
-        if (responders.isEmpty() || !responders.contains(userId))
+        if(responders.isEmpty() || !responders.contains(userId))
         {
             responders.add(userId);
             channelAndWhoResponded.put(channelId, responders);
             return true; // response was successfully tracked
-        } else
+        } else {
-        {
             return false; // response wasn't tracked because there already was an entry
         }
     }
@@ -253,7 +227,7 @@ public class Trivia
     public static void handleMenuSelection(StringSelectInteractionEvent event)
     {
         // check if the user interacting is the same one who ran the command
-        if (!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId())))
+        if(!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId())))
         {
             event.reply("❌ You did not run this command!").setEphemeral(true).queue();
             return;
@@ -278,7 +252,7 @@ public class Trivia
         Message message = event.getMessage();
         MessageChannel channel = message.getChannel();
 
-        if (Trivia.channelsRunningTrivia.contains(channel.getId()))
+        if(Trivia.channelsRunningTrivia.contains(channel.getId()))
         {
             // todo nicer looking
             // todo: also what if the bot stops (database...?)
@@ -286,8 +260,7 @@ public class Trivia
             Message err = event.reply("Trivia is already running here!").complete().retrieveOriginal().complete();
             Cache.getTaskScheduler().schedule(() -> err.delete().queue(), 10, TimeUnit.SECONDS);
             return;
-        } else
+        } else {
-        {
             // todo nicer looking
             event.reply("Starting new Trivia session!").queue();
         }
@@ -304,8 +277,7 @@ public class Trivia
         Trivia.channelsRunningTrivia.add(channel.getId());
     }
 
-    public enum AnswerType
+    public enum AnswerType {
-    {
         CORRECT, WRONG
     }
 

src/main/java/wtf/beatrice/hidekobot/commands/base/UrbanDictionary.java

@@ -25,19 +25,11 @@ import java.util.List;
 public class UrbanDictionary
 {
 
-    private UrbanDictionary()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
     public static LinkedList<String> getCommandLabels()
-    {
+    { return new LinkedList<>(Arrays.asList("urban", "urbandictionary", "ud")); }
-        return new LinkedList<>(Arrays.asList("urban", "urbandictionary", "ud"));
-    }
 
 
-    public static String getBaseUrl()
+    public static String getBaseUrl() {
-    {
         return "https://www.urbandictionary.com/define.php?term=";
     }
 
@@ -59,8 +51,7 @@ public class UrbanDictionary
                 .withEmoji(Emoji.fromFormatted("\uD83D\uDDD1️"));
     }
 
-    public static String getNoArgsError()
+    public static String getNoArgsError() {
-    {
         return "\uD83D\uDE22 I need to know what to search for!";
     }
 
@@ -68,7 +59,7 @@ public class UrbanDictionary
     {
         term = term.replaceAll("[^\\w\\s]", ""); // only keep letters, numbers and spaces
         term = WordUtils.capitalizeFully(term); // Make Every Word Start With A Capital Letter
-        if (forUrl) term = term.replaceAll("\\s+", "+"); // replace all whitespaces with + for the url
+        if(forUrl) term = term.replaceAll("\\s+", "+"); // replace all whitespaces with + for the url
         if (term.length() > 64) term = term.substring(0, 64); // cut it to length to avoid abuse
         return term;
     }
@@ -92,7 +83,7 @@ public class UrbanDictionary
         embedBuilder.addField("\uD83D\uDCD6 Definition", search.getPlaintextMeanings().get(page), false);
         embedBuilder.addField("\uD83D\uDCAD Example", search.getPlaintextExamples().get(page), false);
         embedBuilder.addField("\uD83D\uDCCC Submission",
-                "*Entry " + (page + 1) + " | Sent by " + search.getContributorsNames().get(page) +
+                "*Entry " + (page+1) + " | Sent by " + search.getContributorsNames().get(page) +
                         " on" + search.getSubmissionDates().get(page) + "*",
                 false);
 
@@ -123,8 +114,7 @@ public class UrbanDictionary
         DatabaseSource database = Cache.getDatabaseSource();
 
         // check if the user interacting is the same one who ran the command
-        if (!(database.isUserTrackedFor(event.getUser().getId(), messageId)))
+        if (!(database.isUserTrackedFor(event.getUser().getId(), messageId))) {
-        {
             event.reply("❌ You did not run this command!").setEphemeral(true).queue();
             return;
         }
@@ -146,9 +136,9 @@ public class UrbanDictionary
                 serializedExamples, serializedContributors, serializedDates);
 
         // move to new page
-        if (changeType == ChangeType.NEXT)
+        if(changeType == ChangeType.NEXT)
             page++;
-        else if (changeType == ChangeType.PREVIOUS)
+        else if(changeType == ChangeType.PREVIOUS)
             page--;
 
         term = UrbanDictionary.sanitizeArgs(term, false);
@@ -159,19 +149,17 @@ public class UrbanDictionary
         // get all attached components and check which ones need to be enabled or disabled
         List<ItemComponent> components = new ArrayList<>();
 
-        if (page > 0)
+        if(page > 0)
         {
             components.add(UrbanDictionary.getPreviousPageButton().asEnabled());
-        } else
+        } else {
-        {
             components.add(UrbanDictionary.getPreviousPageButton().asDisabled());
         }
 
-        if (page + 1 == search.getPages())
+        if(page + 1 == search.getPages())
         {
             components.add(UrbanDictionary.getNextPageButton().asDisabled());
-        } else
+        } else {
-        {
             components.add(UrbanDictionary.getNextPageButton().asEnabled());
         }
 
@@ -224,10 +212,10 @@ public class UrbanDictionary
             contributorsNames = new LinkedList<>();
             submissionDates = new LinkedList<>();
 
-            for (Element definition : definitions)
+            for(Element definition : definitions)
             {
                 Elements meaningSingleton = definition.getElementsByClass("meaning");
-                if (meaningSingleton.isEmpty())
+                if(meaningSingleton.isEmpty())
                 {
                     plaintextMeanings.add(" ");
                 } else
@@ -239,13 +227,13 @@ public class UrbanDictionary
                     // this is used to fix eg. &amp; being shown literally instead of being parsed
                     text = StringEscapeUtils.unescapeHtml4(text);
                     // discord only allows 1024 characters for embed fields
-                    if (text.length() > 1024) text = text.substring(0, 1020) + "...";
+                    if(text.length() > 1024) text = text.substring(0, 1020) + "...";
                     plaintextMeanings.add(text);
                 }
 
                 Elements exampleSingleton = definition.getElementsByClass("example");
 
-                if (exampleSingleton.isEmpty())
+                if(exampleSingleton.isEmpty())
                 {
                     plaintextExamples.add(" ");
                 } else
@@ -257,12 +245,12 @@ public class UrbanDictionary
                     // this is used to fix eg. &amp; being shown literally instead of being parsed
                     text = StringEscapeUtils.unescapeHtml4(text);
                     // discord only allows 1024 characters for embed fields
-                    if (text.length() > 1024) text = text.substring(0, 1020) + "...";
+                    if(text.length() > 1024) text = text.substring(0, 1020) + "...";
                     plaintextExamples.add(text);
                 }
 
                 Elements contributorSingleton = definition.getElementsByClass("contributor");
-                if (contributorSingleton.isEmpty())
+                if(contributorSingleton.isEmpty())
                 {
                     contributorsNames.add("Unknown");
                 } else
@@ -275,10 +263,10 @@ public class UrbanDictionary
                             htmlContributor.indexOf("</a>") + 4);
 
                     contributorsNames.add(htmlContributorName
-                            .replaceAll("<.*?>", "")); // remove all html tags
+                            .replaceAll("<.*?>", "")); // remove all html tags;
 
                     submissionDates.add(htmlSubmitDate
-                            .replaceAll("<.*?>", "")); // remove all html tags
+                            .replaceAll("<.*?>", "")); // remove all html tags;
                 }
             }
 
@@ -290,48 +278,39 @@ public class UrbanDictionary
             pages = submissionDates.size();
         }
 
-        public List<String> getPlaintextMeanings()
+        public List<String> getPlaintextMeanings() {
-        {
             return this.plaintextMeanings;
         }
 
-        public List<String> getPlaintextExamples()
+        public List<String> getPlaintextExamples() {
-        {
             return this.plaintextExamples;
         }
 
-        public List<String> getContributorsNames()
+        public List<String> getContributorsNames() {
-        {
             return this.contributorsNames;
         }
 
-        public List<String> getSubmissionDates()
+        public List<String> getSubmissionDates() {
-        {
             return this.submissionDates;
         }
 
-        public String getSerializedMeanings()
+        public String getSerializedMeanings() {
-        {
             return serializedMeanings;
         }
 
-        public String getSerializedExamples()
+        public String getSerializedExamples() {
-        {
             return serializedExamples;
         }
 
-        public String getSerializedContributors()
+        public String getSerializedContributors() {
-        {
             return serializedContributors;
         }
 
-        public String getSerializedDates()
+        public String getSerializedDates() {
-        {
             return serializedDates;
         }
 
-        public int getPages()
+        public int getPages() {
-        {
             return pages;
         }
     }

src/main/java/wtf/beatrice/hidekobot/commands/base/UserPunishment.java

@@ -20,6 +20,7 @@ import wtf.beatrice.hidekobot.util.FormatUtil;
 import java.time.Duration;
 import java.time.temporal.ChronoUnit;
 import java.util.ArrayList;
+import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
 import java.util.concurrent.TimeUnit;
@@ -27,13 +28,8 @@ import java.util.concurrent.TimeUnit;
 public class UserPunishment
 {
 
-    private UserPunishment()
+    private final static Duration maxTimeoutDuration = Duration.of(28, ChronoUnit.DAYS);
-    {
+    private final static Duration minTimeoutDuration = Duration.of(30, ChronoUnit.SECONDS);
-        throw new IllegalStateException("Utility class");
-    }
-
-    private static final Duration maxTimeoutDuration = Duration.of(28, ChronoUnit.DAYS);
-    private static final Duration minTimeoutDuration = Duration.of(30, ChronoUnit.SECONDS);
 
     public static void handle(SlashCommandInteractionEvent event, PunishmentType punishmentType)
     {
@@ -43,24 +39,24 @@ public class UserPunishment
         User targetUser = null;
 
         OptionMapping targetUserArg = event.getOption("target");
-        if (targetUserArg != null)
+        if(targetUserArg != null)
         {
             targetUser = targetUserArg.getAsUser();
         }
 
         List<IMentionable> mentions = null;
-        if (targetUser != null) mentions = new ArrayList<>(Collections.singletonList(targetUser));
+        if(targetUser != null) mentions = new ArrayList<>(Collections.singletonList(targetUser));
 
         String reason = null;
         OptionMapping reasonArg = event.getOption("reason");
-        if (reasonArg != null)
+        if(reasonArg != null)
         {
             reason = reasonArg.getAsString();
         }
 
         String timeDiff = null;
         OptionMapping timeDiffArg = event.getOption("duration");
-        if (timeDiffArg != null)
+        if(timeDiffArg != null)
         {
             timeDiff = timeDiffArg.getAsString();
         }
@@ -72,13 +68,13 @@ public class UserPunishment
         // we should probably rework the it so that it works better in both scenarios.
         String[] reasonSplit = null;
         // generate the arguments array by splitting the string
-        if (reason != null) reasonSplit = reason.split("\\s+");
+        if(reason != null) reasonSplit = reason.split("\\s+");
         //prepend timediff at index 0
-        if (timeDiff != null) reasonSplit = ArrayUtils.insert(0, reasonSplit, timeDiff);
+        if(timeDiff != null) reasonSplit = ArrayUtils.insert(0, reasonSplit, timeDiff);
         // in message-commands, the first arg would contain the user mention. since we have no one mentioned here,
         // because it's in its own argument, we just prepend an empty string. note that this makes relying on the
         // first argument BAD, because it is no longer ensured that it contains the user mention.
-        if (timeDiff != null) reasonSplit = ArrayUtils.insert(0, reasonSplit, "");
+        if(timeDiff != null) reasonSplit = ArrayUtils.insert(0, reasonSplit, "");
 
         MessageResponse response = getResponse(event.getUser(),
                 punishmentType,
@@ -86,9 +82,9 @@ public class UserPunishment
                 mentions,
                 reasonSplit);
 
-        if (response.embed() != null)
+        if(response.embed() != null)
             event.getHook().editOriginalEmbeds(response.embed()).queue();
-        else if (response.content() != null)
+        else if(response.content() != null)
             event.getHook().editOriginal(response.content()).queue();
     }
 
@@ -103,9 +99,9 @@ public class UserPunishment
                 mentions,
                 args);
 
-        if (response.embed() != null)
+        if(response.embed() != null)
             event.getMessage().replyEmbeds(response.embed()).queue();
-        else if (response.content() != null)
+        else if(response.content() != null)
             event.getMessage().reply(response.content()).queue();
     }
 
@@ -118,13 +114,13 @@ public class UserPunishment
         String punishmentTypeName = punishmentType.name().toLowerCase();
 
 
-        if (!(channel instanceof TextChannel))
+        if(!(channel instanceof TextChannel))
         {
             // todo nicer looking with emojis
             return new MessageResponse("Sorry! I can't " + punishmentTypeName + " people in DMs.", null);
         }
 
-        if (mentions == null || mentions.isEmpty())
+        if(mentions == null || mentions.isEmpty())
         {
             // todo nicer looking with emojis
             return new MessageResponse("You have to tell me who to " + punishmentTypeName + "!", null);
@@ -133,8 +129,7 @@ public class UserPunishment
         String mentionedId = mentions.get(0).getId();
         User mentioned = null;
 
-        try
+        try {
-        {
             mentioned = HidekoBot.getAPI().retrieveUserById(mentionedId).complete();
         } catch (RuntimeException ignored)
         {
@@ -148,21 +143,21 @@ public class UserPunishment
         // some commands require an additional parameter before the reason, so in that case, we should start at 2.
         int startingPoint = punishmentType == PunishmentType.TIMEOUT ? 2 : 1;
 
-        if (args != null && args.length > startingPoint)
+        if(args != null && args.length > startingPoint)
         {
-            for (int i = startingPoint; i < args.length; i++)
+            for(int i = startingPoint; i < args.length; i++)
             {
                 String arg = args[i];
                 reasonBuilder.append(arg);
 
-                if (i + 1 != arg.length())
+                if(i + 1 != arg.length())
                     reasonBuilder.append(" "); // separate args with a space except on last iteration.
             }
 
             reason = reasonBuilder.toString();
         }
 
-        if (mentioned == null)
+        if(mentioned == null)
         {
             // todo nicer looking with emojis
             return new MessageResponse("I can't " + punishmentTypeName + " that user!", null);
@@ -172,17 +167,13 @@ public class UserPunishment
         Duration duration = null;
 
         AuditableRestAction<Void> punishmentAction = null;
-        boolean impossible = false;
 
-        try
+        try {
-        {
+            switch (punishmentType) {
-            switch (punishmentType)
-            {
                 case BAN -> punishmentAction = guild.ban(mentioned, 0, TimeUnit.SECONDS);
                 case KICK -> punishmentAction = guild.kick(mentioned);
-                case TIMEOUT ->
+                case TIMEOUT -> {
-                {
+                    if(args != null)
-                    if (args != null)
                     {
                         String durationStr = args[1];
                         duration = FormatUtil.parseDuration(durationStr);
@@ -190,14 +181,14 @@ public class UserPunishment
 
                     boolean isDurationValid = true;
 
-                    if (duration == null) isDurationValid = false;
+                    if(duration == null) isDurationValid = false;
                     else
                     {
-                        if (duration.compareTo(maxTimeoutDuration) > 0) isDurationValid = false;
+                        if(duration.compareTo(maxTimeoutDuration) > 0) isDurationValid = false;
-                        if (minTimeoutDuration.compareTo(duration) > 0) isDurationValid = false;
+                        if(minTimeoutDuration.compareTo(duration) > 0) isDurationValid = false;
                     }
 
-                    if (duration == null || !isDurationValid)
+                    if(duration == null || !isDurationValid)
                     {
                         // todo nicer looking with emojis
                         return new MessageResponse("Sorry, but the specified duration is invalid!", null);
@@ -206,26 +197,16 @@ public class UserPunishment
                     punishmentAction = guild.timeoutFor(mentioned, duration);
                 }
             }
-        } catch (RuntimeException ignored)
+        } catch (RuntimeException ignored) {
-        {
-            impossible = true;
-        }
-
-        if (punishmentAction == null)
-            impossible = true;
-
-        if (impossible)
-        {
             // todo nicer looking with emojis
             return new MessageResponse("Sorry, I couldn't " + punishmentTypeName + " " + mentioned.getAsMention() + "!",
                     null);
         }
 
-        if (!reason.isEmpty() && !reasonBuilder.isEmpty())
+        if(!reason.isEmpty() && !reasonBuilder.isEmpty())
             punishmentAction.reason("[" + author.getAsTag() + "] " + reason);
 
-        try
+        try {
-        {
             punishmentAction.complete();
         } catch (RuntimeException ignored)
         {
@@ -242,10 +223,10 @@ public class UserPunishment
 
         embedBuilder.addField("\uD83D\uDC64 User", mentioned.getAsMention(), false);
         embedBuilder.addField("✂️ By", author.getAsMention(), false);
-        if (duration != null)
+        if(duration != null)
             embedBuilder.addField("⏱️ Duration", FormatUtil.getNiceDuration(duration), false);
 
-        if (reason.isEmpty())
+        if(reason.isEmpty())
             reason = "*No reason specified*";
 
         embedBuilder.addField("\uD83D\uDCD6 Reason", reason, false);
@@ -255,8 +236,7 @@ public class UserPunishment
 
     }
 
-    public enum PunishmentType
+    public enum PunishmentType {
-    {
         KICK("kicked"),
         BAN("banned"),
         TIMEOUT("timed out"),

src/main/java/wtf/beatrice/hidekobot/commands/completer/ProfileImageCommandCompleter.java

@@ -13,25 +13,23 @@ import java.util.List;
 public class ProfileImageCommandCompleter extends SlashArgumentsCompleterImpl
 {
 
-    public ProfileImageCommandCompleter(SlashCommand parentCommand)
+    public ProfileImageCommandCompleter(SlashCommand parentCommand) {
-    {
         super(parentCommand);
     }
 
     @Override
-    public void runCompletion(@NotNull CommandAutoCompleteInteractionEvent event)
+    public void runCompletion(@NotNull CommandAutoCompleteInteractionEvent event) {
-    {
+        if(event.getFocusedOption().getName().equals("size"))
-        if (event.getFocusedOption().getName().equals("size"))
         {
 
             List<Command.Choice> options = new ArrayList<>();
 
-            for (int res : Cache.getSupportedAvatarResolutions())
+            for(int res : Cache.getSupportedAvatarResolutions())
             {
                 String resString = String.valueOf(res);
                 String userInput = event.getFocusedOption().getValue();
 
-                if (resString.startsWith(userInput))
+                if(resString.startsWith(userInput))
                     options.add(new Command.Choice(resString, res));
             }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/AliasCommand.java

@@ -17,49 +17,43 @@ public class AliasCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Arrays.asList("alias", "aliases"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "See other command aliases.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<command>";
     }
 
     @Override
     public void runCommand(MessageReceivedEvent event, String label, String[] args)
     {
-        if (args.length == 0)
+        if(args.length == 0)
         {
             event.getMessage().reply("\uD83D\uDE20 Hey, you have to specify a command!").queue();
             return;
@@ -67,7 +61,7 @@ public class AliasCommand implements MessageCommand
 
         String commandLabel = args[0].toLowerCase();
         MessageCommand command = Cache.getMessageCommandListener().getRegisteredCommand(commandLabel);
-        if (command == null)
+        if(command == null)
         {
             event.getMessage().reply("Unrecognized command: `" + commandLabel + "`!").queue(); // todo prettier
             return;

src/main/java/wtf/beatrice/hidekobot/commands/message/AvatarCommand.java

@@ -6,6 +6,7 @@ import net.dv8tion.jda.api.entities.User;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
+import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.commands.base.ProfileImage;
 import wtf.beatrice.hidekobot.objects.MessageResponse;
@@ -20,48 +21,44 @@ public class AvatarCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("avatar"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get someone's avatar, or your own. You can additionally specify a resolution.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "[mentioned user] [resolution]";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
     @Override
     public void runCommand(MessageReceivedEvent event, String label, String[] args)
     {
+        int[] acceptedSizes = Cache.getSupportedAvatarResolutions();
+
         User user;
         int resolution = -1;
 
@@ -69,26 +66,22 @@ public class AvatarCommand implements MessageCommand
         // (mentions are handled differently by a specific method)
         boolean resFound = false;
 
-        for (String arg : args)
+        for (String arg : args) {
-        {
+            try {
-            try
-            {
                 int givenRes = Integer.parseInt(arg);
                 resolution = ProfileImage.parseResolution(givenRes);
                 resFound = true;
                 break;
-            } catch (NumberFormatException ignored)
+            } catch (NumberFormatException ignored) {
-            {
-                // ignored because we're running a check after this block
             }
         }
 
         // fallback in case we didn't find any specified resolution
-        if (!resFound) resolution = ProfileImage.parseResolution(512);
+        if(!resFound) resolution = ProfileImage.parseResolution(512);
 
         // check if someone is mentioned
         Mentions mentions = event.getMessage().getMentions();
-        if (mentions.getMentions().isEmpty())
+        if(mentions.getMentions().isEmpty())
         {
             user = event.getAuthor();
         } else
@@ -98,14 +91,14 @@ public class AvatarCommand implements MessageCommand
         }
 
         // in case of issues, fallback to the sender
-        if (user == null) user = event.getAuthor();
+        if(user == null) user = event.getAuthor();
 
         // send a response
         MessageResponse response = ProfileImage.buildResponse(resolution, user, ProfileImage.ImageType.AVATAR);
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getMessage().reply(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getMessage().replyEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/message/BanCommand.java

@@ -1,10 +1,19 @@
 package wtf.beatrice.hidekobot.commands.message;
 
+import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.Permission;
+import net.dv8tion.jda.api.entities.IMentionable;
+import net.dv8tion.jda.api.entities.Mentions;
+import net.dv8tion.jda.api.entities.User;
+import net.dv8tion.jda.api.entities.channel.concrete.TextChannel;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
+import net.dv8tion.jda.api.requests.restaction.AuditableRestAction;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
+import wtf.beatrice.hidekobot.Cache;
+import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.commands.base.UserPunishment;
+import wtf.beatrice.hidekobot.objects.MessageResponse;
 import wtf.beatrice.hidekobot.objects.commands.CommandCategory;
 import wtf.beatrice.hidekobot.objects.commands.MessageCommand;
 
@@ -12,47 +21,42 @@ import java.util.ArrayList;
 import java.util.Collections;
 import java.util.LinkedList;
 import java.util.List;
+import java.util.concurrent.TimeUnit;
 
 public class BanCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("ban"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return new ArrayList<Permission>(Collections.singletonList(Permission.BAN_MEMBERS));
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.MODERATION;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Ban the mentioned user.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<mentioned user> [reason]";
     }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/BannerCommand.java

@@ -20,42 +20,36 @@ public class BannerCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("banner"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get someone's profile banner, or your own.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "[mentioned user] [resolution]";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
@@ -69,26 +63,22 @@ public class BannerCommand implements MessageCommand
         // (mentions are handled differently by a specific method)
         boolean resFound = false;
 
-        for (String arg : args)
+        for (String arg : args) {
-        {
+            try {
-            try
-            {
                 int givenRes = Integer.parseInt(arg);
                 resolution = ProfileImage.parseResolution(givenRes);
                 resFound = true;
                 break;
-            } catch (NumberFormatException ignored)
+            } catch (NumberFormatException ignored) {
-            {
-                // ignored because we're running a check after this block
             }
         }
 
         // fallback in case we didn't find any specified resolution
-        if (!resFound) resolution = ProfileImage.parseResolution(512);
+        if(!resFound) resolution = ProfileImage.parseResolution(512);
 
         // check if someone is mentioned
         Mentions mentions = event.getMessage().getMentions();
-        if (mentions.getMentions().isEmpty())
+        if(mentions.getMentions().isEmpty())
         {
             user = event.getAuthor();
         } else
@@ -98,14 +88,14 @@ public class BannerCommand implements MessageCommand
         }
 
         // in case of issues, fallback to the sender
-        if (user == null) user = event.getAuthor();
+        if(user == null) user = event.getAuthor();
 
         // send a response
         MessageResponse response = ProfileImage.buildResponse(resolution, user, ProfileImage.ImageType.BANNER);
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getMessage().reply(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getMessage().replyEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/message/BotInfoCommand.java

@@ -11,6 +11,7 @@ import wtf.beatrice.hidekobot.objects.commands.CommandCategory;
 import wtf.beatrice.hidekobot.objects.commands.MessageCommand;
 
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.LinkedList;
 import java.util.List;
 
@@ -18,54 +19,46 @@ public class BotInfoCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Arrays.asList("botinfo", "info"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get general info about the bot.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return null;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
     @Override
-    public void runCommand(MessageReceivedEvent event, String label, String[] args)
+    public void runCommand(MessageReceivedEvent event, String label, String[] args) {
-    {
 
         // get a list of message commands
         LinkedList<MessageCommand> messageCommands = Cache.getMessageCommandListener().getRegisteredCommands();
         LinkedList<String> commandNames = new LinkedList<>();
-        for (MessageCommand command : messageCommands)
+        for (MessageCommand command : messageCommands) {
-        {
             commandNames.add(command.getCommandLabels().get(0));
         }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/ClearCommand.java

@@ -19,51 +19,44 @@ public class ClearCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList(ClearChat.getLabel()));
     }
 
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() { return Collections.singletonList(ClearChat.getPermission()); }
-    {
-        return Collections.singletonList(ClearChat.getPermission());
-    }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.MODERATION;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Clear the current channel's chat history.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "[amount]";
     }
 
     @Override
     public void runCommand(MessageReceivedEvent event, String label, String[] args)
     {
+        String senderId = event.getMessage().getAuthor().getId();
+
         // check if user is trying to run command in dms.
         String error = ClearChat.checkDMs(event.getChannel());
-        if (error != null)
+        if (error != null) {
-        {
             event.getMessage().reply(error).queue();
             return;
         }
@@ -73,8 +66,7 @@ public class ClearCommand implements MessageCommand
         if (args.length == 0) toDeleteAmount = 1;
         else
         {
-            try
+            try {
-            {
                 toDeleteAmount = Integer.parseInt(args[0]);
             } catch (NumberFormatException e)
             {
@@ -83,11 +75,10 @@ public class ClearCommand implements MessageCommand
         }
 
         // cap the amount to avoid abuse.
-        if (toDeleteAmount > ClearChat.getMaxAmount()) toDeleteAmount = 0;
+        if(toDeleteAmount > ClearChat.getMaxAmount()) toDeleteAmount = 0;
 
         error = ClearChat.checkDeleteAmount(toDeleteAmount);
-        if (error != null)
+        if (error != null) {
-        {
             event.getMessage().reply(error).queue();
             return;
         }

src/main/java/wtf/beatrice/hidekobot/commands/message/CoinFlipCommand.java

@@ -16,48 +16,41 @@ public class CoinFlipCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Arrays.asList("coinflip", "flip", "flipcoin"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // null because it can be used anywhere
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Flip a coin.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return null;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
     @Override
-    public void runCommand(MessageReceivedEvent event, String label, String[] args)
+    public void runCommand(MessageReceivedEvent event, String label, String[] args) {
-    {
 
         // perform coin flip
         event.getMessage().reply(CoinFlip.genRandom())
@@ -66,7 +59,6 @@ public class CoinFlipCommand implements MessageCommand
                 {
                     // set the command as expiring and restrict it to the user who ran it
                     CoinFlip.trackAndRestrict(message, event.getAuthor());
-                }, (error) -> {
+                }, (error) -> {});
-                });
     }
 }

src/main/java/wtf/beatrice/hidekobot/commands/message/DiceRollCommand.java

@@ -16,49 +16,41 @@ import java.util.List;
 public class DiceRollCommand implements MessageCommand
 {
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Arrays.asList("diceroll", "droll", "roll"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
+        return "Roll dice. You can roll multiple dice at the same time." +
-        return """
+                "\nExamples:" +
-                Roll dice. You can roll multiple dice at the same time.
+                "\n - `d8 10` to roll an 8-sided die 10 times." +
-                Examples:
+                "\n - `d12 3 d5 10` to roll a 12-sided die 3 times, and then a 5-sided die 10 times." +
-                 - `d8 10` to roll an 8-sided die 10 times.
+                "\n - `30` to roll a standard 6-sided die 30 times." +
-                 - `d12 3 d5 10` to roll a 12-sided die 3 times, and then a 5-sided die 10 times.
+                "\n - `d10` to roll a 10-sided die once.";
-                 - `30` to roll a standard 6-sided die 30 times.
-                 - `d10` to roll a 10-sided die once.
-                """;
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "[dice size] [rolls]";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
@@ -68,10 +60,10 @@ public class DiceRollCommand implements MessageCommand
 
         MessageResponse response = DiceRoll.buildResponse(event.getAuthor(), args);
 
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getMessage().reply(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getMessage().replyEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/message/HelloCommand.java

@@ -15,41 +15,33 @@ public class HelloCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Arrays.asList("hi", "hello", "heya"));
     }
 
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() { return null; }
-    {
-        return null;
-    }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get pinged by the bot.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return null;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/HelpCommand.java

@@ -11,49 +11,44 @@ import wtf.beatrice.hidekobot.commands.base.Alias;
 import wtf.beatrice.hidekobot.objects.commands.CommandCategory;
 import wtf.beatrice.hidekobot.objects.commands.MessageCommand;
 
-import java.util.*;
+import java.util.Collections;
+import java.util.LinkedHashMap;
+import java.util.LinkedList;
+import java.util.List;
 
 public class HelpCommand implements MessageCommand
 {
 
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("help"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() { return null; }
-    {
-        return null;
-    }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get general help on the bot. Specify a command if you want specific help about that command.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "[command]";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
@@ -62,14 +57,14 @@ public class HelpCommand implements MessageCommand
     {
         LinkedHashMap<CommandCategory, LinkedList<MessageCommand>> commandCategories = new LinkedHashMap<>();
 
-        if (args.length == 0)
+        if(args.length == 0)
         {
-            for (CommandCategory category : CommandCategory.values())
+            for(CommandCategory category : CommandCategory.values())
             {
                 LinkedList<MessageCommand> commandsOfThisCategory = new LinkedList<>();
                 for (MessageCommand command : Cache.getMessageCommandListener().getRegisteredCommands())
                 {
-                    if (command.getCategory().equals(category))
+                    if(command.getCategory().equals(category))
                     {
                         commandsOfThisCategory.add(command);
                     }
@@ -87,18 +82,17 @@ public class HelpCommand implements MessageCommand
                             "\nYou will find a list of commands organized in categories below.",
                     false);
 
-            for (Map.Entry<CommandCategory, LinkedList<MessageCommand>> entry : commandCategories.entrySet())
+            for(CommandCategory category : commandCategories.keySet())
             {
                 StringBuilder commandsList = new StringBuilder();
-                CommandCategory category = entry.getKey();
+                LinkedList<MessageCommand> commandsOfThisCategory = commandCategories.get(category);
-                LinkedList<MessageCommand> commandsOfThisCategory = entry.getValue();
 
-                for (int pos = 0; pos < commandsOfThisCategory.size(); pos++)
+                for(int pos = 0; pos < commandsOfThisCategory.size(); pos++)
                 {
                     MessageCommand command = commandsOfThisCategory.get(pos);
                     commandsList.append("`").append(command.getCommandLabels().get(0)).append("`");
 
-                    if (pos + 1 != commandsOfThisCategory.size())
+                    if(pos + 1 != commandsOfThisCategory.size())
                         commandsList.append(", "); // separate with comma except on last run
                 }
 
@@ -110,12 +104,11 @@ public class HelpCommand implements MessageCommand
             }
 
             event.getMessage().replyEmbeds(embedBuilder.build()).queue();
-        } else
+        } else {
-        {
 
             String commandLabel = args[0].toLowerCase();
             MessageCommand command = Cache.getMessageCommandListener().getRegisteredCommand(commandLabel);
-            if (command == null)
+            if(command == null)
             {
                 event.getMessage().reply("Unrecognized command: `" + commandLabel + "`!").queue(); // todo prettier
                 return;
@@ -124,24 +117,23 @@ public class HelpCommand implements MessageCommand
             commandLabel = command.getCommandLabels().get(0);
             String usage = "`" + Cache.getBotPrefix() + " " + commandLabel;
             String internalUsage = command.getUsage();
-            if (internalUsage != null) usage += " " + internalUsage;
+            if(internalUsage != null) usage += " " + internalUsage;
             usage += "`";
 
             String aliases = Alias.generateNiceAliases(command);
 
             List<Permission> permissions = command.getPermissions();
             StringBuilder permissionsStringBuilder = new StringBuilder();
-            if (permissions == null)
+            if(permissions == null)
             {
                 permissionsStringBuilder = new StringBuilder("Available to everyone");
-            } else
+            } else {
-            {
+                for(int i = 0; i < permissions.size(); i++)
-                for (int i = 0; i < permissions.size(); i++)
                 {
                     Permission permission = permissions.get(i);
                     permissionsStringBuilder.append("**").append(permission.getName()).append("**");
 
-                    if (i + 1 != permissions.size())
+                    if(i + 1 != permissions.size())
                         permissionsStringBuilder.append(", "); // separate with comma expect on last iteration
                 }
             }

src/main/java/wtf/beatrice/hidekobot/commands/message/InviteCommand.java

@@ -19,42 +19,36 @@ public class InviteCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("invite"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null;
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Get the bot's invite link.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return null;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.MODERATION;
     }
 
@@ -67,7 +61,7 @@ public class InviteCommand implements MessageCommand
         Button inviteButton = Invite.getInviteButton();
 
         // if this is a guild, don't spam the invite in public but DM it
-        if (event.getChannelType().isGuild())
+        if(event.getChannelType().isGuild())
         {
             event.getAuthor().openPrivateChannel().queue(privateChannel ->
             {
@@ -76,8 +70,7 @@ public class InviteCommand implements MessageCommand
                         .queue();
                 event.getMessage().addReaction(Emoji.fromUnicode("✅")).queue();
             }, error -> event.getMessage().addReaction(Emoji.fromUnicode("❌")).queue());
-        } else
+        } else {
-        {
             event.getMessage()
                     .replyEmbeds(inviteEmbed)
                     .addActionRow(inviteButton)

src/main/java/wtf/beatrice/hidekobot/commands/message/KickCommand.java

@@ -1,10 +1,19 @@
 package wtf.beatrice.hidekobot.commands.message;
 
+import net.dv8tion.jda.api.EmbedBuilder;
 import net.dv8tion.jda.api.Permission;
+import net.dv8tion.jda.api.entities.IMentionable;
+import net.dv8tion.jda.api.entities.Mentions;
+import net.dv8tion.jda.api.entities.User;
+import net.dv8tion.jda.api.entities.channel.concrete.TextChannel;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
+import net.dv8tion.jda.api.requests.restaction.AuditableRestAction;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
+import wtf.beatrice.hidekobot.Cache;
+import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.commands.base.UserPunishment;
+import wtf.beatrice.hidekobot.objects.MessageResponse;
 import wtf.beatrice.hidekobot.objects.commands.CommandCategory;
 import wtf.beatrice.hidekobot.objects.commands.MessageCommand;
 
@@ -17,42 +26,36 @@ public class KickCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("kick"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return new ArrayList<Permission>(Collections.singletonList(Permission.KICK_MEMBERS));
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.MODERATION;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Kick the mentioned user from the guild.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<mentioned user> [reason]";
     }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/LoveCalculatorCommand.java

@@ -29,35 +29,30 @@ public class LoveCalculatorCommand implements MessageCommand
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; //anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Calculate how much two people love each other. You can mention two people or just one.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<person 1> [person 2]";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
@@ -69,7 +64,7 @@ public class LoveCalculatorCommand implements MessageCommand
         List<IMentionable> mentions = mentionsObj.getMentions();
 
 
-        if (args.length == 0 || mentions.isEmpty())
+        if(args.length == 0 || mentions.isEmpty())
         {
             event.getMessage()
                     .reply("\uD83D\uDE22 I need to know who to check! Please mention them.")
@@ -82,11 +77,10 @@ public class LoveCalculatorCommand implements MessageCommand
         String mentionedUserId = mentions.get(0).getId();
         user1 = HidekoBot.getAPI().retrieveUserById(mentionedUserId).complete();
 
-        if (mentions.size() == 1)
+        if(mentions.size() == 1)
         {
             user2 = event.getAuthor();
-        } else
+        } else {
-        {
             mentionedUserId = mentions.get(1).getId();
             user2 = HidekoBot.getAPI().retrieveUserById(mentionedUserId).complete();
         }

src/main/java/wtf/beatrice/hidekobot/commands/message/MagicBallCommand.java

@@ -15,60 +15,54 @@ public class MagicBallCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return MagicBall.getLabels();
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; // anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Ask a question to the Magic Ball.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<question>";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
     @Override
     public void runCommand(MessageReceivedEvent event, String label, String[] args)
     {
-        if (args.length == 0)
+        if(args.length == 0)
         {
             event.getMessage().reply("You need to specify a question!").queue();
             return;
         }
 
         StringBuilder questionBuilder = new StringBuilder();
-        for (int i = 0; i < args.length; i++)
+        for(int i = 0; i < args.length; i++)
         {
             String arg = args[i];
             questionBuilder.append(arg);
-            if (i + 1 != args.length) // don't add a separator on the last iteration
+            if(i + 1 != args.length) // don't add a separator on the last iteration
                 questionBuilder.append(" ");
         }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/SayCommand.java

@@ -1,7 +1,6 @@
 package wtf.beatrice.hidekobot.commands.message;
 
 import net.dv8tion.jda.api.Permission;
-import net.dv8tion.jda.api.entities.channel.concrete.TextChannel;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
@@ -18,42 +17,34 @@ public class SayCommand implements MessageCommand
 
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("say"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() { return Collections.singletonList(Say.getPermission()); }
-    {
-        return Collections.singletonList(Say.getPermission());
-    }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return true;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Make the bot say something for you.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<text>";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.TOOLS;
     }
 
@@ -62,26 +53,17 @@ public class SayCommand implements MessageCommand
     {
 
         String messageContent;
-        if (args.length != 0 && !args[0].isEmpty())
+        if(args.length != 0 && !args[0].isEmpty())
         {
             messageContent = args[0];
-        } else
+        } else {
-        {
             event.getMessage().reply("\uD83D\uDE20 Hey, you have to tell me what to say!")
                     .queue();
             return;
         }
 
         event.getChannel().sendMessage(messageContent).queue();
+        event.getMessage().delete().queue();
 
-        if (event.getChannel() instanceof TextChannel)
-        {
-            event.getMessage().delete().queue(response -> {
-                // nothing to do with the response
-            }, error -> {
-                // ignore the error if we couldn't delete it, we were probably missing permissions
-                // without this block it would print a stack trace in console
-            });
-        }
     }
 }

src/main/java/wtf/beatrice/hidekobot/commands/message/TimeoutCommand.java

@@ -17,42 +17,36 @@ public class TimeoutCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("timeout"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return new ArrayList<Permission>(Collections.singletonList(Permission.MODERATE_MEMBERS));
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.MODERATION;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Timeout the mentioned user.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<mentioned user> <duration> [reason]";
     }
 

src/main/java/wtf/beatrice/hidekobot/commands/message/TriviaCommand.java

@@ -5,7 +5,6 @@ import net.dv8tion.jda.api.entities.Message;
 import net.dv8tion.jda.api.entities.channel.concrete.TextChannel;
 import net.dv8tion.jda.api.entities.channel.middleman.MessageChannel;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
-import net.dv8tion.jda.api.requests.restaction.MessageCreateAction;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 import wtf.beatrice.hidekobot.Cache;
@@ -23,42 +22,36 @@ public class TriviaCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return new LinkedList<>(Collections.singletonList("trivia"));
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null;
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Start a Trivia session and play with others!";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return null;
     }
 
@@ -67,13 +60,13 @@ public class TriviaCommand implements MessageCommand
     {
         MessageChannel channel = event.getChannel();
 
-        if (!(channel instanceof TextChannel))
+        if(!(channel instanceof TextChannel))
         {
             channel.sendMessage(Trivia.getNoDMsError()).queue();
             return;
         }
 
-        if (Trivia.channelsRunningTrivia.contains(channel.getId()))
+        if(Trivia.channelsRunningTrivia.contains(channel.getId()))
         {
             // todo: also what if the bot stops (database...?)
             // todo: also what if the message is already deleted
@@ -84,20 +77,11 @@ public class TriviaCommand implements MessageCommand
 
         MessageResponse response = Trivia.generateMainScreen();
 
-        Message recvMessage = event.getMessage();
+        event.getMessage().replyEmbeds(response.embed()).addActionRow(response.components()).queue(message ->
-        MessageCreateAction responseAction = null;
-        if (response.content() != null) responseAction = recvMessage.reply(response.content());
-        else if (response.embed() != null) responseAction = recvMessage.replyEmbeds(response.embed());
-
-        if (responseAction != null)
         {
-            if (response.components() != null) responseAction = responseAction.addActionRow(response.components());
-
-            responseAction.queue(message -> {
             Cache.getDatabaseSource().trackRanCommandReply(message, event.getAuthor());
             Cache.getDatabaseSource().queueDisabling(message);
         });
-        }
 
 
     }

src/main/java/wtf/beatrice/hidekobot/commands/message/UrbanDictionaryCommand.java

@@ -21,42 +21,36 @@ public class UrbanDictionaryCommand implements MessageCommand
 {
 
     @Override
-    public LinkedList<String> getCommandLabels()
+    public LinkedList<String> getCommandLabels() {
-    {
         return UrbanDictionary.getCommandLabels();
     }
 
     @Nullable
     @Override
-    public List<Permission> getPermissions()
+    public List<Permission> getPermissions() {
-    {
         return null; //anyone can use it
     }
 
     @Override
-    public boolean passRawArgs()
+    public boolean passRawArgs() {
-    {
         return false;
     }
 
     @NotNull
     @Override
-    public String getDescription()
+    public String getDescription() {
-    {
         return "Look something up in the Urban Dictionary.";
     }
 
     @Nullable
     @Override
-    public String getUsage()
+    public String getUsage() {
-    {
         return "<query>";
     }
 
     @NotNull
     @Override
-    public CommandCategory getCategory()
+    public CommandCategory getCategory() {
-    {
         return CommandCategory.FUN;
     }
 
@@ -64,7 +58,7 @@ public class UrbanDictionaryCommand implements MessageCommand
     @Override
     public void runCommand(MessageReceivedEvent event, String label, String[] args)
     {
-        if (args.length == 0)
+        if(args.length == 0)
         {
             event.getMessage().reply(UrbanDictionary.getNoArgsError()).queue();
             return;
@@ -72,12 +66,11 @@ public class UrbanDictionaryCommand implements MessageCommand
 
         // sanitize args by only keeping letters and numbers, and adding "+" instead of spaces for HTML parsing
         StringBuilder termBuilder = new StringBuilder();
-        for (int i = 0; i < args.length; i++)
+        for (int i = 0; i < args.length; i++) {
-        {
             String arg = args[i];
             termBuilder.append(arg);
 
-            if (i + 1 != args.length) // add spaces between args, but not on the last run
+            if(i + 1 != args.length) // add spaces between args, but not on the last run
                 termBuilder.append(" ");
         }
 
@@ -86,11 +79,9 @@ public class UrbanDictionaryCommand implements MessageCommand
 
         Document doc;
 
-        try
+        try {
-        {
             doc = Jsoup.connect(url).get();
-        } catch (IOException e)
+        } catch (IOException e) {
-        {
             event.getMessage().reply(UrbanDictionary.getTermNotFoundError()).queue();
             return;
         }
@@ -101,7 +92,7 @@ public class UrbanDictionaryCommand implements MessageCommand
 
         // disable next page if we only have one result
         Button nextPageBtnLocal = UrbanDictionary.getNextPageButton();
-        if (search.getPages() == 1) nextPageBtnLocal = nextPageBtnLocal.asDisabled();
+        if(search.getPages() == 1) nextPageBtnLocal = nextPageBtnLocal.asDisabled();
 
         event.getChannel()
                 .sendMessageEmbeds(embed)

src/main/java/wtf/beatrice/hidekobot/commands/slash/AvatarCommand.java

@@ -14,8 +14,7 @@ import wtf.beatrice.hidekobot.objects.commands.SlashCommandImpl;
 public class AvatarCommand extends SlashCommandImpl
 {
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return Commands.slash("avatar", "Get someone's profile picture.")
                 .addOption(OptionType.USER, "user", "User you want to grab the avatar of.")
                 .addOption(OptionType.INTEGER, "size", "The size of the returned image.",
@@ -33,28 +32,26 @@ public class AvatarCommand extends SlashCommandImpl
         int resolution;
 
         OptionMapping userArg = event.getOption("user");
-        if (userArg != null)
+        if(userArg != null)
         {
             user = userArg.getAsUser();
-        } else
+        } else {
-        {
             user = event.getUser();
         }
 
         OptionMapping sizeArg = event.getOption("size");
-        if (sizeArg != null)
+        if(sizeArg != null)
         {
             resolution = ProfileImage.parseResolution(sizeArg.getAsInt());
-        } else
+        } else {
-        {
             resolution = ProfileImage.parseResolution(512);
         }
 
         MessageResponse response = ProfileImage.buildResponse(resolution, user, ProfileImage.ImageType.AVATAR);
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getHook().editOriginal(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getHook().editOriginalEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/slash/BannerCommand.java

@@ -14,8 +14,7 @@ import wtf.beatrice.hidekobot.objects.commands.SlashCommandImpl;
 public class BannerCommand extends SlashCommandImpl
 {
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return Commands.slash("banner", "Get someone's profile banner.")
                 .addOption(OptionType.USER, "user", "User you want to grab the banner of.")
                 .addOption(OptionType.INTEGER, "size", "The size of the returned image.",
@@ -33,28 +32,26 @@ public class BannerCommand extends SlashCommandImpl
         int resolution;
 
         OptionMapping userArg = event.getOption("user");
-        if (userArg != null)
+        if(userArg != null)
         {
             user = userArg.getAsUser();
-        } else
+        } else {
-        {
             user = event.getUser();
         }
 
         OptionMapping sizeArg = event.getOption("size");
-        if (sizeArg != null)
+        if(sizeArg != null)
         {
             resolution = ProfileImage.parseResolution(sizeArg.getAsInt());
-        } else
+        } else {
-        {
             resolution = ProfileImage.parseResolution(512);
         }
 
         MessageResponse response = ProfileImage.buildResponse(resolution, user, ProfileImage.ImageType.BANNER);
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getHook().editOriginal(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getHook().editOriginalEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/slash/BotInfoCommand.java

@@ -16,8 +16,7 @@ import java.util.List;
 public class BotInfoCommand extends SlashCommandImpl
 {
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return Commands.slash("botinfo", "Get info about the bot.");
     }
 
@@ -30,7 +29,7 @@ public class BotInfoCommand extends SlashCommandImpl
         // get a list of slash commands
         List<SlashCommand> registeredCommands = Cache.getSlashCommandListener().getRegisteredCommands();
         LinkedList<String> registeredCommandNames = new LinkedList<>();
-        for (SlashCommand command : registeredCommands)
+        for(SlashCommand command : registeredCommands)
         {
             // node: adding slash so people realize that this is specific about slash commands.
             registeredCommandNames.add("/" + command.getCommandName());

src/main/java/wtf/beatrice/hidekobot/commands/slash/ClearCommand.java

@@ -17,8 +17,7 @@ public class ClearCommand extends SlashCommandImpl
 {
 
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return Commands.slash(ClearChat.getLabel(),
                         ClearChat.getDescription())
                 .addOption(OptionType.INTEGER, "amount", "The amount of messages to delete.")
@@ -33,7 +32,7 @@ public class ClearCommand extends SlashCommandImpl
 
         // check if user is trying to run command in dms.
         String error = ClearChat.checkDMs(event.getChannel());
-        if (error != null)
+        if(error != null)
         {
             event.getHook().editOriginal(error).queue();
             return;
@@ -46,10 +45,10 @@ public class ClearCommand extends SlashCommandImpl
         int toDeleteAmount = amountOption == null ? 1 : amountOption.getAsInt();
 
         // cap the amount to avoid abuse.
-        if (toDeleteAmount > ClearChat.getMaxAmount()) toDeleteAmount = 0;
+        if(toDeleteAmount > ClearChat.getMaxAmount()) toDeleteAmount = 0;
 
         error = ClearChat.checkDeleteAmount(toDeleteAmount);
-        if (error != null)
+        if(error != null)
         {
             event.getHook().editOriginal(error).queue();
             return;

src/main/java/wtf/beatrice/hidekobot/commands/slash/CoinFlipCommand.java

@@ -29,11 +29,12 @@ public class CoinFlipCommand extends SlashCommandImpl
                     interaction.retrieveOriginal().queue((message) ->
                     {
                         CoinFlip.trackAndRestrict(message, event.getUser());
-                    }, (error) -> {
+                    }, (error) -> {});
-                    });
+                }, (error) -> {});
-                }, (error) -> {
-                });
     }
 
 
+
+
+
 }

src/main/java/wtf/beatrice/hidekobot/commands/slash/DiceRollCommand.java

@@ -30,7 +30,7 @@ public class DiceRollCommand extends SlashCommandImpl
 
         OptionMapping textOption = event.getOption("query");
         String messageContent = "";
-        if (textOption != null)
+        if(textOption != null)
         {
              messageContent = textOption.getAsString();
         }
@@ -39,10 +39,10 @@ public class DiceRollCommand extends SlashCommandImpl
 
         MessageResponse response = DiceRoll.buildResponse(event.getUser(), args);
 
-        if (response.content() != null)
+        if(response.content() != null)
         {
             event.getHook().editOriginal(response.content()).queue();
-        } else if (response.embed() != null)
+        } else if(response.embed() != null)
         {
             event.getHook().editOriginalEmbeds(response.embed()).queue();
         }

src/main/java/wtf/beatrice/hidekobot/commands/slash/DieCommand.java

@@ -10,14 +10,12 @@ import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.objects.commands.SlashCommandImpl;
 
 import java.util.concurrent.Executors;
-import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
 
 public class DieCommand extends SlashCommandImpl
 {
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return Commands.slash("die", "Stop the bot's process.")
                 .setDefaultPermissions(DefaultMemberPermissions.DISABLED);
     }
@@ -25,16 +23,12 @@ public class DieCommand extends SlashCommandImpl
     @Override
     public void runSlashCommand(@NotNull SlashCommandInteractionEvent event)
     {
-        if (Cache.getBotOwnerId() != event.getUser().getIdLong())
+        if(Cache.getBotOwnerId() != event.getUser().getIdLong())
         {
             event.reply("Sorry, only the bot owner can run this command!").setEphemeral(true).queue();
-        } else
+        } else {
-        {
             event.reply("Going to sleep! Cya ✨").queue();
-            try (ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor())
+            Executors.newSingleThreadScheduledExecutor().schedule(HidekoBot::shutdown, 3, TimeUnit.SECONDS);
-            {
-                executor.schedule(HidekoBot::shutdown, 3, TimeUnit.SECONDS);
-            }
         }
     }
 }

src/main/java/wtf/beatrice/hidekobot/commands/slash/InviteCommand.java

@@ -30,7 +30,7 @@ public class InviteCommand extends SlashCommandImpl
         ReplyCallbackAction replyCallbackAction = event.deferReply();
 
         // only make message permanent in DMs
-        if (event.getChannelType() != ChannelType.PRIVATE)
+        if(event.getChannelType() != ChannelType.PRIVATE)
         {
             replyCallbackAction = replyCallbackAction.setEphemeral(true);
         }

src/main/java/wtf/beatrice/hidekobot/commands/slash/KickCommand.java

@@ -1,15 +1,25 @@
 package wtf.beatrice.hidekobot.commands.slash;
 
 import net.dv8tion.jda.api.Permission;
+import net.dv8tion.jda.api.entities.IMentionable;
+import net.dv8tion.jda.api.entities.User;
+import net.dv8tion.jda.api.entities.channel.middleman.MessageChannel;
 import net.dv8tion.jda.api.events.interaction.command.SlashCommandInteractionEvent;
 import net.dv8tion.jda.api.interactions.commands.DefaultMemberPermissions;
+import net.dv8tion.jda.api.interactions.commands.OptionMapping;
 import net.dv8tion.jda.api.interactions.commands.OptionType;
 import net.dv8tion.jda.api.interactions.commands.build.CommandData;
 import net.dv8tion.jda.api.interactions.commands.build.Commands;
 import org.jetbrains.annotations.NotNull;
+import wtf.beatrice.hidekobot.commands.base.Say;
 import wtf.beatrice.hidekobot.commands.base.UserPunishment;
+import wtf.beatrice.hidekobot.objects.MessageResponse;
 import wtf.beatrice.hidekobot.objects.commands.SlashCommandImpl;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.List;
+
 public class KickCommand extends SlashCommandImpl
 {
     @Override

src/main/java/wtf/beatrice/hidekobot/commands/slash/LoveCalculatorCommand.java

@@ -37,11 +37,10 @@ public class LoveCalculatorCommand extends SlashCommandImpl
         User firstUser, secondUser;
 
         OptionMapping firsUserArg = event.getOption("first");
-        if (firsUserArg != null)
+        if(firsUserArg != null)
         {
             firstUser = firsUserArg.getAsUser(); //todo null check?
-        } else
+        } else {
-        {
             event.reply("\uD83D\uDE22 I need to know who to check! Please mention them.")
                     .setEphemeral(true)
                     .queue();
@@ -49,11 +48,10 @@ public class LoveCalculatorCommand extends SlashCommandImpl
         }
 
         OptionMapping secondUserArg = event.getOption("second");
-        if (secondUserArg != null)
+        if(secondUserArg != null)
         {
             secondUser = secondUserArg.getAsUser(); //todo null check?
-        } else
+        } else {
-        {
             secondUser = event.getUser();
         }
 

src/main/java/wtf/beatrice/hidekobot/commands/slash/MagicBallCommand.java

@@ -30,12 +30,12 @@ public class MagicBallCommand extends SlashCommandImpl
         // get the asked question
         OptionMapping textOption = event.getOption("question");
         String question = "";
-        if (textOption != null)
+        if(textOption != null)
         {
             question = textOption.getAsString();
         }
 
-        if (textOption == null || question.isEmpty())
+        if(textOption == null || question.isEmpty())
         {
             event.reply("\uD83D\uDE20 Hey, you have to ask me a question!")
                     .setEphemeral(true)

src/main/java/wtf/beatrice/hidekobot/commands/slash/SayCommand.java

@@ -33,12 +33,12 @@ public class SayCommand extends SlashCommandImpl
         // get the text to send
         OptionMapping textOption = event.getOption("text");
         String messageContent = "";
-        if (textOption != null)
+        if(textOption != null)
         {
              messageContent = textOption.getAsString();
         }
 
-        if (textOption == null || messageContent.isEmpty())
+        if(textOption == null || messageContent.isEmpty())
         {
             event.reply("\uD83D\uDE20 Hey, you have to tell me what to say!")
                     .setEphemeral(true)

src/main/java/wtf/beatrice/hidekobot/commands/slash/TriviaCommand.java

@@ -26,13 +26,13 @@ public class TriviaCommand extends SlashCommandImpl
     {
         MessageChannel channel = event.getChannel();
 
-        if (!(channel instanceof TextChannel))
+        if(!(channel instanceof TextChannel))
         {
             event.reply(Trivia.getNoDMsError()).queue();
             return;
         }
 
-        if (Trivia.channelsRunningTrivia.contains(channel.getId()))
+        if(Trivia.channelsRunningTrivia.contains(channel.getId()))
         {
             event.reply(Trivia.getTriviaAlreadyRunningError()).setEphemeral(true).queue();
             return;

src/main/java/wtf/beatrice/hidekobot/commands/slash/UrbanDictionaryCommand.java

@@ -36,12 +36,12 @@ public class UrbanDictionaryCommand extends SlashCommandImpl
         // get the term to look up
         OptionMapping textOption = event.getOption("term");
         String term = "";
-        if (textOption != null)
+        if(textOption != null)
         {
             term = textOption.getAsString();
         }
 
-        if (textOption == null || term.isEmpty())
+        if(textOption == null || term.isEmpty())
         {
             event.reply(UrbanDictionary.getNoArgsError())
                     .setEphemeral(true)
@@ -54,11 +54,9 @@ public class UrbanDictionaryCommand extends SlashCommandImpl
 
         Document doc;
 
-        try
+        try {
-        {
             doc = Jsoup.connect(url).get();
-        } catch (IOException e)
+        } catch (IOException e) {
-        {
             event.reply(UrbanDictionary.getTermNotFoundError())
                     .setEphemeral(true)
                     .queue();
@@ -71,7 +69,7 @@ public class UrbanDictionaryCommand extends SlashCommandImpl
 
         // disable next page if we only have one result
         Button nextPageBtnLocal = UrbanDictionary.getNextPageButton();
-        if (search.getPages() == 1) nextPageBtnLocal = nextPageBtnLocal.asDisabled();
+        if(search.getPages() == 1) nextPageBtnLocal = nextPageBtnLocal.asDisabled();
 
         ActionRow actionRow = ActionRow.of(UrbanDictionary.getPreviousPageButton().asDisabled(),
                 //disabled by default because we're on page 0

src/main/java/wtf/beatrice/hidekobot/datasources/ConfigurationEntry.java

@@ -7,27 +7,18 @@ public enum ConfigurationEntry
     BOT_OWNER_ID("bot-owner-id", 100000000000000000L),
     BOT_COLOR("bot-color", "PINK"),
     HEARTBEAT_LINK("heartbeat-link", "https://your-heartbeat-api.com/api/push/apikey?status=up&msg=OK&ping="),
-    RANDOM_ORG_API_KEY("random-org-api-key", "00000000-0000-0000-0000-000000000000"),
 
     ;
 
 
     private String path;
     private Object defaultValue;
-
     ConfigurationEntry(String path, Object defaultValue)
     {
         this.path = path;
         this.defaultValue = defaultValue;
     }
 
-    public String getPath()
+    public String getPath() { return path; }
-    {
+    public Object getDefaultValue() { return defaultValue; }
-        return path;
-    }
-
-    public Object getDefaultValue()
-    {
-        return defaultValue;
-    }
 }

src/main/java/wtf/beatrice/hidekobot/datasources/ConfigurationSource.java

@@ -1,26 +1,26 @@
 package wtf.beatrice.hidekobot.datasources;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import org.yaml.snakeyaml.DumperOptions;
-import org.yaml.snakeyaml.LoaderOptions;
 import org.yaml.snakeyaml.Yaml;
 import org.yaml.snakeyaml.constructor.SafeConstructor;
 import wtf.beatrice.hidekobot.HidekoBot;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.io.*;
 import java.util.LinkedHashMap;
-import java.util.Map;
 
 public class ConfigurationSource
 {
+
+
     private final LinkedHashMap<String, Object> configurationEntries = new LinkedHashMap<>();
-    private static final Logger LOGGER = LoggerFactory.getLogger(ConfigurationSource.class);
+    private final Logger logger;
     private final String configFilePath;
 
     public ConfigurationSource(String configFilePath)
     {
         this.configFilePath = configFilePath;
+        logger = new Logger(getClass());
     }
 
     public void initConfig()
@@ -32,52 +32,40 @@ public class ConfigurationSource
          * we used to have a config.yml file in the "resources" folder, but that is no longer necessary.
          */
         LinkedHashMap<String, Object> internalConfigContents = new LinkedHashMap<>(); // map holding all file entries
-        for (ConfigurationEntry entry : ConfigurationEntry.values())
+        for(ConfigurationEntry entry : ConfigurationEntry.values())
         {
             internalConfigContents.put(entry.getPath(), entry.getDefaultValue());
         }
 
-        if (internalConfigContents.isEmpty())
+        if(internalConfigContents.isEmpty())
         {
-            LOGGER.error("Error reading internal configuration!");
+            logger.log("Error reading internal configuration!");
             HidekoBot.shutdown();
             return;
         }
 
         // check if config files exists in filesystem
         File fsConfigFile = new File(configFilePath);
-        if (!fsConfigFile.exists())
+        if(!fsConfigFile.exists())
         {
             // try to create config file
-            try
+            try { fsConfigFile.createNewFile(); }
-            {
+            catch (IOException e) {
-                if (!fsConfigFile.createNewFile())
+                logger.log("Error creating configuration file!");
-                {
+                logger.log(e.getMessage());
-                    LOGGER.error("We tried creating a file that already exists!");
-                    HidekoBot.shutdown();
-                    return;
-                }
-            } catch (IOException e)
-            {
-                LOGGER.error("Error creating configuration file!", e);
                 HidekoBot.shutdown();
                 return;
             }
         }
         // load the YAML file from the filesystem
-        LoaderOptions options = new LoaderOptions();
+        Yaml fsConfigYaml = new Yaml(new SafeConstructor());
-        Yaml fsConfigYaml = new Yaml(new SafeConstructor(options));
         LinkedHashMap<String, Object> fsConfigContents = null; // map holding all file entries
         try (InputStream fsConfigStream = new FileInputStream(fsConfigFile))
-        {
+        { fsConfigContents = fsConfigYaml.load(fsConfigStream); }
-            fsConfigContents = fsConfigYaml.load(fsConfigStream);
+        catch (IOException e) { logger.log(e.getMessage()); }
-        } catch (IOException e)
-        {
-            LOGGER.error(e.getMessage());
-        }
 
 
-        if (fsConfigContents == null) // if file contents are empty or corrupted...
+        if(fsConfigContents == null) // if file contents are empty or corrupted...
         {
             // "clean" them (this effectively forces a config file reset)
             fsConfigContents = new LinkedHashMap<>();
@@ -85,10 +73,10 @@ public class ConfigurationSource
 
         // check for missing keys
         boolean missingKeys = false;
-        for (String key : internalConfigContents.keySet())
+        for(String key : internalConfigContents.keySet())
         {
             // if key is missing
-            if (!fsConfigContents.containsKey(key))
+            if(!fsConfigContents.containsKey(key))
             {
                 // quit and flag it, as we need to complete the file with the missing ones
                 missingKeys = true;
@@ -97,29 +85,24 @@ public class ConfigurationSource
         }
 
         // if keys are missing
-        if (missingKeys)
+        if(missingKeys)
         {
             // create a new mixed map that will take existing values from the non-missing keys
             // and fill everything else with the default values
             LinkedHashMap<String, Object> filledEntries = new LinkedHashMap<>();
-
+            for(String key : internalConfigContents.keySet())
-            for (Map.Entry<String, Object> entry : internalConfigContents.entrySet())
             {
-                String key = entry.getKey();
+                if(fsConfigContents.containsKey(key))
-
-                if (fsConfigContents.containsKey(key))
                 {
                     // if the key already exists, copy the original value
                     filledEntries.put(key, fsConfigContents.get(key));
-                } else
+                } else {
-                {
                     // else, copy the value from the example config file
-                    filledEntries.put(key, entry.getValue());
+                    filledEntries.put(key, internalConfigContents.get(key));
                 }
             }
 
-            try
+            try {
-            {
                 // new writer to actually write the contents to the file
                 PrintWriter missingKeysWriter = new PrintWriter(fsConfigFile);
 
@@ -132,17 +115,15 @@ public class ConfigurationSource
                 // create the yaml object and dump the values to filesystem
                 Yaml yaml = new Yaml(dumperOptions);
                 yaml.dump(filledEntries, missingKeysWriter);
-            } catch (FileNotFoundException e)
+            } catch (FileNotFoundException e) {
-            {
+                logger.log(e.getMessage());
-                LOGGER.error(e.getMessage());
                 HidekoBot.shutdown();
                 return;
             }
 
             // finally, dump all entries to cache.
             loadConfig(filledEntries);
-        } else
+        } else {
-        {
             // if no key is missing, just cache all entries and values from filesystem.
             loadConfig(fsConfigContents);
         }
@@ -153,7 +134,6 @@ public class ConfigurationSource
     {
         this.configurationEntries.putAll(configurationEntries);
     }
-
     public Object getConfigValue(ConfigurationEntry key)
     {
         return configurationEntries.get(key.getPath());

src/main/java/wtf/beatrice/hidekobot/datasources/DatabaseSource.java

@@ -3,8 +3,8 @@ package wtf.beatrice.hidekobot.datasources;
 import net.dv8tion.jda.api.entities.Message;
 import net.dv8tion.jda.api.entities.User;
 import net.dv8tion.jda.api.entities.channel.ChannelType;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.Cache;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.sql.*;
 import java.time.LocalDateTime;
@@ -15,35 +15,29 @@ import java.util.List;
 public class DatabaseSource
 {
 
-    private static final org.slf4j.Logger LOGGER = LoggerFactory.getLogger(DatabaseSource.class);
+    private final static String sqliteURL = "jdbc:sqlite:%path%";
-    private static final String JDBC_URL = "jdbc:sqlite:%path%";
     private Connection dbConnection = null;
     private final String dbPath;
+    private final Logger logger;
 
     public DatabaseSource(String dbPath)
     {
         this.dbPath = dbPath;
-    }
+        this.logger = new Logger(getClass());
-
-    private void logException(SQLException e)
-    {
-        LOGGER.error("Database Exception", e);
     }
 
     public boolean connect()
     {
-        String url = JDBC_URL.replace("%path%", dbPath);
+        String url = sqliteURL.replace("%path%", dbPath);
 
-        if (!close()) return false;
+        if(!close()) return false;
 
-        try
+        try {
-        {
             dbConnection = DriverManager.getConnection(url);
-            LOGGER.info("Database connection established!");
+            logger.log("Database connection established!");
             return true;
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
             return false;
         }
 
@@ -54,15 +48,13 @@ public class DatabaseSource
     {
         if (dbConnection != null)
         {
-            try
+            try {
-            {
+                if(!dbConnection.isClosed())
-                if (!dbConnection.isClosed())
                 {
                     dbConnection.close();
                 }
-            } catch (SQLException e)
+            } catch (SQLException e) {
-            {
+                e.printStackTrace();
-                logException(e);
                 return false;
             }
 
@@ -98,48 +90,43 @@ public class DatabaseSource
      */
 
     //todo: javadocs
+
     public boolean initDb()
     {
         List<String> newTables = new ArrayList<>();
 
-        newTables.add("""
+        newTables.add("CREATE TABLE IF NOT EXISTS pending_disabled_messages (" +
-                CREATE TABLE IF NOT EXISTS pending_disabled_messages (
+                "guild_id TEXT NOT NULL, " +
-                guild_id TEXT NOT NULL,
+                "channel_id TEXT NOT NULL," +
-                channel_id TEXT NOT NULL,
+                "message_id TEXT NOT NULL," +
-                message_id TEXT NOT NULL,
+                "expiry_timestamp TEXT NOT NULL " +
-                expiry_timestamp TEXT NOT NULL);
+                ");");
-                """);
 
-        newTables.add("""
+        newTables.add("CREATE TABLE IF NOT EXISTS command_runners (" +
-                CREATE TABLE IF NOT EXISTS command_runners (
+                "guild_id TEXT NOT NULL, " +
-                guild_id TEXT NOT NULL,
+                "channel_id TEXT NOT NULL," + // channel the command was run in
-                channel_id TEXT NOT NULL,
+                "message_id TEXT NOT NULL," + // message id of the bot's response
-                message_id TEXT NOT NULL,
+                "user_id TEXT NOT NULL, " + // user who ran the command
-                user_id TEXT NOT NULL,
+                "channel_type TEXT NOT NULL" + // channel type (PRIVATE, FORUM, ...)
-                channel_type TEXT NOT NULL);
+                ");");
-                """);
 
-        newTables.add("""
+        newTables.add("CREATE TABLE IF NOT EXISTS urban_dictionary (" +
-                CREATE TABLE IF NOT EXISTS urban_dictionary (
+                "message_id TEXT NOT NULL, " + // message id of the bot's response
-                message_id TEXT NOT NULL,
+                "page INTEGER NOT NULL," + // page that we are currently on
-                page INTEGER NOT NULL,
+                "meanings TEXT NOT NULL," + // list of all meanings, serialized and encoded
-                meanings TEXT NOT NULL,
+                "examples TEXT NOT NULL, " + // list of all examples, serialized and encoded
-                examples TEXT NOT NULL,
+                "contributors TEXT NOT NULL, " + // list of all contributors, serialized and encoded
-                contributors TEXT NOT NULL,
+                "dates TEXT NOT NULL, " + // list of all submission dates, serialized and encoded
-                dates TEXT NOT NULL,
+                "term TEXT NOT NULL" + // the term that was searched
-                term TEXT NOT NULL
+                ");");
-                );
-                """);
 
-        for (String sql : newTables)
+        for(String sql : newTables)
-        {
-            try (Statement stmt = dbConnection.createStatement())
         {
+            try (Statement stmt = dbConnection.createStatement()) {
                 // execute the statement
                 stmt.execute(sql);
-            } catch (SQLException e)
+            } catch (SQLException e) {
-            {
+                e.printStackTrace();
-                logException(e);
                 return false;
             }
         }
@@ -153,11 +140,10 @@ public class DatabaseSource
         String guildId;
 
         ChannelType channelType = message.getChannelType();
-        if (!(channelType.isGuild()))
+        if(!(channelType.isGuild()))
         {
             guildId = userId;
-        } else
+        } else {
-        {
             guildId = message.getGuild().getId();
         }
 
@@ -165,13 +151,11 @@ public class DatabaseSource
         String messageId = message.getId();
 
 
-        String query = """
+        String query = "INSERT INTO command_runners " +
-                INSERT INTO command_runners
+                "(guild_id, channel_id, message_id, user_id, channel_type) VALUES " +
-                (guild_id, channel_id, message_id, user_id, channel_type) VALUES
+                " (?, ?, ?, ?, ?);";
-                 (?, ?, ?, ?, ?);
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, guildId);
             preparedStatement.setString(2, channelId);
@@ -184,7 +168,7 @@ public class DatabaseSource
             return true;
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
         }
 
         return false;
@@ -193,32 +177,29 @@ public class DatabaseSource
     public boolean isUserTrackedFor(String userId, String messageId)
     {
         String trackedUserId = getTrackedReplyUserId(messageId);
-        if (trackedUserId == null) return false;
+        if(trackedUserId == null) return false;
         return userId.equals(trackedUserId);
     }
 
     public ChannelType getTrackedMessageChannelType(String messageId)
     {
-        String query = """
+        String query = "SELECT channel_type " +
-                SELECT channel_type
+                "FROM command_runners " +
-                FROM command_runners
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 String channelTypeName = resultSet.getString("channel_type");
                 return ChannelType.valueOf(channelTypeName);
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -227,25 +208,22 @@ public class DatabaseSource
 
     public String getTrackedReplyUserId(String messageId)
     {
-        String query = """
+        String query = "SELECT user_id " +
-                SELECT user_id
+                "FROM command_runners " +
-                FROM command_runners
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("user_id");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -258,11 +236,10 @@ public class DatabaseSource
         String guildId;
 
         ChannelType channelType = message.getChannelType();
-        if (!(channelType.isGuild()))
+        if(!(channelType.isGuild()))
         {
             guildId = "PRIVATE";
-        } else
+        } else {
-        {
             guildId = message.getGuild().getId();
         }
 
@@ -271,13 +248,11 @@ public class DatabaseSource
         DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofPattern(Cache.getExpiryTimestampFormat());
         String expiryTimeFormatted = dateTimeFormatter.format(expiryTime);
 
-        String query = """
+        String query = "INSERT INTO pending_disabled_messages " +
-                INSERT INTO pending_disabled_messages
+                "(guild_id, channel_id, message_id, expiry_timestamp) VALUES " +
-                (guild_id, channel_id, message_id, expiry_timestamp) VALUES
+                " (?, ?, ?, ?);";
-                 (?, ?, ?, ?);
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, guildId);
             preparedStatement.setString(2, channelId);
@@ -289,7 +264,7 @@ public class DatabaseSource
             return true;
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
         }
 
         return false;
@@ -299,22 +274,19 @@ public class DatabaseSource
     {
         List<String> messages = new ArrayList<>();
 
-        String query = """
+        String query = "SELECT message_id " +
-                SELECT message_id
+                "FROM pending_disabled_messages ";
-                FROM pending_disabled_messages;
-                """;
 
         try (Statement statement = dbConnection.createStatement())
         {
             ResultSet resultSet = statement.executeQuery(query);
-            if (resultSet.isClosed()) return messages;
+            if(resultSet.isClosed()) return messages;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 messages.add(resultSet.getString("message_id"));
             }
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return messages;
@@ -324,35 +296,35 @@ public class DatabaseSource
     {
         String query = "DELETE FROM pending_disabled_messages WHERE message_id = ?;";
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             preparedStatement.execute();
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
             return false;
         }
 
         query = "DELETE FROM command_runners WHERE message_id = ?;";
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             preparedStatement.execute();
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
             return false;
         }
 
         query = "DELETE FROM urban_dictionary WHERE message_id = ?;";
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             preparedStatement.execute();
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
             return false;
         }
 
@@ -361,25 +333,22 @@ public class DatabaseSource
 
     public String getQueuedExpiringMessageExpiryDate(String messageId)
     {
-        String query = """
+        String query = "SELECT expiry_timestamp " +
-                SELECT expiry_timestamp
+                "FROM pending_disabled_messages " +
-                FROM pending_disabled_messages
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("expiry_timestamp");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -387,25 +356,22 @@ public class DatabaseSource
 
     public String getQueuedExpiringMessageChannel(String messageId)
     {
-        String query = """
+        String query = "SELECT channel_id " +
-                SELECT channel_id
+                "FROM pending_disabled_messages " +
-                FROM pending_disabled_messages
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("channel_id");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -413,25 +379,22 @@ public class DatabaseSource
 
     public String getQueuedExpiringMessageGuild(String messageId)
     {
-        String query = """
+        String query = "SELECT guild_id " +
-                SELECT guild_id
+                "FROM pending_disabled_messages " +
-                FROM pending_disabled_messages
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("guild_id");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -442,13 +405,11 @@ public class DatabaseSource
                               Message message, String term)
     {
 
-        String query = """
+        String query = "INSERT INTO urban_dictionary " +
-                INSERT INTO urban_dictionary
+                "(message_id, page, meanings, examples, contributors, dates, term) VALUES " +
-                (message_id, page, meanings, examples, contributors, dates, term) VALUES
+                " (?, ?, ?, ?, ?, ?, ?);";
-                 (?, ?, ?, ?, ?, ?, ?);
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, message.getId());
             preparedStatement.setInt(2, 0);
@@ -463,7 +424,7 @@ public class DatabaseSource
             return true;
         } catch (SQLException e)
         {
-            logException(e);
+            e.printStackTrace();
         }
 
         return false;
@@ -471,25 +432,22 @@ public class DatabaseSource
 
     public int getUrbanPage(String messageId)
     {
-        String query = """
+        String query = "SELECT page " +
-                SELECT page
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return 0;
+            if(resultSet.isClosed()) return 0;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getInt("page");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return 0;
@@ -497,25 +455,22 @@ public class DatabaseSource
 
     public String getUrbanMeanings(String messageId)
     {
-        String query = """
+        String query = "SELECT meanings " +
-                SELECT meanings
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("meanings");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -523,25 +478,22 @@ public class DatabaseSource
 
     public String getUrbanExamples(String messageId)
     {
-        String query = """
+        String query = "SELECT examples " +
-                SELECT examples
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("examples");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -549,25 +501,22 @@ public class DatabaseSource
 
     public String getUrbanContributors(String messageId)
     {
-        String query = """
+        String query = "SELECT contributors " +
-                SELECT contributors
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("contributors");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -575,25 +524,22 @@ public class DatabaseSource
 
     public String getUrbanDates(String messageId)
     {
-        String query = """
+        String query = "SELECT dates " +
-                SELECT dates
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("dates");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -601,25 +547,22 @@ public class DatabaseSource
 
     public String getUrbanTerm(String messageId)
     {
-        String query = """
+        String query = "SELECT term " +
-                SELECT term
+                "FROM urban_dictionary " +
-                FROM urban_dictionary
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, messageId);
             ResultSet resultSet = preparedStatement.executeQuery();
-            if (resultSet.isClosed()) return null;
+            if(resultSet.isClosed()) return null;
-            while (resultSet.next())
+            while(resultSet.next())
             {
                 return resultSet.getString("term");
             }
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return null;
@@ -627,13 +570,11 @@ public class DatabaseSource
 
     public boolean setUrbanPage(String messageId, int page)
     {
-        String query = """
+        String query = "UPDATE urban_dictionary " +
-                UPDATE urban_dictionary
+                "SET page = ? " +
-                SET page = ? 
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setInt(1, page);
             preparedStatement.setString(2, messageId);
@@ -641,9 +582,8 @@ public class DatabaseSource
 
             return true;
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return false;
@@ -656,13 +596,11 @@ public class DatabaseSource
         DateTimeFormatter dateTimeFormatter = DateTimeFormatter.ofPattern(Cache.getExpiryTimestampFormat());
         String expiryTimeFormatted = dateTimeFormatter.format(expiryTime);
 
-        String query = """
+        String query = "UPDATE pending_disabled_messages " +
-                UPDATE pending_disabled_messages
+                "SET expiry_timestamp = ? " +
-                SET expiry_timestamp = ? 
+                "WHERE message_id = ?;";
-                WHERE message_id = ?;
-                """;
 
-        try (PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
+        try(PreparedStatement preparedStatement = dbConnection.prepareStatement(query))
         {
             preparedStatement.setString(1, expiryTimeFormatted);
             preparedStatement.setString(2, messageId);
@@ -670,9 +608,8 @@ public class DatabaseSource
 
             return true;
 
-        } catch (SQLException e)
+        } catch (SQLException e) {
-        {
+            e.printStackTrace();
-            logException(e);
         }
 
         return false;

src/main/java/wtf/beatrice/hidekobot/datasources/PropertiesSource.java

@@ -1,8 +1,7 @@
 package wtf.beatrice.hidekobot.datasources;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.HidekoBot;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.io.IOException;
 import java.io.InputStream;
@@ -13,7 +12,7 @@ public class PropertiesSource
 
     private Properties properties = null;
     private final String fileName = "default.properties";
-    private static final Logger LOGGER = LoggerFactory.getLogger(PropertiesSource.class);
+    private final Logger logger = new Logger(getClass());
 
     public void load()
     {
@@ -25,16 +24,14 @@ public class PropertiesSource
         {
             properties.load(internalPropertiesStream);
 
-        } catch (IOException e)
+        }
-        {
+        catch (IOException e) {
-            LOGGER.error(e.getMessage());
+            logger.log(e.getMessage());
             HidekoBot.shutdown();
             return;
         }
     }
 
     public String getProperty(String property)
-    {
+    { return properties == null ? "" : properties.getProperty(property); }
-        return properties == null ? "" : properties.getProperty(property);
-    }
 }

src/main/java/wtf/beatrice/hidekobot/listeners/ButtonInteractionListener.java

@@ -2,8 +2,6 @@ package wtf.beatrice.hidekobot.listeners;
 
 import net.dv8tion.jda.api.events.interaction.component.ButtonInteractionEvent;
 import net.dv8tion.jda.api.hooks.ListenerAdapter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.commands.base.CoinFlip;
 import wtf.beatrice.hidekobot.commands.base.Trivia;
 import wtf.beatrice.hidekobot.commands.base.UrbanDictionary;
@@ -12,14 +10,11 @@ import wtf.beatrice.hidekobot.util.CommandUtil;
 public class ButtonInteractionListener extends ListenerAdapter
 {
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(ButtonInteractionListener.class);
-
     @Override
     public void onButtonInteraction(ButtonInteractionEvent event)
     {
 
-        switch (event.getComponentId().toLowerCase())
+        switch (event.getComponentId().toLowerCase()) {
-        {
 
             // coinflip
             case "coinflip_reflip" -> CoinFlip.buttonReFlip(event);
@@ -36,10 +31,6 @@ public class ButtonInteractionListener extends ListenerAdapter
             case "trivia_wrong_1", "trivia_wrong_2", "trivia_wrong_3" ->
                     Trivia.handleAnswer(event, Trivia.AnswerType.WRONG);
 
-            // error handling
-            default -> LOGGER.warn("Received unhandled {}", event.getClass().getSimpleName());
-
-
         }
 
     }

src/main/java/wtf/beatrice/hidekobot/listeners/MessageCommandListener.java

@@ -11,6 +11,7 @@ import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.objects.commands.CommandCategory;
 import wtf.beatrice.hidekobot.objects.commands.MessageCommand;
 import wtf.beatrice.hidekobot.objects.comparators.MessageCommandAliasesComparator;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.util.*;
 
@@ -19,13 +20,13 @@ public class MessageCommandListener extends ListenerAdapter
 
     // map storing command labels and command object alphabetically.
     private final TreeMap<LinkedList<String>, MessageCommand> registeredCommands =
-            new TreeMap<>(new MessageCommandAliasesComparator());
+            new TreeMap<LinkedList<String>, MessageCommand>(new MessageCommandAliasesComparator());
 
     // map commands and their categories.
     // this is not strictly needed but it's better to have it so we avoid looping every time we need to check the cat.
     LinkedHashMap<CommandCategory, LinkedList<MessageCommand>> commandCategories = new LinkedHashMap<>();
 
-    private static final String COMMAND_REGEX = "(?i)^(hideko|hde)\\b";
+    private final String commandRegex = "(?i)^(hideko|hde)\\b";
     // (?i) -> case insensitive flag
     // ^ -> start of string (not in middle of a sentence)
     // \b -> the word has to end here
@@ -38,16 +39,12 @@ public class MessageCommandListener extends ListenerAdapter
 
     public MessageCommand getRegisteredCommand(String label)
     {
-        for (Map.Entry<LinkedList<String>, MessageCommand> entry : registeredCommands.entrySet())
+        for(LinkedList<String> aliases : registeredCommands.keySet())
         {
-            LinkedList<String> aliases = entry.getKey();
+            for(String currentAlias : aliases)
-
-            for (String currentAlias : aliases)
             {
-                if (label.equals(currentAlias))
+                if(label.equals(currentAlias))
-                {
+                { return registeredCommands.get(aliases); }
-                    return entry.getValue();
-                }
             }
         }
 
@@ -55,25 +52,26 @@ public class MessageCommandListener extends ListenerAdapter
     }
 
     public LinkedList<MessageCommand> getRegisteredCommands()
-    {
+    { return new LinkedList<>(registeredCommands.values()); }
-        return new LinkedList<>(registeredCommands.values());
+
-    }
+
+    private final Logger logger = new Logger(MessageCommandListener.class);
 
     @Override
     public void onMessageReceived(@NotNull MessageReceivedEvent event)
     {
         // check if a bot is sending this message, and ignore it
-        if (event.getAuthor().isBot()) return;
+        if(event.getAuthor().isBot()) return;
 
         // warning: we are getting the RAW value of the message content, not the DISPLAY value!
         String eventMessage = event.getMessage().getContentRaw();
 
         // check if the sent message matches the bot activation regex (prefix, name, ...)
-        if (!eventMessage.toLowerCase().matches("(?s)" + COMMAND_REGEX + ".*"))
+        if(!eventMessage.toLowerCase().matches(commandRegex + "((.|\\n)*)"))
             return;
 
         // generate args from the string
-        String argsString = eventMessage.replaceAll(COMMAND_REGEX + "\\s*", "");
+        String argsString = eventMessage.replaceAll(commandRegex + "\\s*", "");
 
 
         // if no args were specified apart from the bot prefix
@@ -81,7 +79,7 @@ public class MessageCommandListener extends ListenerAdapter
         // and that element is the whole string passed as a single argument, which would be empty in this case
         // (or contain text in other cases like "string split ," if the passed text doesn't contain any comma ->
         // it will be the whole text as a single element.
-        if (argsString.isEmpty())
+        if(argsString.isEmpty())
         {
             event.getMessage()
                     .reply("Hello there! ✨ Type `" + Cache.getBotPrefix() + " help` to get started!")
@@ -96,7 +94,7 @@ public class MessageCommandListener extends ListenerAdapter
         String commandLabel = argsRaw[0];
         MessageCommand commandObject = getRegisteredCommand(commandLabel);
 
-        if (commandObject == null)
+        if(commandObject == null)
         {
             /* temporarily disabled because when people talk about the bot, it replies with this spammy message.
 
@@ -111,33 +109,37 @@ public class MessageCommandListener extends ListenerAdapter
 
         // permissions check
         List<Permission> requiredPermissions = commandObject.getPermissions();
-        if (requiredPermissions != null && !requiredPermissions.isEmpty())
+        if(requiredPermissions != null && !requiredPermissions.isEmpty())
         {
-            if (channelType.isGuild()) //todo: what about forum post
+            if(channelType.isGuild()) //todo: what about forum post
             {
                 Member member = event.getMember();
                 GuildChannel channel = event.getGuildChannel(); //todo: what about forum post
-                if (member != null && !member.hasPermission(channel, requiredPermissions))
+                if(member != null)
+                {
+                    if(!member.hasPermission(channel, requiredPermissions))
                     {
                         event.getMessage()
                                 .reply("You do not have permissions to run this command!")
                                 .queue(); // todo prettier
                         // todo: queue message deletion in 15 seconds or so
                         return;
+                    }
 
                 }
             }
         }
 
         String[] commandArgs;
-        if (commandObject.passRawArgs())
+        if(commandObject.passRawArgs())
         {
 
             // remove first argument, which is the command label
             argsString = argsString.replaceAll("^[\\S]+\\s*", "");
             // pass all other arguments as a single argument as the first array element
             commandArgs = new String[]{argsString};
-        } else
+        }
+        else
         {
             // copy all split arguments to the array, except from the command label
             commandArgs = Arrays.copyOfRange(argsRaw, 1, argsRaw.length);

src/main/java/wtf/beatrice/hidekobot/listeners/MessageLogger.java

@@ -6,17 +6,16 @@ import net.dv8tion.jda.api.entities.channel.concrete.TextChannel;
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent;
 import net.dv8tion.jda.api.hooks.ListenerAdapter;
 import org.jetbrains.annotations.NotNull;
-import org.slf4j.Logger;
+import wtf.beatrice.hidekobot.util.Logger;
-import org.slf4j.LoggerFactory;
 
 public class MessageLogger extends ListenerAdapter
 {
     // this class only gets loaded as a listener if verbosity is set to true on startup.
 
-    private static final String GUILD_MESSAGE_LOG_FORMAT = "[%guild%] [#%channel%] %user%: %message%";
+    private final static String guildChannelFormat = "[%guild%] [#%channel%] %user%: %message%";
-    private static final String DIRECT_MESSAGE_LOG_FORMAT = "[DM] %user%: %message%";
+    private final static String dmFormat = "[DM] %user%: %message%";
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(MessageLogger.class);
+    private final Logger logger = new Logger(MessageLogger.class);
 
     @Override
     public void onMessageReceived(@NotNull MessageReceivedEvent event)
@@ -25,30 +24,31 @@ public class MessageLogger extends ListenerAdapter
         String userName = event.getAuthor().getAsTag();
         String message = event.getMessage().getContentDisplay();
 
-        if (event.getChannel() instanceof TextChannel channel)
+        if(event.getChannel() instanceof TextChannel)
         {
-            String guildName = channel.getGuild().getName();
+            String guildName = ((TextChannel) event.getChannel()).getGuild().getName();
             String channelName = event.getChannel().getName();
 
-            toLog = GUILD_MESSAGE_LOG_FORMAT
+            toLog = guildChannelFormat
                     .replace("%guild%", guildName)
                     .replace("%channel%", channelName);
-        } else if (event.getChannel() instanceof PrivateChannel)
+        }
+        else if(event.getChannel() instanceof PrivateChannel)
         {
-            toLog = DIRECT_MESSAGE_LOG_FORMAT;
+            toLog = dmFormat;
         }
 
         toLog = toLog
                 .replace("%user%", userName)
                 .replace("%message%", message);
 
-        LOGGER.info(toLog);
+        logger.log(toLog);
 
-        if (!event.getMessage().getAttachments().isEmpty())
+        if(!event.getMessage().getAttachments().isEmpty())
         {
-            for (Message.Attachment atch : event.getMessage().getAttachments())
+            for(Message.Attachment atch : event.getMessage().getAttachments())
             {
-                LOGGER.info(atch.getUrl());
+                logger.log(atch.getUrl());
             }
         }
     }

src/main/java/wtf/beatrice/hidekobot/listeners/SelectMenuInteractionListener.java

@@ -2,26 +2,18 @@ package wtf.beatrice.hidekobot.listeners;
 
 import net.dv8tion.jda.api.events.interaction.component.StringSelectInteractionEvent;
 import net.dv8tion.jda.api.hooks.ListenerAdapter;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.commands.base.Trivia;
 
 public class SelectMenuInteractionListener extends ListenerAdapter
 {
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(SelectMenuInteractionListener.class);
-
     @Override
     public void onStringSelectInteraction(StringSelectInteractionEvent event)
     {
-        switch (event.getComponentId().toLowerCase())
+        switch (event.getComponentId().toLowerCase()) {
-        {
 
             // trivia
             case "trivia_categories" -> Trivia.handleMenuSelection(event);
-
-            // error handling
-            default -> LOGGER.warn("Received unhandled {}", event.getClass().getSimpleName());
         }
     }
 }

src/main/java/wtf/beatrice/hidekobot/listeners/SlashCommandCompletionListener.java

@@ -21,21 +21,16 @@ public class SlashCommandCompletionListener extends ListenerAdapter
     }
 
     public SlashArgumentsCompleter getRegisteredCompleter(String label)
-    {
+    { return registeredCompleters.get(label); }
-        return registeredCompleters.get(label);
-    }
 
     public LinkedList<SlashArgumentsCompleter> getRegisteredCompleters()
-    {
+    { return new LinkedList<>(registeredCompleters.values()); }
-        return new LinkedList<>(registeredCompleters.values());
-    }
-
     @Override
     public void onCommandAutoCompleteInteraction(CommandAutoCompleteInteractionEvent event)
     {
         String commandName = event.getName().toLowerCase();
         SlashArgumentsCompleter completer = registeredCompleters.get(commandName);
-        if (completer == null) return;
+        if(completer == null) return;
 
         // not running in a thread because nothing heavy should be done here...
         completer.runCompletion(event);

src/main/java/wtf/beatrice/hidekobot/listeners/SlashCommandListener.java

@@ -21,21 +21,17 @@ public class SlashCommandListener extends ListenerAdapter
     }
 
     public SlashCommand getRegisteredCommand(String label)
-    {
+    { return registeredCommands.get(label); }
-        return registeredCommands.get(label);
-    }
 
     public LinkedList<SlashCommand> getRegisteredCommands()
-    {
+    { return new LinkedList<>(registeredCommands.values()); }
-        return new LinkedList<>(registeredCommands.values());
-    }
 
     @Override
     public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent event)
     {
         String commandName = event.getName().toLowerCase();
         SlashCommand command = registeredCommands.get(commandName);
-        if (command == null) return;
+        if(command == null) return;
 
 
         // finally run the command, in a new thread to avoid locking the main one.

src/main/java/wtf/beatrice/hidekobot/objects/MessageResponse.java

@@ -4,41 +4,8 @@ import net.dv8tion.jda.api.entities.MessageEmbed;
 import net.dv8tion.jda.api.interactions.components.ItemComponent;
 import org.jetbrains.annotations.Nullable;
 
-import java.util.Arrays;
-import java.util.Objects;
-
 public record MessageResponse(@Nullable String content,
                               @Nullable MessageEmbed embed,
-                              @Nullable ItemComponent... components)
+                              @Nullable ItemComponent... components) {
-{
-
-    @Override
-    public boolean equals(Object o)
-    {
-        if (this == o) return true;
-        if (o == null || getClass() != o.getClass()) return false;
-        MessageResponse response = (MessageResponse) o;
-        return Objects.equals(content, response.content) &&
-                Objects.equals(embed, response.embed) &&
-                Arrays.equals(components, response.components);
-    }
-
-    @Override
-    public int hashCode()
-    {
-        int result = Objects.hash(content, embed);
-        result = 31 * result + Arrays.hashCode(components);
-        return result;
-    }
-
-    @Override
-    public String toString()
-    {
-        return "MessageResponse{" +
-                "content=" + content +
-                ", embed=" + embed +
-                ", components=" + Arrays.toString(components) +
-                '}';
-    }
 
 }

src/main/java/wtf/beatrice/hidekobot/objects/commands/CommandCategory.java

@@ -9,14 +9,10 @@ public enum CommandCategory
     ;
 
     private String emoji;
-
     CommandCategory(String emoji)
     {
         this.emoji = emoji;
     }
 
-    public String getEmoji()
+    public String getEmoji() { return emoji; }
-    {
-        return emoji;
-    }
 }

src/main/java/wtf/beatrice/hidekobot/objects/commands/MessageCommand.java

@@ -70,7 +70,9 @@ public interface MessageCommand
      *
      * @param event the received message event. It should not be used for parsing message contents data as
      *              the arguments already account for it in a better way.
+     *
      * @param label the command label that was used, taken from all available command aliases.
+     *
      * @param args a pre-formatted list of arguments, excluding the bot prefix and the command name.
      *             This is useful because command logic won't have to change in case the bot prefix is changed,
      *             removed, or we switch to another method of triggering commands (ping, trigger words, ...).

src/main/java/wtf/beatrice/hidekobot/objects/commands/SlashArgumentsCompleter.java

@@ -12,7 +12,6 @@ public interface SlashArgumentsCompleter
      * @return the command object.
      */
     SlashCommand getCommand();
-
     /**
      * Run the argument-completion logic by parsing the event and replying accordingly.
      *

src/main/java/wtf/beatrice/hidekobot/objects/commands/SlashArgumentsCompleterImpl.java

@@ -6,16 +6,13 @@ import org.jetbrains.annotations.NotNull;
 public class SlashArgumentsCompleterImpl implements SlashArgumentsCompleter
 {
     private final SlashCommand parentCommand;
-
     public SlashArgumentsCompleterImpl(SlashCommand parentCommand)
     {
         this.parentCommand = parentCommand;
     }
 
     public SlashCommand getCommand()
-    {
+    { return parentCommand; }
-        return parentCommand;
-    }
 
     public void runCompletion(@NotNull CommandAutoCompleteInteractionEvent event)
     {

src/main/java/wtf/beatrice/hidekobot/objects/commands/SlashCommand.java

@@ -21,7 +21,6 @@ public interface SlashCommand
      * @return the command data object.
      */
     CommandData getSlashCommandData();
-
     /**
      * Run the command logic by parsing the event and replying accordingly.
      *

src/main/java/wtf/beatrice/hidekobot/objects/commands/SlashCommandImpl.java

@@ -8,20 +8,17 @@ public class SlashCommandImpl implements SlashCommand
 {
 
     @Override
-    public String getCommandName()
+    public String getCommandName() {
-    {
         return getSlashCommandData().getName();
     }
 
     @Override
-    public CommandData getSlashCommandData()
+    public CommandData getSlashCommandData() {
-    {
         return null;
     }
 
     @Override
-    public void runSlashCommand(@NotNull SlashCommandInteractionEvent event)
+    public void runSlashCommand(@NotNull SlashCommandInteractionEvent event) {
-    {
         event.reply("Base command implementation").queue();
     }
 }

src/main/java/wtf/beatrice/hidekobot/objects/comparators/MessageCommandAliasesComparator.java

@@ -6,15 +6,13 @@ import java.util.LinkedList;
 /**
  * This class gets two linked lists, and compares their first value alphabetically.
  */
-public class MessageCommandAliasesComparator implements Comparator<LinkedList<String>>
+public class MessageCommandAliasesComparator implements Comparator<LinkedList<String>> {
-{
 
     @Override
-    public int compare(LinkedList<String> linkedList, LinkedList<String> t1)
+    public int compare(LinkedList<String> linkedList, LinkedList<String> t1) {
-    {
 
-        if (linkedList.isEmpty()) return 0;
+        if(linkedList.isEmpty()) return 0;
-        if (t1.isEmpty()) return 0;
+        if(t1.isEmpty()) return 0;
 
         return linkedList.get(0).compareTo(t1.get(0));
     }

src/main/java/wtf/beatrice/hidekobot/objects/comparators/TriviaCategoryComparator.java

@@ -7,12 +7,10 @@ import java.util.Comparator;
 /**
  * This class gets two trivia categories, and compares them by their name.
  */
-public class TriviaCategoryComparator implements Comparator<TriviaCategory>
+public class TriviaCategoryComparator implements Comparator<TriviaCategory> {
-{
 
     @Override
-    public int compare(TriviaCategory o1, TriviaCategory o2)
+    public int compare(TriviaCategory o1, TriviaCategory o2) {
-    {
         return CharSequence.compare(o1.categoryName(), o2.categoryName());
     }
 }

src/main/java/wtf/beatrice/hidekobot/objects/comparators/TriviaScoreComparator.java

@@ -7,12 +7,10 @@ import java.util.Comparator;
 /**
  * This class gets two trivia scores, and compares their score.
  */
-public class TriviaScoreComparator implements Comparator<TriviaScore>
+public class TriviaScoreComparator implements Comparator<TriviaScore> {
-{
 
     @Override
-    public int compare(TriviaScore o1, TriviaScore o2)
+    public int compare(TriviaScore o1, TriviaScore o2) {
-    {
         return Integer.compare(o2.getScore(), o1.getScore()); // inverted, because higher number should come first
     }
 }

src/main/java/wtf/beatrice/hidekobot/objects/fun/TriviaCategory.java

@@ -1,6 +1,5 @@
 package wtf.beatrice.hidekobot.objects.fun;
 
-public record TriviaCategory(String categoryName, int categoryId)
+public record TriviaCategory(String categoryName, int categoryId) {
-{
 
 }

src/main/java/wtf/beatrice/hidekobot/objects/fun/TriviaQuestion.java

@@ -3,7 +3,6 @@ package wtf.beatrice.hidekobot.objects.fun;
 import java.util.List;
 
 public record TriviaQuestion(String question, String correctAnswer,
-                             List<String> wrongAnswers)
+                             List<String> wrongAnswers) {
-{
 
 }

src/main/java/wtf/beatrice/hidekobot/objects/fun/TriviaScore.java

@@ -18,15 +18,9 @@ public class TriviaScore
         score += add;
     }
 
-    public int getScore()
+    public int getScore() { return score; }
-    {
-        return score;
-    }
 
-    public User getUser()
+    public User getUser() { return user; }
-    {
-        return user;
-    }
 
     @Override
     public String toString()

src/main/java/wtf/beatrice/hidekobot/runnables/ExpiredMessageTask.java

@@ -1,20 +1,18 @@
 package wtf.beatrice.hidekobot.runnables;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.datasources.DatabaseSource;
 import wtf.beatrice.hidekobot.util.CommandUtil;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.List;
 
-public class ExpiredMessageTask implements Runnable
+public class ExpiredMessageTask implements Runnable {
-{
 
     private final DateTimeFormatter formatter;
-    private static final Logger LOGGER = LoggerFactory.getLogger(ExpiredMessageTask.class);
+    private final Logger logger;
     private DatabaseSource databaseSource;
 
 
@@ -23,28 +21,28 @@ public class ExpiredMessageTask implements Runnable
         String format = Cache.getExpiryTimestampFormat();
         formatter = DateTimeFormatter.ofPattern(format);
         databaseSource = Cache.getDatabaseSource();
+        logger = new Logger(getClass());
     }
 
 
     @Override
-    public void run()
+    public void run() {
-    {
 
         databaseSource = Cache.getDatabaseSource();
-        if (databaseSource == null) return;
+        if(databaseSource == null) return;
 
         List<String> expiringMessages = Cache.getDatabaseSource().getQueuedExpiringMessages();
-        if (expiringMessages == null || expiringMessages.isEmpty()) return;
+        if(expiringMessages == null || expiringMessages.isEmpty()) return;
 
         LocalDateTime now = LocalDateTime.now();
 
-        for (String messageId : expiringMessages)
+        for(String messageId : expiringMessages)
         {
 
-            if (Cache.isVerbose()) LOGGER.info("expired check: {}", messageId);
+            if(Cache.isVerbose()) logger.log("expired check: " + messageId);
 
             String expiryTimestamp = databaseSource.getQueuedExpiringMessageExpiryDate(messageId);
-            if (expiryTimestamp == null || expiryTimestamp.isEmpty()) // if missing timestamp
+            if(expiryTimestamp == null || expiryTimestamp.equals("")) // if missing timestamp
             {
                 // count it as already expired
                 databaseSource.untrackExpiredMessage(messageId);
@@ -54,9 +52,9 @@ public class ExpiredMessageTask implements Runnable
 
 
             LocalDateTime expiryDate = LocalDateTime.parse(expiryTimestamp, formatter);
-            if (now.isAfter(expiryDate))
+            if(now.isAfter(expiryDate))
             {
-                if (Cache.isVerbose()) LOGGER.info("expired: {}", messageId);
+                if(Cache.isVerbose()) logger.log("expired: " + messageId);
                 CommandUtil.disableExpired(messageId);
             }
         }

src/main/java/wtf/beatrice/hidekobot/runnables/HeartBeatTask.java

@@ -1,8 +1,7 @@
 package wtf.beatrice.hidekobot.runnables;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.Cache;
+import wtf.beatrice.hidekobot.util.Logger;
 
 import java.io.IOException;
 import java.net.HttpURLConnection;
@@ -10,16 +9,22 @@ import java.net.URL;
 
 public class HeartBeatTask implements Runnable
 {
-    private static final Logger LOGGER = LoggerFactory.getLogger(HeartBeatTask.class);
+    private final Logger logger;
+
+    public HeartBeatTask()
+    {
+        logger = new Logger(getClass());
+    }
+
+
 
     @Override
     public void run()
     {
         String urlString = Cache.getFullHeartBeatLink();
-        if (urlString == null || urlString.isEmpty()) return;
+        if(urlString == null || urlString.isEmpty()) return;
 
-        try
+        try {
-        {
 
             URL heartbeatUrl = new URL(urlString);
 
@@ -29,18 +34,18 @@ public class HeartBeatTask implements Runnable
             connection.setReadTimeout(5000);
 
             int responseCode = connection.getResponseCode();
-            if (200 <= responseCode && responseCode < 300)
+            if(200 <= responseCode && responseCode < 300)
             {
                 // only log ok response codes when verbosity is enabled
-                if (Cache.isVerbose()) LOGGER.info("Heartbeat response code: {}", responseCode);
+                if(Cache.isVerbose()) logger.log("Heartbeat response code: " + responseCode);
-            } else
+            }
+            else
             {
-                LOGGER.error("Heartbeat returned problematic response code: {}", responseCode);
+                logger.log("Heartbeat returned problematic response code: " + responseCode);
             }
 
-        } catch (IOException e)
+        } catch (IOException e) {
-        {
+            logger.log("Error while trying to push heartbeat: " + e.getMessage());
-            LOGGER.error("Error while trying to push heartbeat", e);
         }
 
     }

src/main/java/wtf/beatrice/hidekobot/runnables/RandomOrgSeedTask.java

@@ -1,30 +0,0 @@
-package wtf.beatrice.hidekobot.runnables;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import wtf.beatrice.hidekobot.Cache;
-import wtf.beatrice.hidekobot.util.RandomUtil;
-
-/**
- * This runnable pulls a random seed from random.org and used it to feed a SecureRandom,
- * if the integration is enabled.
- * <br/>
- * This is necessary since we are not directly accessing random.org for each random number we
- * need, and thus, only the seed is random - not the algorithm applied to it to compute the numbers.
- */
-public class RandomOrgSeedTask implements Runnable
-{
-
-    private static final Logger LOGGER = LoggerFactory.getLogger(RandomOrgSeedTask.class);
-
-    @Override
-    public void run()
-    {
-        if (RandomUtil.isRandomOrgKeyValid())
-        {
-            if (Cache.isVerbose()) LOGGER.info("Updating Random seed from random.org...");
-            RandomUtil.initRandomOrg();
-            if (Cache.isVerbose()) LOGGER.info("Random.org seed updated!");
-        }
-    }
-}

src/main/java/wtf/beatrice/hidekobot/runnables/RandomSeedTask.java

@@ -0,0 +1,12 @@
+package wtf.beatrice.hidekobot.runnables;
+
+import wtf.beatrice.hidekobot.Cache;
+
+public class RandomSeedTask implements Runnable
+{
+
+    @Override
+    public void run() {
+        Cache.setRandomSeed(System.currentTimeMillis());
+    }
+}

src/main/java/wtf/beatrice/hidekobot/runnables/StatusUpdateTask.java

@@ -22,8 +22,7 @@ public class StatusUpdateTask implements Runnable
     );
 
     @Override
-    public void run()
+    public void run() {
-    {
         int randomPos = RandomUtil.getRandomNumber(0, statuses.size() - 1);
         String status = statuses.get(randomPos) + " | " + Cache.getBotPrefix() + " help";
         HidekoBot.getAPI().getPresence().setActivity(Activity.playing(status));

src/main/java/wtf/beatrice/hidekobot/runnables/TriviaTask.java

@@ -52,12 +52,12 @@ public class TriviaTask implements Runnable
     public void run()
     {
 
-        if (previousMessage != null)
+        if(previousMessage != null)
         {
             // todo: we shouldn't use this method, since it messes with the database... look at coin reflip
             CommandUtil.disableExpired(previousMessage.getId());
 
-            String previousCorrectAnswer = questions.get(iteration - 1).correctAnswer();
+            String previousCorrectAnswer = questions.get(iteration-1).correctAnswer();
 
             // we need this to be thread-locking to avoid getting out of sync with the rest of the trivia features
             previousMessage.reply("The correct answer was: **" + previousCorrectAnswer + "**!").complete();
@@ -67,7 +67,7 @@ public class TriviaTask implements Runnable
             Trivia.channelAndWhoResponded.put(previousMessage.getChannel().getId(), new ArrayList<>());
         }
 
-        if (iteration >= questions.size())
+        if(iteration >= questions.size())
         {
 
             String scoreboardText = "\uD83D\uDC23 Trivia session is over!";
@@ -77,30 +77,28 @@ public class TriviaTask implements Runnable
             StringBuilder othersBuilder = new StringBuilder();
 
             LinkedList<TriviaScore> triviaScores = Trivia.channelAndScores.get(channel.getId());
-            if (triviaScores == null) triviaScores = new LinkedList<>();
+            if(triviaScores == null) triviaScores = new LinkedList<>();
             else triviaScores.sort(new TriviaScoreComparator());
 
             int pos = 0;
             Integer previousScore = null;
-            for (TriviaScore triviaScore : triviaScores)
+            for(TriviaScore triviaScore : triviaScores)
             {
-                if (pos > 10) break; // cap at top 10
+                if(pos > 10) break; // cap at top 10
 
                 String user = triviaScore.getUser().getAsMention();
                 int score = triviaScore.getScore();
-                if (previousScore == null)
+                if(previousScore == null)
                 {
                     previousScore = score;
                     topScore = score;
                     pos = 1;
-                } else
+                } else {
-                {
+                    if(score != previousScore) pos++;
-                    if (score != previousScore) pos++;
                 }
 
-                if (pos == 1) winners.add(user);
+                if(pos == 1) winners.add(user);
-                else
+                else {
-                {
                     othersBuilder.append("\n").append(pos)
                             .append(" | ").append(user)
                             .append(": ").append(score).append(" points");
@@ -108,15 +106,14 @@ public class TriviaTask implements Runnable
             }
 
             StringBuilder winnersBuilder = new StringBuilder();
-            for (int i = 0; i < winners.size(); i++)
+            for(int i = 0; i < winners.size(); i++)
             {
                 String winner = winners.get(i);
                 winnersBuilder.append(winner);
-                if (i + 1 != winners.size())
+                if(i + 1 != winners.size())
                 {
                     winnersBuilder.append(", "); // separate with comma except on last run
-                } else
+                } else {
-                {
                     winnersBuilder.append(": ").append(topScore).append(" points \uD83C\uDF89");
                 }
             }
@@ -130,10 +127,10 @@ public class TriviaTask implements Runnable
             EmbedBuilder scoreboardBuilder = new EmbedBuilder();
             scoreboardBuilder.setColor(Cache.getBotColor());
             scoreboardBuilder.setTitle("\uD83C\uDF1F Trivia Scoreboard");
-            if (!winnersString.isEmpty()) scoreboardBuilder.addField(winnersTitle, winnersString, false);
+            if(!winnersString.isEmpty()) scoreboardBuilder.addField(winnersTitle, winnersString, false);
             else scoreboardBuilder.addField("\uD83D\uDE22 Sad Trivia",
                     "No one played \uD83D\uDE2D", false);
-            if (!othersString.isEmpty()) scoreboardBuilder.addField("☁️ Others", othersString, false);
+            if(!othersString.isEmpty()) scoreboardBuilder.addField("☁️ Others", othersString, false);
 
             channel.sendMessage(scoreboardText).addEmbeds(scoreboardBuilder.build()).queue();
 
@@ -156,7 +153,7 @@ public class TriviaTask implements Runnable
         answerButtons.add(correctAnswerButton);
 
         int i = 0; // we need to add a number because buttons can't have the same id
-        for (String wrongAnswer : currentTriviaQuestion.wrongAnswers())
+        for(String wrongAnswer : currentTriviaQuestion.wrongAnswers())
         {
             i++;
             Button wrongAnswerButton = Button.primary("trivia_wrong_" + i, wrongAnswer);
@@ -169,9 +166,9 @@ public class TriviaTask implements Runnable
                 "\uD83D\uDFE2", "\uD83D\uDFE1", "\uD83D\uDFE4", "\uD83D\uDFE3", "\uD83D\uDFE0");
 
         // add emojis to buttons
-        for (int emojiPos = 0; emojiPos < buttonEmojis.size(); emojiPos++)
+        for(int emojiPos = 0; emojiPos < buttonEmojis.size(); emojiPos++)
         {
-            if (emojiPos == answerButtons.size()) break;
+            if(emojiPos == answerButtons.size()) break;
 
             String emoji = buttonEmojis.get(emojiPos);
             Button button = answerButtons.get(emojiPos);
@@ -183,7 +180,7 @@ public class TriviaTask implements Runnable
 
         embedBuilder.setColor(Cache.getBotColor());
         embedBuilder.setTitle("\uD83C\uDFB2 Trivia - " + category.categoryName() +
-                " (" + (iteration + 1) + "/" + questions.size() + ")");
+                " (" + (iteration+1) + "/" + questions.size() + ")");
 
         embedBuilder.addField("❓ Question", currentTriviaQuestion.question(), false);
 

src/main/java/wtf/beatrice/hidekobot/util/CommandUtil.java

@@ -1,6 +1,5 @@
 package wtf.beatrice.hidekobot.util;
 
-
 import net.dv8tion.jda.api.JDA;
 import net.dv8tion.jda.api.entities.Guild;
 import net.dv8tion.jda.api.entities.Message;
@@ -12,8 +11,6 @@ import net.dv8tion.jda.api.interactions.commands.Command;
 import net.dv8tion.jda.api.interactions.commands.build.CommandData;
 import net.dv8tion.jda.api.interactions.components.LayoutComponent;
 import net.dv8tion.jda.api.requests.RestAction;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 import wtf.beatrice.hidekobot.Cache;
 import wtf.beatrice.hidekobot.HidekoBot;
 import wtf.beatrice.hidekobot.datasources.DatabaseSource;
@@ -25,12 +22,7 @@ import java.util.List;
 public class CommandUtil
 {
 
-    private static final Logger LOGGER = LoggerFactory.getLogger(CommandUtil.class);
+    private static final Logger logger = new Logger(CommandUtil.class);
-
-    private CommandUtil()
-    {
-        throw new IllegalStateException("Utility class");
-    }
 
     /**
      * Function to delete a message when a user clicks the "delete" button attached to that message.
@@ -41,8 +33,7 @@ public class CommandUtil
     public static void delete(ButtonInteractionEvent event)
     {
         // check if the user interacting is the same one who ran the command
-        if (!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId())))
+        if (!(Cache.getDatabaseSource().isUserTrackedFor(event.getUser().getId(), event.getMessageId()))) {
-        {
             event.reply("❌ You did not run this command!").setEphemeral(true).queue();
             return;
         }
@@ -65,10 +56,8 @@ public class CommandUtil
 
         // populate commands list from registered commands
         List<CommandData> allCommands = new ArrayList<>();
-        for (SlashCommand cmd : Cache.getSlashCommandListener().getRegisteredCommands())
+        for(SlashCommand cmd : Cache.getSlashCommandListener().getRegisteredCommands())
-        {
+        { allCommands.add(cmd.getSlashCommandData()); }
-            allCommands.add(cmd.getSlashCommandData());
-        }
 
         JDA jdaInstance = HidekoBot.getAPI();
 
@@ -77,22 +66,22 @@ public class CommandUtil
 
         boolean update = false;
 
-        if (force)
+        if(force)
         {
             update = true;
         } else
         {
 
             // for each command that we have already registered...
-            for (Command currRegCmd : registeredCommands)
+            for(Command currRegCmd : registeredCommands)
             {
                 boolean found = false;
 
                 // iterate through all "recognized" commands
-                for (CommandData cmdData : allCommands)
+                for(CommandData cmdData : allCommands)
                 {
                     // if we find the same command...
-                    if (cmdData.getName().equals(currRegCmd.getName()))
+                    if(cmdData.getName().equals(currRegCmd.getName()))
                     {
                         // quit the loop since we found it.
                         found = true;
@@ -102,7 +91,7 @@ public class CommandUtil
 
                 // if no match was found, we need to send an updated command list because
                 // an old command was probably removed.
-                if (!found)
+                if(!found)
                 {
                     update = true;
 
@@ -112,18 +101,18 @@ public class CommandUtil
             }
 
             // if an update is not already queued...
-            if (!update)
+            if(!update)
             {
                 // for each "recognized" valid command
-                for (CommandData currCmdData : allCommands)
+                for(CommandData currCmdData : allCommands)
                 {
                     boolean found = false;
 
                     // iterate through all already registered commands.
-                    for (Command cmd : registeredCommands)
+                    for(Command cmd : registeredCommands)
                     {
                         // if this command was already registered...
-                        if (cmd.getName().equals(currCmdData.getName()))
+                        if(cmd.getName().equals(currCmdData.getName()))
                         {
                             // quit the loop since we found a match.
                             found = true;
@@ -133,7 +122,7 @@ public class CommandUtil
 
                     // if no match was found, we need to send an updated command list because
                     // a new command was probably added.
-                    if (!found)
+                    if(!found)
                     {
                         update = true;
 
@@ -144,13 +133,13 @@ public class CommandUtil
             }
         }
 
-        LOGGER.info("Found {} commands.", registeredCommands.size());
+        logger.log("Found " + registeredCommands.size() + " commands.");
 
-        if (update)
+        if(update)
         {
             // send updated command list.
             jdaInstance.updateCommands().addCommands(allCommands).queue();
-            LOGGER.info("Commands updated. New total: {}.", allCommands.size());
+            logger.log("Commands updated. New total: " + allCommands.size() + ".");
         }
     }
 
@@ -173,18 +162,18 @@ public class CommandUtil
         MessageChannel textChannel = null;
 
         // this should never happen, but only message channels are supported.
-        if (!msgChannelType.isMessage())
+        if(!msgChannelType.isMessage())
         {
             databaseSource.untrackExpiredMessage(messageId);
             return;
         }
 
         // if this is a DM
-        if (!(msgChannelType.isGuild()))
+        if(!(msgChannelType.isGuild()))
         {
             String userId = databaseSource.getTrackedReplyUserId(messageId);
             User user = userId == null ? null : HidekoBot.getAPI().retrieveUserById(userId).complete();
-            if (user == null)
+            if(user == null)
             {
                 // if user is not found, consider it expired
                 // (deleted profile, or blocked the bot)
@@ -193,11 +182,12 @@ public class CommandUtil
             }
 
             textChannel = user.openPrivateChannel().complete();
-        } else
+        }
+        else
         {
             String guildId = databaseSource.getQueuedExpiringMessageGuild(messageId);
             Guild guild = guildId == null ? null : HidekoBot.getAPI().getGuildById(guildId);
-            if (guild == null)
+            if(guild == null)
             {
                 // if guild is not found, consider it expired
                 // (server was deleted or bot was kicked)
@@ -207,7 +197,7 @@ public class CommandUtil
             textChannel = guild.getTextChannelById(channelId);
         }
 
-        if (textChannel == null)
+        if(textChannel == null)
         {
             // if channel is not found, count it as expired
             // (channel was deleted or bot permissions restricted)
@@ -218,11 +208,11 @@ public class CommandUtil
         RestAction<Message> retrieveAction = textChannel.retrieveMessageById(messageId);
 
 
-        if (Cache.isVerbose()) LOGGER.info("cleaning up: {}", messageId);
+        if(Cache.isVerbose()) logger.log("cleaning up: " + messageId);
 
         retrieveAction.queue(
                 message -> {
-                    if (message == null)
+                    if(message == null)
                     {
                         databaseSource.untrackExpiredMessage(messageId);
                         return;

src/main/java/wtf/beatrice/hidekobot/util/FormatUtil.java

@@ -4,41 +4,18 @@ import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 
 import java.time.Duration;
+import java.time.Instant;
 import java.time.LocalDateTime;
 import java.time.temporal.ChronoUnit;
+import java.time.temporal.TemporalAccessor;
 import java.time.temporal.TemporalUnit;
 import java.util.Arrays;
+import java.util.Locale;
+import java.util.concurrent.TimeUnit;
 
 public class FormatUtil
 {
 
-    private FormatUtil()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
-    // cosmetic string to print on startup.
-    private static final String LOGO = """
-            \s
-                 ██╗░░██╗██╗██████╗░███████╗██╗░░██╗░█████╗░
-                 ██║░░██║██║██╔══██╗██╔════╝██║░██╔╝██╔══██╗
-                 ███████║██║██║░░██║█████╗░░█████═╝░██║░░██║
-                 ██╔══██║██║██║░░██║██╔══╝░░██╔═██╗░██║░░██║
-                 ██║░░██║██║██████╔╝███████╗██║░╚██╗╚█████╔╝
-                 ╚═╝░░╚═╝╚═╝╚═════╝░╚══════╝╚═╝░░╚═╝░╚════╝░
-            \s""";
-
-
-    /**
-     * Returns ASCII art saying the bot name.
-     *
-     * @return a String containing the logo
-     */
-    public static String getLogo()
-    {
-        return LOGO;
-    }
-
     /**
      * Generate a nicely formatted time-diff String that omits unnecessary data
      * (e.g. 0 days, 0 hours, 4 minutes, 32 seconds -> 4m 32s)
@@ -63,28 +40,32 @@ public class FormatUtil
      */
     public static String getNiceDuration(Duration duration)
     {
-        long days = duration.toDays();
-        long hours = duration.toHoursPart();
-        long minutes = duration.toMinutesPart();
         long seconds = duration.toSecondsPart();
+        long minutes = duration.toMinutesPart();
+        long hours = duration.toHoursPart();
+        long days = duration.toDays();
 
-        StringBuilder sb = new StringBuilder();
+        StringBuilder uptimeStringBuilder = new StringBuilder();
-        if (days > 0)
+        if(days == 0)
         {
-            sb.append(days).append("d ");
+            if(hours == 0)
-            sb.append(hours).append("h ");
-            sb.append(minutes).append("m ");
-        } else if (hours > 0)
             {
-            sb.append(hours).append("h ");
+                if(minutes == 0)
-            sb.append(minutes).append("m ");
+                {} else { // i know this if has an empty body but i feel like this reads better
-        } else if (minutes > 0)
+                    uptimeStringBuilder.append(minutes).append("m ");
-        {
-            sb.append(minutes).append("m ");
                 }
-        sb.append(seconds).append("s");
+            } else {
+                uptimeStringBuilder.append(hours).append("h ");
+                uptimeStringBuilder.append(minutes).append("m ");
+            }
+        } else {
+            uptimeStringBuilder.append(days).append("d ");
+            uptimeStringBuilder.append(hours).append("h ");
+            uptimeStringBuilder.append(minutes).append("m ");
+        }
+        uptimeStringBuilder.append(seconds).append("s ");
 
-        return sb.toString();
+        return uptimeStringBuilder.toString();
     }
 
     /**
@@ -98,7 +79,7 @@ public class FormatUtil
     public static Duration parseDuration(String duration)
     {
         // sanitize a bit to avoid cluttering with garbled strings
-        if (duration.length() > 16) duration = duration.substring(0, 16);
+        if(duration.length() > 16) duration = duration.substring(0, 16);
         duration = duration.replaceAll("[^\\w]", ""); //only keep digits and word characters
         duration = duration.toLowerCase();
 
@@ -106,11 +87,8 @@ public class FormatUtil
         eg: 3d, 33hours, 32048dojg, 3d2h5m22s.
         it does not match if the [digits and characters] blocks are missing.
         eg: 33, asd, 3g5hj, 4 asd.
-
-        {1,10} is used to limit the size of the input to parse, to avoid stack overflows.
-        no one should be typing more than 10 arguments, or more than 10 digits for a single argument anyway.
          */
-        if (!duration.matches("(\\d{1,10}[a-zA-Z]{1,10}){1,10}"))
+        if(!duration.matches("(\\d+[a-zA-Z]+)+"))
             return null;
 
         String[] durationTimes = duration.split("[a-zA-Z]+");
@@ -121,7 +99,7 @@ public class FormatUtil
 
         Duration fullDuration = Duration.ZERO;
 
-        for (int i = 0; i < durationTimes.length; i++)
+        for(int i = 0; i < durationTimes.length; i++)
         {
             String durationTimeStr = durationTimes[i];
             String durationUnitStr = durationUnits[i];
@@ -129,7 +107,7 @@ public class FormatUtil
             int durationValue = Integer.parseInt(durationTimeStr);
             TemporalUnit unit = parseTimeUnit(durationUnitStr);
 
-            if (unit != null)
+            if(unit != null)
                 fullDuration = fullDuration.plus(durationValue, unit);
             else return null; // if we failed finding the time unit, instantly quit with failed parsing.
         }
@@ -143,7 +121,7 @@ public class FormatUtil
         // we won't do any sanitization, because this is a private method, and
         // we are only accessing it with things that we know for sure are already sanitized.
         unitName = unitName.toLowerCase();
-        TemporalUnit timeUnit;
+        TemporalUnit timeUnit = null;
 
         /*
         parsing table
@@ -161,7 +139,6 @@ public class FormatUtil
             case "m", "mi", "min", "minute", "minutes" -> timeUnit = ChronoUnit.MINUTES;
             case "h", "ho", "hr", "hour", "hours" -> timeUnit = ChronoUnit.HOURS;
             case "d", "day", "days" -> timeUnit = ChronoUnit.DAYS;
-            default -> timeUnit = null;
         }
 
         return timeUnit;

src/main/java/wtf/beatrice/hidekobot/util/Logger.java

@@ -3,22 +3,29 @@ package wtf.beatrice.hidekobot.util;
 import java.time.LocalDateTime;
 import java.time.format.DateTimeFormatter;
 import java.util.concurrent.Executors;
-import java.util.concurrent.ScheduledExecutorService;
 import java.util.concurrent.TimeUnit;
 
-@Deprecated(since = "0.5.16", forRemoval = true)
+public class Logger
-public class Logger<T>
 {
 
+    // cosmetic string to print on startup.
+    private String logo =
+            "██╗░░██╗██╗██████╗░███████╗██╗░░██╗░█████╗░\n" +
+            "██║░░██║██║██╔══██╗██╔════╝██║░██╔╝██╔══██╗\n" +
+            "███████║██║██║░░██║█████╗░░█████═╝░██║░░██║\n" +
+            "██╔══██║██║██║░░██║██╔══╝░░██╔═██╗░██║░░██║\n" +
+            "██║░░██║██║██████╔╝███████╗██║░╚██╗╚█████╔╝\n" +
+            "╚═╝░░╚═╝╚═╝╚═════╝░╚══════╝╚═╝░░╚═╝░╚════╝░";
+
     // objects that we need to have for a properly formatted message
-    private final String className;
+    private String className;
     private final String format = "[%date% %time%] [%class%] %message%";
-    private final DateTimeFormatter dateFormatter = DateTimeFormatter.ofPattern("yyyy-MM-dd");
+    private final DateTimeFormatter dateFormatter = DateTimeFormatter.ofPattern("YYYY-MM-dd");
     private final DateTimeFormatter timeFormatter = DateTimeFormatter.ofPattern("HH:mm:ss");
 
 
     // when initializing a new logger, save variables in that instance
-    public Logger(Class<T> logClass)
+    public Logger(Class logClass)
     {
         className = logClass.getSimpleName();
     }
@@ -50,10 +57,7 @@ public class Logger<T>
     {
         // create a new scheduled executor with an anonymous runnable...
         //... after waiting <delay> seconds.
-        try (ScheduledExecutorService executor = Executors.newSingleThreadScheduledExecutor())
+        Executors.newSingleThreadScheduledExecutor().schedule(() -> log(message), delay, TimeUnit.SECONDS);
-        {
-            executor.schedule(() -> log(message), delay, TimeUnit.SECONDS);
-        }
 
     }
 
@@ -67,5 +71,14 @@ public class Logger<T>
         System.out.println(message);
     }
 
+    /**
+     * Returns ASCII art saying the bot name.
+     *
+     * @return a String containing the logo
+     */
+    public String getLogo()
+    {
+        return logo;
+    }
 
 }

src/main/java/wtf/beatrice/hidekobot/util/RandomUtil.java

@@ -1,24 +1,9 @@
 package wtf.beatrice.hidekobot.util;
 
-import org.random.util.RandomOrgRandom;
 import wtf.beatrice.hidekobot.Cache;
-import wtf.beatrice.hidekobot.datasources.ConfigurationEntry;
-
-import java.security.SecureRandom;
-import java.util.Random;
 
 public class RandomUtil
 {
-
-    private RandomUtil()
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
-    // the Random instance that we should always use when looking for an RNG based thing.
-    // the seed is updated periodically, if the random.org integration is enabled.
-    private static Random randomInstance = new SecureRandom();
-
     /**
      * Returns a random integer picked in a range.
      *
@@ -28,8 +13,8 @@ public class RandomUtil
      */
     public static int getRandomNumber(int min, int max)
     {
-        if (min == max) return min; // dumbass
+        if(min == max) return min; // dumbass
-        if (min > max) // swap em
+        if(min > max) // swap em
         {
             min = min - max;
             max = min + max;
@@ -41,44 +26,9 @@ public class RandomUtil
         int difference = (max - min) + 1;
 
         // find a number between 0 and our range (eg. 5 -> 8 = 0 -> 3)
-        int randomTemp = getRandom().nextInt(difference);
+        int randomTemp = Cache.getRandom().nextInt(difference);
 
         // add the minimum value, so we are sure to be in the original range (0->5, 1->6, 2->7, 3->8)
         return randomTemp + min;
     }
-
-
-    public static Random getRandom()
-    {
-        return randomInstance;
-    }
-
-    public static void initRandomOrg()
-    {
-        /* we use the random.org instance to generate 160 random bytes.
-        then, we're feeding those 160 bytes as a seed for a SecureRandom.
-
-        this is preferred to calling the RandomOrgRandom directly every time,
-        because it has to query the api and (1) takes a long time, especially with big
-        dice rolls, and (2) you'd run in the limits extremely quickly if the bot
-        was run publicly for everyone to use.
-         */
-        String apiKey = Cache.getRandomOrgApiKey();
-
-        RandomOrgRandom randomOrg = new RandomOrgRandom(apiKey);
-        byte[] randomBytes = new byte[160];
-        randomOrg.nextBytes(randomBytes);
-
-        randomInstance = new SecureRandom(randomBytes);
-    }
-
-
-    public static boolean isRandomOrgKeyValid()
-    {
-        String apiKey = Cache.getRandomOrgApiKey();
-
-        return apiKey != null &&
-                !apiKey.isEmpty() &&
-                !apiKey.equals(ConfigurationEntry.RANDOM_ORG_API_KEY.getDefaultValue());
-    }
 }

src/main/java/wtf/beatrice/hidekobot/util/SerializationUtil.java

@@ -10,38 +10,29 @@ import java.util.List;
 public class SerializationUtil
 {
 
-    private SerializationUtil()
+    public static String serializeBase64(List dataList) {
-    {
-        throw new IllegalStateException("Utility class");
-    }
-
-    public static <T> String serializeBase64(List<T> dataList)
-    {
 
         try (ByteArrayOutputStream bo = new ByteArrayOutputStream();
-             ObjectOutputStream so = new ObjectOutputStream(bo))
+             ObjectOutputStream so = new ObjectOutputStream(bo)) {
-        {
             so.writeObject(dataList);
             so.flush();
             return Base64.getEncoder().encodeToString(bo.toByteArray());
-        } catch (IOException e)
+        }
-        {
+        catch (IOException e) {
             throw new SerializationException("Error during serialization", e);
         }
     }
 
-    public static <T> LinkedList<T> deserializeBase64(String dataStr)
+    public static LinkedList deserializeBase64(String dataStr) {
-    {
 
         byte[] b = Base64.getDecoder().decode(dataStr);
         ByteArrayInputStream bi = new ByteArrayInputStream(b);
         ObjectInputStream si;
-        try
+        try {
-        {
             si = new ObjectInputStream(bi);
             return LinkedList.class.cast(si.readObject());
-        } catch (IOException | ClassNotFoundException e)
+        }
-        {
+        catch (IOException | ClassNotFoundException e) {
             throw new SerializationException("Error during deserialization", e);
         }
     }

src/main/resources/default.properties

@@ -1,2 +1 @@
 bot.version=${project.version}
-repo.base_url=https://git.beatrice.wtf/bea/HidekoBot/

suppressions.xml

@@ -1,12 +0,0 @@
-<?xml version="1.0" encoding="UTF-8"?>
-<suppressions xmlns="https://jeremylong.github.io/DependencyCheck/dependency-suppression.1.3.xsd">
-    <!--
-    <suppress>
-        <notes><![CDATA[
-           file name: snakeyaml-1.33.jar
-        ]]></notes>
-        <packageUrl regex="true">^pkg:maven/org\.yaml/snakeyaml@.*$</packageUrl>
-        <cve>CVE-2021-4235</cve>
-    </suppress>
-    -->
-</suppressions>