Discussion :

[gtraxx]

Bonjour à tous, je suis entrain d'écrire une petite classe pour mes fichiers sitemap.
Je tente désespérément de créer un fichier .gz pour Google mais celui-ci est toujours vide .
Il contient bien le fichier en question mais entièrement vierge, ce n'est pas vraiment le but .
Voici la partie qui déconne :

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
protected function makeFile($file, $data) {
		if((int)$this->GZCompressionLevel !== 0) {
			if(!extension_loaded('zlib')) {
				throw new Exception('Unable to find zlib extension');
				return false;
			}
			if(!$fp = fopen($data, "w")) {
				throw new Exception('Unable to open sitemap file : '.$file);
				return false;
			}
			$datafile = fread($fp, filesize($data));
			fclose($fp);
			$mode = 'w' . (int)$this->GZCompressionLevel;
			if(!$zp = @gzopen($file, $mode)) {
				throw new Exception('Unable to create/update GZIP sitemap file : '.$file);
				return false;
			}
			gzwrite($zp, $datafile);
			gzclose($zp);
		} else {
			if(!@file_put_contents($file, $datafile)) {
				throw new Exception('Unable to create/update sitemap file : '.$file);
				return false;
			}
		}
		return true;
	}

Sa coince au niveau du fread() et filesize() car la taille est toujours de zéro hors le fichier xml fais 369 ko .

Warning: fread() [function.fread]: Length parameter must be greater than 0 in

C'est vraiment bizarre ou c'est moi qui lit le fichier comme une poire

[karl3i]

Salut,

avant le vif du sujet, quelques remarques sur le code :

• Il ne sert à rien de mettre des return après un throw, car ce code n'est pas exécuté. Après un throw, c'est le catch qui sera exécuté, ou rien s'il n'y a pas de catch (uncaught exception)
• Evite les "@", il vaut mieux voir les erreurs de fonction apparaître en période de dév

Ensuite, avec ton code, tu écrasais systématiquement le fichier que tu voulais gziper.

Voici ton code modifié et qui fonctionne chez moi :

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
 
<?php
 
 
$data = "C:/toto.txt";
$file = "c:/toto.gz";
 
if(!extension_loaded('zlib')) {
  throw new Exception('Unable to find zlib extension');
}
 
if(!$fp = fopen($data, "r")) {
  throw new Exception('Unable to open sitemap file : '.$file);
}
 
$filesize = filesize($data);
 
if($filesize===false)
throw new Exception("filesize error");
 
echo "Filesize: $filesize";
 
$datafile = fread($fp, $filesize);
fclose($fp);
$mode = 'w' . "9";
if(!$zp = @gzopen($file, $mode)) {
  throw new Exception('Unable to create/update GZIP sitemap file : '.$file);
}
gzwrite($zp, $datafile);
gzclose($zp);
 
 
?>

Karl3i.

[gtraxx]

j'ai fais ceci :

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
if(!extension_loaded('zlib')) {
				throw new Exception('Unable to find zlib extension');
			}
			if(!$fp = fopen($data, "r")) {
				throw new Exception('Unable to open sitemap file : '.$file);
			}
			$filesize = filesize($data);
			if($filesize === false){
				throw new Exception("filesize error");
			}
			echo "Filesize: $filesize";
 
			$datafile = fread($fp, filesize($filesize));
			fclose($fp);
			$mode = 'w' . (int)$this->GZCompressionLevel;
			if(!$zp = gzopen($file, $mode)) {
				throw new Exception('Unable to create/update GZIP sitemap file : '.$file);
			}
			gzwrite($zp, $datafile);
			gzclose($zp);

Il me retourne :

Filesize: 490
Warning: filesize() [function.filesize]: stat failed for 490 in /var/www/html/site/lib/sitemap.php on line 76

Warning: fread() [function.fread]: Length parameter must be greater than 0 in /var/www/html/site/lib/sitemap.php on line 76

EDIT: j'ai rien dit mon double filesize retournais forcement une erreur
Honte à moi
merci