begin process at 2012 02 12 03:49:56
  Trouver un code source :
 
dans
 

471 commentaire(s) de codefalse sur des sources sur tout CodeS-SourceS

Déposé sur Blog complet : news, images, coms. simple d'utilisation et d'...

Regarde de ce côté là :

http://fr.wikipedia.org/wiki/Mod%C3%A8le-Vue-Contr%C3%B4leur
http://fr.wikipedia.org/wiki/DRY
http://fr.wikipedia.org/wiki/KISS-principe
http://fr.wikipedia.org/wiki/Poo
http://fr.wikipedia.org/wiki/Frameworks

Tu simplifiera ton code, tu le rendras plus facilement lisible, plus pérenne, plus propre, et tout le monde pourra l'utiliser comme bon leur semble.

Courage.
Posté le : 10/06/2009 12:43:10

Déposé sur Générateur de date dans toutes les langues du monde en (php) ...

Tu devrais regarder du côté de DateTime qui permet grandement de simplifier le travail sur les dates, en tenant compte du fuseau horaire, du pays, etc.

http://fr.php.net/datetime
Posté le : 21/04/2009 09:49:07

Déposé sur Exemple d'une administration de gestion d'utilisateur et de g...

Bon, je vais donner mon avis sur ton script.
Déjà, je vois que tu utilise un moteur de template, c'est très bien.

Au niveau des droits ("grades"), tu ne devrait pas te baser sur des droits écrits en durs, mais quelque chose existant dans la base de donnée, de façon à avoir la possibilité de gérer les droits sans avoir à réécrire le code.

Par contre, gros point de pénalité sur ton utilisation de la base de donnée.
Utilise PDO, outre l'avantage d'avoir une classe abstraite qui t'évite d'en refaire une, PDO permet d'utiliser la base de donnée de son choix, juste en choisissant le driver souhaité dans le fichier de configuration (et un code bien adapté).
Regarde de ce côté :
http://blog.reflectiv.net/2008/07/23/les-fonctions-mysql_-sont-a-considerer-comme-obsoletes/
http://blog.reflectiv.net/2008/10/21/nutilisez-plus-les-fonctions-mysql_/

Ca parle des requêtes mysql_, mais mysqli est dans le même ordre d'idée.

Par ailleurs, plutôt que de récuperer des variables globales dans tes classes (notament mydb), tu devrait passer ce dont ta classe à besoin en paramètre, c'est plus ce qui s'apparente à de la programmation orienté objet.

Je te conseille de jeter un oeil à ce système, qui propose l'idée de la gestion des droits directement implémenté dans la base de donnée. Tu peux t'appuyer là-dessus pour modifier ton code :
http://net.tutsplus.com/tutorials/php/a-better-login-system/

Bonne continuation ;)
Posté le : 06/04/2009 10:18:24

Déposé sur Organigramme php dynamique

Non, malheureusement ce n'est que du zip de toléré :s
Et si tu indique où télécharger l'archive du framework js, et que tu ne fournis que ce que tu as fait, le zip est toujours trop grand ?
Posté le : 31/03/2009 10:09:57

Déposé sur Copier un repertoire ou deplacer un repertoire

Sur ta source, ou plutôt fonction, il manque de nombreux éléments de vérification importantes, telles que vérifier si le répertoire dans lequel tu déplace à les droits en écriture, si la copie/suppression à réussi, etc.

De plus, avec l'arrivée de PHP 5.2 (dois-je vraiment dire la date?), est arrivé la SPL, qui te permet dans ton cas de parcourir un répertoire de manière beaucoup plus performante que les vieux opendir/readdir. Regarde du côté de DirectoryIterator.

Le nom de ta fonction ne convient pas. En effet, outre le fait qu'elle ne soit pas anglaise (mais ce n'est pas un problème en soi ;)), le terme "déplacer" perd tout son intérêt lorsque tu met le dernier paramètre à false. En effet, tu ne fait plus un déplacement mais une copie !

