From 9c2a73b108f34333743962ed569d29a8da9acd41 Mon Sep 17 00:00:00 2001
From: basti <basti@DESKTOP-3F3CQL3>
Date: Fri, 11 Dec 2020 17:03:56 +0100
Subject: [PATCH] Cleaning

---
 Projet_POO/src/bdd/Liaison.java              |  1 +
 Projet_POO/src/defaut/Main.java              |  8 --------
 Projet_POO/src/liste/TraitementCmdListe.java |  5 ++++-
 Projet_POO/src/liste/UdpBroadcastClient.java | 10 +++++++++-
 Projet_POO/src/liste/UdpBroadcastServer.java | 17 ++++++++++++++---
 Projet_POO/src/nom/GestionnaireNom.java      |  5 +++--
 Projet_POO/src/ui/ListUI.java                |  6 +++++-
 Projet_POO/src/ui/Login_RegisterUI.java      |  5 ++++-
 Projet_POO/src/ui/NomUI.java                 |  8 +++++---
 9 files changed, 45 insertions(+), 20 deletions(-)

diff --git a/Projet_POO/src/bdd/Liaison.java b/Projet_POO/src/bdd/Liaison.java
index ab06a27..c7fa573 100644
--- a/Projet_POO/src/bdd/Liaison.java
+++ b/Projet_POO/src/bdd/Liaison.java
@@ -32,6 +32,7 @@ public class Liaison {
 			con = dataSource.getConnection();
 			
 		} catch (SQLException e) {
+			System.out.println("Erreur : Liaison BDD");
 			e.printStackTrace();
 		}
 		
diff --git a/Projet_POO/src/defaut/Main.java b/Projet_POO/src/defaut/Main.java
index b922a23..fff3974 100644
--- a/Projet_POO/src/defaut/Main.java
+++ b/Projet_POO/src/defaut/Main.java
@@ -21,14 +21,6 @@ public class Main {
 		GestionnaireListeUtilisateur.instance().majListe2();  //enlever le 2 //TODO
 		
 		
-		
-		try {
-			Thread.sleep(0);
-		} catch (InterruptedException e) {
-			// TODO Auto-generated catch block
-			e.printStackTrace();
-		}
-		
 		//on lance le UI
 		Thread t = new Thread(new Login_RegisterUI());
 		t.start();
diff --git a/Projet_POO/src/liste/TraitementCmdListe.java b/Projet_POO/src/liste/TraitementCmdListe.java
index d6e0a7f..2ebee00 100644
--- a/Projet_POO/src/liste/TraitementCmdListe.java
+++ b/Projet_POO/src/liste/TraitementCmdListe.java
@@ -65,7 +65,10 @@ public class TraitementCmdListe implements Runnable{
 				semaphore.release();
 				
 				
-			} catch (Exception e) {}
+			} catch (Exception e) {
+				System.out.println("Erreur : run TraitementCmdListe");
+				e.printStackTrace();
+			}
 		
 		}
 	
