Revenir au code

De retour sur un code un peu trop longtemps délaissé (un peu comme ce blog), je tombe sur ces lignes de PHP dans un bloc d'initialisation de test.

$Membre = new CMembres(0);
$Membre->LierAGroupe(1);

La variable $membre n'est utilisée nulle part ailleurs, pourtant quand je désactive l'une des lignes ou les deux, un test plante. Donc ces lignes sont utiles, mais en quoi?

Le "1" identifie un groupe créé un peu plus haut, c'est clair. la classe CMembres sert à manipuler la table éponyme et ses dépendances (les tables qui lui apportent des informations relationnelles). Par contre le "0"...
A cet endroit là, ce doit être un identifiant de membre, de personne inscrite, sauf qu'il n'y a par construction pas de membre zéro... sauf... sauf... le membre "visiteur non enregistré", qui est un membre particulier : il a des droits d'accès sans pour autant avoir un compte propre, d'où l'idée de lui donner l'identifiant zéro.

Réaction immédiate : création d'une constante explicite, et le code devient

$Membre = new CMembres(MEMBRE_VISITEUR);
$Membre->LierAGroupe(1);

Seconde étape, cherchez dans mon code les zéros qui portent cette information de "membre visiteur", en les distinguant par exemple des zéros qui signifient "taille nulle"... En fait après relecture rapide je ne pense pas en avoir oublié, et ils étaient tous soit dans des tests qui sécurisent les droits d'accès, soit dans la classe Session qui a pour mission d'accompagner l'internaute en fournissant les données nécessaires. Donc une répartition plutôt logique et limitée, ce qui est plutôt positif et montre une faible dispersion de cette notion de "visiteur".

C'était mes réflexions du moment sur le code. D'un côté quand on relit son propre code on le voit un peu plus comme le verront les autres, et on le rend meilleur. De l'autre une notion explicite est mieux qu'une implicite, et moins elle est dispersée mieux on pourra la surveiller.