bug menu

WEB-INF/src/controleurs/Menu.java

@@ -4,6 +4,7 @@ import java.io.IOException;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+
 import dao.FilDAO;
 import dao.MessageDAO;
 import dao.ParticipantFilDAO;
@@ -15,6 +16,8 @@ import jakarta.servlet.http.*;
 import jakarta.servlet.annotation.WebServlet;
 import org.apache.tomcat.jakartaee.commons.lang3.StringEscapeUtils;
 
+//Servlet qui permet de rediriger l'utilisateur vers le login.html si il n'est pas connecté, et si il l'est,
+// on récupère sa liste de fils et on redirige vers la JSP menu
 @WebServlet("/Menu")
 public class Menu extends HttpServlet {
 
@@ -23,54 +26,45 @@ public class Menu extends HttpServlet {
         res.setContentType("text/html;charset=UTF-8");
         HttpSession session = req.getSession(true);
         Utilisateur utilisateur = (Utilisateur) session.getAttribute("utilisateur");
-
         if (utilisateur == null) {
             res.sendRedirect("login.html");
-            return;
-        }
-
-        FilDAO filDAO = new FilDAO();
-        ParticipantFilDAO participantFilDAO = new ParticipantFilDAO();
-        List<Fil> fils = filDAO.getFilsByUser(utilisateur.getId());
-        req.setAttribute("fils", fils);
-
-        String filIdParam = StringEscapeUtils.escapeHtml4(req.getParameter("filId"));
-        if (filIdParam == null || filIdParam.isEmpty()) {
-            res.sendRedirect("Menu");
-            return;
-        }
+        } else {
+            FilDAO filDAO = new FilDAO();
+            ParticipantFilDAO participantFilDAO = new ParticipantFilDAO();
+            List<Fil> fils = filDAO.getFilsByUser(utilisateur.getId());
+            req.setAttribute("fils", fils);
+            String rese = StringEscapeUtils.escapeHtml4(req.getParameter("filId"));
+            if (rese != null) {
+                if (participantFilDAO.partFil(Integer.parseInt(rese), utilisateur.getId())) {
+                    MessageDAO mDAO = new MessageDAO();
+                    mDAO.checkTimeMessage();
+                    List<dto.Message> messages = mDAO.getLastMessages(Integer.parseInt(rese));
+                    req.setAttribute("message", messages);
+                    req.setAttribute("filId", rese);
 
-        try {
-            int filId = Integer.parseInt(filIdParam);
-            if (participantFilDAO.partFil(filId, utilisateur.getId())) {
-                MessageDAO mDAO = new MessageDAO();
-                mDAO.checkTimeMessage();
-                List<dto.Message> messages = mDAO.getLastMessages(filId);
-                req.setAttribute("message", messages);
-                req.setAttribute("filId", filIdParam);
-
-                Map<Integer, String> userNames = new HashMap<>();
-                UtilisateurDAO utilisateurDAO = new UtilisateurDAO();
-                for (dto.Message mess : messages) {
-                    if (!userNames.containsKey(mess.getUid())) {
-                        Utilisateur user = utilisateurDAO.getUserById(mess.getUid());
-                        if (user != null) {
+                    Map<Integer, String> userNames = new HashMap<>();
+                    UtilisateurDAO utilisateurDAO = new UtilisateurDAO();
+                    for (dto.Message mess : messages) {
+                        if (!userNames.containsKey(mess.getUid())) {
+                            Utilisateur user = utilisateurDAO.getUserById(mess.getUid());
                             userNames.put(mess.getUid(), user.getPrenom() + " " + user.getNom());
                         }
                     }
+
+                    // Vérifier si l'utilisateur est admin du fil
+                    boolean isAdmin = participantFilDAO.isAdmin(utilisateur, Integer.parseInt(rese));
+
+                    // Envoyer les données à la JSP
+                    req.setAttribute("isAdmin", isAdmin);
+
+                    req.setAttribute("userNames", userNames);
+
+                    List<Utilisateur> liste = participantFilDAO.getUsersByFilId(Integer.parseInt(rese));
+
+                    req.setAttribute("listeUsers", liste);
                 }
-                boolean isAdmin = participantFilDAO.isAdmin(utilisateur, filId);
-                req.setAttribute("isAdmin", isAdmin);
-                List<Utilisateur> liste = participantFilDAO.getUsersByFilId(filId);
-                req.setAttribute("listeUsers", liste);
-            } else {
-                res.sendRedirect("Menu");
-                return;
             }
-        } catch (NumberFormatException e) {
-            res.sendRedirect("Menu");
-            return;
+            req.getRequestDispatcher("menu.jsp").forward(req, res);
         }
-        req.getRequestDispatcher("menu.jsp").forward(req, res);
     }
 }