code slightly reworked

.gitignore

@@ -1,3 +1,4 @@
 bin/
 .classpath
 .project
+.settings/

src/fr/topeka/levelSync/listener/EventsListener.java

@@ -28,19 +28,15 @@ public class EventsListener implements Listener {
 				@Override
 				public void run() {
 					Player p = event.getPlayer();
-					if(sql.checkConnection()) {
-						if(p.isOnline()) {
 					try {
+						sql.checkConnection();
 						float[] values = sql.getPlayerLevel(p);
 						p.setExp(values[1]);
 						p.setLevel((int) values[0]);
 					} catch (SQLException e) {
+						p.sendMessage("[LevelSync] " + plugin.msg_syncError);
+						plugin.getLogger().warning("[LevelSync] " + plugin.msg_syncError);
 						e.printStackTrace();
-								p.sendMessage("[LevelSync] " + plugin.msg_syncError);
-							}
-						}
-					}else {
-						p.sendMessage("[LevelSync] " + plugin.msg_syncError);
 					}
 				}
 			}, 20L);
@@ -51,17 +47,14 @@ public class EventsListener implements Listener {
 		Bukkit.getScheduler().runTaskAsynchronously(plugin, new Runnable() {
 			@Override
 			public void run() {
-				Player p = event.getPlayer();
-				if(sql.checkConnection()) {
 				try {
-						sql.setPlayerLevel(p);
+					Player p = event.getPlayer();
+					sql.checkConnection();
+					sql.sendPlayerLevel(p);
 				}catch(SQLException e) {
-						p.sendMessage("[LevelSync] " + plugin.msg_dbError);
+					plugin.getLogger().warning("[LevelSync] " + plugin.msg_dbError);
 					e.printStackTrace();
 				}
-				}else {
-					p.sendMessage("[LevelSync] " + plugin.msg_dbError);
-				}
 			}		
 		});
 	}

src/fr/topeka/levelSync/sql/Sql.java

@@ -57,7 +57,7 @@ public class Sql {
 		return values;
 	}
 	
-	public void setPlayerLevel(Player player) throws SQLException{
+	public void sendPlayerLevel(Player player) throws SQLException{
 		UUID uuid = player.getUniqueId();
 		int level = player.getLevel();
 		float xp = player.getExp();
@@ -73,27 +73,9 @@ public class Sql {
 		}
 	}
 	
-	public boolean checkConnection() {
-		try {
-			if(connection == null || connection.isClosed() || !connection.isValid(3)) {
-				return reConnect();
-			}
-			return true;
-		} catch (SQLException e) {
-			e.printStackTrace();
-			plugin.getLogger().warning("Could not reconnect to database ! error: " + e.getMessage());
-			return false;
-		}
-	}
-	
-	public boolean reConnect() {
-		try {
-			this.connect();
-			return true;
-		}catch(SQLException e) {
-			e.printStackTrace();
-			return false;
-		}
+	public void checkConnection() throws SQLException {
+		if(connection == null || connection.isClosed() || !connection.isValid(1))
+			connect();
 	}
 	
 	public void closeConnection() {
@@ -102,8 +84,6 @@ public class Sql {
 				connection.close();
 				connection = null;
 			}
-	}catch(SQLException e) {
-		e.printStackTrace();
-	}
+	}catch(SQLException ignored) {}
 	}
 }

src/fr/topeka/levelSync/tasks/SaveDataTask.java

@@ -21,7 +21,7 @@ public class SaveDataTask extends BukkitRunnable {
 		try {
 			main.getLogger().info("[LevelSync] Starting saving data task");
 			for(Player p : main.getServer().getOnlinePlayers()) {
-				main.getSql().setPlayerLevel(p);
+				main.getSql().sendPlayerLevel(p);
 				n++;
 			}
 			main.getLogger().info("[LevelSync] saving data task done for " + n + " players");