Discussion :

[Yuki_Hime]

Bonjour,
Je me retrouve face à un problème d'optimisation de mon script.
Je veux vérifier si mon fichier ne contient absolument aucun identifiant!!

Jusqu'ici je ne m'étais jamais retrouvé avec ce cas de figure, une suffisament impotante pour avoir le message "Maximum execution time of 30 seconds"

Pourriez-vous m'aider à optimiser mon script

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
 
	case "description":
		$file = fopen("../tmp/derniersResultats.txt", "r") or die("<strong>Can not open file </strong>");	
 
		if ($file) {
		$cmpt=0;
		$empty="";
			while (!feof($file)) {
			$cmpt++;
			$empty=$empty.fgets($file, 4096); 
			}	
		}
		fclose($file);
 
		$_SESSION['CMPT'] = $cmpt - 1;
 
		if ($empty==""){
			$_SESSION['ERROR'] = ('Your set is empty,<br> please type in an other request');
			header("location:../pages/acceuil2.php") or die("Can not go to the home page");
		}
		else get_description("../tmp/derniersResultats.txt") or die("Can not go to the right path" );
	break;

[Celira]

Tu pourrais peut-être utiliser la fonction file qui lit tout le fichier dans un tableau. Il te suffira ensuite de tester si le tableau est vide et d'en récupérer la taille pour obtenir ton compteur. (Je ne garantis pas que ce soit une optimisation, mais c'est au moins une autre façon de faire...)

[Yuki_Hime]

Hmm oui , mais parfois j'ai des fichiers de 1Mo d'ID, et je me retrouve avec un temps d'exécution trop long

Pourrais tu m'aider a faire un script tu genre si les 3 premières ligne sont vides alors out sinon on continue
j'ai écrit pour le moment

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
	case "description":
		$file = fopen("../tmp/derniersResultats.txt", "r") or die("<strong>Can not open file </strong>");	
 
		if ($file) {
		$cmpt=0;
		$empty="";
			while (!feof($file)) {
			$cmpt++; // j'ai besoin du nombre total d'ID
                        }
                         for ($cmpt = 0; $cmpt < 3; $cmpt++;){
			$empty=$empty.fgets($file, 4096); 
			}	
		}
		fclose($file);
 
		$_SESSION['CMPT'] = $cmpt - 1;
 
		if ($empty==""){
			$_SESSION['ERROR'] = ('Your set is empty,<br> please type in an other request');
			header("location:../pages/acceuil2.php") or die("Can not go to the home page");
		}
		else get_description("../tmp/derniersResultats.txt") or die("Can not go to the right path" );
	break;

Est ce correcte ??

[Inazo]

Bonjour à tous,

Alors ici ce n'est pas correct :

Code :

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

1
2
3
4
5
6
7
 
while (!feof($file)) {
			$cmpt++; // j'ai besoin du nombre total d'ID
                        }
                         for ($cmpt = 0; $cmpt < 3; $cmpt++;){
			$empty=$empty.fgets($file, 4096); 
			}

Pourquoi ? Tout simplement que tu vas tourner en rond, car dans ta boucle While tu ne lit pas les lignes du fichier donc tu ne seras jamais en feof, donc le script tournera en boucle sans fin...

Je vais regarder ton code un peu plus et voir si on peut trouver une solution.

Cordialement,

[Yuki_Hime]

Merci bcp

Je n'avais pas vu la boucle infinie.........;

[Inazo]

J'ai une petite question :

si les 3 premières ligne sont vides alors out

Pourquoi les 3 premières ?

De plus je n'ai pas bien compris le but de ta manoeuvre...

Cordialement,

[Yuki_Hime]

juste vérifier que le fichier contient bien au moins un identifiant !

[Inazo]

J'en perd mon latin (que je n'ai pas d'ailleurs) :

Bonjour,
Je me retrouve face à un problème d'optimisation de mon script.
Je veux vérifier si mon fichier ne contient absolument aucun identifiant!!

juste vérifier que le fichier contient bien au moins un identifiant !

Au moins un ou aucun ? Car si tu en veut au moins un il faut que tu regarde si c'est un identifiant dès que tu lis une ligne de ton fichier.

Donc dans ton while :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
while (!feof($file)) {
			$cmpt++;
                         //-- ICI tu vérifie que ta ligne contient ou pas un ID
			 $LigneCourante = fgets($file, 4096); 
 
                         //-- Donc je ne sais pas comment tu regarde que c'est un ID ou pas et si tu ne devrais avoir que des ID dans ce fichier ou pas, enfin c'est un peu flou ^^
 
                         $empty .= $LigneCourante;
			}

Cordialement,

[Yuki_Hime]

thx u !!!

[Yuki_Hime]

j'ai trouvé plus simple avec filesize et en alliant avec la fonction file que Celira a suggéré

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
		$filename = '../tmp/derniersResultats.txt';	
		$size = filesize($filename);
		if($size == 0){
			$_SESSION['ERROR'] = ('Your set is empty,<br> please type in an other request');
			header("location:../pages/acceuil2.php") or die("Can not go to the home page");		
		}
		else{
		$lines = file('../tmp/derniersResultats.txt');
		$_SESSION['CMPT'] = count($lines);
 
		get_description("../tmp/derniersResultats.txt") or die("Can not go to the right path" );
		}