Récupération de couple de valeur

**Premium** · 31/05/2006, 10h48

Bonjour,

j'ai un fichier de ce type par exemple(le nombre de ligne n'est pas connue à l'avance)

Code :

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

1
2
3
4
 
134.154.6.2,276.143.76.65
165.212.243.54,127.54.9.5
206.165.232.18,65.143.123.221

j'ai écrit ce code pour lire le fichier et afficher le couple mais il n'est pas correct
J'aurais besoin d'aide pour le compléter
Merci

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>
 
#define F "fichier"
 
 
static void purge(FILE *fp)
{
   int c;
   while ((c = fgetc(fp)) != '\n' && c != EOF)
   {
   }
}
 
static void clean (char *s, FILE *fp)
{
   /* search ... */
   char *p = strchr (s, '\n'); /* <string.h> */
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      purge (fp);
   }
}
 
 
typedef struct {
  char *duoIP[2];
} t_couple;
 
 
int main(void)
{
  int nb = 0 ,size = 5;
  FILE *fp = fopen("voisin","r");
 
  t_couple *tabCouple;
  tabCouple = (t_couple*)malloc(sizeof(t_couple));
  if(tabCouple == NULL)
    {
      printf("erreur allocation\n");
      exit(1);
    }
 
  if (fp != NULL)
    {
      char line[BUFSIZ];
 
      while (fgets(line, sizeof line, fp) != NULL)
	{
	  clean(line, fp);
	  char *p = line;
	  if(nb == size)
	    {
	      size += 16;
 
	      tabCouple = realloc(tabCouple, size * sizeof(t_couple));
	      if (tabCouple == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	    }
 
	  int ip1[4],ip2[4];
	  int n = sscanf(p, "%d.%d.%d.%d,%d.%d.%d.%d", ip1, ip1 + 1, ip1 + 2, ip1 + 3, ip2, ip2 + 1, ip2 + 2, ip2 + 3);
	  if(n == 9)
	    {
	      tabCouple[nb].duoIP[0] = malloc((strlen(ip1)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[0] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      strcpy(tabCouple[nb].duoIP[0],p);
 
	      tabCouple[nb].duoIP[1] = malloc((strlen(ip2)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[1] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      strcpy(tabCouple[nb].duoIP[1],p);
 
	      nb++;
	    }
	  else
	    {
	      printf ("format error\n");
	      break;
	    }
	}
      fclose (fp), fp = NULL;
    }
  else
    {
      perror (F);
    }
  assert (fp == NULL);
 
  int i;
  for(i=0; i<nb; i++)
    printf("couple : %s et %s\n", tabCouple[i].duoIP[0], tabCouple[i].duoIP[1]);
 
  return 0;
}

**fearyourself** · 31/05/2006, 11h00

Faudrait-il regarder les warnings d'abord non?

test.c: In function 'main':
test.c:76: warning: passing argument 1 of 'strlen' from incompatible pointer type
test.c:85: warning: passing argument 1 of 'strlen' from incompatible pointer type

Jc

**Jack_serious** · 31/05/2006, 11h00

Code :

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

1
2
3
4
 
134.154.6.2,276.143.76.65
165.212.243.54,127.54.9.5
206.165.232.18,65.143.123.221

Code :

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

1
2
3
 
	  int n = sscanf(p, "%d.%d.%d.%d,%d.%d.%d.%d", ip1, ip1 + 1, ip1 + 2, ip1 + 3, ip2, ip2 + 1, ip2 + 2, ip2 + 3);
	  if(n == 9)

Pour commencer change la valeur par 8.

En effet sscanf() te retourne le nombre d'items assignes. (pas francais ca).
Ici tu as 8 fois "%d". Donc si le format de ton fichier est correct, sscanf() et retournera 8, et non 9.

**Premium** · 31/05/2006, 11h03

Envoyé par fearyourself

Faudrait-il regarder les warnings d'abord non?

Jc

Bonjour,

Comment est-ce qu'on convertit un entier en chaine de caractère?

**Premium** · 31/05/2006, 11h06

Envoyé par Jack_serious

Code :

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

1
2
3
4
 
134.154.6.2,276.143.76.65
165.212.243.54,127.54.9.5
206.165.232.18,65.143.123.221

Code :

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

1
2
3
 
	  int n = sscanf(p, "%d.%d.%d.%d,%d.%d.%d.%d", ip1, ip1 + 1, ip1 + 2, ip1 + 3, ip2, ip2 + 1, ip2 + 2, ip2 + 3);
	  if(n == 9)

Pour commencer change la valeur par 8.

En effet sscanf() te retourne le nombre d'items assignes. (pas francais ca).
Ici tu as 8 fois "%d". Donc si le format de ton fichier est correct, sscanf() et retournera 8, et non 9.

Et la virgule comprise dans le fichier n'a pas à être lu ?

**gnto** · 31/05/2006, 11h06

Envoyé par Premium

Bonjour,

Comment est-ce qu'on convertit un entier en chaine de caractère?

la fonction itoa de mémoire

**fearyourself** · 31/05/2006, 11h08

Envoyé par gnto

Transtypage !!

N'importe quoi.

Bonjour,

Comment est-ce qu'on convertit un entier en chaine de caractère?

On utilise itoa ou sprintf.

Jc

**gnto** · 31/05/2006, 11h10

désolé j'ai modifé entre temps j'avais mal lu

**gnto** · 31/05/2006, 11h12

Envoyé par Premium

Bonjour,

Code :

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

1
2
3
4
5
6
7
 
char line[BUFSIZ];
 
while (fgets(line, sizeof line, fp) != NULL)
{ 
...
}

BUFSIZ == sizeof line nan ?

**gnto** · 31/05/2006, 11h17

Code :

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

1
2
3
4
 typedef struct {
    char duoIP[2][16];
    int nbDuoIP;
  } t_couple;

t'éviterais les allocations dynamiques

**Premium** · 31/05/2006, 11h18

Alors j'ai modifié le code de sorte de pouvoir récupérer les 2 valeurs : la première est dans line, la seconde dans p.
A l'execution, j'obtiens :

Code :

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

format error

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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <assert.h>
 
#define F "fichier"
 
 
static void purge(FILE *fp)
{
   int c;
   while ((c = fgetc(fp)) != '\n' && c != EOF)
   {
   }
}
 
static void clean (char *s, FILE *fp)
{
   /* search ... */
   char *p = strchr (s, '\n'); /* <string.h> */
   if (p != NULL)
   {
      /* ... and kill */
      *p = 0;
   }
   else
   {
      purge (fp);
   }
}
 
 
typedef struct {
  char *duoIP[2];
} t_couple;
 
 
int main(void)
{
  int nb = 0 ,size = 5;
  FILE *fp = fopen("fichier","r");
 
  t_couple *tabCouple;
  tabCouple = (t_couple*)malloc(sizeof(t_couple));
  if(tabCouple == NULL)
    {
      printf("erreur allocation\n");
      exit(1);
    }
 
  if (fp != NULL)
    {
      char line[BUFSIZ];
 
      while (fgets(line, sizeof line, fp) != NULL)
	{
	  clean(line, fp);
	  char *p = strtok(line, ",");
	  if(p == NULL)
	    {
	      printf("erreur\n");
	      exit(1);
	    }
	  *p = '\0';
	  p++;
 
	  if(nb == size)
	    {
	      size += 16;
 
	      tabCouple = realloc(tabCouple, size * sizeof(t_couple));
	      if (tabCouple == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	    }
 
	  int ip1[4],ip2[4];
	  int n = sscanf(p, "%d.%d.%d.%d,%d.%d.%d.%d", ip1, ip1 + 1, ip1 + 2, ip1 + 3, ip2, ip2 + 1, ip2 + 2, ip2 + 3);
	  if(n == 8)
	    {
	      tabCouple[nb].duoIP[0] = malloc((strlen(line)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[0] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      strcpy(tabCouple[nb].duoIP[0],line);
 
	      tabCouple[nb].duoIP[1] = malloc((strlen(p)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[1] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      strcpy(tabCouple[nb].duoIP[1],p);
 
	      nb++;
	    }
	  else
	    {
	      printf ("format error\n");
	      break;
	    }
	}
      fclose (fp), fp = NULL;
    }
  else
    {
      perror (F);
    }
  assert (fp == NULL);
 
  int i;
  for(i=0; i<nb; i++)
    printf("couple : %s et %s\n", tabCouple[i].duoIP[0], tabCouple[i].duoIP[1]);
 
  return 0;
}

**Premium** · 31/05/2006, 11h19

Envoyé par gnto

Code :

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

1
2
3
4
 typedef struct {
    char duoIP[2][16];
    int nbDuoIP;
  } t_couple;

t'éviterais les allocations dynamiques

Le problème est qu'on ne connait pas le nombre de ligne à l'avance

**gangsoleil** · 31/05/2006, 11h20

Envoyé par Premium

Et la virgule comprise dans le fichier n'a pas à être lu ?

Si, elle va être lue. Mais si tu regardes le man de sscanf, tu pourras y lire :

Ces fonctions renvoient le nombre d'éléments d'entrées correctement assignés.

C'est à dire si tu préfères, dans un langage plus courant (et donc moins correct), le nombre de "%" qui seront correctement lus par sscanf.

**gnto** · 31/05/2006, 11h22

Envoyé par Premium

Le problème est qu'on ne connait pas le nombre de ligne à l'avance

oui c'est donc pour ca que tu alloues dynamiquement juste le tableau de t_couple mais pas les double chaines d'IP
je parle de ça

Code :

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

char * duoIP[2];

en ça

Code :

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

char duoIP[2][16];

parceque tu sais qu'une adresse IP c'est 16 caracteres

**Premium** · 31/05/2006, 11h35

J'ai mis des printfs un peu partout et ça m'indique que cette partie du code est fausse :
Je n'ai pas l'affichage de sscanf ok

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
if(n == 8)
	    {
	      printf("sscanf ok\n");
	      tabCouple[nb].duoIP[0] = malloc((strlen(line)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[0] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      printf("Copie1\n");
	      strcpy(tabCouple[nb].duoIP[0],line);
 
	      tabCouple[nb].duoIP[1] = malloc((strlen(p)+1) * sizeof(t_couple));
	      if (tabCouple[nb].duoIP[1] == NULL)
		{
		  perror("erreur allocation\n");
		  free(tabCouple); tabCouple = NULL;
		  exit(1);
		}
	      printf("Copie2\n");
	      strcpy(tabCouple[nb].duoIP[1],p);
 
	      nb++;
	    }

**gnto** · 31/05/2006, 11h43

Code :

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

strcpy(tabCouple[nb].duoIP[0],line);

Je comprends pas la philosophie de ton code
tu mets la ligne, donc les deux IP séparé d'une virgule dans la premier chaine qui s'appel duoIP[0].

Perso je croyais que un élements de duoIP représenté une IP sous forme de chaine de caratères et pas une ligne avec 2 IP séparé par une virgule

**gnto** · 31/05/2006, 11h48

je viens de voir une fonction dont il faut etre très méfiant sur son utilisation

Le strtok(), moi je le déconseille. D'ailleur regarde que valent p et line après utilisation de strtok

**Premium** · 31/05/2006, 12h03

Envoyé par gnto

Code :

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

strcpy(tabCouple[nb].duoIP[0],line);

Je comprends pas la philosophie de ton code
tu mets la ligne, donc les deux IP séparé d'une virgule dans la premier chaine qui s'appel duoIP[0].

Perso je croyais que un élements de duoIP représenté une IP sous forme de chaine de caratères et pas une ligne avec 2 IP séparé par une virgule

Ce que je veux faire c'est lire une ligne de cette forme : 1,2 (les chiffres sont des IPs)
et mettre dans duoIP[0] 1 et dans duoIP[1] 2

Ce n'est apparamment pas ce que fait le programme

J'ai fait l'affichage de line, je n'obtiens rien et pour p, cela m'affiche la 1ère valeur du couple tronqué du 1er chiffre.

Si quelqu'un pouvait m'aider, à corriger le code

**gege2061** · 31/05/2006, 12h26

Bonjour,

c'est beaucoup plus simple d'utiliser fgets, avec str_istr et str_dup, ça se fait très bien

**Premium** · 31/05/2006, 13h16

Salut,

alors depuis tout à l'heure je bloque ...
Je voudrais juste savoir comment à partir d'une chaine de caractere
chaine = "1.54.65.176,132.54.76.56"
créer 2 chaine
chaine1 = "1.54.65.176"
et
chaine2 = "132.54.76.56"

Merci

Récupération de couple de valeur

C

Discussions similaires

Partager

Partager