Renettoyage et ajout/déplacement de contrôles dans le contrôleur de partie

app/controller/PartieController.php

@@ -28,12 +28,26 @@ class PartieController extends Controller
         $data = $_POST;
 
         $scores = explode(";", $data["scores"]);
-        if (count($scores) != count($data["joueurs"])){
+        $ns = count($scores);
+        $nj = count($data["joueurs"]);
+        if ($ns != $nj){
             //appeler une fonction d'erreur
             echo "<h1>Erreur : scores et joueurs incohérents</h1>";
             die();
         }
 
+        for ($j=0;$j<$nj;$j++){
+            if(!Profil::checkExists($data['joueurs'][$j])){
+                echo "<h1>Erreur : un des joueurs est entré est introuvable dans la BD.</h1>";
+                die();
+            }
+        }
+
+        if(!Jeu::checkExists($data['jeu'])){
+            echo "<h1>Erreur : le jeu entré est introuvable dans la BD.</h1>";
+            die();
+        }
+
         $pb = Partie::addPartie($data['joueurs'], $scores, $data['jeu']);
         if(!$pb)
             header('Location:valid'); // Redirection vers la page OK
@@ -46,16 +60,11 @@ class PartieController extends Controller
                 echo "<h1>Erreur de connexion à la BD.</h1>";
                 die();
                 break;
-            case 3:
-                echo "<h1>Erreur : un des joueurs est entré est introuvable dans la BD.</h1>";
-                die();
-                break;
-            case 4:
-                echo "<h1>Erreur : le jeu entré est introuvable dans la BD.</h1>";
+            default:
+                echo "<h1>Erreur non gérée.</h1>";
                 die();
                 break;
         }
-        
     }
 
 }
\ No newline at end of file

app/model/Partie.php

@@ -99,7 +99,7 @@ class Partie extends Model
 
         if ($stmt0->execute(array(":nom_jeu" => $nom_jeu))) {
             // lecture du slug du jeu
-            if ($row = $stmt0->fetch()) {
+            $row = $stmt0->fetch();
             $id_jeu = $row[0];
             if (!Jeu::checkExists($id_jeu)) {
                 return 1;
@@ -119,17 +119,11 @@ class Partie extends Model
                     $id_joueur[$j] = $row[0];
                     // lecture des slugs des joueur
                     if ($stmt2->execute(array(":joueur" => $joueurs[$j]))) {
-                            if ($row = $stmt2->fetch()) {
+                        $row = $stmt2->fetch();
                         $id_joueur[$j] = $row[0];
-                            }
-                            else
-                                return 3;
-                        }
-                        else{
-                            echo "test";
+                    } else
                         return 2;
                 }
-                    }
 
                 $old_score = array_fill(0, $m, 0);
 
@@ -145,12 +139,9 @@ class Partie extends Model
                             $stmtN = $db->prepare($sqlN);
                             if ($stmtN->execute(array(":id_joueur" => $id_joueur[$j], ":id_jeu" => $id_jeu, ":score" => 1000))) {
 
-                                }
-                                else{
-                                    echo test;
+                            } else
                                 return 2;
                         }
-                            }
 
                     }
                 }
@@ -174,33 +165,16 @@ class Partie extends Model
                             // on met à jour le classement des autres joueurs
                             if ($stmt6->execute(array(":new_classement" => $classement_jeu))) {
                                 return 0;
-                                }
-                                else{
-                                    echo "test";
+                            } else
                                 return 2;
-                                }
-                            }
-                            else{
-                                echo "test";
+                        } else
                             return 2;
-                            }
-                        }
-                        else{
-                            echo "test";
+                    } else
                         return 2;
                 }
-                    }
-                }
-                else{
-                    echo "test1";
+            } else
                 return 2;
-                }
-
-            }
-            else
-                return 4;
-        }
-        else
+        } else
             return 2;
         return 0;
     }