From b2359280894ef51bdee882c693fa6ee0247d44a1 Mon Sep 17 00:00:00 2001 From: Sti-jn Date: Sun, 4 Apr 2021 19:24:43 +0200 Subject: [PATCH] Re-did formatting, removed some unused code and rewrote some excessive if statements --- .../java/com/liteflags/auth/AuthTimer.java | 70 +-- .../com/liteflags/auth/Authentication.java | 38 +- .../java/com/liteflags/commands/AuthCMD.java | 4 - .../java/com/liteflags/commands/FlagCMD.java | 464 +++++++++--------- src/main/java/com/liteflags/data/Data.java | 4 - .../com/liteflags/data/database/Database.java | 379 +++++++------- .../data/database/DatabaseConnection.java | 65 +-- .../com/liteflags/data/database/Methods.java | 165 +++---- .../com/liteflags/data/maps/MapCache.java | 4 +- .../java/com/liteflags/events/ChatEvent.java | 78 +-- .../java/com/liteflags/events/LoginEvent.java | 40 +- .../com/liteflags/events/LogoutEvent.java | 35 +- .../java/com/liteflags/events/MoveEvent.java | 22 +- .../java/com/liteflags/util/Utilities.java | 125 ++--- 14 files changed, 724 insertions(+), 769 deletions(-) delete mode 100644 src/main/java/com/liteflags/commands/AuthCMD.java delete mode 100644 src/main/java/com/liteflags/data/Data.java diff --git a/src/main/java/com/liteflags/auth/AuthTimer.java b/src/main/java/com/liteflags/auth/AuthTimer.java index a3ad44a..ae9d747 100644 --- a/src/main/java/com/liteflags/auth/AuthTimer.java +++ b/src/main/java/com/liteflags/auth/AuthTimer.java @@ -3,51 +3,53 @@ package com.liteflags.auth; import com.liteflags.LiteFlags; import com.liteflags.data.maps.MapCache; import com.liteflags.util.Utilities; + import java.util.HashMap; import java.util.Map; import java.util.UUID; + import org.bukkit.Bukkit; import org.bukkit.entity.Player; public class AuthTimer { - public LiteFlags flags; - public static Map taskID = new HashMap(); + public LiteFlags flags; + public static Map taskID = new HashMap(); - public AuthTimer(LiteFlags flags) { - this.flags = flags; - } + public AuthTimer(LiteFlags flags) { + this.flags = flags; + } - public static void startTimer(final Player player) { - int tid = LiteFlags.getInstance().getServer().getScheduler().scheduleSyncRepeatingTask(LiteFlags.getInstance(), new Runnable() { - int timeRemaining = LiteFlags.getInstance().getConfig().getInt("Authenticate_Timer"); + public static void startTimer(final Player player) { + int tid = LiteFlags.getInstance().getServer().getScheduler().scheduleSyncRepeatingTask(LiteFlags.getInstance(), new Runnable() { + int timeRemaining = LiteFlags.getInstance().getConfig().getInt("Authenticate_Timer"); + + public void run() { + if (this.timeRemaining <= 0) { + if (player.hasPermission("liteflags.authentication.success")) { + AuthTimer.endTask(player); + } else { + player.sendMessage(Utilities.format(LiteFlags.getInstance().getConfig().getString("Messages.Authenticate").replace("%code%", (CharSequence) MapCache.reauthedPlayers.get(player.getUniqueId().toString())))); + Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), LiteFlags.getInstance().getConfig().getString("Authenticate_Timer_Command").replace("%player%", player.getName()).replace("%code%", (CharSequence) MapCache.reauthedPlayers.get(player.getUniqueId().toString()))); + this.timeRemaining = LiteFlags.getInstance().getConfig().getInt("Authenticate_Timer"); + } + } else if (player.hasPermission("liteflags.authentication.success")) { + AuthTimer.endTask(player); + } else { + --this.timeRemaining; + } - public void run() { - if (this.timeRemaining <= 0) { - if (player.hasPermission("liteflags.authentication.success")) { - AuthTimer.endTask(player); - } else { - player.sendMessage(Utilities.format(LiteFlags.getInstance().getConfig().getString("Messages.Authenticate").replace("%code%", (CharSequence)MapCache.reauthedPlayers.get(player.getUniqueId().toString())))); - Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), LiteFlags.getInstance().getConfig().getString("Authenticate_Timer_Command").replace("%player%", player.getName()).replace("%code%", (CharSequence)MapCache.reauthedPlayers.get(player.getUniqueId().toString()))); - this.timeRemaining = LiteFlags.getInstance().getConfig().getInt("Authenticate_Timer"); - } - } else if (player.hasPermission("liteflags.authentication.success")) { - AuthTimer.endTask(player); - } else { - --this.timeRemaining; } + }, 0L, 20L); + Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), LiteFlags.getInstance().getConfig().getString("Authenticate_Timer_Command").replace("%player%", player.getName()).replace("%code%", (CharSequence) MapCache.reauthedPlayers.get(player.getUniqueId().toString()))); + taskID.put(player.getUniqueId(), tid); + } - } - }, 0L, 20L); - Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), LiteFlags.getInstance().getConfig().getString("Authenticate_Timer_Command").replace("%player%", player.getName()).replace("%code%", (CharSequence)MapCache.reauthedPlayers.get(player.getUniqueId().toString()))); - taskID.put(player.getUniqueId(), tid); - } + public static void endTask(Player player) { + if (taskID.containsKey(player.getUniqueId())) { + int tid = taskID.get(player.getUniqueId()); + LiteFlags.getInstance().getServer().getScheduler().cancelTask(tid); + taskID.remove(player.getUniqueId()); + } - public static void endTask(Player player) { - if (taskID.containsKey(player.getUniqueId())) { - int tid = taskID.get(player.getUniqueId()); - LiteFlags.getInstance().getServer().getScheduler().cancelTask(tid); - taskID.remove(player.getUniqueId()); - } - - } + } } diff --git a/src/main/java/com/liteflags/auth/Authentication.java b/src/main/java/com/liteflags/auth/Authentication.java index a8f1ef1..d643960 100644 --- a/src/main/java/com/liteflags/auth/Authentication.java +++ b/src/main/java/com/liteflags/auth/Authentication.java @@ -3,32 +3,34 @@ package com.liteflags.auth; import com.liteflags.LiteFlags; import com.liteflags.data.maps.MapCache; import com.liteflags.util.Utilities; + import java.util.Random; import java.util.UUID; + import org.bukkit.Bukkit; import org.bukkit.entity.Player; public class Authentication { - public static String getAuthKey() { - String randChars = "ABCDEFGHIJKLMNPQRSTUVWXYZ123456789abcdefghijklmnopqrstuvwsyz"; - StringBuilder salt = new StringBuilder(); - Random rnd = new Random(); + public static String getAuthKey() { + String randChars = "ABCDEFGHIJKLMNPQRSTUVWXYZ123456789abcdefghijklmnopqrstuvwsyz"; + StringBuilder salt = new StringBuilder(); + Random rnd = new Random(); - while(salt.length() < 6) { - int index = (int)(rnd.nextFloat() * (float)randChars.length()); - salt.append(randChars.charAt(index)); - } + while (salt.length() < 6) { + int index = (int) (rnd.nextFloat() * (float) randChars.length()); + salt.append(randChars.charAt(index)); + } - return "." + salt.toString(); - } + return "." + salt.toString(); + } - public static void checkAuthStatus(UUID uuid) { - Player player = Bukkit.getPlayer(uuid); - if (!player.hasPermission("liteflags.authentication.success")) { - MapCache.reauthedPlayers.put(player.getUniqueId().toString(), getAuthKey()); - AuthTimer.startTimer(player); - player.sendMessage(Utilities.format(LiteFlags.getInstance().getConfig().getString("Messages.Authenticate").replace("%code%", (CharSequence)MapCache.reauthedPlayers.get(player.getUniqueId().toString())))); - } + public static void checkAuthStatus(UUID uuid) { + Player player = Bukkit.getPlayer(uuid); + if (!player.hasPermission("liteflags.authentication.success")) { + MapCache.reauthedPlayers.put(player.getUniqueId().toString(), getAuthKey()); + AuthTimer.startTimer(player); + player.sendMessage(Utilities.format(LiteFlags.getInstance().getConfig().getString("Messages.Authenticate").replace("%code%", (CharSequence) MapCache.reauthedPlayers.get(player.getUniqueId().toString())))); + } - } + } } diff --git a/src/main/java/com/liteflags/commands/AuthCMD.java b/src/main/java/com/liteflags/commands/AuthCMD.java deleted file mode 100644 index 1fa92f9..0000000 --- a/src/main/java/com/liteflags/commands/AuthCMD.java +++ /dev/null @@ -1,4 +0,0 @@ -package com.liteflags.commands; - -public class AuthCMD { -} diff --git a/src/main/java/com/liteflags/commands/FlagCMD.java b/src/main/java/com/liteflags/commands/FlagCMD.java index e1788db..e8e246d 100644 --- a/src/main/java/com/liteflags/commands/FlagCMD.java +++ b/src/main/java/com/liteflags/commands/FlagCMD.java @@ -5,12 +5,14 @@ import com.liteflags.data.database.Database; import com.liteflags.data.database.Methods; import com.liteflags.data.maps.MapCache; import com.liteflags.util.Utilities; + import java.sql.ResultSet; import java.sql.SQLException; import java.util.Arrays; import java.util.Iterator; import java.util.List; import java.util.concurrent.TimeUnit; + import net.md_5.bungee.api.chat.ClickEvent; import net.md_5.bungee.api.chat.ComponentBuilder; import net.md_5.bungee.api.chat.HoverEvent; @@ -25,259 +27,257 @@ import org.bukkit.command.CommandSender; import org.bukkit.entity.Player; public class FlagCMD implements CommandExecutor { - private LiteFlags flags; + private LiteFlags flags; - public FlagCMD(LiteFlags flags) { - this.flags = flags; - } + public FlagCMD(LiteFlags flags) { + this.flags = flags; + } - public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) { - OfflinePlayer target; - String removeTimeLetter; - if (cmd.getName().equalsIgnoreCase("flag") && sender instanceof Player) { - if (sender.hasPermission("liteflags.staff")) { - if (args.length == 0) { - sender.sendMessage(ChatColor.RED + "Invalid command. /flag help"); - } + public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) { + OfflinePlayer target; + String removeTimeLetter; + if (cmd.getName().equalsIgnoreCase("flag") && sender instanceof Player) { + if (sender.hasPermission("liteflags.staff")) { + if (args.length == 0) { + sender.sendMessage(ChatColor.RED + "Invalid command. /flag help"); + } - if (args.length == 1 && args[0].equalsIgnoreCase("help")) { - sender.sendMessage(ChatColor.YELLOW + "/flaglist | /fl " + ChatColor.GRAY + "- Shows all the players with active flags"); - sender.sendMessage(ChatColor.YELLOW + "/flaglist | /fl " + ChatColor.GRAY + "- Shows all flags a player has"); - sender.sendMessage(ChatColor.YELLOW + "/flag