diff --git a/Projet_POO/src/liste/UdpBroadcastClient.java b/Projet_POO/src/liste/UdpBroadcastClient.java
index 843d63e..75f4ed4 100644
--- a/Projet_POO/src/liste/UdpBroadcastClient.java
+++ b/Projet_POO/src/liste/UdpBroadcastClient.java
@@ -11,6 +11,7 @@ public class UdpBroadcastClient implements Runnable{
 	
 	private final int serverPort;
 	private final String message;
+	private DatagramSocket serverSocket;
 	
 	//Constructeur
 	public UdpBroadcastClient(int serverPort, String message) {
@@ -21,7 +22,7 @@ public class UdpBroadcastClient implements Runnable{
 
 	public void run() {
 		try{
-			DatagramSocket serverSocket = new DatagramSocket();
+			serverSocket = new DatagramSocket();
 			serverSocket.setBroadcast(true); 
 			DatagramPacket datagramPacket = new DatagramPacket(
 					message.getBytes(),
@@ -39,6 +40,13 @@ public class UdpBroadcastClient implements Runnable{
 			e.printStackTrace();
 		} catch (IOException e) {
 			e.printStackTrace();
+		} finally {
+			try {
+				serverSocket.close();
+			} catch (Exception e) {
+				System.out.println("Erreur : pas de Client Socket à fermer (UDP)");
+				e.printStackTrace();
+			}
 		}
 		
 		
diff --git a/Projet_POO/src/liste/UdpBroadcastServer.java b/Projet_POO/src/liste/UdpBroadcastServer.java
index 7e22f0e..c245b4e 100644
--- a/Projet_POO/src/liste/UdpBroadcastServer.java
+++ b/Projet_POO/src/liste/UdpBroadcastServer.java
@@ -10,6 +10,7 @@ import java.util.concurrent.Semaphore;
 public class UdpBroadcastServer implements Runnable{
 	
 	private final int serverPort;
+	private DatagramSocket serverSocket;
 	
 	
 	public UdpBroadcastServer(int serverPort) {
@@ -18,8 +19,8 @@ public class UdpBroadcastServer implements Runnable{
 	
 	
 	public void run() {
-		try(DatagramSocket serverSocket = new DatagramSocket(serverPort)){ //fermeture ne cas de coupure abrupte ?? //TODO
-			
+		try{
+			serverSocket = new DatagramSocket(serverPort);
 			Semaphore semaphore = new Semaphore(1, true);
 			
 			byte[] buffer = new byte[ConstanteListeUtilisateur.SIZE_ORDRE];
@@ -38,9 +39,19 @@ public class UdpBroadcastServer implements Runnable{
 			}
 			
 		} catch (SocketException e) {
+			System.out.println("Erreur : Socket UDP Server");
 			e.printStackTrace();
 		} catch (IOException e) {
-			System.out.println("Timedout. CLient is closing.");
+			System.out.println("Erreur : IO UDP Sever");
+			e.printStackTrace();
+		} finally {
+			try {
+				serverSocket.close();
+			} catch (Exception e) {
+				System.out.println("Erreur : pas de Server Socket à fermer (UDP)");
+				e.printStackTrace();
+			}
+				
 		}
 	}
 	
diff --git a/Projet_POO/src/nom/GestionnaireNom.java b/Projet_POO/src/nom/GestionnaireNom.java
index a839b40..0da95ef 100644
--- a/Projet_POO/src/nom/GestionnaireNom.java
+++ b/Projet_POO/src/nom/GestionnaireNom.java
@@ -28,7 +28,8 @@ public class GestionnaireNom{
 				ipUtilisateur = InetAddress.getLocalHost().getHostName();
 			} 
 			catch (Exception e) {
-				System.out.println("Problème IP");
+				System.out.println("Erreur : inetAddres LocalHost (UserIp)");
+				e.printStackTrace();
 			}
 		}
 		
@@ -104,7 +105,7 @@ public class GestionnaireNom{
 	// renvoie "true" si le nom est disponible
 	public boolean verifierDisponibiliteNom(String nom) {
 		for (int i=0; i<GestionnaireListeUtilisateur.instance().getListeUtilisateur().size(); i++) {
-			if (GestionnaireListeUtilisateur.instance().getListeUtilisateur().get(i).nom.toLowerCase().equals(nom.toLowerCase())) {
+			if (GestionnaireListeUtilisateur.instance().getListeUtilisateur().get(i).nom.equals(nom)) {
 				return false;
 			}	
 		}
diff --git a/Projet_POO/src/ui/ListUI.java b/Projet_POO/src/ui/ListUI.java
index eca5893..ac4d3b7 100644
--- a/Projet_POO/src/ui/ListUI.java
+++ b/Projet_POO/src/ui/ListUI.java
@@ -48,7 +48,9 @@ public class ListUI extends JFrame implements Runnable{
 			
 			
 		} catch (Exception e) {
+			System.out.println("Erreur : run ListUI");
 			e.printStackTrace();
+			dispose();
 		}
 	}
 	
@@ -171,8 +173,10 @@ public class ListUI extends JFrame implements Runnable{
 			gestionnaireClavardage gc = gestionnaireClavardage.instance();
 			gc.createSession(nom);
 			
+			}catch (Exception e) {
+				System.out.println("Erreur : connect ListUI");
+				e.printStackTrace();
 			}
-			catch (Exception e) {}
 		
 	}
 	
diff --git a/Projet_POO/src/ui/Login_RegisterUI.java b/Projet_POO/src/ui/Login_RegisterUI.java
index 5c158c9..cbce3a3 100644
--- a/Projet_POO/src/ui/Login_RegisterUI.java
+++ b/Projet_POO/src/ui/Login_RegisterUI.java
@@ -50,14 +50,16 @@ public class Login_RegisterUI extends JFrame implements Runnable{
 	
 	public void run() {
 		try {
-			
 			Login_RegisterUI frame = new Login_RegisterUI();
 			frame.setVisible(true);
 			
 			//center the frame
 			frame.setLocationRelativeTo(null);
+			
 		} catch (Exception e) {
+			System.out.println("Erreur : run Login_RegisterUI");
 			e.printStackTrace();
+			dispose();
 		}
 	}
 
@@ -280,6 +282,7 @@ public class Login_RegisterUI extends JFrame implements Runnable{
 			
 			
 		} catch (SQLException e) {
+			System.out.println("Erreur : BDD");
 			e.printStackTrace();
 		}
 		
diff --git a/Projet_POO/src/ui/NomUI.java b/Projet_POO/src/ui/NomUI.java
index 2402020..8e9a31d 100644
--- a/Projet_POO/src/ui/NomUI.java
+++ b/Projet_POO/src/ui/NomUI.java
@@ -44,10 +44,12 @@ public class NomUI extends JFrame implements Runnable{
 			frame.setVisible(true);
 			
 
-			System.out.println( GestionnaireNom.instance().getId() );
+			//System.out.println( GestionnaireNom.instance().getId() );
 			
 		} catch (Exception e) {
+			System.out.println("Erreur : run NomUI");
 			e.printStackTrace();
+			dispose();
 		}
 	}
 
@@ -119,7 +121,7 @@ public class NomUI extends JFrame implements Runnable{
 	// On choisit son Username
 	private void nommer(String nom) {
 		// Si l'on se renomme on peut annuler en entrant rien ou Reprendre le même Username
-		if( (GestionnaireNom.instance().getNom() != null) && (nom.isBlank() || nom.equals(GestionnaireNom.instance().getNom()) ) ) {
+		if( (GestionnaireNom.instance().getNom() != null) && ( nom.isBlank() || nom.equals(GestionnaireNom.instance().getNom()) ) ) {
 			dispose();
 			
 			Thread t = new Thread(new ListUI());
@@ -128,7 +130,7 @@ public class NomUI extends JFrame implements Runnable{
 		}
 		else {
 			//Sinon on essaye de prendre un nom
-			if (GestionnaireNom.instance().verifierDisponibiliteNom(nom) || nom.toLowerCase().equals(GestionnaireNom.instance().getNom().toLowerCase()) ) {
+			if (GestionnaireNom.instance().verifierDisponibiliteNom(nom)) {
 				//Celui-ci ne peut pas être "vide"
 				if (!nom.isBlank()) {
 					GestionnaireNom.instance().nommer2(nom); //enlever le 2 à la fin //TODO