Discussion :

[laurentSc]

Bonsoir,

je lis un fichier CSV et je compte analyser chaque colonne. Ce que je pensais faire, c'est créer une classe pour chaque colonne, comme ça :

Code php :

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

1
2
3
4
5
6
7
 
while (false !== ($fields = fgetcsv($handle, $buffer, $this->sep)) ) {     
                $line_csv=[];    
                $line_csv[0]=new Field0($fields[0]);
                $line_csv[1]=new Field1($fields[1]);
                ...
                $line_csv[23]=new Field23($fields[23]);

donc pour la colonne X $line_csv[X]=new FieldX($fields[X]); avec la classe FieldX :
FieldX :

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
<?php
namespace Classes\Olivier;
 
 
class FieldX
{
    private $name;
   /* Constructor */
public function __construct($value)
  {
    if (is_string($value))
        $this->name=$value;
  }
 
public function __tostring()
  {
      return($this->name);
  }    
} //end FieldX

Ca fait un code sacrément lourd (et plein de boulot). Comme je pense (et espère) qu'on peut faire mieux, j'ai pas été loin. Quelle amélioration me proposez-vous ?

[rawsrc]

salut Laurent

parser un CSV avec 24 colonnes=24 classes ?

tu dérailles ou quoi ?
Ce n'est parce que tu t'es mis à la prog objet qu'il faut te sentir obligé de pondre des classes à tout bout de champ !
Bon sens, raisonnement et simplicité : voilà, ce que tu dois utiliser en premier.

Alors fais toi une simple fonction de validation d'un ligne du csv et puis c'est tout : utilise les closures si tu veux mais par pitié ne t'embarque pas dans 24 classes !

[laurentSc]

Non, je ne déraille pas et il était clair que c'était lourdingue, donc je me suis très vite arrêté mais je n'avais pas d'idée comment faire simple et j'avoue que je sèche encore ; peux-tu préciser comment utiliser les closures ?

[rawsrc]

liste d'abord les tests de validation que tu souhaites faire.

[laurentSc]

En fait, pour l'instant, je ne vois que des tests de type (is_string ou is_integer ou une date (je sais faire)) avec éventuellement au préalable une manipulation (exemple : un champ "id" avec un préfixe INC, donc d'abord supprimer le préfixe puis vérifier si le reste est bien integer).

[rawsrc]

pour les tests natifs, t'as besoin de rien pour ceux qui nécessitent un peu plus, tu peux faire dans ce style (closure) :

Code :

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

1
2
3
4
5
6
7
8
9
$is_date = function($date): bool {
    $formats = ['M d, Y', 'M d Y', 'M d', 'M-d'];
    foreach ($formats as $f) {
        if (\DateTime::createFromFormat($f, $date) !== false) {
            return true;
        }
    }
    return false;
};