Discussion :

[Argetloum]

Bonjour à tous .
Alors voilà, fortement intéressé par le php et le dev Web, je m'entraîne un peu sur divers scripts !
Et en voilà un où j'ai un soucis que je n'arrive pas à comprendre:

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
<?php
 
	function displayCSV() {
 
		$fichier = "files/uploadedDoc.csv";
		if (file_exists($fichier)) {
		 $fp = fopen("$fichier", "r");
		}
		else { /* le fichier n'existe pas */
		   echo "Fichier introuvable !<br/>Arret de l'importation.";
		   exit();
		}
		$row = 0;
		while (!feof($fp)){ 
 
			$ligne = fgets($fp,4096); 
 
 
			$liste = explode( ";",$ligne);
			$nbrColonnes = count($liste);
			$tableauTitre = array($nbrColonnes);
			$tableauVariables = array($nbrColonnes);
			$row++;
			if($row == 1) {
				echo "---------------------------------------------<br/>
					Affichage des titres de colonnes: <br/>
					---------------------------------------------<br/><br/>";
				for($i = 0; $i < $nbrColonnes; $i++) {
 
					$tableauTitre[$i] = $liste[$i];
					echo $tableauTitre[$i]."<br/>";
 
				}
				echo "<br/>
					---------------------------------------------<br/>";
				echo "Importation des variables par titre: <br/>
				---------------------------------------------<br/><br/>";
			}
			else {
				$row2 = $row - 1;
				echo $row2."eme Ligne: <br/>";
				for($j = 0; $j < $nbrColonnes; $j++) {
 
					$tableauVariables[$j] = $liste[$j];
					echo $tableauTitre[$j]." = ".$tableauVariables[$j]."<br/>";
 
				}
			}
 
		}
	}
?>

Je n'arrive pas à comprendre l'erreur:
Elle se situe dans la seconde boucle "for".
Notice: Undefined offset: 1 in C:\xxx\displayCSV.php on line 45

Et cette erreur est répétitive pour chacune des colonnes de mon fichier csv ...


Merci d'avance à ceux qui trouveront une réponse à mon problème

[Séb.]

Code :

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

1
2
$tableauTitre = array($nbrColonnes);
$tableauVariables = array($nbrColonnes);

Ici tu ne dis pas que $tableauTitre/Variables pourront contenir $nbrColonnes valeurs, tu ne fais que les initialiser avec 1 entier valant $nbrColonnes (ce qui ne sert à rien et n'a aucun sens dans ton script).

Le pb vient du fait que $tableauTitre est réinitialisé à chaque boucle du while( ), tu perds son contenu après le 1er tour, d'où les "undefined index" s'ensuivant.

[stealth35]

pour parser du csv c'est avec fgetcsv

[Argetloum]

Oups, je viens de corriger ^^.
Erreur bête !
Et oui, j'avais déjà vu pour fgetcsv, mais comme j'étais déjà parti avec un simple fopen, pourquoi ne pas continuer ? ^^

Merci pour tout en tout cas

[stealth35]

Oups, je viens de corriger ^^.
Erreur bête !
Et oui, j'avais déjà vu pour fgetcsv, mais comme j'étais déjà parti avec un simple fopen, pourquoi ne pas continuer ? ^^

Merci pour tout en tout cas

fgetcsv s'utilise avec fopen, c'est le fgets + explose qui n'est pas bon (enfin pas fait pour)

[Séb.]

fgetcsv s'utilise avec fopen, c'est le fgets + explose qui n'est pas bon (enfin pas fait pour)

Pas fait pour, t'y vas fort C'est juste qu'il y a plus simple et plus efficace.

[Séb.]

Vu que tu découvres PHP je me permets de te coller le script tel que je l'aurais écrit
Ce qui est intéressant pour toi est notamment le foreach au lieu du for, le implode( ) qui peut remplacer parfois avantageusement les for/foreach, délimiter les chaînes avec des ' plutôt que des " quand elle ne contiennent pas de variables (évite un parsing inutile), la syntaxe heredoc (<<<) qui permet de créer assez facilement une variable avec un bloc de texte conséquent.

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
<?php
 
function displayCSV($file)
{
    $fp = fopen($file, 'r') ;
 
    if ( $fp === FALSE ) {
        return FALSE ;
    }
 
    $row = 0 ;
    while ( ! feof($fp) ) {
 
        $line = fgets($fp) ;
        $values = explode(';', $line) ;
 
        if ( $row === 0 ) {
            $headers = $values ;
            echo '-------------------<br>' ;
            echo 'TITRES<br>' ;
            echo '-------------------<br><br>' ;
            echo implode('<br>', $headers), '<br><br>' ;
            echo '-------------------<br>' ;
            echo 'VALEURS<br>' ;
            echo '-------------------<br><br>' ;
        } else {
            echo "Ligne n° $row :<br>" ;
            foreach ( $values as $i => $value ) {
                echo "- $headers[$i] = $value<br>" ;
            }
            echo '<br>' ;
        }
 
        $row++ ;
    }
 
    fclose($fp) ;
 
    return TRUE ;
}
 
$data = <<<EOS
Pays;Capitale;Population;Continent
Italie;Rome;61M;Europe
Japon;Tokyo;125M;Asie
France;Paris;65M;Europe
USA;Washington;310M;Amérique
EOS;
 
$file = 'data.csv' ;
file_put_contents($file, $data) ;
displayCSV($file) ;