La fonction load_activities_and_players est une gros block monolithique qui fait de nombreuses opérations. Pire, l'ordre dans lequel les fonctions de chargement doivent être appelées est subtil. Chaque modification du fichier d'entrée est difficile, et rend la fonction load_activities_and_players plus difficile à lire encore.
Il faudrait donc :
Créer des fonctions de lecture load_activities and load_players indépendantes. Ces fonctions se content simplement de charger les données des fichiers CSV sans faire le moindre traitement.
Faire une des fonctions traitant les données pour notamment :
Remplacer les noms d'activités, organisatrices, blacklists, etc, par des objets Player ou Activity.
La fonction
load_activities_and_playersest une gros block monolithique qui fait de nombreuses opérations. Pire, l'ordre dans lequel les fonctions de chargement doivent être appelées est subtil. Chaque modification du fichier d'entrée est difficile, et rend la fonctionload_activities_and_playersplus difficile à lire encore.Il faudrait donc :
load_activitiesandload_playersindépendantes. Ces fonctions se content simplement de charger les données des fichiers CSV sans faire le moindre traitement.PlayerouActivity.