Discussion :

[jean1190]

Bonjour,

Difficile d'expliquer mon problème simplement avec le titre.

Mon objectif à atteindre est le suivant :
- Récupérer les données d'un fichier .CSV (généré par Excel)
- Les mettre dans un tableau
- Parcourir ce tableau pour remplir une table de ma base de données avec plusieurs champs.

Je bloque sur ce troisième point. Lorsque j'insère mes données, il m'arrive un truc chelou. Après avoir inséré correctement les éléments de la 1ère colonne, je me retrouve à remplir la 2ème colonne avec une quantité d'éléments vides au départ qui correspondent en fait aux éléments que j'ai remplis dans la 1ère.

En gros voilà ce que j'obtiens :
1 vide
2 vide
3 vide
4 vide
5 vide
a
b
c
Alors que je voudrais :
1 a
2 b
etc...

Et 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
30
31
32
33
34
35
36
37
38
39
40
41
42
				$mycsvfile = "CAP_quadri_3_2008_VFIN.csv";
 
				// j'ouvre le fichier
				$fh = fopen ($mycsvfile, 'r');
				// Je charge tout le fichier dans une variable $contents
				$contents = fread($fh, filesize($mycsvfile));
				// Je ferme le handle de fichier
				fclose ($fh);
 
				// J'utilise une fonction pour tout exploser
				$alldata = explodedata(trim($contents));
 
				// Pour chacune des lignes de $aldata
				for ($x=0;$x<count($alldata);$x++) {
					for ($y=0;$y<count($alldata[0]);$y++) {
						$evenement = $alldata[$x][$y];
						echo " $evenement ";
						if ($x==1)
						{
							$resultat = mysql_query("INSERT INTO edt (date) VALUES ('".$evenement."')") or die ('Erreur dans la requête : '.mysql_error());
						}
						else if ($x==2)
						{
							$resultat = mysql_query("INSERT INTO edt (jour) VALUES ('".$evenement."')") or die ('Erreur dans la requête : '.mysql_error());
						}
					}
				}
 
				//Fonction EXPLODEDATA
				function explodedata($contents) {
				//Initialisation des tableaux
				$mydata = array();
				$mycontents = array();
				//Je découpe toutes les lignes
				$mydata = split("\n|\r\n", $contents);
				//Pour chacun des lignes
				for ($x=0;$x<count($mydata);$x++) {
					//Je splitte la ligne via les points-virgules
					$mycontents[$x] = split (";", $mydata[$x]);
				}
				return ($mycontents);
				}

Je suis pas très doué et j'ai du mal à comprendre pourquoi il me fait ça

Merci !

[sabotage]

Bonjour,

Il y a surement moyen de debugger ton code actuel mais je te conseillerais plutot d'utiliser les fonctions CSV de PHP comme fgetcsv().

[jean1190]

J'ai essayé avec fgetcsv, et ça ne change rien pour moi. Je récupère très bien mon tableau comme je le désire (le echo $evenement marche tres bien et affiche tout dans l'ordre).

En fait, je viens de comprendre qu'à chaque insert into je crée une ligne entière, donc normal qu'il ne puisse pas revenir à la 1ère ligne au prochain insert into, même si c'est pour remplir la 2ème colonne.
C'est là dessus que je coince !

[sabotage]

D'ou l'interet des fonctions csv qui récupérent ligne par ligne, toutes les colonnes.
Je veux dire que ton code actuel est une artillerie pour reproduire ce que la fonction fait déjà.

De plus parcourir un tableau avec des boucles for/count c'est beaucoup plus lent qu'un foreach(), surtout quand le count est dans la boucle.

[jeca]

Bonjour,

Montre la structure de ton .CSV, ainsi que celle de la table.

[jean1190]

D'ou l'interet des fonctions csv qui récupérent ligne par ligne, toutes les colonnes.
Je veux dire que ton code actuel est une artillerie pour reproduire ce que la fonction fait déjà.

De plus parcourir un tableau avec des boucles for/count c'est beaucoup plus lent qu'un foreach(), surtout quand le count est dans la boucle.

Ok ok... Je vais me forcer à approfondir de ce côté là
Merci !

[narutobaka]

plop plop

ca te donnera quelque chose du style

Code :

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

1
2
3
4
5
 
foreach ($alldata as $key => $value) {
// echo "Clé : $key; Valeur : $value<br />\n";
//ta requête d'insertion des valeurs ou $key représente $x et $value représente //$y dans $alldata[$x][$y]
}

http://de2.php.net/manual/fr/control...es.foreach.php

par contre je vois pas du tout à quoi servent tes conditionnelles :

Code :

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

1
2
3
4
5
6
if ($x==1)
{
}
else if ($x==2)
{
}

Surtout que si tu voulais faire une insertions dans le champs 1 si $x==1 et dans le champs 2 si $x==2 cela ne fonctionnerai pas .

Il aurait fallut utilisé la technique du $x%2 qui vaut 1 pour les valeur impair de $x et qui vaut 0 pour les valeurs paire de $x.

Autre chose la tu effectues la création de 2 lignes dans ta base pour une ligne de ton tableau. 2 INSERT INTO => 2 lignes créés dans ta base.

Quoiqu'il en soit avec le foreach tu n'as qu'à récupérer les valeurs de $key et $values pour ton INSERT INTO.

voili voilou

++

[jean1190]

En fait c'était débile ma façon de faire et complètement incohérent.

J'ai réduis à un seul for :

Code :

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

1
2
for ($i=0;$i<count($alldata[0]);$i++) {
$resultat= mysql_query("INSERT INTO edt (col1, col2, col3...) VALUES ('".$alldata[0][$i]."','".$alldata[1][$i]."', ...)")

C'est moche, TRES moche, mais ça marche.
Donc je vais essayer d'améliorer en utilisant fgetcsv etc...

[Edit] Merci pour le foreach, je vais changer de suite