Discussion :

[Marc22]

Bonjour, depuis ce matin je n'arrive pas à faire partir ces warning :

Warning: getimagesize(C:/wamp/www/...) [function.getimagesize]: failed to open stream: Permission denied in C:\wamp\www\...

Warning: Division by zero in C:\wamp\www\...

Warning: Division by zero in C:\wamp\www\...

Le "division by 0" vient du fait que l'image n'existe pas, mais je n'arrive pas à lever l'exception avec un file_exists par exemple ou autre chose.

Voilà à mon avis les lignes de codes où je dois changer quelque chose, j'ai déjà essayé de rajouter && $file!=0 mais ça m'affiche page blanche

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
	while (($file = readdir($dh)) !== false && $file!="") {
 
		if ($file != '.' && $file != '..' && $file!='') {

[ThomasR]

Bonjour,

Montres-nous plutôt que le code où tu appelles getimagesize car si tu rentres dans le if de ton while, c'est que ton fichier existe et est valide.

[Marc22]

d'accord et merci pour ton aide, pour expliquer, je veux me servir d'un script pour redimensionner des images trop importantes d'un dossier "images" dans un dossier "miniatures"

je divise le code en 3 pour faire plus clair :

la fonction recadrement() que j'ai faite pour réduire une image en la gardant dans ses proportions (je récupère largeur et hauteur et de la nouvelle image que j'entrerai dans le width et height de la balise img)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
function recadrement($pathIm, $largeur, $hauteur){ 
 
		if(!file_exists($pathIm))
			$pathIm=$_SERVER["DOCUMENT_ROOT"]."/TEST/images/standard.jpg";
 
		$size=getimagesize($pathIm);
 
		if($size[0]/$largeur>=$size[1]/$hauteur){
			$rapport=$size[0]/$largeur;						
		}
		else{
			$rapport=$size[1]/$hauteur;
		}
 
		$x=$size[0]/$rapport;
		$y=$size[1]/$rapport;
 
		$dim=array("width"=>$x, "height"=>$y);
 
		return $dim;
}

Ca c'est la fonction get_thumbnail() que l'on m'a passé et que j'ai débuggé par des echo car je n'ai pas l'habitude de manipuler opendir etc...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
function get_thumbnail($imagefile,$dim){
$width=50;$height=50;
	$cfgThumbnailDir = $_SERVER["DOCUMENT_ROOT"]."/TEST/images/miniatures/";
	$cfgDocumentRoot = $_SERVER["DOCUMENT_ROOT"]."/TEST/images/";
 
	if(substr($imagefile,0,1)=="/") $imagefile=substr($imagefile,1); // supprime le caractère '/' éventuel en debut de chaine
	$file =  $cfgDocumentRoot.$imagefile; // chemin complet de l'image
 
	$tab=explode("//",$file);
	$file=$tab[0]."/".$tab[1];
 
	echo $file."<br>";
	if(!file_exists($file)) return(false);
	$ext = substr($file,strrpos($file,"."));
 
	// récupère la taille de l'image d'origine:
 
 
	$size = getimagesize($file);
	$ow = $size[0];
	$oh = $size[1];
	// calcul de la taille de la vignette
	if($height==0 && $width==0) {$width = $ow; $height = $oh; }
	if($height==0) $height=round($oh*($width/$ow));
	if($width==0) $width=round($ow*($height/$oh));
 
 
 
	$dst=$cfgThumbnailDir.$imagefile;
	$ext = substr($dst,strrpos($dst,"."));
	if($ext!=".jpeg" && $ext!=".jpg" && $ext!=".gif") return(false);
	//$dst = str_replace($ext,"_".$width."_".$height.$ext,$dst);
	$ret = substr($dst,strlen($cfgDocumentRoot)-1);
 
	$im = imagecreatefromjpeg( "$file" );
	$ox = imagesx( $im );
	$oy = imagesy( $im );
	$nx = $dim["width"];
	$ny = $dim["height"];
	$nm = imagecreatetruecolor( $nx, $ny );
	imagecopyresized( $nm, $im, 0, 0, 0, 0, $nx, $ny, $ox, $oy );
	imagejpeg( $nm, "$dst" );
 
	// Vérifions si la vignette existe déjà et si elle plus récente que l'image d'origine :
	//echo "$dst ?...";
 
	if(file_exists($dst)){
		$dateImage = filemtime ($file); 
		$dateThumb = filemtime ($dst); 
		//echo "<br>date de '$file'=$dateImage";
		//echo "<br>date de '$dst'=$dateThumb";
		if($dateThumb > $dateImage) return($ret); // on retourne le chemin de la vignette existante
	}
 
	$command = "convert $file -geometry ".$width."x".$height."! $dst";
	//echo "<br>command = $command";
	exec($command); // création de la vignette
	return($ret); // on retourne le chemin de la vignette créée
}

Et enfin le code qui manipule les deux fonctions :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
$chemin = $_SERVER["DOCUMENT_ROOT"]."/TEST/images/";
 
if ($dh = opendir($chemin)) {
 
	while (($file = readdir($dh)) !== false && $file!="") {
 
		if ($file != '.' && $file != '..' && $file!='') {
 
			$path=$chemin.$file;
 
			$tab=explode("//",$path);
			$path=$tab[0]."/".$tab[1];
 
			if (file_exists($path)) {
 
				$dim=recadrement($path,148,148); 
				get_thumbnail($file,$dim);
			}
		}
	}
	closedir($dh);
}

Bon il y a des trucs que j'ai remanié, mis en commentaire, parce que j'essayais de m'en sortir mais j'aurais peut être pas dû y toucher.

Je précise que ça marche, j'obtiens bien mes images recadrées comme il faut. Mais à la fin ça fait encore un tour si je peux me permettre l'expression alors qu'il n'y a plus de fichier dans le dossier. (ni de fichier caché je pense puis il aurait un nom)

Et comme je risque de m'en servir souvent de ce script, j'aimerais qu'il soit sans erreur.

Merci

[ThomasR]

Bonjour,

Ton file_exists dans ta boucle while ne sert à rien, puisque le fichier existe forcément (t'es en train de le parcourir).

Rajoutes ceci avant ton getimagesize.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
if(!file_exists($pathIm)) return;
$size=getimagesize($pathIm);

[Marc22]

Hélas toujours le même warning me disant

Warning: getimagesize(C:/wamp/www/TEST/images/miniatures) [function.getimagesize]: failed to open stream: Permission denied in C:\wamp\www\TEST\script.php on line 8

la ligne 8 correspondant à ceci de la fonction encadrement()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$size=getimagesize($pathIm);

Et après forcément si il n'a pas l'image, il doit prendre la dimension hauteur comme largeur comme un 0 et j'ai donc :

Warning: Division by zero in C:\wamp\www\TEST\script.php on line 17

[ThomasR]

ha

Je sais d'où vient le souci, tu ne différencie pas les dossiers des fichiers lorsque tu parcour ton arbo

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
$chemin = $_SERVER["DOCUMENT_ROOT"]."/TEST/images/";
 
if ($dh = opendir($chemin)) {
 
	while (($file = readdir($dh)) !== false && $file!="") {
 
		if ($file != '.' && $file != '..' && $file!='' && !is_dir($chemin.$file)) {
 
			$path=$chemin.$file;
 
			$tab=explode("//",$path);
			$path=$tab[0]."/".$tab[1];
 
			$dim=recadrement($path,148,148); 
                        get_thumbnail($file,$dim);		
		}
	}
	closedir($dh);
}

[Marc22]

D'accord merci beaucoup

Néanmoins, j'ai encore un petit soucis mais c'est parce que j'ai encore l'histoire du slash en trop comme je l'ai traité que après dans le code mais je le règlerai demain.

Bonne soirée !