Mon programme bugue

Version imprimable

Mon programme bugue

J'ai écrit ce petit programme mais mon compilatur refuse de le compiler :

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
#include <stdio.h>
int c=0 ;
char path[50]= "*" ;
char path2[50][100] ;
char b[100][200], d, f, g, h , a ;
FILE *fp, *fp2[100] ;
main () 
{
     if ((fp=fopen (path, "r")) == NULL)
     {
                   printf("Erreur1\n") ;
                   exit(1) ;
     }
     while ((a = fget(fp)) != EOF)
     {
           gets (b[c], 200, fp) ;
           d= 97 ;
           d= f = g = h ;
           if (d != 122)
              d++ ;
           else
           {
               if (f=122) 
               {
                          if(g=122) ;
                          {
                                    if(h=122) 
                                              exit(1) ;
                                              }}
                                    else
                                        h++ ;
                          else 
                               g++ ;
               else
                   f++ ;
               }
               path2[c] = d ;
               path2[c] = f ;
               path2[c] = g ;
               path2[c] = h ;
               path2[c] = "*" ;
               if ((fp2[c] = fopen(path2[c], "w")) != NULL)
               {
                           puts (b[c], 200, fp2[c]) ;
                           close (fp2[c]) ;
                           c++ ;
               }
               else
               {
                   printf("Erreur2\n") ;
                   exit(1) ;
               }
               close (fp) ;
 
     }

Il est censé prendre les 200 premiers caractères d'un fichier et les mettre dans un autre et ainsi de suite jusqu'à la fin du fichier.

Merci d'avance !

PS: les * sont des adresses de fichiers que j'ai préféré retirer !

EDIT : désolé, j'ai oublié d'afficher les messages d'erreurs :

16 * too many arguments to function `gets'
32 * syntax error before "else"
37 * incompatible types in assignment
38 * incompatible types in assignment
39 * incompatible types in assignment
40 * incompatible types in assignment
41 * incompatible types in assignment
44 * too many arguments to function `puts'
55 * syntax error at end of input

Mon compilo est Dev-C++ !

Re: Mon programme bugue

Citation:

Envoyé par CCin

J'ai écrit ce petit programme mais mon compilatur refuse de le compiler :

Et tu attends quoi pour corriger les fautes ?

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
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
 
#include <stdio.h>
#include <stdlib.h>
 
/* -ed- pourquoi des globales ?
int c=0 ;
char path[50]= "*" ;
char path2[50][100] ;
char b[100][200], d, f, g, h , a ;
FILE *fp, *fp2[100] ;
*/
 
/* -ed- le type retourne de main() doit etre explicite (C99) */
int main (void)
{
    int c=0 ;
    char path[50]= "*" ;
    char path2[50][100] ;
    char b[100][200], d, f, g, h , a ;
    FILE *fp, *fp2[100] ;
 
    if ((fp=fopen (path, "r")) == NULL)
    {
        printf("Erreur1\n") ;
 
        /* -ed- manque <stdlib.h>
        */
        exit(1) ;
    }
    /* -ed-
    while ((a = fget(fp)) != EOF)
    fgets() n'existe pas. Tu veux sans doute dire fgets()... 
    */
    while (fgets (b[c], 200, fp) != NULL)
    {
        /* -ed-
         gets (b[c], 200, fp) ;
         
         Il ne faut pas utiliser gets() qyu est un bug. 
         De plus, les parametres sont errones. Tu voulais sans doute dire 
         fgets()...
         
         en fait l'usage veut que l'on place ce fgets() dans le test du while.
        */
        d= 97 ;
        d= f = g = h ;
        /* -ed-
        deux affectations successives de la meme variable ? 
        C'est louche 
        */
 
        if (d != 122)
            d++ ;
        else
        {
            if (f=122)
            {
                if(g=122)
                    /* -ed-
                                        ;
                     
                    En principe, pas de ';' apres un if()... 
                    */
                {
                    if(h=122)
                        exit(1) ;
                }
            }
            else
                h++ ;
            /* -ed on ne sait pas a quel if ces else se referent...
                        else
                            g++ ;
                        else
                            f++ ;
            */
        }
        /* -ed-
        path2[c] = d ;
        path2[c] = f ;
        path2[c] = g ;
        path2[c] = h ;
        path2[c] = "*" ;
         
        path2 etant un tableau de char a 2 diension, 
        je ne vois pas ce que tu cherches a faire. 
        En tout cas, c'est plutot
         
        path2[c][...] = d ;
        etc.
         
        */
 
        if ((fp2[c] = fopen(path2[c], "w")) != NULL)
        {
            /* -ed-
               puts (b[c], 200, fp2[c]) ;
               
               mauvais parametres. Tu pensais peut etre a printf () ?
               */
            close (fp2[c]) ;
            c++ ;
        }
        else
        {
            printf("Erreur2\n") ;
            exit(1) ;
        }
        close (fp) ;
 
    }
 
}

08/10/2005, 12h36
gl

Re: Mon programme bugue

Citation:

Envoyé par CCin

16 * too many arguments to function `gets'

Il y a trop d'arguent dans l'appel a la fonction gets(). Le prototype de gets est :

Code:

char* gets(char *);

De plus il est fortement deconseille d'utiliser gets(). fgets() est plus sure.

Citation:

Envoyé par CCin

32 * syntax error before "else"

Il y a un ; apres le if a la ligne

Code:

if(g=122) ;

.

Citation:

Envoyé par CCin

37 * incompatible types in assignment
38 * incompatible types in assignment
39 * incompatible types in assignment
40 * incompatible types in assignment
41 * incompatible types in assignment

path2 est un tableau a deux dimensions, or doit tu cherches a affectuer un char a une ligne complete avec un =.
Soit tu veux modifier un seul caractere et dans ce cas il faut indiquer la deuxieme dimension.
Soit tu cherches a copier une chaine complete et il faut alors utiliser strcpy ou une fonction equivalente et fournir une chaine a copier.

Citation:

Envoyé par CCin

44 * too many arguments to function `puts'

Le prototype de puts est :

Code:

int puts(const char*)

Citation:

Envoyé par CCin

55 * syntax error at end of input

Il manque des accolades fermantes

De plus en survolant ton code, j'ai vu pas mal de trucs etranges:

* les = dans les if, ne serait-ce pas plutot ==
* d, f, g et h sont initialisees de maniere tres suprenante
* utilisation de close avec des fichiers ouvert par fopen
* utilisation de gets et puts pour travailler sur des fichiers
* utilisation de tableaux la ou ce n'est pas necessaire
* utilisation de variables globales

Pour copier un fichier dans un autre, je te renvois a la FAQ : http://c.developpez.com/faq/c/?page=...ICHIERS_copier
08/10/2005, 16h06
CCin
EDIT : après avoir relu et corrigé, il y a encore deux petites choses qui me titillent :

j'ai un livre sous les mains qui me prétend que d'écrire :
Code:

1 2 3 4 5 6 while ((c=fget(fp)) != EOF) { ..... .... .... }
indique au fichier que l'opération doit se poursuivre jusqu'au dernier caractère.
Pouvez vous m'en dire plus puisque vous semblez ignorer fget.

Ensuite, j'ai toujours l'erreur :

Citation:

45 * incompatible types in assignment

ce qui paraît logique puisque j'essaye d'introduire des variables de type "int" dans un tableau de types "char" mais, toujours selon le livre, le code suivant :
Code:

1 2 3 4 5 6 7 8 include <stdio.h> main() { char a; int b; b=5 ; a = b ; }
stockerait dans "b" le code ASCII équivalent à "a".

Est-ce vrai ?

Merci
08/10/2005, 17h49
gl
Citation:
Envoyé par CCin

Code:

1 2 3 4 5 6 while ((c=fget(fp)) != EOF) { ..... .... .... }

indique au fichier que l'opération doit se poursuivre jusqu'au dernier caractère.
Pouvez vous m'en dire plus puisque vous semblez ignorer fget.
fget n'existe pas en C standard, c'est plutot fgetc. Cette fonction lit un fichier caractere par caractere et retourne EOF en cas d'echec de lecture.
Code:

1 2 3 4 5 6 while ((c=fgetc(fp)) != EOF) { ..... .... .... }
permet donc de lire le fichier. A chaque iteration de la boucle, c contient le caractere lu.

Citation:

Envoyé par CCin

Ensuite, j'ai toujours l'erreur :

Citation:

45 * incompatible types in assignment

A quelle ligne cette erreur correspond ?
08/10/2005, 17h53
CCin
A celle là

Code:

path2[c] = d ;

mais celles qui suivent déclenchent la même erreur :
Code:

1 2 3 4 5 path2[c] = f ; path2[c] = g ; path2[c] = h ; path2[c] = "*" ;
Citation:
Citation:

while ((c=fget(fp)) != EOF)
{
.....
....
....
}

indique au fichier que l'opération doit se poursuivre jusqu'au dernier caractère.
Pouvez vous m'en dire plus puisque vous semblez ignorer fget.

Citation:

fget n'existe pas en C standard, c'est plutot fgetc. Cette fonction lit un fichier caractere par caractere et retourne EOF en cas d'echec de lecture.

Code:

1 2 3 4 5 6 7 while ((c=fgetc(fp)) != EOF) { ..... .... .... }

permet donc de lire le fichier. A chaque iteration de la boucle, c contient le caractere lu.
Non, non la fonction est bien fget dans le livre !
08/10/2005, 18h09
Emmanuel Delahaye

Citation:

Envoyé par CCin

Non, non la fonction est bien fget dans le livre !

Quel livre ? En tout cas, cette fonction n'est pas standard.
08/10/2005, 18h12
CCin

Mais tout compte fait, je pense qu'il s'agit d'une erreur de frappe de leur part car il ne la mentionne pas dans l'index des fonctions à la fin.

En tout cas, je crois avoir trouvé la solution à mon problème.

Merci !
08/10/2005, 18h40
gl
Citation:
Envoyé par CCin

A celle là

Code:

path2[c] = d ;

mais celles qui suivent déclenchent la même erreur :

Code:

1 2 3 4 5 path2[c] = f ; path2[c] = g ; path2[c] = h ; path2[c] = "*" ;
path2 est un tableau de char a deux dimensions donc path2[2] est un tableau de char ou d est un int. Ca ne peut pas rentrer, tu ne peux pas egaliser un tableau avec un element.
Relis mon premier message ou celui d'Emmanuel, ce probleme a deja ete explique.

J'ai totalement changé mon code en espérant que celui ci marcherait mais je recois des erreurs qu je trouve incompréhensibles.
Voici mon code :

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
 
#include <string.h>
#include <stdio.h>
int a[200], b = 97, c = 0, d = 0, e = 0, cra, i ;
char path[50] = "*" ;
/* ligne 5 */ char path2[600][70] ;
char o ;
FILE *fp, *fp2[600], 
main (void)
/* ligne n9 */ {    /* on affecte la valeur 97 a tous les éléments du tableau "a" */    
     for(i=0, i != 200, i++)
             a[i] = 97 ;
     /* on ouvre notre fichier */
	if((fp=fopen(path, "r") == NULL) 
		printf("Erreur1\n");
	/* on effectue l'oération 600 fois au maximum */
	while (e != 600)
	{     /* c prend la valeur du caractère pointé à chaque tour */
		c[e][d]= fgetc(fp);
		if (d == 199)
			e++ ;
		d++ ;
		/* on affecte à "a" des valeurs aléatoires entre 97(a) et 122(z) sans qu'elle ne soit jamais les mêmes */
		if (a != 122)
			a++ ;
		else 
		{
			b++ ;
			while ( ,a[c] != 97, a[c]--)
				c++ ;
			b++ ;
		}
		/* si tous les éléments du tableau "a" ont été affectés */
		if (a[10] == NULL)
		{
			/* on enregistre tous les caractères contenus de "a" puis celle de "b" */
            while ( ,c!= -1, c--) 
			{	
				o= strlen(path2[c]) ;
/* ligne n39 */				path2[e][o]= "*" ;
				o= strlen(path2[c]) ;
				path2[e][o]= a[c] ;
				o= strlen(path2[c])  ;
				path2[e][o]= b ;
			}
		}
		/* autrement, on enregistre normalement à chaque boucle les caractères "a" et "b" */
		else 
		{
			o= strlen(path2[c]) ;
			path2[e][o]= a ;
			o= strlen(path2[c]) ;
			path2[e][o]= b ;
	/* on ouvre un autre fichier à chaque boucle et on enregistre les caractres récupérés dans l'autre puis on le ferme */
	if((fp2[e]= fopen(path2[e], "w" )
		printf("Erreur2\n") ;
	fputs (c[e], fp2[e] ;
	e++ ;
	close(fp2) ;
	}
/* on ferme notre premier fichier */
close(fp) ;
}

Voici ces erreurs :

9 * syntax error before '{' token ( :? )
39 * variable-size type declared outside of any function
39 * variable-sized object may not be initialized
39 * conflicting types for 'path2'
5 * previous declaration of 'path2' was here

Voilà, et toutes ces erreurs revienent une vingtaine de fois au moins.
J'aimerais juste savoir leur signification pour pouvoir toutes les corriger parce que là raiment je ne vois pas.

D'avance, merci !

PS : j'ai indiqué les numéros de lignes qui n'allaient pour vous simplifier la tâche.

Citation:

Envoyé par CCin

J'ai totalement changé mon code en espérant que celui ci marcherait mais je recois des erreurs qu je trouve incompréhensibles.
Voici mon code :

Mes commentaires : -ed-

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
 
#include <string.h>
#include <stdio.h>
int a[200], b = 97, c = 0, d = 0, e = 0, cra, i ;
char path[50] = "*" ;
/* ligne 5 */
char path2[600][70] ;
char o ;
 
/* -ed ',' au lieu de ';' */
FILE *fp, *fp2[600];
 
/* -ed- manque 'int' */
int main (void)
/* ligne n9 */ {    /* on affecte la valeur 97 a tous les éléments du tableau "a" */
    /* -ed-
         for(i=0, i != 200, i++)
     
    ',' au lieu de ';'
    */
    for(i=0; i != 200; i++)
        a[i] = 97 ;
    /* on ouvre notre fichier */
 
    /* -ed- manque une parenthese... */
    if((fp=fopen(path, "r")) == NULL)
        printf("Erreur1\n");
 
    /* on effectue l'oération 600 fois au maximum */
    while (e != 600)
    {     /* c prend la valeur du caractère pointé à chaque tour */
 
        /* -ed-
              c[e][d]= fgetc(fp);
        c est un int, et non un tableau a deux dimensions. 
        */
        c= fgetc(fp);
        if (d == 199)
            e++ ;
        d++ ;
        /* on affecte à "a" des valeurs aléatoires entre 97(a) et 122(z) sans qu'elle ne soit jamais les mêmes */
 
        /* -ed-
        visiblement, tu es dans la confusion... 
        a est un tableau. Le comparer a un int n'a aucun sens... 
        */
        if (a != 122)
            /* -ed- L'incrementer non plus... */
            a++ ;
        else
        {
            b++ ;
 
            /* -ed- tu confonds while et for ? C'est quoi ces virgules. 
            while ( ,a[c] != 97, a[c]--)

STOP. Le compilateur n'est pas là pour apprendre le C à ta place. C'est à toi d'apprendre la syntaxe et de comprendre ce que tu tapes. Visiblement, tu tapes au hasard sans refléchir en comptant sur les messages du compilateur pour t'aider à coder. Curieusement, ça marche... si on est expérimenté (parce qu'on comprend le sens des messages). Sinon, y'a pas de secret, il faut apprendre le langage et sa syntaxe.

Par contre, je te conseille la rédaction/compilation incrémentale. Chaque ligne de code écrite doit générer un code compilable, (voire exécutable.). Ca permet de tester au fur et à mesure et de ne pas se noyer dans 3000 erreurs.

09/10/2005, 12h00
CCin
Oui je crois que tu as raison, je crois que je ferais mieux de réviser (d'apprendre ?? :? ) la syntaxe.

Mais en fait, les erreurs de syntaxe sont aussi dû au fait que j'ai d'abord écrit mon code sur papier puis je l'ai amélioré une fois rentré dans l'ordi (d'ou les "while" à la place des "for" quand je me suis rendu compte que je pouvais utiliser un "for" à la place du "while").

Merci quand même !

PS : Qu'entends-tu par :
Code:

1 2 3 4 5 /* -ed- for(i=0, i != 200, i++) ',' au lieu de ';' */
je ne vois pas ou j'avais mis ";" ??
09/10/2005, 12h34
jowo
Citation:
Envoyé par CCin

Oui je crois que tu as raison, je crois que je ferais mieux de réviser (d'apprendre ?? :? ) la syntaxe.

Mais en fait, les erreurs de syntaxe sont aussi dû au fait que j'ai d'abord écrit mon code sur papier puis je l'ai amélioré une fois rentré dans l'ordi (d'ou les "while" à la place des "for" quand je me suis rendu compte que je pouvais utiliser un "for" à la place du "while").

Merci quand même !

PS : Qu'entends-tu par :

Code:

1 2 3 4 5 /* -ed- for(i=0, i != 200, i++) ',' au lieu de ';' */

je ne vois pas ou j'avais mis ";" ??
Bonjour,

Justement on utilise le ";" pour séparer les 3 parties d'un for (Initialisation; Condition; Commande). La virgule "," est utilisée pour séprarée les différentes parties dans l'initialisation ou dans la commande.
Code:

1 2 3 4 for (size_t i = 0, j=0; i < strlen(szX) && j < strlen(szY); ++i, ++j) { .../.... }
10/10/2005, 19h07
CCin

Oh là là, merci beaucoup !

Tous ces messages d'erreurs étaient dû au fait que je mettais des "," au lieu des ";".

Ouf ! Maintenant, ca marche ! :D