Problème de crash dans un programme en C

Version imprimable

Bonjour,

Je suis en première année d'école d’ingénieur et j'ai un projet a réaliser.
Le problème ce trouve au niveau des fonction pour déplacer les chats et les souris mais je n'arrive pas a l'identifier et cela fait crash mon programme.
Toute type d'aide est la bienvenue :).

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
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void placementsouris(int M[7][7]){
    int i;
        for(i=0;i<=6;i++){
        M[0][i]=1;
        M[1][i]=1;
        M[2][i]=1;
        M[3][i]=0;
        M[4][i]=0;
        M[5][i]=0;
        M[6][i]=0;
    }
}
void plateau(int M[7][7]){
    int i,j;
    printf("   ");
    for(i=0;i<=6;i++){
        printf("%i   ",i+1);
    }
    printf("\n\n");
    for(i=0;i<=6;i++){
        printf("%i  ",i+1);
        for(j=0;j<=6;j++){
            if(i==5 && j==0 || i==5 && j==1 || i==6 && j==0 || i==6 && j==1 || i==5 && j==5 || i==5 && j==6 || i==6 && j==5 || i==6 && j==6){
                printf("    ");
            }
            else{
            if(M[i][j]==0){
                printf("0");
            }
            if(M[i][j]==1){
                printf("S");
            }
            if(M[i][j]==2){
                printf("C");
            }
            printf("   ");
            }
        }
        printf("\n\n");
    }
}
void placementchat(int M[7][7]){
    int chatrestant=3,i,j;
    do{
    plateau(M);
    printf("il vous reste %i chat a placer: \n",chatrestant-1);
    printf("ligne :");
    scanf("%i",&i);
    printf("Colone : ");
    scanf("%i",&j);
    if(i==4 || i==5 && j==1 || i==5 && j==2 || i==5 && j==6 || i==5 && j==7){
            M[i-1][j-1]=2;
            chatrestant--;
    }
    }while(chatrestant>0);
}
void deplacementsouris(int M[7][7]){
    int i,j,a,b,bon;
    bon=0;
    do{
    printf("choisissez une souris a deplacer\n");
    printf("ligne:");
    scanf("%i",&i);
    printf("colonne:");
    scanf("%i",&j);
    i -- ;
    j -- ;
    if(M[i][j]==1){
    if(M[i-1][j]==0 || M[i-1][j+1]==0 || M[i][j+1]==0 || M[i+1][j]==0 || M[i+1][j+1]==0 || M[i+1][j-1]==0 || M[i][j-1]==0 || M[i-1][j-1]==0){
        printf("choisissez un deplacement\n");
        printf("ligne:");
        scanf("%i",&a);
        printf("colonne:");
        scanf("%i",&b);
        a-- ;
        b-- ;
        if(M[i-1][j]==0 || M[i-1][j+1]==0 || M[i][j+1]==0 || M[i+1][j]==0 || M[i+1][j+1]==0){
            M[i][j]=0;
            M[a][b]=1;
            bon=1;
        }
    }
    else{
        printf("cette souris ne peut pas être déplacé\n");
    }
    plateau(M);
    }
    else{
        printf("ce n'est pas l'emplacement d'une souris\n");
    }
    }while(bon==0);
}
void detecchat(int M[7][7],int i1,int j1,int i2,int j2,int i3,int j3){
    int i,j,detec;
    detec=3;
    for(i=0;i<=6;i++){
        for(j=0;j<=6;j++){
            if(M[i][j]==2){
                if(detec==3){
                    i1=i;
                    j1=j;
                    detec--;
                }
                if(detec==2){
                    i2=i;
                    j2=j;
                    detec--;
                }
                if(detec==1){
                    i3=i;
                    j3=j;
                    detec--;
                }
            }
        }
    }
}
void possiblemange(int M[7][7],int i1,int j1,int i2,int j2,int i3,int j3,int p){
    if(M[i1+1][j1+1]==1 && M[i1+2][j1+2]==0 || M[i1+1][j1]==1 && M[i1+2][j1]==0 || M[i1+1][j1-1]==1 && M[i1+2][j1-2]==0 || M[i1][j1+1]==1 && M[i1][j1+2]==0 || M[i1][j1-1]==1 && M[i1][j1-2]==0 || M[i1-1][j1+1]==1 && M[i1-2][j1+2]==0 || M[i1-1][j1]==1 && M[i1-2][j1]==0 || M[i1-1][j1-1]==1 && M[i1-2][j1-2]==0){
        p++;
    }
    if(M[i2+1][j2+1]==1 && M[i2+2][j2+2]==0 || M[i2+1][j2]==1 && M[i2+2][j2]==0 || M[i2+1][j2-1]==1 && M[i2+2][j2-2]==0 || M[i2][j2+1]==1 && M[i2][j2+2]==0 || M[i2][j2-1]==1 && M[i2][j2-2]==0 || M[i2-1][j2+1]==1 && M[i2-2][j2+2]==0 || M[i2-1][j2]==1 && M[i2-2][j2]==0 || M[i2-1][j2-1]==1 && M[i2-2][j2-2]==0){
        p++;
    }
    if(M[i3+1][j3+1]==1 && M[i3+2][j3+2]==0 || M[i3+1][j3]==1 && M[i3+2][j3]==0 || M[i3+1][j3-1]==1 && M[i3+2][j3-2]==0 || M[i3][j3+1]==1 && M[i3][j3+2]==0 || M[i3][j3-1]==1 && M[i3][j3-2]==0 || M[i3-1][j3+1]==1 && M[i3-2][j3+2]==0 || M[i3-1][j3]==1 && M[i3-2][j3]==0 || M[i3-1][j3-1]==1 && M[i3-2][j3-2]==0){
        p++;
    }
}
void choixetdeplacement(int M[7][7],int i1,int j1,int i2,int j2,int i3,int j3,int p){
    int a,b,c,d,i,j;
    printf("choisissez un chat à déplacer:\n");
    printf("Ligne:");
    scanf("%i",a);
    printf("Colonne:");
    scanf("%i",b);
    if(M[a][b]==M[i1][j1] || M[a][b]==M[i2][j2] || M[a][b]==M[i3][j3]){
        printf("Choisissez un déplacement a faire\n");
        printf("Ligne:");
        scanf("%i",c);
        printf("Colonne:");
        scanf("%i",d);
 
        }
    }
 
