MÉDIATHÈQUE PHP MYSQL : ANYTHEK

Commentaire de petifa le 05/03/2009 20:12:07

Slt
Ça m'a l'air intéressant comme projet. Je la mettrais sur mon pc quand je pourrais et je te commenterai tout ca.
A vrai dire j'ai aussi fait une "mediatech" mais version Java utilisant des fichiers plutôt que des BD.
Il est sur un site?

Commentaire de codefalse le 05/03/2009 23:50:16 administrateur CS

Bonsoir :)

Quelques points et quelque remarques.
Déjà, vous commentez votre code, c'est bien, mais c'est inutile de mettre la mention copyright, author et version pour chaque version. Une seule fois, au début du fichier suffit. (Dans le cas de plusieurs auteurs, mettre plusieurs fois la balise @author, ou les séparer par des virgules suffira largement).

Vous n'utilisez pas les indicateurs de portée pour les fonctions. Je sais que par défaut, ils sont publiques, et que "ce n'est pas nécéssaire, de ce fait, de les mettres". Mais cela ne change rien au code, et ca permet de mieux comprendre le code de manière plus rapide ! :)

Ensuite vous utilisez quelques méthodes magiques (__construct), ce qui est bien, mais plus bas, je vois "function toString". Il existe une méthode magique "__toString ()" qui fasse, en mieux, ce que vous proposez ici.

De plus, faire une méthode __destruct juste pour appeler le parent, ca ne sert à rien. De plus, si on remonte les appels, on finit par voir que la classe Media, au final, ne fait rien, dans ce __destruct...

Ensuite, vous implémentez l'interface DbInteraction dans toute les classes filles de Media ainsi que Media. C'est inutile !
Vous implémentez l'interface dans la classe mère Media, et vous laissez vos classes filles implémenter la fonction.
De plus, dans votre exemple actuel (si j'ai bien compris), la classe media récupere les données dans la table media, la classe fille Video, dans la table video, etc.
Ce qui fait que vous faites plusieurs requêtes sql alors qu'une seule pourrait être faite.

A propos des methodes magiques, vous faites un getCollector/setCollector. Il existe des méthodes magiques __get, __set. Mais peut-être que votre structure prévoyait l'utilisation de méthodes de ce type, ce qui n'engage que vous :)

Voilà pour mes quelques conseils. Mis à part cela, vous avez fait un boulot qui semble intéressant. Reste plus qu'à le tester ! :)

Et dans ce type d'application, ce que je demande toujours :
UNE DEMO !!! Faites nous profiter de votre travail de manière facile et rapidement accessible :)

Bonne continuation !

Commentaire de mfaraday le 08/03/2009 14:18:52

Salut

Bon j'ai résolu mon problème de bdd...

Alors dans un premier temps, des caractères bizarres ("") apparaissent et quand on regarde dans les listes déroulantes, les accents sont remplacés par d'autres caractères (dans ce cas je pense que c'est lié à l'utf8)

Au niveau du code j'ai rien vu de particulier.

Juste une petite chose, '' est plus rapide que "". Enfin c'est ce que j'ai lu plusieurs fois sur le site :)

Sinon bon code j'aime le principe.

Florian