Modification d'une fonction

**fstlirmm** · 30/12/2009, 17h40

Salut,
Je veux modifier une fonction de lecture d'un fichier des données en entrée. La fonction ne peut lire que des données codées en numéros [1 2 3], je veux la transformer pour lire tout les types des données (char), par exp [a b].
Voilà la fonction:

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
 
void freq_1_itemsets( FILE *fp ) 
{ 
char s[3000], code[10]; 
int n, size =0, i, j, length; 
 
 
t=NULL; 
 
while(fgets( s, 3000, fp )) 
{ 
length = strlen(s)-1; 
for(i=0,j=0,size=0;i<length;i++) 
{ 
while((i<length)&&(s[i]!=' ')) 
code[j++]=s[i++]; 
 
code[j]='\0'; 
if(j) 
{ 
n = atoi(code); 
size++; 
 
if(n > nbitems) 
{ 
nbitems = n; 
t = (int*)realloc(t,(nbitems+1)*sizeof(int)); 
for(j=dernier+1;j<nbitems;j++) 
t[j]=0; 
 
dernier = nbitems; 
t[nbitems]=1; 
} 
else 
t[n]++; 
 
j=0; 
} 
 
 
} 
if(size > maxsizetr) 
maxsizetr = size; 
 
nblines++; 
} 
 
}

L'algorithme est écrit en C.
Merci pour vos aides.

**diogene** · 30/12/2009, 22h13

Bienvenue sur le forum,

Avant de venir à l'extension de ton problème, il faudrait améliorer ton code. On peut lui reprocher :
- D'utiliser un paquet de variables globales (ou non définies) : t, nbitems, dernier, maxsizetr, nblines
- De ne pas respecter l'indentation ce qui le rend illisible. En voici une version plus claire :

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
void freq_1_itemsets( FILE *fp ) 
{ 
   char s[3000], code[10]; 
   int n, size =0, i, j, length;  
   t=NULL; 
   while(fgets( s, 3000, fp )) 
   { 
      length = strlen(s)-1; 
      for(i=0,j=0,size=0;i<length;i++) 
      { 
         while((i<length)&&(s[i]!=' ')) code[j++]=s[i++]; 
         code[j]='\0'; 
         if(j) 
         { 
            n = atoi(code); 
            size++;  
            if(n > nbitems) 
            { 
               nbitems = n; 
               t = (int*)realloc(t,(nbitems+1)*sizeof(int)); 
               for(j=dernier+1;j<nbitems;j++) t[j]=0; 
               dernier = nbitems; 
               t[nbitems]=1; 
            } 
            else t[n]++;  
            j=0; 
         }  
      } 
      if(size > maxsizetr) maxsizetr = size;  
      nblines++; 
   } 
}

Pour ta question, il faut que tu précises un peu ce que tu entends par "tous les types des données". Veux-tu dire que tu souhaites stocker des char ? des chaines de caractères ? Séparées comment ?

**fstlirmm** · 31/12/2009, 12h04

Salut,
Merci pour l’amélioration de la fonction.
En fait, je veux stocker des chaines de caractères séparées par des espaces.
Comme, par exemple :

http 30 10.2.3.150
tcpmux 32 14.24.58.190
auth 80 14.24.58.190

Merci pour vos aides.

**diogene** · 31/12/2009, 12h08

Sur une ligne, est-ce que il y a un nombre fixé à priori de chaines (comme dans ton exemple 3) ou le nombre de chaines peut-il varier d'une ligne à l'autre ?

**fstlirmm** · 31/12/2009, 14h27

Sur chaque ligne, le nombre de chaines reste fixe (comme dans l'exemple 3).

**diogene** · 31/12/2009, 16h12

Dans ce cas, on peut envisager quelque chose du genre :

- Définir un type de données contenant 3 chaines de caractères (ici allouées dynamiquement)

Code :

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

1
2
3
4
5
6
typedef struct
{
   char* champ1;
   char* champ2;
   char* champ3;
} Data ;

- Définir un type de données représentant un tableau de Data

Code :

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

1
2
3
4
5
6
typedef struct
{
   Data * data;
   size_t capacite;
   size_t nbData;
}TabData;

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
- Créer un objet t de type TabData et l'initialiser à {NULL,0,0}

- Lire le fichier ligne par ligne : fgets(s,....)
      - Pour chaque ligne
          - Si t.nbData == t.capacite, allouer des Data supplémentaires 
            (realloc) et mettre à jour t.capacite 
          - Lire les 3 champs de s : sscanf(s,"%s%s%s",...).
          - Pour chaque champ, allouer de la mémoire pour le stocker 
            (strlen()+1) dans le prochain Data libre de t (t.data[nbData])
             et copier le champ dedans (strcpy)
          - incrémenter t.nbData
- Redimensionner t.data si  t.capacite > t.nbData et mettre à jour t.capacite 

Pour détruire l'objet t :
- Pour chaque Data du tableau, libérer (free) la mémoire allouée pour les trois champs
- Liberer la mémoire allouée pour le tableau

**fstlirmm** · 01/01/2010, 11h15

Salut,
STP, je veux bien que tu m'ecrit tes explications là dans la fonction (un vrai code).

- Créer un objet t de type TabData et l'initialiser à {NULL,0,0}

- Lire le fichier ligne par ligne : fgets(s,....)
- Pour chaque ligne
- Si t.nbData == t.capacite, allouer des Data supplémentaires
(realloc) et mettre à jour t.capacite
- Lire les 3 champs de s : sscanf(s,"%s%s%s",...).
- Pour chaque champ, allouer de la mémoire pour le stocker
(strlen()+1) dans le prochain Data libre de t (t.data[nbData])
et copier le champ dedans (strcpy)
- incrémenter t.nbData
- Redimensionner t.data si t.capacite > t.nbData et mettre à jour t.capacite

Pour détruire l'objet t :
- Pour chaque Data du tableau, libérer (free) la mémoire allouée pour les trois champs
- Liberer la mémoire allouée pour le tableau

Merci d'avance.

**diogene** · 01/01/2010, 13h00

STP, je veux bien que tu m'ecrit tes explications là dans la fonction (un vrai code).

Bonjour et bonne année

Si tu veux apprendre le langage, ce serait un bien mauvais service à te rendre que de te fournir le code.

Essaie et on t'aidera à y arriver.

Modification d'une fonction

C

Discussions similaires

Partager

Partager