REDIMENSIONNEMENT D'UNE IMAGES, GIF, PNG, JPEG, JPG

Signaler
Messages postés
137
Date d'inscription
samedi 21 février 2004
Statut
Membre
Dernière intervention
16 décembre 2008
-
Messages postés
1
Date d'inscription
vendredi 27 avril 2007
Statut
Membre
Dernière intervention
13 septembre 2007
-
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/40758-redimensionnement-d-une-images-gif-png-jpeg-jpg

Messages postés
1
Date d'inscription
vendredi 27 avril 2007
Statut
Membre
Dernière intervention
13 septembre 2007

Salut,

j'ai télèchargé le script pour voir, mais il y a une erreur à la ligne :

$size empty( $size ) || strpos( strtolower( $size ), 'x' ) FALSE ? 100x100 : $size;

@+
Messages postés
10
Date d'inscription
jeudi 5 août 2004
Statut
Membre
Dernière intervention
20 décembre 2006

Lol do_image est une function ! Juste ! Elle n'est pas dans une class, donc pas de public avant. Je l'avais dans une class pour ma part, erreur de correction...
Messages postés
10
Date d'inscription
jeudi 5 août 2004
Statut
Membre
Dernière intervention
20 décembre 2006

Public pk, parce que c'est bien clair. On sait de quoi on parle, on pourrait ne rien mettre oui, automatiquement cela est considéré en public je sais. C'est histoire de bien faire les choses.

// Citation
D'ailleur tu as une petit erreur dans cette même condition qui ne renvoie false que si $del == 1..

Merci pour l'interrogation du type qui ne renvoie FALSE que si $del est égal à 1, je n'avais pas remarqué.

// Citation
Tu fais deux fois la vérification de l'image... une fois par rapport à l'extension et une seconde fois pas rapport au type retourné par getimagesize()... c'est inutile le second test suffit... et plutot que de faire une condition à part tu pourrais faire... vu que tu utilise un switch...

Oui je fais 2 fois le contrôle du type de l'image, une fois par l'extension et une fois par le type même de l'image, je trouve nécessaire en cas de problème de savoir où chercher, c'est totalement volontaire !

// Citation
Sinon je trouve que l'utilisation de mime_content_type() n'est pas justifiée et limite la portabilité du script pour pas grand chose... autant faire...

Pour les headers c'est un petit détail je trouve mais je prend note.


// Citation
par contre il pourrait être bien de rajouter...
if(!empty($show) && headers_sent())
return false;

Par contre je ne comprends pas pourquoi contrôler que les headers sont envoyés vu que c'est un enregistrement ? Cela peut-être fait avant ou après les headers envoyés

// Citation
Utilise empty() plutot que $maVar == 1 ou $maVar != 1

Tu me dis utilise plustot cette fonction, j'ai eu pas mal d'erreur du à cette fonction que je n'apprècie pas beaucoup et j'ai remarqué que d'autres sources contrôle toujours avec une comparaison ($maVar == 1 | $maVar !=1 ) et la fonction empty() mais jamais seul. Enfin cela reste un avis personnel peut-etre.


// Citation
Pour finir ton unset($img, $tmp, $src); à la fin est totalement inutile vu que les variables sont locales php les supprime automatiquement à la fin de l'éxécution de la fonction... .. .

Ca c'est du folklore ! C'est vrai mais j'aime bien supprimer ce que je fait une fois terminé, c'est une assurance qui ne sert à rien sûrement.

Merci pour ta critique très objectives
Messages postés
1293
Date d'inscription
mardi 9 novembre 2004
Statut
Membre
Dernière intervention
21 mai 2015

Zut erreurs de frappe dans mon commentaire précédant...

if(!empty($del)) et non if(!empty($del) et il faut virer le...

# @header("Content-type: ". mime_content_type( $src ) );

@ tchaOo°
Afficher les 6 commentaires