fichiers bug indétectable

Version imprimable

bon voila, j'ai retourner le problème dans tout les sens et je voit pas pourquoi le programme se comporte mal...

on as deux fichiers file1 et file2
un autre fichier fileR qui contiendra tout les élements qui sont à la fois dans file1 et dans file2.
fileR = file1 /\ file2

Code:

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
 
void intersection(const char* file1, const char* file2) {
   FILE *p_file1 = fopen(file1,"r");	// mode lecture seule
	if (!p_file1) {printf("Erreur! Le fichier n'existe pas!\n"); return;}
	FILE *p_file2 = fopen(file2,"r");
	if (!p_file2) {printf("Erreur! Le fichier n'existe pas!\n"); return;}
 
	printf("Fichier rÃ©sultat.\n");
	char Name[MAX];
	FILE *p_fileR = fopen(saisie(Name),"w");
	if (!p_fileR) {printf ("Erreur de création\n"); return;}
 
	t_element dataf1 = lire_element (p_file1);
	while (!feof(p_file1))
	{
      rewind(p_file2);  // se placer en début de fichier
      t_element dataf2 = lire_element (p_file2);
      while (!feof(p_file2)) {
         if (comparer_data(dataf1, dataf2)) {
            fprintf(p_fileR, "%s %s %d\n", dataf2.nom, dataf2.prenom, dataf2.taille);
            break;
         }
         dataf2 = lire_element (p_file2);
      }
		dataf1 = lire_element (p_file1);
	}
	fclose(p_file2); fclose(p_file1); fclose(p_fileR);
	printf("enregistrement terminÃ©.\n");
	return;
}

le problème, c'est que le fichier résultat contient les éléments de file2... et je ne comprend pas ou est l'erreur.

25/03/2007, 22h31
Skyounet

Il manque la moitié des fonctions. Ton code est crade au possible.
25/03/2007, 23h06
Emmanuel Delahaye

Citation:

Envoyé par flolem

bon voila, j'ai retourner le problème dans tout les sens et je voit pas pourquoi le programme se comporte mal...<...>

Ce code n'est pas compilable.
25/03/2007, 23h21
tinico

Ton programme a l'air de fonctionner, mais il faut que tu nous donnes tes fonctions comparer_data() et lire_element().

A mon avis, cela vient de l'un de ses deux fonction, surement comparer_data() !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
t_element lire_element (FILE *p_file) {
   t_element data;
   fscanf(p_file, "%s", data.nom);
   fscanf(p_file, "%s", data.prenom);
   fscanf(p_file, "%d", &data.taille);
   return data;
}
 
int comparer_data (const t_element data1, const t_element data2) {
   return (data1.taille == data2.taille &&
            !strcmp(data1.nom, data2.nom) &&
            !strcmp(data1.prenom, data2.prenom));
}

mais ces deux fonctions sont utilisés autre part dans le programme et marche trés bien...

26/03/2007, 23h59
Thierry Chappuis
Citation:
Envoyé par Emmanuel Delahaye

Ben oui, classique. Manque un ';'

Code:

1 2 3 4 5 t_element lire_element (FILE *p_file) { <...> };
Tu mets un ';' à la fin de la définition d'une fonction?

Thierry
27/03/2007, 00h35
Emmanuel Delahaye

Citation:

Envoyé par mujigka

Tu mets un ';' à la fin de la définition d'une fonction?

Thierry

Je fatigue. J'ai vu une définition de structure... Ca va pas bien...

Citation:

Envoyé par flolem

mais ces deux fonctions sont utilisés autre part dans le programme et marche trés bien...

Montre du code compilable...

Ceci est correct :

Code:

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
 
#include <stdio.h>
#include <string.h>
 
typedef struct
{
   char nom[32];
   char prenom[32];
   int taille;
}
t_element;
 
t_element lire_element (FILE * p_file)
{
   t_element data;
   fscanf (p_file, "%s", data.nom);
   fscanf (p_file, "%s", data.prenom);
   fscanf (p_file, "%d", &data.taille);
   return data;
}
 
int comparer_data (const t_element data1, const t_element data2)
{
   return (data1.taille == data2.taille &&
           !strcmp (data1.nom, data2.nom) &&
           !strcmp (data1.prenom, data2.prenom));
}
 
int main (void)
{
   return 0;
}

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define MAX 260
 
typedef struct{
	char nom[MAX], prenom[MAX];
	int taille;
} t_element;
 
char* saisie (char* nom_fichier) {
	printf("Saisir nom du fichier: ");
	scanf ("%s", nom_fichier);
	return nom_fichier;
}
 
