fgetcsv trompé malgré le délimiteur

**laurentSc** · 14/12/2021, 11h36

Bonjour,

Mon fichier CSV contient :

"ACTIVATEDATE","DEACTIVATEDATE"
"Jan 01, 2021",""

Donc le délimiteur est ".
Pour le lire, j'ai

Code php :

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

1
2
while (($row = CSVParser::getRow()) !== false) {
            $lineCSV = array_combine(CSVParser::getHeader(), explode(",",$row[0]));

avec

Code php :

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

1
2
3
public static function getRow(int $buffer = 4096) {
        return(fgetcsv(self::$handle, $buffer, self::$separator, self::$enclosure,""));
    }

et

Problème du explode : s'il y a une virgule dans $row[0], il va extraire 2 éléments au lieu d'un seul. Je dois faire un explode car tous les éléments d'une ligne du CSV sont contenus dans la même cellule. Le souci ici est que le header va contenir 2 éléments mais la ligne 3. Comment faut-il faire, SVP ?

**Séb.** · 14/12/2021, 14h25

Tu ne devrais pas avoir à faire de explode()

Il faut que $separator vale ,
et $delimiter "

Est-ce le cas ?

**laurentSc** · 14/12/2021, 14h42

Presque ; j'ai vu d'où vient le problème ; j'avais raccourci la ligne par souci de clarté, mais voici une ligne complète :

SESA1000,"(none)","Software Engineering","SESA84441","Atlassian","JIRA Global Instance","","True","Jan 01, 2021","","","Spain","Miguel","Abad","SCHNEIDER ELECTRIC ESPANA, S.A.U.","FKP8","","Global Finance","BARCELONA (BAC DE RODA)","miquel.abad@se.com","Alistair","Mckelvie","alistair.mckelvie@se.com"

Tous ces éléments sont regroupés dans une seule cellule du CSV. On voit que tous les éléments sont encadrés par le délimiteur ", sauf le premier !

Je vais m'absenter environ une heure...

**Séb.** · 14/12/2021, 14h48

Presque

Que valent self::$delimiter/$separator dans ton script ?

**laurentSc** · 14/12/2021, 23h02

Bonsoir, en fait, en voulant réparer, j'ai tout cassé et la dernière sauvegarde date du 30 novembre

Donc demain, j'essaie de remettre en ordre et je reprendrai cette discussion...Bonne nuit

**laurentSc** · 16/12/2021, 10h26

Tous comptes faits, y avait pas grand chose à faire, mais y a quand même 2 jours qui se sont écoulés

On a les propriétés $enclosure='"' et $separator=','. Quand tu parles de $delimiter, je suppose qu'il s'agit en fait de $enclosure...

**Séb.** · 16/12/2021, 14h27

On a les propriétés $enclosure='"' et $separator=','

Montre-nous où dans ton code tu obtiens ces valeurs stp.

**laurentSc** · 16/12/2021, 14h38

Code php :

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
53
54
55
 public static function detectSyntaxFromHeader(): bool
    {
        if (isset(self::$handle)) {
            $row = fgets(self::$handle);
            $row=strtolower($row); //mettre header en minuscules
        } // doit contenir les en-têtes des colonnes}
        if (empty($row)) {
            self::$errors[] = 'The file is empty';
            return false;
        }
 
        // séparateur
        if (mb_stripos($row, ',') !== false) {
            self::$separator = ',';
        } elseif (mb_stripos($row, ';') !== false) {
            self::$separator = ';';
        } else {
            self::$errors[] = 'Unable to detect the CSV field separator';
            return false;
        }
 
        $char = mb_substr($row, 0, 1); // premier caractère
        if ($char === "\xef\xbb\xbf") {  //if first character is BOM, one begins with the next character
            $row  = mb_substr($row, 1);
            $char = mb_substr($row, 0, 1);
        }
 
        if (($char === '"') && (mb_stripos($row, '"'.self::$separator.'"') !== false)) {
            self::$enclosure = '"';
        } elseif (($char === "'") && (mb_stripos($row, "'".self::$separator."'") !== false)) {
            self::$enclosure = "'";
        }
 
        if (self::$enclosure) {
            // on vérifie que le premier séparateur trouvé est bien entouré de lettres ASCII
            $i = mb_stripos($row, self::$separator);
            if ($i) {
                $fc = mb_substr($row, $i - 1, 1);
                $lc = mb_substr($row, $i + 1, 1);
                if (ctype_alpha($fc) && ctype_alpha($lc)) {
                    self::$enclosure = '';
                }
            }
        }
        else  self::$enclosure = '';
 
        if (self::$enclosure === false) {
            self::$errors[] = 'Unable to detect the CSV text enclosure or this file has no header';
            return false;
        } else {
            // transform header columns into array
            self::$header = explode(self::$separator, str_replace(self::$enclosure, '', trim($row, "\r\n")));
            return true;
        }
    }

**laurentSc** · 16/12/2021, 14h59

Sinon, j'ai adopté un moyen alambiqué pour m'en sortir :

Code php :

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
 while (($line_csv = fgetcsv(CSVParser::getHandle(), $buffer, CSVParser::getSeparator(), $enclosure,"")) !== false) {
 
            //on lit une ligne du CSV, on l'analyse et si pas d'erreur, en bdd
 
             // découper line_csv en un tableau sép="," si besoin
            if (sizeof($line_csv)==1)
                $line_csv=explode('","', $line_csv[0]);
            //découper 1e cellule avec sép=,
            $first_cell = explode(',',$line_csv[0]);
 
            if (isset($first_cell[1]))
            {
                // on enlève le 1er elt de line_csv et on met les 2 à la place
                array_shift($line_csv);
                array_unshift($line_csv, $first_cell[0], $first_cell[1]);
            }

y a plus d'explode et je dois traiter aussi les cas où ça se présente bien...

**Séb.** · 16/12/2021, 16h14

Envoyé par laurentSc

Code php :

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
53
54
55
 public static function detectSyntaxFromHeader(): bool
    {
        if (isset(self::$handle)) {
            $row = fgets(self::$handle);
            $row=strtolower($row); //mettre header en minuscules
        } // doit contenir les en-têtes des colonnes}
        if (empty($row)) {
            self::$errors[] = 'The file is empty';
            return false;
        }
 
        // séparateur
        if (mb_stripos($row, ',') !== false) {
            self::$separator = ',';
        } elseif (mb_stripos($row, ';') !== false) {
            self::$separator = ';';
        } else {
            self::$errors[] = 'Unable to detect the CSV field separator';
            return false;
        }
 
        $char = mb_substr($row, 0, 1); // premier caractère
        if ($char === "\xef\xbb\xbf") {  //if first character is BOM, one begins with the next character
            $row  = mb_substr($row, 1);
            $char = mb_substr($row, 0, 1);
        }
 
        if (($char === '"') && (mb_stripos($row, '"'.self::$separator.'"') !== false)) {
            self::$enclosure = '"';
        } elseif (($char === "'") && (mb_stripos($row, "'".self::$separator."'") !== false)) {
            self::$enclosure = "'";
        }
 
        if (self::$enclosure) {
            // on vérifie que le premier séparateur trouvé est bien entouré de lettres ASCII
            $i = mb_stripos($row, self::$separator);
            if ($i) {
                $fc = mb_substr($row, $i - 1, 1);
                $lc = mb_substr($row, $i + 1, 1);
                if (ctype_alpha($fc) && ctype_alpha($lc)) {
                    self::$enclosure = '';
                }
            }
        }
        else  self::$enclosure = '';
 
        if (self::$enclosure === false) {
            self::$errors[] = 'Unable to detect the CSV text enclosure or this file has no header';
            return false;
        } else {
            // transform header columns into array
            self::$header = explode(self::$separator, str_replace(self::$enclosure, '', trim($row, "\r\n")));
            return true;
        }
    }

Je ne vois rien ici indiquant comment tu obtiens, pour ce fichier, les valeurs suivantes " On a les propriétés $enclosure='"' et $separator=',' "
J'ai besoin du débug car je doute que ce soit réellement ces valeurs, justement
Ou alors ton fichier CSV n'est pas tel que tu le crois... Comment vérifies-tu ce que ton fichier CSV contient ? Tu l'ouvres avec Excel ?

Et l'algo me paraît tout à faire insuffisant pour détecter comment ton CSV est construit
=> Tu considères le séparateur , si ton en-tête contient au moins 1 , même s'il y a 50 ; et cela sans tenir compte du délimiteur

**laurentSc** · 16/12/2021, 16h48

Honnêtement, j'utilise ce code codé par quelqu'un d'autre (un ancien modo d'ici).

Pour savoir que le délimiteur est un double-quote " et le séparateur est une virgule ,, je fais un debug pas à pas avec PHPSTORM et je peux lire la valeur des variables.

J'édite également mon fichier CSV avec cet IDE ; c'est donc ainsi que je vérifie ces valeurs.

Comme tu n'as pas l'air d'aimer ce code, as-tu mieux à me proposer ?

fgetcsv trompé malgré le délimiteur

Langage PHP

Discussions similaires

Partager

Partager