void deplacementchat(int M[7][7]){
    int i1,j1,i2,j2,i3,j3,p;
    p=0;
    detecchat(M,i1,j1,i2,j2,i3,j3);
    possiblemange(M,i1,j1,i2,j2,i3,j3,p);
    choixetdeplacement(M,i1,j1,i2,j2,i3,j3,p);
}
 
int jeu()
{
    system("color 02");
    int i;
    int M[7][7];
    placementsouris(M);
    placementchat(M);
    deplacementsouris(M);
    deplacementchat(M);
}
 
 
void choix_mode(){
 
    int x ;
 
    printf("\n1 - 1 vs 1 \n2 - IA Facile \n3 - IA difficile \n\n");
    printf("Votre Choix : ");
    scanf("%i",&x);
    while(x<1 || x>3){
                printf("\n1 - 1 vs 1 \n2 - IA Facile \n3 - IA difficile \n");
                printf("Votre Choix : ");
            scanf("%i",&x);
    }
    switch(x){
        case 1 : jeu() ;
        break;
        case 2 : ;
        break;
        case 3 : ;
        break;
    }
}
 
void regle(){
        printf(" ");
        choix_mode();
 
}
int main(){
    int c;
system("color 02");
printf("\nLe Jeu du chat et de la souris\n\n\n ");
 
 
    printf("   Que voulez vous faire ?\n");
    printf("   ________________________\n");
    printf("   |    1 - jouer         |\n   |    2 - les regles    |\n");
    printf("   ////////////////////////\n\n");
    printf("   Votre choix :");
        scanf("%i",&c);
    while(c<1 || c>3){
                printf("tapez 1 pour jouer\n2 pour voir les regles\n");
                printf("Votre choix : ");
            scanf("%i",&c);
    }
    switch(c){
        case 1 : choix_mode() ;
        break;
        case 2 : regle();
        break;
    }
}

