Discussion :

[un passant]

salut, j'ai ecrit un code qui a partir d'un fichier d'entree, remplit une structure de donnee et une liste chainee.
la structure de donnee est de type GRI et s'appelle geo
la liste chainee s'appelle g
voila le code, je voudrais juste savoir si ma manipulation de pointeurs est correcte avant de tester. merci pour votre aide

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
struct gri
{
    int n,nt,nb,nl,nr;
    float x,y;
    struct gri *next, *next1, *next2;      
};
typedef struct gri GRI;
GRI * geo;

main()
{
    char inputfile[80];     
    int taille, i;     
    FILE *fp;
//Lecture du nom de fichier
    puts("Enter file name:");
    gets(inputfile);  
    fp = fopen(inputfile, "r");  
//initialisation du pointeur
    GRI *g=NULL;
//initialisation du pointeur
    GRI *g=NULL;
    GRI *grd=NULL;
//allocation de memoire
    grd=(GRI*)calloc(1,sizeof(GRI));
    if(grd==NULL)
    {
        exit(1);              
    }
    g=grd;
    fscanf(fp,"%d",&taille);
    geo = (GRI*)calloc(taille,sizeof(GRI));
//remplissage de ma structure et liste chainee
    i=0;
    while ( !feof(fp) )
    {
		fscanf(fp,"%d %d %d %d %d %d %d %f %f "
				  ,&geo[i].n,&g->next1,&g->next2,&geo[i].nl,&geo[i].nr,&geo[i].nb,&geo[i].nt
				  ,&geo[i].x,&geo[i].y);
        i++;
        g->next=(GRI*)calloc(1,sizeof(GRI));
        g=g->next;          
    }
    return 0;      
}

J'obtiens cette erreur apres compilation pour la ligne en gras dans le code:
C:\Documents and Settings\mcutp\Desktop\Finite Difference Source\ess.cpp:: error: invalid conversion from `int' to `gri*'

[Médinoc]

Bonjour,

1. Déjà, tu programmes en C, donc renomme-moi ce fichier de .cpp à .c
   Tu vas devoir monter tes déclarations de variables en début de bloc, puisque tu es sous Visual Studio, qui ne supporte pas le standard C99.
2. Inclusions de <stdio.h> et <stdlib.h> manquantes.
3. La fonction main() retourne un int. Toujours.
4. La fonction gets() est un bug et ne doit pas être utilisée. Lui préférer fgets() ou même scanf() avec la taille (scanf("%80s", inputfile) ici).
5. La variable g est déclarée deux fois.
6. En C, le cast du retour de malloc() ou calloc() est inutile et même déconseillé.
7. Mauvaise utilisation de feof(). http://emmanuel-delahaye.developpez....s.htm#fichiers
8. Je ne vois pas ce qui peut causer l'erreur que tu mentionnes. Vérifie qu'elle ne disparait pas une fois les autres erreurs corrigées, mais j'en doute.

[un passant]

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
#include <stdio.h>
#include <stdlib.h>
 
struct gri
{
    int n,nl,nr,nb,nt,c;
    float x,y,dl,dr,db,dt;
    struct gri *next, *next1, *next2;      
};
typedef struct gri GRI;
GRI * geo;
 
int main(void)
{
    char inputfile[80];     
    int taille, i;    
    FILE *fp;
//Lecture du nom de fichier
    puts("Enter file name:");
    scanf("%80s", inputfile);  
    fp = fopen(inputfile, "r");  
//initialisation du pointeur
    GRI *g=NULL;
    GRI *grd=NULL;
//allocation de memoire
    grd=calloc(1,sizeof(GRI));
    if(grd==NULL)
    {
        exit(1);              
    }
    g=grd;
    fscanf(fp,"%d",&taille);
    geo=calloc(taille,sizeof(GRI));
//remplissage de ma structure et liste chainee
    i=0;
    while (fp!=NULL)
    {
		fscanf(fp,"%d %d %d %d %d %d %f %f",&geo[i].n,&g->next1->n,&g->next2->n,&geo[i].nl,&geo[i].nr,&geo[i].nb,&geo[i].nt,&geo[i].x,&geo[i].y);
        i++;
        printf("%d\t%d\n",g->next1->n,g->next2->n);
        g->next=(GRI*)calloc(1,sizeof(GRI));
        g=g->next;   
        if (feof(fp) || ferror(fp))
        {
            break;
        }
    }
    system("PAUSE");
    return 0;      
}

Apres compilation, je n'ai acune erreur mais lors de l'execution le prgoramme rencontre une erreur.
est ce que le remplissage de ma liste chainee est correct?

[Médinoc]

Lors de ton fscanf(), les pointeurs next1 et next2 ne sont pas encore alloués.
De plus, le test d'erreur doit être sur le retour de fscanf() : Les valeurs de feof() et ferror() sont indéfinies tant que fscanf() n'a pas retourné une valeur anormale.

[un passant]

donc quand j'ecris

Code :

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

1
2
grd=calloc(1,sizeof(GRI));
g=grd;

je n'alloue pas de memoire pour next1 et next2?
ils appartiennent tous les 2 a la structure grd.
tu ferais comment a ma place

[Médinoc]

Quand tu fais ça, il y a de la place pour les pointeurs eux-mêmes, mais pas pour les structures pointées.

Malheureusement, je ne peux pas te conseiller car je ne sais pas exactement ce qu'est censé contenir le fichier texte, ni à quoi sert chaque champ de la structure, etc.