Pb php : ImageCreateFromJPEG

/*
Bijour,
j'ai un ti pb en php :/ quelqu'un pourrait-il m'aider? voici mon pb

$donnees est de type "fichier.jpg"
et $directory est un répertoire que l'ont utilise dans la suite de la fonction
*/

function redimension($donnees,$directory)
{

$repository="../medias/grd/";
$fichier=$repository.$donnees;

$type=strtolower(strstr($fichier,"."));
if ($type==".jpg")
{
$image = ImageCreateFromJPEG($fichier);
}

$largeur = ImageSX($image) ;
$hauteur = ImageSY($image) ;
$new_largeur=150;
$new_hauteur=120;

$timbre = ImageCreate($new_largeur,$new_hauteur);
ImageCopyResized($timbre,$image,0,0,0,0,$new_largeur,$new_hauteur,$largeur,$hauteur);


// suite de la fonction

}

/*
Cette fonction fonctionne bien lorsque $repository est de type "media/grd/" mais des que l'on insère les ".." pour aller dans le directory precedent dans l'arborescence il ne passe plus */

Erreur :

Warning: imagesx(): supplied argument is not a valid Image resource in mapage.php on line 40

Warning: imagesy(): supplied argument is not a valid Image resource in mapage.php on line 41

Warning: imagecopyresized(): supplied argument is not a valid Image resource in mapage.php on line 46


Est-il possible que j'accede a ce repertoire "../medias/grd/" sachant qu'il est impossible (point de vu sécurité) de mettre ma page php dans le dossier media ?

Merci beaucoup de votre aide.

Trascal

*/

hehe mais c normal ! c'est l'utilisation de ta fonction strstr qui n'est pas bonne.

en fait avec cette fonction tu coupes ta chaine à partir de la 1ère occurence de "."

dans ton cas tu devrais utiliser substr (c'est ce que je ferais) comme ca :

$type=strtolower(substr($fichier, -4));

là tu lis les 4 derniers caractères de la chaine.

sinon ton code est très bien ; m'enfin si tu veux + d'inspiration tu peux regarder ce code que j'ai fait :
http://www.phpcs.com/code.aspx?ID=13368
ce code génère des vignettes d'image, donc c'est un peu similaire.

note que c'est beaucoup mieux pour que la qualité des images d'utiliser ImageCopyResampled plutot que ImageCopyResized, mais ImageCopyResampled nécessite la libraire GD 2.

Houzéf@ - www.123toto.com

autre chose : ne mets jamais de majuscule dans les fonctions que t'as utilisées ici ; elles sont toutes faites pour etre en minuscule.

la preuve ici :
[ Lien ]

Houzéf@ - www.123toto.com

salut Houzéf@,

Merci beaucoup effectivement je n'y avait pas pensé :)
je vais regarder ton code au calme tout à l'heure ;)
et cela change quoi au niveau de la fonction si je rajoute des majuscule a la place des minuscule? je penserai a laisser les fonction en minuscule ;)

bonne journée

Trascal

pour les minuscules je sais pas si ca peut vraiment perturber l'exécution ; je ne pense pas. mais vu que ces fonctions ont été définies en minuscules, autant toujours laisser les minuscules meme si ca peut paraitre moins lisible.

Houzéf@ - www.123toto.com

Pour ce qui est de
$type=strtolower(substr($fichier, -4));, c'est vraiment bête... et les fichiers une extension de 2 caractères ? tu perds une lettre...
Plutot faire

$type=strtolower(substr($fichier,0,strrpos($fichier,".")));

Pour l'appel des fonctions, tu regardes sur www.php.net... si elles sont en minuscule alors tu les déclares en minuscule.

non ca n'a rien de bete puisque il vérifie uniquement si la fin du nom est ".jpg". donc on se moque des ext à 2 caractères...

Houzéf@ - www.123toto.com

oh, y veut la fin, bah ca revient au même mais c'est pas bon comme jte dis si l'extension c'est .js mettons

il faut se code :
$type=strtolower(substr($fichier,strrpos($fichier,".")+1));

le pb c'est que là on veut seulement savoir si c'est ".jpg" donc seuls les 4 derniers caractères comptent et dans ce cas mon code est + rapide que le tien, c tout juste une question de performances (et aussi mon code est + petit).

Houzéf@ - www.123toto.com

oui mais pas fonctionnel à 100 % !!!!!!!!!!
tu préfères un code qui marche 1 fois sur 2 ou un code qui est un peu plus lent et plus gros (lent c'est relatif la... 0.000000001 sec :P) mais qui fonctionne a tout coup ?