Correction sur la page statistiques.php

[droumanet]

J'ai constaté que le fichier fait presque 1500 lignes, ce qui est trop. Il s'agit ici de nombreuses répétitions de code : c'est dangereux, car s'il y a un bogue sur une partie du code, il faut penser à corriger plusieurs parties.

D'autre part, cela génère des problèmes de performances, car l'appel à la base est réalisé plusieurs fois :

        require_once('Modele.php');
        $pdo = connexion();
        if(isset($_GET['filtreAnnee']) and $_GET['filtreAnnee'] != "") {
            $filtreAnnee = $_GET['filtreAnnee'];
            $Etudiants = getEtudiantAnneeOption($filtreAnnee, NULL, NULL);
        } else {
            $Etudiants = getEtudiants();
        }

Une fois la variable remplie, elle est accessible dans toute la page, donc inutile de recommencer : $Etudiants

Enfin, il y a souvent les mêmes calculs avec seulement un paramètre ou deux qui changent : une fonction serait plus propre (plusieurs appels à la même fonction, avec un ou deux paramètres différents).

[droumanet]

• [x] Appeler une seule fois le code accédant à la base de données

Exemple d'amélioration pour les variables

• [ ] Créer un tableau "fille[3] contenant : nombreFilleSIO1 en fille[1] et nombreFilleSIO2 en fille[2]
• [ ] Remplir le tableau en une seule boucle (un seul foreach et des conditions). l'index pour 1 ou 2 correspond à $Etudiant['premiereAnnee']
• [ ] Obtenir le total de fille en faisant fille[1]+fille[2]

Concernant la ou les fonctions :

• [ ] Créer une fonction PHP statsEtudiants(annee) { }
• [ ] Placer dans cette fonction le code souvent répété
• [ ] Utiliser l'attribut annee pour choisir la statistique voulue