Quelques remarques :
https://github.com/Honomatopic/Recen...eation.php#L18
Tu ne vérifies pas le contenu de ton formulaire avant d'envoyer ca a la bdd. Par exemple on peut t'envoyer toto comme email et ca devrait passer.
Tu créé un objet Personne avec toutes tes données POST, et tu change les valeurs immédiatement après avec htmlspecialchar($POST).
Pour moi c'est l'un ou l'autre pas les deux.
Dans ta classe Personne, je pense que tu peux utiliser les "magic" method de PHP pour ne pas les réécrire tous. (mais je ne suis pas sur que ca soit mieux que ce que tu fais).
Comme dit précédement, les commentaires devraient contenir des informations fonctionnelles lisibles par un référent métier : Par exemple
"Ajout de la notion de code postal dans l'adresse d'une personne."
Dans 6 mois tu aura autrement le plus grand mal à te souvenir de la raison pour laquelle tu as fait cette modification. Un ocntexte fonctionnel, voir un lien vers une tache a effectuer, aident vraiment dans un contexte professionnel.
Pour moi, ca ne veut pas dire que tu n'es pas capable de progresser, mais il faut que tu entres une boite en junior, que tu regardes le code des autres et qu'on ne fasse du code review régulièrement. C'est le serpent qui se mord la queue tu me diraset je n'ai pas de solution pour cela.
Mais en entretien, je pense que la bonne attitude serait de parler de ta motivation, de ton envie d'apprendre, de progresser, de ton besoin d'un référent pour grandir plus vite, plus que sur ta capacité technique immédiate.
Au pire, tu prends un ou 2 ans avec une paye faible ou tu apprends, quitte a faire une bascule de salaire quand tu auras intégré les principes de qualité attendus d'un dev.
Partager