Erreur de segmentation

Question

Bonjour à tous, je suis étudiant en informatique, et j'ai un algorithme à écrire. J'ai voulu le mettre en c pour le vérifier mais j'ai une erreur de segmentation. La compilation se passe bien mais à l'excution une erreur de ce type apparait. Petite précision, je suis sous linux (j'ai essayé sous daube mais rien ne de passe la-dessus). Merci à tous. Voici la source: #include #include int * calcul(void) { int p[3]; int t[5][5] = {{5, 9, 6 , 10, 4},{7, 8, 15, 2, 12}, {14, 1, 3, 11, 6},{16, 4, 8, 9, 13},{12, 6, 18, 5, 3}}; int i,j,k,l,m,n,max,minimax; max = t[1][1]; for (j = 2; j <= 5; j++) { if (t[1][j] > max) { max = t[1][j]; k = 1; l = j; } } minimax = max; for (i = 2; i <= 5; i++) { max = t[i][1]; m = i; n= j; for (j = 2; j <= 5; i++) { if (t[i][j] > max) { max = t[i][j]; m = i; n = j; } } if (max < minimax) { minimax = max; k = m; l = n; } } p[1] = k; p[2] = l; p[3] = minimax; return *p; } int main (void) { int *m = calcul(); printf ("La valeur minimum des maximum est %d et pour coordonnees [%d][%d]. ", m[3], m[2], m[3]); return 0;

BruNews · Answer

Le compilo de ton linull ne gueule pas ???

Tu d&#233;clares func int* et tu retournes une valeur *p (vaut p[0]) au lieu d'une adresse.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

BruNews · Answer

et aussi, comment veux tu retourner l'adresse d'une variable LOCALE &#224; une focntion. Cette variable n'existe sur la pile que le temps de la fonction, elle a toute chance d'&#234;tre &#233;cras&#233;e sur la pile en sortie de fonction.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

cosmobob · Answer

salut,

en + t'as pas bien pig&#233; qu'en C les indices des tableaux vont de 0 a n-1 :

tu fais un buffer overflow qd t'&#233;cris : int p[3]; puis p[3] = minimax;

si tu &#233;cris int p[3]; ca va de p[0] &#224; p[2].

pareil pour le tableau t c'est de t[0][0] &#224; t[4][4].



a+

Snacksou · Answer

Merci &#224; vous deux pour vos r&#233;ponses. Merci cosmobob, en effet j'avais
oubli&#233; qu'on partait de 0 car en cours d'algo y nous apprenent &#224; partir
de 1  ce qui me porte toujours &#224;
confusion.  BruNews, je vois &#224; peut pr&#232;s ton 1er post bien que je
ne soit pas sur qu'il faille retourner une adresse &#224; moins que
.....pour ton 2&#232;me, je ne vois pas vraiment comment faire alors si je
ne peux pas retourner l'adresse du pointeur vers la tableau p si
celui-ci reste local (y me semble que la but des pointeurs c'est
justement de travailler directement sur des valeurs et non pas sur une
copie  ). Merci &#224; vous  (ceci dit j'ai toujours pas r&#233;solu mon prob)

BruNews · Answer

valeur, copie... tu ne serais pas pass&#233; par vb ou autre bidouillerie de ce genre ???

un pointeur stocke une adresse m&#233;moire, pour lire ou affecter ce qu'il y a &#224; cette adresse on d&#233;r&#233;f&#233;rence avec '*' (ex *p). Il n'y a pas notion de copie ici.

ciao...
http://dev.winsysdev.com
BruNews, MVP VC++

cosmobob · Answer

pour r&#233;soudre ton pb:

au d&#233;but de ta fonction calcul:

static  int p[3]; // le tableau p persiste en dehors du contexte local de ta fonction



et a la fin:



return p;





a+

Snacksou · Answer

Heu non je crois pas Brunews, j'essaye d'appliquer ce que j'apprends
(mais j'avoue j'en chie avec ces con de pointeurs), mais je suis tout &#224;
fait d'accord, il n'y a pas de copie (justement l'interet des
pointeurs) dans le cas de mon tableau p. Je viens d'essyer ta solution
cosmobob mais j'ai toujours une erreur de segmentation.

Snacksou · Answer

je mets quand même le nouveau code source: #include #include int * calcul(void) { static int p[3]; int t[5][5] = {{5, 9, 6 , 10, 4},{7, 8, 15, 2, 12}, {14, 1, 3, 11, 6},{16, 4, 8, 9, 13},{12, 6, 18, 5, 3}}; int i,j,k,l,m,n,max,minimax; max = t[0][0]; for (j = 1; j <= 4; j++) { if (t[0][j] > max) { max = t[0][j]; k = 0; l = j; } } minimax = max; for (i = 1; i <= 4; i++) { max = t[i][0]; m = i; n = 0; for (j = 1; j <= 4; i++) { if (t[i][j] > max) { max = t[i][j]; m = i; n = j; } } if (max < minimax) { minimax = max; k = m; l = n; } } p[0] = k; p[1] = l; p[2] = minimax; return p; } int main (void) { int *m = calcul(); printf ("La valeur minimum des maximum est %d et pour coordonnees [%d][%d]. ", m[2], m[1], m[0]); return 0; } et merci à tout ceux qui prennent de leur temps pour me répondre

Snacksou · Answer

d&#233;sol&#233; pour les 40 messages &#224; la suite mais j'ai pas vu de fonction "editer",  je vous mets tout les fichiers correspondant &#224; mon probl&#232;me ici.

Snacksou · Answer

re tout le monde, d&#233;sol&#233; d'avoir fait le boulet mais apr&#232;s de multiple
essais, j'ai enfin trouv&#233;, dans la 2&#232;me boucle pour ce n'est pas "for
(j = 1; j <= 4; i++)" mais "for (j = 1; j <= 4; J++)". Normal l'erreur puisqu'il essaye de trouver une valeur du tableau qui n'existe pas  .Encore merci &#224; ceux qui m'ont aid&#233;. @+

Erreur de segmentation

10 réponses

Votre réponse

Discussions similaires