Bonjour à tous,

je débute en php en partant de 0 et j'aimerais avoir votre avis sur la "qualité" de mon code qui j'imagine est plutôt du genre dégeulasse et certainement plein d'énormités pour le coup .

J'ai codé pour la partie backend d'une appli android de gestion de frais un petit fichier php, si vous pouviez me faire un petit feedback il serait très apprécié :

https://pastebin.com/Q8AewG1Q

Ps: les mots de passe ne sont pas pour l'instant hashés dans la base de données mais le seront ultérieurement d'où le système d'identification crado.

Merci d'avance

Tant que le développeur s'y retrouve.

Je ne connais pas le Php (ça fait 10 ans que j'ai pas codé avec ce langage) :
- gère tes dates (formatage inclus) avec l'API de date & de formatage que le langage dispose. NE RÉINVENTE PAS UNE QUELCONQUE PARTIE DES DATES (QUEL QUE SOIT LE LANGAGE !)
- prends l'habitude d'utiliser des triples égal ( === ) plutôt que des doubles pour la plupart de tes comparaisons
- crée des méthodes (une méthode n'a qu'une unique responsabilité (principes SOLID & KISS))
- si tu as besoin d'un commentaire pour comprendre un appel de méthode, ta méthode est sans doute mal nommée (je pense aux lignes 88 & 89 de ton pastebin - un getId retourne... des types de frais, sémantiquement, ça me pose problème)
- la gestion d'erreur : pour le moment, tu prints juste quand y'a un souci. Ne faudrait-il pas faire plus ? (exemple : Prévenir l'utilisateur ? Arrêter le batch ? Permettre de corriger les saisies ?...)
- tu peux simplifier ta méthode estPremierFraisMois en une seule ligne : return getFicheFrais($mPDO, $idVisiteur, $dateFrais) == null; (ou quel que soit le truc de nullité en Php)
- la méthode estPremierFraisMois fait un "get" au lieu de simplement compter s'il y a des frais sur le mois (pour vérifier simplement la présence sans retourner trop d'informations inutilisées)

Hormis ça :
- bons commentaires
- ça a l'air de faire le taff
- c'est compréhensible pour un français

Merci d'avoir pris le temps de jeter un coup d'oeil
Je prends bonne note de tes remarques et vais m'atteler à améliorer tout ça !

Fichier beaucoup trop long, divise ton code en plusieurs fichiers.