From d67026edba838b5e6de7fb17ecc17f95b66d49f8 Mon Sep 17 00:00:00 2001 From: MagicBot Date: Mon, 22 May 2023 12:35:15 -0400 Subject: [PATCH] Refactor to remove abstraction. --- src/engine/db/handlers/dbRaceHandler.java | 124 ++++++++++++++++++---- 1 file changed, 101 insertions(+), 23 deletions(-) diff --git a/src/engine/db/handlers/dbRaceHandler.java b/src/engine/db/handlers/dbRaceHandler.java index df83d381..a8c2727c 100644 --- a/src/engine/db/handlers/dbRaceHandler.java +++ b/src/engine/db/handlers/dbRaceHandler.java @@ -9,9 +9,12 @@ package engine.db.handlers; +import engine.gameManager.DbManager; import engine.objects.Race; import org.pmw.tinylog.Logger; +import java.sql.Connection; +import java.sql.PreparedStatement; import java.sql.ResultSet; import java.sql.SQLException; import java.util.HashSet; @@ -23,33 +26,110 @@ public class dbRaceHandler extends dbHandlerBase { } public HashSet BEARD_COLORS_FOR_RACE(final int id) { - prepareCallable("SELECT `color` FROM `static_rune_racebeardcolor` WHERE `RaceID` = ?"); - setInt(1, id); - return getIntegerList(1); + + HashSet integerSet = new HashSet<>(); + + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT `color` FROM `static_rune_racebeardcolor` WHERE `RaceID` = ?")) { + + preparedStatement.setInt(1, id); + + ResultSet rs = preparedStatement.executeQuery(); + + while (rs.next()) + integerSet.add(rs.getInt(1)); + + } catch (SQLException e) { + Logger.error(e); + } + + return integerSet; } public HashSet BEARD_STYLES_FOR_RACE(final int id) { - prepareCallable("SELECT `beardStyle` FROM `static_rune_racebeardstyle` WHERE `RaceID` = ?"); - setInt(1, id); - return getIntegerList(1); + + HashSet integerSet = new HashSet<>(); + + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT `beardStyle` FROM `static_rune_racebeardstyle` WHERE `RaceID` = ?")) { + + preparedStatement.setInt(1, id); + + ResultSet rs = preparedStatement.executeQuery(); + + while (rs.next()) + integerSet.add(rs.getInt(1)); + + } catch (SQLException e) { + Logger.error(e); + } + + return integerSet; } public HashSet HAIR_COLORS_FOR_RACE(final int id) { - prepareCallable("SELECT `color` FROM `static_rune_racehaircolor` WHERE `RaceID` = ?"); - setInt(1, id); - return getIntegerList(1); + + HashSet integerSet = new HashSet<>(); + + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT `color` FROM `static_rune_racehaircolor` WHERE `RaceID` = ?")) { + + preparedStatement.setInt(1, id); + + ResultSet rs = preparedStatement.executeQuery(); + + while (rs.next()) + integerSet.add(rs.getInt(1)); + + } catch (SQLException e) { + Logger.error(e); + } + + return integerSet; + } public HashSet HAIR_STYLES_FOR_RACE(final int id) { - prepareCallable("SELECT `hairStyle` FROM `static_rune_racehairstyle` WHERE `RaceID` = ?"); - setInt(1, id); - return getIntegerList(1); + + HashSet integerSet = new HashSet<>(); + + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT `hairStyle` FROM `static_rune_racehairstyle` WHERE `RaceID` = ?")) { + + preparedStatement.setInt(1, id); + + ResultSet rs = preparedStatement.executeQuery(); + + while (rs.next()) + integerSet.add(rs.getInt(1)); + + } catch (SQLException e) { + Logger.error(e); + } + + return integerSet; + } public HashSet SKIN_COLOR_FOR_RACE(final int id) { - prepareCallable("SELECT `color` FROM `static_rune_raceskincolor` WHERE `RaceID` = ?"); - setInt(1, id); - return getIntegerList(1); + + HashSet integerSet = new HashSet<>(); + + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT `color` FROM `static_rune_raceskincolor` WHERE `RaceID` = ?")) { + + preparedStatement.setInt(1, id); + + ResultSet rs = preparedStatement.executeQuery(); + + while (rs.next()) + integerSet.add(rs.getInt(1)); + + } catch (SQLException e) { + Logger.error(e); + } + + return integerSet; } public ConcurrentHashMap LOAD_ALL_RACES() { @@ -60,10 +140,10 @@ public class dbRaceHandler extends dbHandlerBase { races = new ConcurrentHashMap<>(); int recordsRead = 0; - prepareCallable("SELECT * FROM static_rune_race"); + try (Connection connection = DbManager.getConnection(); + PreparedStatement preparedStatement = connection.prepareStatement("SELECT * FROM static_rune_race")) { - try { - ResultSet rs = executeQuery(); + ResultSet rs = preparedStatement.executeQuery(); while (rs.next()) { @@ -73,13 +153,11 @@ public class dbRaceHandler extends dbHandlerBase { races.put(thisRace.getRaceRuneID(), thisRace); } - Logger.info("read: " + recordsRead + " cached: " + races.size()); - } catch (SQLException e) { - Logger.error( e.toString()); - } finally { - closeCallable(); + Logger.error(e); } + + Logger.info("read: " + recordsRead + " cached: " + races.size()); return races; } }