13/06/2020, 19h27
Xelland
Mets ton code dans les balises code + indentation du code afin d'améliorer la lisibilité (:oops: tu as modifié entre temps :) )
C'est possible d'avoir une description plus précise du problème ? (je n'ai pas testé le code)

la fonction detecchat est "bizarre"

extrait :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 if(M[i][j]==2){ if(detec==3){ i1=i; j1=j; detec--; } if(detec==2){ i2=i; j2=j; detec--; } if(detec==1){ i3=i; j3=j; detec--; } }
en gros au 1er passage où (M[i][j]==2), on va exécuter les 3 cas, donc pourquoi tester ? (ou il manque "else" ?)
pour les passages suivants, detec est négatif, donc tu finis les 2 boucles sans rien faire

pas regardé le reste du code, trop illisible sans l'identation

Bonjour,

Sans les balises, le code est illisible. Je l'ai soumis à mon compilateur qui me signales tous les problèmes suivants:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
1>code.c(136,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(138,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(142,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(144,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(133,21): warning C4101: 'j'*: variable locale non référencée
1>code.c(133,18): warning C4101: 'i'*: variable locale non référencée
1>code.c(160,6): warning C4101: 'i'*: variable locale non référencée
1>code.c(136): error C4700: variable locale 'a' non initialisée utilisée
1>code.c(138): error C4700: variable locale 'b' non initialisée utilisée
1>code.c(142): error C4700: variable locale 'c' non initialisée utilisée
1>code.c(144): error C4700: variable locale 'd' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j3' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i3' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j2' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i2' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j1' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i1' non initialisée utilisée
1>code.c(166): warning C4716: 'jeu' : doit retourner une valeur

Ton compilateur doit être mal configuré pour ne pas t'indiquer ces problèmes qu'il te faut tous corriger.

14/06/2020, 09h52
Sve@r
Bonjour
Citation:
Envoyé par freelancer01

Code:

1 2 int a; scanf("%i",a);
Il n'y a rien qui te choque ???

Citation:

Envoyé par dalfab

Ton compilateur doit être mal configuré pour ne pas t'indiquer ces problèmes qu'il te faut tous corriger.

Ben malheureusement... par défaut gcc ne dit absolument rien sur ces soucis. Effectivement si on active les options qui vont bien là il décolle comme une fusée mais sinon...:?

Citation:

Envoyé par dalfab

Bonjour,

Sans les balises, le code est illisible. Je l'ai soumis à mon compilateur qui me signales tous les problèmes suivants:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
1>code.c(136,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(138,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(142,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(144,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(133,21): warning C4101: 'j'*: variable locale non référencée
1>code.c(133,18): warning C4101: 'i'*: variable locale non référencée
1>code.c(160,6): warning C4101: 'i'*: variable locale non référencée
1>code.c(136): error C4700: variable locale 'a' non initialisée utilisée
1>code.c(138): error C4700: variable locale 'b' non initialisée utilisée
1>code.c(142): error C4700: variable locale 'c' non initialisée utilisée
1>code.c(144): error C4700: variable locale 'd' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j3' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i3' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j2' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i2' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'j1' non initialisée utilisée
1>code.c(152): error C4700: variable locale 'i1' non initialisée utilisée
1>code.c(166): warning C4716: 'jeu' : doit retourner une valeur

Ton compilateur doit être mal configuré pour ne pas t'indiquer ces problèmes qu'il te faut tous corriger.

Bonjour,
Effectivement mon compilateur ne me donnais aucune erreur et je suis débutant en programmation donc je ne comprend pas ce que veux dire int*, j'ai essayé de chercher mais je n'ai pas trouver? pouvez vous m'éclairer ?

14/06/2020, 15h34
freelancer01

Citation:

Envoyé par Sve@r

Bonjour

Il n'y a rien qui te choque ???

Ben malheureusement... par défaut gcc ne dit absolument rien sur ces soucis. Effectivement si on active les options qui vont bien là il décolle comme une fusée mais sinon...:?

Bonjour,
Effectivement il manque "&" avant le "a" mais il y a un seul endroit ou il manque et le programme ne marche toujours pas ... et faut t'il mettre %d a la place de %i ?
14/06/2020, 16h19
Sve@r

Citation:

Envoyé par freelancer01

Effectivement il manque "&" avant le "a" et faut t'il mettre %d a la place de %i ?

Ben oui. "%i" je ne connaissais pas mais bon, il y a tant d'environnements de programmation C que je me disais que peut-être c'était un truc un peu exotique. Donc pour un int, c'est "%d" et "&a" pour la variable. Mais bon, ça c'est une erreur qui m'a sauté aux yeux suite au post de dalfab. Ce qui signifie que ce n'est pas parce que j'ai vu celle-là que c'est la seule erreur. Programmer, c'est déjà écrire dans une syntaxe correcte. C'est une des choses que vérifie le compilateur (surtout si tu lui mets options de vérification plus poussées comme "-Wall -Werror"). Mais c'est aussi créer un algorithme qui réponde au problème. Si l'algo n'est pas bon, alors ton programme tout syntaxiquement correct qu'il soit, ne fonctionnera pas. Et en informatique, les sophismes (raisonnements faux mais qui semblent corrects comme par exemple "si A entraine B alors non(A) entraine non(B)") sont nombreux.
Et enfin c'est aussi ne pas écrire d'instructions correctes mais illégales comme par exemple taper dans l'indice 25 d'un tableau de 10 car pour être le plus rapide possible, le C ne vérifie pas ce genre de détail. Sa philosophie c'est "le programmeur sait ce qu'il fait". Donc si cette instruction est demandée, le C l'exécutera quelle que soit la conséquence (et bien souvent la conséquence c'est que le programme continue à tourner mais produit des résultats totalement farfelus qu'on nomme "UB" pour "Undefined Behavior").

Citation:

Envoyé par freelancer01

Effectivement mon compilateur ne me donnais aucune erreur et je suis débutant en programmation donc je ne comprend pas ce que veux dire int*,

Ah oui, effectivement. Désolé, savoir ce qu'est un int* (ou un float* ou un n'importquoi*) est essentiel en programmation C. Et on ne peut pas t'enseigner en 10mn une des notions les plus fondamentales du C. Il faut que tu ouvres un tuto et que tu te plonges dans l'apprentissage, le rôle et l'utilité des pointeurs. Pour résuméer très rapidement, scanf() doit recevoir l'adresse de la variable à remplir. Si cette variable est un int, alors son adresse est un int*. C'est ce que dit le compilateur.
Ceci dit, je suis étonné qu'on donne un tel truc à un débutant. Donc soit tu es un vrai débutant et ton prof t'a donné un projet de trop haut niveau ; mais dans ce cas tous ceux de ta classe sont confrontés au même souci et vous pouvez donc aller vous plaindre à la direction de cette école, soit tu es débutant parce que t'as pas étudié les cours précédents. Et dans ce cas...

Citation:

Envoyé par freelancer01

Citation:

Envoyé par dalfab

[C]1>code.c(136,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(138,8): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(142,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'
1>code.c(144,9): warning C4477: 'scanf'*: la chaîne de format '%i' nécessite un argument de type 'int *', mais l'argument variadique 1 est de type 'int'

mais il y a un seul endroit ou il manque

Mouais, un seul :applo: Sans déconner quoi, même pas foutu de lire un post et tu veux être ingénieur ???
14/06/2020, 17h00
freelancer01

Je ne sais pas ce que c'est un int* car ma prof ne me la tout simplement pas expliquée n'y mentionné dans son cours. Donc ce n'est pas moi qui n'étudie pas mais simplement que mon cours ne le mentionne pas.
J’irai me renseigner sur ce sujet car comme vous le dites c'est l'une des notions les plus fondamentales et que vous avez l'aire assez expérimenté donc vos conseilles sont les bienvenue mais concernant la dernière ligne je ne suis pas bête quand je disais un seul endroit je parlais d'une seul fonction que j'ai retiré de mon programme pour identifier le problème mais cela ne changeais rien.
Merci d'avoir répondu a mes message et d'avoir tenté de m’éclaircir.
14/06/2020, 17h50
Sve@r

Citation:

Envoyé par freelancer01

J’irai me renseigner sur ce sujet

Pas vraiment besoin pour ce TP. Simplement n'oublie plus le "&" devant tes variables pour le scanf() (sauf si cette variable est un tableau de char pour stocker une string). Et (culture perso), "étoile" dans un type de variable signifie "adresse de". Donc "int *" c'est "adresse d'un int". Pour le reste, il te faut maintenant examiner chaque fonction une à une. Afficher ce qu''elle reçoit, ce qu'elle calcule, ce qu'elle renvoie, vérifier si ce qu'elle renvoie est conforme à ce qui devrait. Bref faire du débug...