J'aurais plus vu une classe statique, avec une méthode cp et une méthode mv, qui appèlent toute deux une méthode privée qui s'occupe de déplacer récursivement ou pas un fichier/répertoire, en fonction des paramètres données.
Avec ceci j'aurai joué sur la classe DirectoryIterator pour le parcours de dossier/fichier et j'aurais lancé des exceptions si je n'avais pas le droit de copier/déplacer/supprimer un élément.

Voilà quelques idées d'améliorations ;)
Posté le : 27/03/2009 10:09:06

Déposé sur Organigramme php dynamique

D'ailleurs la source est manquante.
Il faut l'ajouter sinon elle sera supprimé (c'est la règle sur CS)
Posté le : 26/03/2009 10:30:41

Déposé sur Gestion de fichiers avec la spl

Que dire ! Pas mal de choses ont déjà été mentionnée.
Je voulais ajouter un malus parce que mon pseudo était pas mentionné, mais tu te rattrape dans ton dernier message ... mais j'hésite quand même ... bon ok je blague :p

Pour ce qui est du MVC, je rejoint Neige et Malalam. Tu parle ensuite qu'en général, tu boucle sur un array.
J'aimerai juste mentionner que je commence à voir les tableaux comme de la peste. Surtout sur une structure ou tu ne peux pas gérer la taille du tableau.
Si le dossier contient 5 fichiers, ca va pas être méchant. En revanche, si ton dossier en possède des milliers/millions, tu risque d'avoir des problèmes de mémoire.

La spl te fournit une itération à la volée, pourquoi ne pas en profiter ?
Pourquoi parcourir (boucler) le dossier pour mettre le contenu dans un tableau, et ensuite re-boucler sur ce tableau pour afficher le contenu ?

Si, en tant que moteur de template, tu utilise php (ce que je fait, car à l'origine, php à été réalisé pour faire office de moteur de template, ndlr), tu peux passer un objet qui implémente iterator et qui parcours le dossier.

Par exemple tu peux instancier tout ton système est passer ton objet à ton moteur de template. Ou si tu veux rajouter des restrictions, tu peux faire une classe spécifique pour ton moteur de template, et passer un objet iterator (__construct (iterator $oTonObjet)) afin que quand tu boucle sur la classe spécifique, celle ci bouclera sur l'objet donné dans le constructeur.
Ca à l'avantage d'être générique (itérateur pour un dossier, pour un contenu de fichier (par lignes), contenu d'une base de donnée, etc, etc, etc).

En tout cas très bon travail. L'intérêt que tu montre à la SPL montre tout simplement que tu te tiens à jour (la SPL n'est là que depuis PHP 5.2), et que tu ne nous ponds pas des classes en php 4 :p
Posté le : 18/03/2009 15:05:52

Déposé sur Télécharger les videos de youtube avec permalien depuis youtube

@Destinyfr : Ce n'est pas une classe ;)

@Webdev : Tu parle de la source ou de la proposition de @Destinyfr ?
Posté le : 13/03/2009 10:08:53

Déposé sur Sqlal : classe sql pour mysql, mysqli, sqlite, sql server, et...

Et sinon il y a PDO ;)
http://php.net/pdo
Posté le : 10/03/2009 12:14:03

Déposé sur Médiathèque php mysql : anythek

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 !
Posté le : 05/03/2009 23:50:16



Nos sponsors


Sondage...

CalendriCode

Février 2012
LMMJVSD
  12345
6789101112
13141516171819
20212223242526
272829    

Consulter la suite du CalendriCode

Photothèque

 
Développement réalisé par Nicolas SOREL (Nix) avec l'aide de : Cyril DURAND et Emmanuel (EBArtSoft), Merci à Vincent pour ses précieux conseils.
CodeS-SourceS.com© Toute reproduction même partielle est interdite sauf accord écrit du Webmaster
CodeS-SourceS.com© est une marque déposée tous droits réservés

Google Coop CodeS-SourceS Google Coop CodeS-SourceS
Temps d'éxécution de la page : 1,326 sec (4)

Nous contacter | Annoncer sur CodeS-SourceS | Mentions légales