Version imprimable
Ca, c'est un tableau de 11 pointeurs sur des "char *".Code:char * file[11];
tu es sûre de vouloir mettre "*" ?
De plus tu déclares un tableau de 11 caractères pour stocker 11 caractères, ça va coincer, parce qu'il faut obligatoirement avec 1 caractère de plus que la chaîne désirée pour stocke le caractère de fin de chaine ('\0')
Pourquoi tu fais des fopen avec "r" alors que Neckara t'a dit de faire "rb" ? C'est pas le même fichier ?
Ce problème vient probablement de ta variable file que tu déclare comme un pointeur sur une chaine de caractère de taille 11. Or la fonction sscanf attend en premier paramètre une variable de type char *.
De plus tu essai de lire dans la chaine file sans avoir écrit à l'intérieur.
Edit : doublé par mala.
Je pense que tu confond sscanf et sprintf.
j'avais zappé, tu as raison !!!! c'est sprintf.Citation:
Envoyé par MythOnirie
merci et pour le segment fault je l'ai débuggé avec gdb et voici le résultat
Starting program: /home/remi/jeu
[Thread debugging using libthread_db enabled]
Program received signal SIGSEGV, Segmentation fault.
_IO_new_fclose (fp=0x0) at iofclose.c:52
52 iofclose.c: Aucun fichier ou dossier de ce type.
in iofclose.c
c'est affreux ce code !!
tu as suivi ce qu'on t'a dit ?
sscanf fait exactement l'inverse de ce que tu veux (sprintf)
dans fonctions dans des fonctions : très mauvaise idée, surtout pour un débutant. :aie:Code:fichier = fopen(fgets( *file, sizeof(*file), stdin), "r" );
PS : surtout, retire ton code pour qu'on comprenne encore moins :aie:
C'est bon comme ça ?
mais l'erreur de segmentation est toujours présente qu'est ce que je peux faire pour l'enleverCode:
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
Non, ce n'est pas bon. Et ca ne risque pas de le devenir vu que tu ne comprends pas ce que tu fais.
A mon sens, tu es comme une perssone qui a du sable, du ciment, de l'eau et des cailloux, et qui cherche a monter un mur de beton : tu as tous les ingredients, mais pas la moindre idee de comment faire. Et tu ne cherches pas a faire d'abord un essai, tu veux construire ton mur. Et tu ne tiens pas compte de ce que te disent les professionnels qui te conseillent.
Code:
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/* fichiers.c ---------- Par: remi Rôle : fonctions de lecture / écriture de fichiers de niveau. */ #include <stdlib.h> #include <stdio.h> #include <SDL/SDL.h> #include <SDL/SDL_image.h> #include "constantes.h" #include "fichiers.h" int chargerNiveau(int niveau[][NB_BLOCS_HAUTEUR]) { FILE *fichier = NULL; FILE *test = NULL; char ligneFichier[NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1] ; /* pas besoin de = "" */ int i = 0, j = 0, n = 0; //on ouvre le fichier test.txt et on le stocke dans test test=fopen("test.txt","r"); //on lis test.txt et on le stocke dans la variable n n=fgetc(test); char file[21]; /* Definition de variable au milieu du code : interdit en C, mauvaise idee dans les autres langages */ sprintf(file,"niveau%s.lvl", n ); fclose(fichier); /* C'est test qu'il faut fermer, pas fichier.*/ if(!fichier) /* Utile uniquement si tu ouvres "fichier", par exemple avec fichier = fopen (file, "r"); ou quelque chose du genre */ { fprintf(stderr,"No such file or directory : %s !"); /* Et le %s, il se remplit tout seul ??? */ exit(EXIT_FAILURE); } fichier = fopen(fgets( file, sizeof(*file), stdin), "r" ); /* Oula, fallait oser. Sais-tu ce que tu fais ici ? Pourquoi ne pas separer les deux appels de fonction : d'abord fgets, puis ensuite seulement fopen. Enfin, a supposer que ce soit ce que tu veux faire, ce qui m'etonnerait. */ fgets(ligneFichier, NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1, fichier); for(i = 0; i < NB_BLOCS_LARGEUR; i++) { for(j = 0; j < NB_BLOCS_HAUTEUR; j++) { niveau[j][i] = ligneFichier[(i * NB_BLOCS_LARGEUR) + j] - '0'; } } fclose(fichier); return 1; } int sauvegarderNiveau(int niveau[][NB_BLOCS_HAUTEUR]) { FILE* fichier = NULL; int i = 0, j = 0; fichier = fopen("niveaux.lvl", "w"); if(fichier != NULL) { for(i = 0; i < NB_BLOCS_LARGEUR; i++) { for(j = 0; j < NB_BLOCS_HAUTEUR; j++) { fprintf(fichier, "%d", niveau[j][i]); } } fclose(fichier); } else { printf("Impossible d'ouvrir le fichier niveaux.lvl"); return 0; } return 1; }
Installer Diablo III et le lancer : ton erreur disparaitra tant que tu joueras.Citation:
mais l'erreur de segmentation est toujours présente qu'est ce que je peux faire pour l'enlever
Ok merci ce que vous voulez dire c'est que je ne suis pas capable de faire ça et que je devrai revoir les bases c'est ça ?
Citation:
Envoyé par Rémi1995
C'est quand même clair non?Citation:
Non, ce n'est pas bon. Et ca ne risque pas de le devenir vu que tu ne comprends pas ce que tu fais.
A mon sens, tu es comme une perssone qui a du sable, du ciment, de l'eau et des cailloux, et qui cherche a monter un mur de beton : tu as tous les ingredients, mais pas la moindre idee de comment faire. Et tu ne cherches pas a faire d'abord un essai, tu veux construire ton mur. Et tu ne tiens pas compte de ce que te disent les professionnels qui te conseillent.
En gros il te dit :
- Tu dois apprendre les bases ;
- Tu dois essayer de comprendre comment utiliser les fonctions que tu as à ta disposition ;
- Tu dois faire des tests plutôt que de te rajouter ton code dans un projet complétement boggué. Ainsi tu pourras mieux comprendre les fonctions que tu utilises ;
- Tu n'écoutes pas ce qu'on te dit.
ok merci je vais faire tout ça
Bonjour,
J'ai peut-etre ete un peu sec dans ma reponse, et je m'en excuse. Ce que je voulais dire, c'est que cet exercice est visiblement trop complique pour votre niveau actuel en C, et que plutot que de chercher a trouver une solution, il serait probablement plus benefique de changer d'exercice.Citation:
Envoyé par Rémi1995
De nombreux cours/tutoriels sont disponibles ici : http://c.developpez.com/cours
Parcourez-les, prenez ceux pour debutant : meme si vous avez l'impression de connaitre, ca vous fera toujours decouvrir des choses, et revoir les bases que vous connaissez.
Ensuite, faites des exercices simples. Par exemple au lieu de chercher a lire des fichiers de niveau, pourquoi ne pas commencer par lire un seul fichier, et afficher son contenu ?
Petit a petit, en progressant, vous pourrez refaire cet exercice avec plusieurs fichiers, et non seulement vous serez capables de le resoudre, mais en plus vous comprendrez les problemes que vous rencontrez aujourd'hui.
merci du conseil j'ai étais un peu trop rapide pour l’apprentissage du c
mais par j'ai quand même essayé de suivre les conseils de gangsoleil et voici ce que ça donne
si s'est bien ça qu'il fallait faire ce qui m’étonnerai car ça crée une erreur de segmentCode:
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
mais je suivrais l'autre conseil de suivre un tuto et de faire des exo à mon niveau
sauvegarderNiveau() :
La fonction veut écrire la suite des int contenus dans un tableau 2D dans un fichier texte par fprintf(fichier, "%d", niveau[j][i]);.
Supposons que cette suite soit 1, 2, 4, 8, 16, 32... Que contiendra le fichier ? Cela dépend du format utilisé par fprintf(). Ici, il indique un entier tout simplement. On aura donc
Ceci peut poser problème à la relecture pour retrouver chaque nombre individuel. Donc, il est prudent de mettre un séparateur entre chaque nombre, par exemple un blanc, une ',', ou un retour à la ligne '\n'. Dans ce dernier cas, on aura fprintf(fichier, "%d\n", niveau[j][i]);ce qui donnera sur le fichierCitation:
12481632...
Maintenant qu'on a choisi un format d'écriture dans le fichier (et vérifié par un éditeur de texte que le fichier produit est correct), on peut s'intéresser à son pendant chargerNiveau()Citation:
1
2
4
8
16
32
....
chargerNiveau() :
- Déterminer le fichier à lire. Son numéro est inscrit dans le fichier "test.txt", par exemple 10
- Ouvrir le fichier et le lire. On connait parfaitement maintenant le format de ce qui est écrit dedans. Par exemple, si on a utilisé un blanc ou un '\n' comme séparateur, on peut avoir un simple format "%d" :Code:
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
Ce code simple a l'inconvénient de ne pas détecter des erreurs possibles dans le format du fichier (normalement, il n'y en a pas mais...), par exemple présence de caractères intempestifs, fichier trop petit qui ne contient pas un nombre suffisant de nombres,...Code:
2
3
4
5
6
7
8
9
10
11
12
On peut le sécuriser en testant la valeur retournée par fscanf() (qui doit être égale à 1 dans notre cas), ce qui permet de savoir qu'on a bien lu un nombre, qu'il n'y a pas eu d'erreurs et qu'on n'a pas atteint inopinément la fin du fichier.
Voir la doc de fscanf() pour les détails.
j'ai fais comme m'a dit diogene
mais il m'affiche segment faultCode:
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
j'ai lancer gdb et il m'affiche
Starting program: /home/remi/jeu
[Thread debugging using libthread_db enabled]
Program received signal SIGSEGV, Segmentation fault.
_IO_fgets (buf=0xbffff016 "\030", n=145, fp=0x0) at iofgets.c:52
52 iofgets.c: Aucun fichier ou dossier de ce type.
in iofgets.c
sauf que j'ai du mal à comprendre se qu'il raconte mais d'après ce que j'ai compris et lu sur d'autre forum Il y à peu être un r buffer overflow à l'adresse 0xbffff016.
Non ! Relis attentivement mon post précédent en essayant de comprendre.Citation:
j'ai fais comme m'a dit diogene
Ton code est absurde : comment sauvegarder le niveau dans un fichier en LISANT ce fichier (fscanf())? Il faudrait mieux ECRIRE dans ce fichier les valeurs à sauvegarder (fprintf())
Je viens de le corriger et voici le nouveau code
mais il m'affiche toujours segment faultCode:
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
et quand je lance gdb il m'affiche [Thread debugging using libthread_db enabled]
Program received signal SIGSEGV, Segmentation fault.
_IO_fgets (buf=0xbffff006 "\030", n=145, fp=0x0) at iofgets.c:52
52 iofgets.c: Aucun fichier ou dossier de ce type.
in iofgets.c
Déjà pour cette ligne :
Soit tu teste la valeur de n soit tu fais en sorte que file soit un char[19+11] mais la meilleurs solution est de tester que n ne dépasse pas 10^(tailleDeFile - 11).Code:sprintf(file,"niveau%d.lvl", n );
EDIT : sinon je te conseillerais de relire tout ce qui t'as été dit.
Regarde aussi comment tu lis et comment tu écris dans ton fichier de niveau...
Lors de la lecture, tu considère que tu as écris des caractères pour stocker ton nombre (donc de '0' à '9') fgets(ligneFichier, NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1, fichier); et dans l'écriture tu considère que tu as écris des int (donc de 0 à 10^19 et quelques) fprintf(fichier, "%d",&niveau[j][i]);....
Il faut vraiment que tu prennes le temps de lire (et de comprendre) à tête reposée ce qu'on t'as déjà dit.
tu veux dire que si par exemple n=20 il ne faut pas qu'il dépasse 10^9 ?Citation:
Soit tu teste la valeur de n soit tu fais en sorte que file soit un char[19+11] mais la meilleurs solution est de tester que n ne dépasse pas 10^(tailleDeN - 11).
je vais le faire.
j'ai corrigé comme ça vu que j'ai lu le fichier comme ça
je l'écris comme çaCode:fgets(ligneFichier, NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1, fichier);
c'est bon ?Code:fprintf(fichier, "%d",NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1);
Je vois ce que tu essaye de dire mais ta phrase est mal formulée.Citation:
Envoyé par Rémi1995
Si la taille physique de ta chaîne de caractère est 20 il ne faut pas que n > 10^9.
Non là c'est vraiment n'importe quoi.Citation:
Envoyé par Rémi1995
Quand on programme il faut comprendre un minimum ce que l'on fait.
Programmer ce n'est pas essayer au hasard jusqu'à ce que ça marche.
fprintf va écrire un int soit 0 à 19 caractères cet int vaudra NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1.
fgets va lire NB_BLOCS_LARGEUR * NB_BLOCS_HAUTEUR + 1 caractères.
Un conseil, laisse tomber.
Prend le temps de lire les tutoriels de base, de faire quelques hello world.
Ensuite fait un petit programme simple en prenant bien le temps de lire correctement le man pour chaque fonctions que tu ne connais pas très bien.
Parce que là, à moins qu'on te fasse ton programme à ta place, tu t'en sortira jamais.
Tu aurais beaucoup plus vite fait d'apprendre correctement le C puis de te repencher sur ton projet dans quelques mois que d'essayer de le faire maintenant.
ok merci c'est se que je suis en train de faire mais je ne sais pas si je dois tout recommencer du début ?