Discussion :

[tiago77]

Bonjour

je cherche à optimiser ma requete.

je m'explique avec la requête ci dessous j'ai une erreur "Maximum execution time of 120 seconds exceeded ".
si je retire les conditions if elseif et else (ligne 28 à 35). la requête fonctionne en en moins de 5 secondes et sans erreur.

Comment optimiser ce code dont le but est de récupérer les données de la première ligne du fichier CSV pour créer ma table.

je vous remercie d'avance pour votre aide

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
require('_connexion.php');
if (isset($_POST["envoyer"])) {
$table = pathinfo ($_FILES["file"]["name"], PATHINFO_FILENAME);
$fich=str_replace('_', '',$table);
$fichier=str_replace('-', '_',$fich);
$file = $_FILES["file"]["tmp_name"];
$delimiteur=";";
$taille = 1000;
var_dump($table);
var_dump($fichier);
var_dump($file);
 
 
 
// get structure from csv and insert db
ini_set('auto_detect_line_endings',TRUE);
 
if (($handle = fopen($file,'r')) !==FALSE )
{
// ... obtenir le contenu de la ligne
	$data = fgetcsv($handle,$taille,$delimiteur);
            $num = count($data);
            var_dump($data);
 
 
for($f=0;$f<$num; $f++) 
{
	if ($f=0){
	 $fields[] = $data[$f].' int(11)';
	}
	elseif ($f=1){
     $fields[] = $data[$f].' DATETIME';
	}
	else {
     $fields[] = $data[$f].' VARCHAR(45)';
   }
 //var_dump($fields);
 
} 
try
{
$sql='CREATE TABLE '.$fichier.' ('. implode(',', $fields) .');';
 var_dump($sql);
 $req=$pdo->query($sql);
}
catch(exception $e)
{
	die('erreur : '.$e->getMessage());
}
}
fclose($handle);
}

[MaitrePylos]

Je sais pas si c'est plus rapide, à tester

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
 
$handle = file($file);
 
		if($file){
 
			list($int,$datetime,$varchar) = explode($delimiteur,$file[0]);
			$int .=' int(11)';
			$datetime .= ' DATETIME';
			$varchar .= ' VARCHAR(45)';
		try
		{
		$sql='CREATE TABLE '.$fichier.' ('.$int.','.$datetime.','.$varchar.');';
		 var_dump($sql);
		 $req=$pdo->query($sql);
		}
		catch(exception $e)
		{
			die('erreur : '.$e->getMessage());
		}
	}

[tiago77]

bonjour MaitrePylos,

Merci de ta réponse

Avec ta solution je ne créer que 3 colonnes. Dans mon fichier j'ai 7 colonnes et cela peut être variable suivant les fichiers.

Ce que je sais c'est que la première colonne est en INT, la deuxième est en DATETIME et que le reste des colonnes est en VARCHAR(45).

merci d'avoir essayer.

[MaitrePylos]

Du coup, vous pouvez alors faire votre implode, la différence réside dans la lecture du fichier un file() plutôt qu'un fopen(), je ne sais si l'un est plus rapide que l'autre

Dans le file() , j'ai directement un tableau , tandis que le fopen(), on doit encore passer le fgetcsv().

[tiago77]

bonjour,

le file sera plus rapide que le fopen avec la lecture de la ligne.
néanmoins je suis toujours obligé des conditions

si ligne = 0
je fait un explode(;, ligne)
puis encore une condition pour les paramètres de colonne
if col=0 -> data = int
if col=1 -> data = DATETIME
sinon -> data = VARCHAR

sinon

insert into
ou
load data local infile avec ignore 1 lines


j'ai pas pu mettre mon code, car mon code tourne en boucle sur la première boucle depuis hier soir sur mon autre PC et j'arrive pas a l’arrêter !!!

Que pensez vous de ma logique ? est elle trop compliqué?

Merci a vous

[MaitrePylos]

Que si cela boucle depuis hier c'est qu'il y a un souci, normalement PHP est limité à 30 secondes.