TYPE DU FICHER

Commentaire de yannvag le 13/07/2006 17:09:57

Pour une meilleure colorisation syntaxique, allez voir sur
http://yannvag.free.fr/PHPManual/hightlightphp.php
et copiez/collez votre code ;-) .

Commentaire de MyXiLo le 13/07/2006 17:54:14

( Il vaudrait mieux retourner $file_extension plutot que $file_extension_tab, n'est-ce pas ?
Il serait egalement preferable, je pense, d'utiliser le array_pop pour la rapidite. )

Commentaire de garfield90 le 13/07/2006 18:35:35

achement compliquer ta fonction et cela pour pas grand chose ( extension d'un fichier ne signifie rien, a part peut etre pour le systeme de fichier de windows )

rien ne t'empeche d'avoir un fichier texte ( au hasard PHP ) et dire que c'est une image ( toujours au hasard, jpg) et l'inverse est aussi vrai.
Imagine les répercutions d'un test basé uniquement sur ca

si ton fichier s'appelle .htaccess ( htaccess n'est pas une extension a ma connaissance )
tu fais un raccourci rapide entre le type et l'extension ( par exemple un fichier php et de type texte, j'espere ne pas dire de connerie sur ce coup la )

de plus tu utilise une fonction eregi ( expression rationnelle ) pour déterminé la présence ou non d'un point
-> a voir si tu n'as pas fait une erreur sur ta regle, je connais pas les ereg, j'utilise plutot preg, et le point est a échappé (faire précédé de \) car il correspond a n'importe quel caractere
-> un simple strpos suffirait, normalement c'est moins gourmand en ressource

Pour finir, encore un hello world (vu X fois sur le web et Y fois sur code source) copyrighter en plus.

Pour ta défense, tu lui a mis un niveau débutant, il en vaut pas plus
tu fais relativement des tests sur ce qu'il y a a faire, par contre c'est pas réellement optimisé

PS : ca doit pas marché

tu as mis
return $file_extension_tab; au lieu
return $file_extension;

voila, amuse toi autant que tu peux avec les fonctions PHP mais c'est pas parce que tu découvres une nouvelle que tu dois absolument l'utilisé ( je pense en particulier a eregi ) car il existe généralement des fonctions moins gourmande dans certains cas.

Conclusion:
continues dans cette voie, évites les copyrights sur des fonctions aussi simple et bon courage

Commentaire de MyXiLo le 13/07/2006 20:07:04

Garfield90 > yannvag n'a pas pretendu que son code pouvait servir a un test concernant le type de fichier, il a dit qu'il avait cree une fonction permettant de retourner l'extension d'un fichier (malgre l'abus de langage dans le nom de la fonction), ce qui n'a absolument rien a voir. L'objectif de cette fonction n'est pas du tout denue d'interet meme si la fonction en elle meme est assez peu optimisee, en effet.

Sinon la fonction actuelle doit poser un probleme lorsque la chaine passee en argument est vide puisque explode retourne false dans ce cas > http://fr.php.net/manual/fr/function.explode.php

Que pensez vous de ceci ?
function file_type($file)
{
$file=basename($file);
return (strpos($file, ".") && $file!="") ? strtolower((array_pop(explode(".",$file))) : "NULL";
}

Commentaire de coucou747 le 13/07/2006 20:12:43 administrateur CS

return "NULL"

ouah... tu cartonnes...

tu savais que NULL et "NULL" c'était différent ?....

comme 0, "", false et NULL...

Commentaire de guill76 le 13/07/2006 20:18:44

ah excuses moi myXylo j'avais pas vu ton post.

Commentaire de coucou747 le 14/07/2006 01:47:48 administrateur CS

entre nous, là, il renvoi l'extension et non le type...

Commentaire de malalam le 14/07/2006 11:24:15 administrateur CS

kankrelune => "l'extension conditionne la façon dont est éxécuté," tu raisonnes en Microsoftien, là. Et peu de serveurs web tournent sous Windows.

Bref...si je ne suis pas convaincu de l'utilité de cette fonction, je suis par contre convaincu qu'elle est mal codée. L'exemple de Kankrelune est meilleur.

Disons que ça montre comment extirper une partie de chaîne dans une autre chaîne, à partir d'un motif...ce qui est toujours utile. Malheureusement, un coup d'oeil à la doc montre des tas d'exemples dans le genre.

Commentaire de kankrelune le 14/07/2006 15:24:29

"tu raisonnes en Microsoftien, là. Et peu de serveurs web tournent sous Windows."

Non je raisonne au niveau d'apache... dans le httpd.conf le mimetype est associé à l'extension que tu veux via AddType... si tu veux que tes fichiers html soient interprétés comme du php c'est tout à fait possible... .. .

AddType application/x-httpd-php .html .htm

Quand à l'utilitée c'est pratigue de savoire quel fichier on as, pour y appliquer le traitement adéquate, par exemple lors de l'exploration d'un répertoire ou à l'upload... .. .

@ tchaOo°

Commentaire de kankrelune le 14/07/2006 15:45:35

Merde je connaissais pas celle là... comme quoi on en apprend tous les jours sur php... .. . ;o)

@ tchaOo°

Commentaire de guill76 le 14/07/2006 16:25:12

oh j'avais pas vu strpos au lieu de strrpos ,  désolé!

Commentaire de Evangun le 14/07/2006 20:53:37

Salut, en toute objectivité, je pense que ce genre de fonctions devrait être supprimé du site: effectivement pathinfo() fait déjà tt le boulot, et ce n'est pas avec ce genre de source que je conseillerais d'apprendre le php... et ça encombre le site aussi.

Commentaire de coucou747 le 14/07/2006 21:40:42 administrateur CS

#  * Begin : 13/07/06, Jeudi, 17h43
# * Merci de laisser le copyright

quand on voit l'heure ou la source a été postée... on se dit que pour un bout de code qui porte un (c)... il n'a pas été très très travaillé...

Commentaire de coucou747 le 15/07/2006 11:50:50 administrateur CS

?? une extention ne fait pas forcément trois caractères...

un .gz par exemples, un .data, un .tar.gz... un .php3 un .html

Commentaire de guill76 le 16/07/2006 01:19:39

c'est pas grave coucou, je crois tu m'as pas suivi là dessus.