Ajouter un commentaire

fredericmazue

Bravo :!: :D

Maintenant quelques remarques si tu me le permets.
Bon ça marche mais...

1) c'est vraiment pas beau de créer une table et de faire une boucle pour ajouter des colonnes. Normalement tu crées toutes les colonnes d'un coup avec une seule requête. Pour moult raisons un ajout de colonnes dans une table est quelque chose qui devrait n'être fait qu'exceptionnellement.

2) toutes tes colonnes ont un type varchar(50) même pour les dates, valeurs numériques et autres. C'est pas bien beau non plus.
Et puis tu perds une place folle dans la base

Ton code marche c'est entendu, mais ce que je veux te dire c'est que si tu le montres à un entretien d'embauche, tu vas être recalé à coup sûr.

Je ne te dis pas ça pour te faire de la peine. Comprends moi bien
Je vois que tu as bossé dur pour faire ce que tu as fait. Je te dis ça simplement pour te signaler des améliorations que tu devrais apporter, maintenant que tu as déjà une base de code qui fonctionne.

Filtered HTML

Plain text

CAPTCHA
Cette question permet de vérifier que vous n'êtes pas un robot spammeur :-)
 ZZZZZ  L      GGG   N   N  H  H 
Z L G NN N H H
Z L G GG N N N HHHH
Z L G G N NN H H
ZZZZZ LLLL GGG N N H H