Construction d'un formulaire

[Sephi-Chan]

En vrac :

• Tu devrais utiliser une seule langue (l'anglais) pour coder (les nom des tables, de colonnes, de variables, idéalement les commentaires, etc.) ;
  
• Tu devrais utiliser une seule syntaxe pour tes variable ($camelCase ou $snake_case) et n'utiliser qu'elle (là on voit clairement que tu as fait du copier-coller) ;
  
• Tu devrais aligner tes = ou ne pas le faire, mais pas mélanger les deux styles (là aussi on voit le copier-coller) ;
  
• Tu ne devrais pas utiliser d'abréviation (comme proprio) mais toujours les mêmes mots, cohérents ;
  
• Tu ne devrais pas utiliser de pseudo comme clé (ici "pseudo_proprio") : seulement des ID (par exemple "owner_id");
  
• Tu ne devrais pas utiliser une chaîne complexe pour stocker tes états (ici "Prête au combat") mais une simple chaîne (comme "ready" ou bien un nombre) stockée dans une constante PHP, puis toujours utiliser cette constante (un peu comme tu le fais pour PREFIX) ;
  
• Tu devrais utiliser sprintf pour générer tes requêtes SQL et ainsi éviter des concaténation au milieu de la requête ;
  
• Tu ne devrais pas mettre de commentaires triviaux comme "Execution de la requete" ou "Triage de la requète", qui est mal nommé (ce n'est pas un "triage") en plus de contenir une faute de français (requête, pas requète) ;
  
• Tu devrais utiliser seulement mysql_fetch_assoc ;
  
• Tu devrais utiliser un format de sérialisation (json_encode et json_decode) plutôt que de faire la tienne à base de tirets ;
  
• echo $_POST['attack']['target'].''; quel est l'intérêt de concaténer une chaîne vide ?
  
• Tu devrais utiliser les guillemets doubles quand tu écris une chaîne qui nécessite de l'échappement (ou fermer les balises PHP quand tu génères du HTML) ;
  

Voilà en gros.