Discussion :

[Asdorve]

Bonjour,

J'ai écrit une fonction qui permet de rechercher des lignes particulières dans un fichier texte.
Or, le fichier commence a être volumineux (13000 lignes pour le moment) et le temps de traitement est de plus en plus long.

Voici le code:

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
 
function interdit($file, $section, $etape)
{
		/*Ouvre le fichier et retourne un tableau contenant une ligne par élément*/
        $lines = file($file);
 
        /*On parcourt le tableau $lines et on affiche le contenu de chaque ligne précédée de son numéro*/
        foreach ($lines as $lineNumber => $lineContent)
        {				
            if ((substr($lineContent, 0, 1) != "#")) //on ne prend pas la lignes de commentaires
            {
                //on découpe la ligne
                $tab = preg_split("/;/", $lineContent);
                //si la section est dans une ligne du fichier            
                if(($tab[2].";".$tab[3]) == ($etape.";".$section))
                {
						$positionProduit = array_search($tab[1],  array_reverse($_SESSION['panier']['libelle'], true));
 
                        if ($positionProduit !== false)
                        {                                        
                                if($tab[0]==$_SESSION['panier']['etape'][$positionProduit])
                                {
                                        return true;	
                                }                                
                        }
                }
            }            
        }    
}

Avez-vous des suggestions pour optimiser le temps de traitement?

Merci

xavier

[Spartacusply]

Déjà mettre en cache le résultat de la fonction file me paraitrait pas mal. Il faudrait même y rajouter directement l'enlèvement des commentaires et l'utilisation de preg_split. Si ton fichier est un csv, ce qui a l'air d'être le cas, tu peux te simplifier la tâche en utilisant fgetcsv.

Le array_reverse ne sert strictement à rien aussi.

Une dernière remarque, dans les règles de l'art il faudrait mettre tout ça dans une base de données et simplifier la recherche avec les requêtes SQL ce qui serait bien plus performant.

[Asdorve]

Effectivement, je viens de basculer le contenu de mon fichier dans une base de données, et ça va nettement mieux!!

Merci du conseil.