Fix formatting in SQLDatabaseManager

This commit is contained in:
TfT_02 2014-07-27 12:32:48 +02:00
parent 16b40c3eba
commit 49f2bf5452

View File

@ -14,8 +14,6 @@ import java.util.Map;
import java.util.Properties;
import java.util.UUID;
import snaq.db.ConnectionPool;
import com.gmail.nossr50.mcMMO;
import com.gmail.nossr50.config.Config;
import com.gmail.nossr50.datatypes.MobHealthbarType;
@ -28,6 +26,8 @@ import com.gmail.nossr50.datatypes.skills.SkillType;
import com.gmail.nossr50.runnables.database.UUIDUpdateAsyncTask;
import com.gmail.nossr50.util.Misc;
import snaq.db.ConnectionPool;
public final class SQLDatabaseManager implements DatabaseManager {
private static final String ALL_QUERY_VERSION = "taming+mining+woodcutting+repair+unarmed+herbalism+excavation+archery+swords+axes+acrobatics+fishing+alchemy";
private String connectionString;
@ -60,7 +60,8 @@ public final class SQLDatabaseManager implements DatabaseManager {
connectionProperties);
connectionPool.init(); // Init first connection
connectionPool.registerShutdownHook(); // Auto release on jvm exit just in case
} catch (ClassNotFoundException e) {
}
catch (ClassNotFoundException e) {
// TODO tft do something here
e.printStackTrace();
}
@ -94,27 +95,32 @@ public final class SQLDatabaseManager implements DatabaseManager {
"JOIN " + tablePrefix + "skills s ON (u.id = s.user_id) " +
"JOIN " + tablePrefix + "cooldowns c ON (u.id = c.user_id) " +
"WHERE (s.taming+s.mining+s.woodcutting+s.repair+s.unarmed+s.herbalism+s.excavation+s.archery+s.swords+s.axes+s.acrobatics+s.fishing) = 0");
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -152,27 +158,32 @@ public final class SQLDatabaseManager implements DatabaseManager {
"JOIN " + tablePrefix + "skills s ON (u.id = s.user_id) " +
"JOIN " + tablePrefix + "cooldowns c ON (u.id = c.user_id) " +
"WHERE ((NOW() - lastlogin * " + Misc.TIME_CONVERSION_FACTOR + ") > " + PURGE_TIME + ")");
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -203,20 +214,24 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.setString(1, playerName);
success = statement.executeUpdate() != 0;
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -316,20 +331,24 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.setString(1, profile.getMobHealthbarType() == null ? Config.getInstance().getMobHealthbarDefault().name() : profile.getMobHealthbarType().name());
statement.setInt(2, id);
success = (statement.executeUpdate() != 0);
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -362,27 +381,32 @@ public final class SQLDatabaseManager implements DatabaseManager {
stats.add(new PlayerStat(column.get(1), Integer.valueOf(column.get(0))));
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -472,27 +496,32 @@ public final class SQLDatabaseManager implements DatabaseManager {
resultSet.close();
statement.close();
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -507,13 +536,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
try {
connection = connectionPool.getConnection(VALID_TIMEOUT);
newUser(connection, playerName, uuid);
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -538,20 +570,24 @@ public final class SQLDatabaseManager implements DatabaseManager {
}
writeMissingRows(connection, resultSet.getInt(1));
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -604,30 +640,36 @@ public final class SQLDatabaseManager implements DatabaseManager {
try {
PlayerProfile ret = loadFromResult(playerName, resultSet);
return ret;
} catch (SQLException e) {
}
catch (SQLException e) {
}
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -713,31 +755,37 @@ public final class SQLDatabaseManager implements DatabaseManager {
}
return profile;
} catch (SQLException e) {
}
catch (SQLException e) {
}
}
resultSet.close();
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -783,33 +831,39 @@ public final class SQLDatabaseManager implements DatabaseManager {
resultSet.next();
destination.saveUser(loadFromResult(playerName, resultSet));
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
convertedUsers++;
Misc.printProgress(convertedUsers, progressInterval, startMillis);
}
} catch (SQLException e) {
}
catch (SQLException e) {
printErrors(e);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -830,21 +884,25 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.setString(2, userName);
statement.execute();
return true;
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
return false;
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -880,21 +938,25 @@ public final class SQLDatabaseManager implements DatabaseManager {
}
return true;
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
return false;
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -915,27 +977,32 @@ public final class SQLDatabaseManager implements DatabaseManager {
while (resultSet.next()) {
users.add(resultSet.getString("user"));
}
} catch (SQLException e) {
}
catch (SQLException e) {
printErrors(e);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1029,20 +1096,24 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.executeUpdate("DELETE FROM `" + tablePrefix + "huds` WHERE NOT EXISTS (SELECT * FROM `" + tablePrefix + "users` `u` WHERE `" + tablePrefix + "huds`.`user_id` = `u`.`id`)");
statement.executeUpdate("DELETE FROM `" + tablePrefix + "cooldowns` WHERE NOT EXISTS (SELECT * FROM `" + tablePrefix + "users` `u` WHERE `" + tablePrefix + "cooldowns`.`user_id` = `u`.`id`)");
statement.executeUpdate("DELETE FROM `" + tablePrefix + "skills` WHERE NOT EXISTS (SELECT * FROM `" + tablePrefix + "users` `u` WHERE `" + tablePrefix + "skills`.`user_id` = `u`.`id`)");
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (connection != null) {
try {
connection.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1053,8 +1124,7 @@ public final class SQLDatabaseManager implements DatabaseManager {
/**
* Check database structure for necessary upgrades.
*
* @param upgrade
* Upgrade to attempt to apply
* @param upgrade Upgrade to attempt to apply
*/
private void checkDatabaseStructure(Connection connection, UpgradeType upgrade) {
if (!mcMMO.getUpgradeManager().shouldUpgrade(upgrade)) {
@ -1106,13 +1176,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
}
mcMMO.getUpgradeManager().setUpgradeCompleted(upgrade);
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1143,13 +1216,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.setString(2, Config.getInstance().getMobHealthbarDefault().name());
statement.execute();
statement.close();
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1218,13 +1294,15 @@ public final class SQLDatabaseManager implements DatabaseManager {
try {
mobHealthbarType = MobHealthbarType.valueOf(result.getString(OFFSET_OTHER + 2));
} catch (Exception e) {
}
catch (Exception e) {
mobHealthbarType = Config.getInstance().getMobHealthbarDefault();
}
try {
uuid = UUID.fromString(result.getString(OFFSET_OTHER + 3));
} catch (Exception e) {
}
catch (Exception e) {
uuid = null;
}
@ -1244,7 +1322,8 @@ public final class SQLDatabaseManager implements DatabaseManager {
private void checkUpgradeAddAlchemy(final Statement statement) throws SQLException {
try {
statement.executeQuery("SELECT `alchemy` FROM `" + tablePrefix + "skills` LIMIT 1");
} catch (SQLException ex) {
}
catch (SQLException ex) {
mcMMO.p.getLogger().info("Updating mcMMO MySQL tables for Alchemy...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "skills` ADD `alchemy` int(10) NOT NULL DEFAULT '0'");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "experience` ADD `alchemy` int(10) NOT NULL DEFAULT '0'");
@ -1254,7 +1333,8 @@ public final class SQLDatabaseManager implements DatabaseManager {
private void checkUpgradeAddBlastMiningCooldown(final Statement statement) throws SQLException {
try {
statement.executeQuery("SELECT `blast_mining` FROM `" + tablePrefix + "cooldowns` LIMIT 1");
} catch (SQLException ex) {
}
catch (SQLException ex) {
mcMMO.p.getLogger().info("Updating mcMMO MySQL tables for Blast Mining...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "cooldowns` ADD `blast_mining` int(32) NOT NULL DEFAULT '0'");
}
@ -1263,7 +1343,8 @@ public final class SQLDatabaseManager implements DatabaseManager {
private void checkUpgradeAddFishing(final Statement statement) throws SQLException {
try {
statement.executeQuery("SELECT `fishing` FROM `" + tablePrefix + "skills` LIMIT 1");
} catch (SQLException ex) {
}
catch (SQLException ex) {
mcMMO.p.getLogger().info("Updating mcMMO MySQL tables for Fishing...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "skills` ADD `fishing` int(10) NOT NULL DEFAULT '0'");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "experience` ADD `fishing` int(10) NOT NULL DEFAULT '0'");
@ -1273,7 +1354,8 @@ public final class SQLDatabaseManager implements DatabaseManager {
private void checkUpgradeAddMobHealthbars(final Statement statement) throws SQLException {
try {
statement.executeQuery("SELECT `mobhealthbar` FROM `" + tablePrefix + "huds` LIMIT 1");
} catch (SQLException ex) {
}
catch (SQLException ex) {
mcMMO.p.getLogger().info("Updating mcMMO MySQL tables for mob healthbars...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "huds` ADD `mobhealthbar` varchar(50) NOT NULL DEFAULT '" + Config.getInstance().getMobHealthbarDefault() + "'");
}
@ -1294,18 +1376,22 @@ public final class SQLDatabaseManager implements DatabaseManager {
try {
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "skills` ADD INDEX `idx_" + skill_name + "` (`" + skill_name + "`) USING BTREE");
} catch (SQLException ex) {
}
catch (SQLException ex) {
// Ignore
}
}
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1334,13 +1420,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "users` ADD `uuid` varchar(36) NULL DEFAULT NULL");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "users` ADD UNIQUE INDEX `uuid` (`uuid`) USING BTREE");
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1352,13 +1441,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
while (resultSet.next()) {
names.add(resultSet.getString("user"));
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1389,13 +1481,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
mcMMO.p.getLogger().info("Removing party name from users table...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "users` DROP COLUMN `party`");
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1422,13 +1517,16 @@ public final class SQLDatabaseManager implements DatabaseManager {
mcMMO.p.getLogger().info("Removing Spout HUD type from huds table...");
statement.executeUpdate("ALTER TABLE `" + tablePrefix + "huds` DROP COLUMN `hudtype`");
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1456,26 +1554,31 @@ public final class SQLDatabaseManager implements DatabaseManager {
try {
cachedUserIDs.put(UUID.fromString(resultSet.getString("uuid")), id);
} catch (Exception e) {
}
catch (Exception e) {
}
return id;
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
@ -1505,20 +1608,24 @@ public final class SQLDatabaseManager implements DatabaseManager {
return id;
}
} catch (SQLException ex) {
}
catch (SQLException ex) {
printErrors(ex);
} finally {
}
finally {
if (resultSet != null) {
try {
resultSet.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}
if (statement != null) {
try {
statement.close();
} catch (SQLException e) {
}
catch (SQLException e) {
// Ignore
}
}