t_element lire_element (FILE *p_file) {
   t_element data;
   fscanf(p_file, "%s", data.nom);
   fscanf(p_file, "%s", data.prenom);
   fscanf(p_file, "%d", &data.taille);
   return data;
}
 
// fonction retourne 0 si les deux enregistrement sont différent
int comparer_data (const t_element data1, const t_element data2) {
   return (data1.taille == data2.taille &&
            !strcmp(data1.nom, data2.nom) &&
            !strcmp(data1.prenom, data2.prenom));
}
 
int main (void) {
  char FileName[MAX];
  intersection(saisie(FileName),saisie(FileName));
  return 0;
}
 
void intersection(const char* file1, const char* file2) {
   FILE *p_file1 = fopen(file1,"r");	// mode lecture seule
   if (!p_file1) {
     printf("Erreur! Le fichier n'existe pas!\n"); return;
  }
   FILE *p_file2 = fopen(file2,"r");
   if (!p_file2) {
      printf("Erreur! Le fichier n'existe pas!\n"); return;
  }
 
	printf("Fichier résultat.\n");
	char Name[MAX];
	FILE *p_fileR = fopen(saisie(Name),"w");
	if (!p_fileR) {
           printf ("Erreur de création\n"); return;
        }
 
	t_element dataf1 = lire_element (p_file1);
	while (!feof(p_file1))
	{
          rewind(p_file2);  // se placer en début de fichier
          t_element dataf2 = lire_element (p_file2);
          while (!feof(p_file2)) {
            if (comparer_data(dataf1, dataf2)) {
              fprintf(p_fileR, "%s %s %d\n", dataf2.nom, dataf2.prenom, dataf2.taille);
              break;
            }
            dataf2 = lire_element (p_file2);
          }
          dataf1 = lire_element (p_file1);
	}
	fclose(p_file2); fclose(p_file1); fclose(p_fileR);
	printf("enregistrement terminé.\n");
	return;
}

ce code devrait compiler.
il faut préalable avoir deux nouveau fichier texte dans la racine de l'exe.

par exemple:
> toto.txt
nom1 prenom1 1
n2 p2 2

> titi.txt
nom1 prenom1 1
n1 p1 1

normallement en appelant:
intersection("toto.txt", "titi.txt");
le fichier résultat ne devrait contenir que "nom1 prenom1 1"
or il contient tout le contenu de titi.txt

Moi j'obtiens ceci :
Code:

1 2 3 4 5 6 inter.c: In function 'main': inter.c:34: warning: implicit declaration of function 'intersection' inter.c:33: warning: unused variable 'FileName' inter.c: At top level: inter.c:38: warning: conflicting types for 'intersection' inter.c:34: warning: previous implicit declaration of 'intersection' was here
Du coup, j'ai rajouté ceci au début du fichier :

Code:

void intersection(const char* file1, const char* file2);

Et le code a tourné et mon fichier résultat avait seulement :

Citation:

nom1 prenom1 1

Jc

27/03/2007, 17h31
Florian.L

mhhh c'est bizarre... chez moi ça marche pas :(

t'a essayer en rajoutant d'autres lignes sur les fichiers test voir si ça marche toujours...
27/03/2007, 17h42
stephl

Il y a au moins un petit problème dans le code proposé. Dans main(), l'appel:

Code:

intersection(saisie(FileName),saisie(FileName));

utilise le même buffer pour stocker les deux noms de fichier. Forcément, l'un des noms écrasera l'autre et au final, intersection() sera appelée avec deux fois le même nom de fichier.
27/03/2007, 19h08
Florian.L

Citation:

Envoyé par stephl

Il y a au moins un petit problème dans le code proposé. Dans main(), l'appel:

Code:

intersection(saisie(FileName),saisie(FileName));

utilise le même buffer pour stocker les deux noms de fichier. Forcément, l'un des noms écrasera l'autre et au final, intersection() sera appelée avec deux fois le même nom de fichier.

mhhh faut croire que l'erreur vient d'ici.

j'ai essayer le programme en enlevant la saisie et en mettant directement les noms des fichiers et ça marché trés bien...

bon bah merci à toi :)

perdu pas mal d'heure sur un problème pareil, c'est triste:?
28/03/2007, 12h10
fearyourself

Citation:

Envoyé par stephl

Il y a au moins un petit problème dans le code proposé. Dans main(), l'appel:

Code:

intersection(saisie(FileName),saisie(FileName));

utilise le même buffer pour stocker les deux noms de fichier. Forcément, l'un des noms écrasera l'autre et au final, intersection() sera appelée avec deux fois le même nom de fichier.

En effet, j'avais aussi modifier cette partie du code et j'ai mis les noms des fichiers :oops:

Jc