Commentaires de coucou747 sur tout CodeS-SourceS

3224 commentaire(s) de coucou747 sur des sources sur tout CodeS-SourceS

Le : 14/05/2008 02:03:10
Source : [PHP 5.1] CLASS STRING : NOUVEL EXEMPLE SUR LA SPL

je passe juste comme ca :)
rien a dire sur la classe, si ce n'est qu'elle ne s'occupe que de String, et non des types de bases, (genre les types de bases ont une representation json, ils sont serializable, on peut les passer en sql, etc...) mais c'est juste une evolution possible et pas une critique :)

sinon, j'ai vu des langages ou certains prennaient un malin plaisir a definir /* comme un operateur couple a */ (je sais plus trop comment, mais c'etait marrant)

bref, quand on voit du francais dans un code, peu importe ce qu'il y a devant, que ca soit #, ;, rem, ', ou //, on sait que c'est un commentaire...

Le : 13/05/2008 15:06:45
Source : ESPACE MEMBRE V2

# $retour = mysql_query("SELECT * FROM EM_site");
# $donn = mysql_fetch_array($retour);

tu ne fais aucune verification d'erreur... si ca se trouve, qqn fait ces interrogations sans avoir installe la base de donnnee...

Le : 13/05/2008 15:04:31
Source : LIVRE D'OR

salut
$base = mysql_connect("", "", "");
tu pourrais faire un fichier de configuration, et verifier que tu as bien reussi a te connecter...
mysql_query($sql);
idem ici, aucune verification d'erreur.

Bref, ca manque d'originalite, et de rigueur.

Le : 08/05/2008 03:46:39
Source : RECODE AST GRAMMAIRE DU PHP

en fait, suite a une discution sur irc, j'ai vu que continue prennait un paran, je l'ignorais jusqu'a present, le fait que continue prenne un parametre peut permettre de coder un php - tail recursif.

Le : 07/05/2008 19:10:01
Source : EXTRACTEUR DE VARIABLES DE FORMULAIRES

on dit 'anciens' et pas "anciens" et 'puristes' et pas "puristes" stp.

*me => []

Le : 07/05/2008 14:31:52
Source : EXTRACTEUR DE VARIABLES DE FORMULAIRES

ce que je disais pour l'entreprise, c'est que le client s'interesse plus au prix qu'il y met, et a ce que lui rapportera le site, qu'aux sources du site... Si le site est joli, ne contient pas de failles, mais est lent et a un code illisible, ca peut convennir...

Il est clair qu'entreprise et Open Source n'ont pas les memes objectifs... T'as des entreprises qui jouent leur reputation donc qui font du code serieux, mais c'est toujours pour repondre a un marche qu'elles developpent...

Apres, t'as des entreprises qui font de l'open source, et qui savent tres bien mixer les deux (Zend par exemple, qui fait un code porc pour php, car de toute facon, ils sont les seuls a maintennir ce logiciel, mais ils font des choses correctes, et attirantes pour les plugins pour php, histoire d'attirer les contribution.)

Le : 06/05/2008 21:51:50
Source : EXTRACTEUR DE VARIABLES DE FORMULAIRES

"je prefere: echo "salut $nom";"
moi echo 'salut'.$nom; ou echo 'salut'.$_POST['nom']; ca depend du contexte.

c'est justement en entreprise que la propreté est parfois négligée... logiquement, l'open source se doit d'etre propre (par souci de partage, compréhension, et honte devant toute la communauté qui lit ton code.)

la fin de ton commentaire n'est pas appropriee... si pour un petit code tu fais porc, alors pour un gros, on peut s'attendre au pire...

Le : 06/05/2008 19:37:08
Source : GÉNÉRATION DU CODE HTML POUR UN TABLEAU (HÉRITE DE LA CLASSE ND_TEMPLATE)

bah en fait, tu n'utilise pas l'heritage la... pourtant, ta class en herite... donc oui t'as une erreur de conception (ou d'utilisation de l'heritage je sais pas...)

apres, perso, j'ai tout lu en diagonale, donc je ne sais pas si tu devrais faire un heritage (propre, je veux dire, et l'utiliser ensuite), ou faire autrement, j'en ai aucune idee...

Le : 06/05/2008 18:22:57
Source : GÉNÉRATION DU CODE HTML POUR UN TABLEAU (HÉRITE DE LA CLASSE ND_TEMPLATE)

# public function __construct($array){
# $this->template = new ND_Template();

pourquoi tu ne fais pas : parent::__construct(); ?

Le : 05/05/2008 22:58:32
Source : RECODE AST GRAMMAIRE DU PHP

non, faudrait faire des calculs d'inference de types pour ca, et dans un langage comme le php, ca ne peut pas se faire aussi facilement...