Discussion :

[olafsergent]

Bonjour à tous,

J'ai un problème lorsque j'exécute mon programme qui doit permettre d'afficher d'abord un tableau de valeurs aléatoires puis une matrice. Une erreur de segmentation se produit quand le programme essaie d'afficher la matrice et lorsque qu celle-ci est libéré.

Voici les fichiers :
exo4.h

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
#ifndef EXO4_H
#define EXO4_H
/* Pour les tableaux */
int *AllocTab(int n);
void RandTab( int *tab, int n, int a, int b );
void FreeTab( int *tab );
 
/* Pour les matrices */
int** AllocMat( int nbRows, int nbCols );
void RandMat( int **mat, int nbRows, int nbCols, int a, int b );
void FreeMat( int **mat, int nbRows );
 
#endif

exo4.c

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
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
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
#include "EXO4.h"
 
int main(int argc, char **argv)
{
  int n, a, b, i, j;
  int nbRows, nbCols;
 
  printf("Taille du tableau: ");
  scanf("%d", &n);
  printf("Génération de nombre aléatoire entre a et b:");
  printf("\na: ");
  scanf("%d", &a);
  printf("\nb: ");
  scanf("%d", &b);
 
  /* Allocation du tableau statique */
  int *tab=AllocTab(n);
  /* Remplissage du tableau */
  RandTab(tab, n, a, b);
  /* Libération du tableau */
  FreeTab(tab);
  /* Affichage du tableau */
  for(i=0; i<n; i++)
  {
    printf("\n%d", tab[i]);
  }
 
  printf("\nNombre de lignes et colonnes de la matrice:");
  printf("\nnbRows: ");
  scanf("%d", &nbRows);
  printf("\nnbCols: ");
  scanf("%d", &nbCols);
 
  /* Allocation de matrices */
  int **mat=AllocMat(nbRows, nbCols);
  /* Remplissage de la matrice */
  RandMat(mat, nbRows, nbCols, a, b);
  /* Affichage matrice */
  for (i=0; i<nbRows; i++)
  {
    for(j=0; j<nbCols; j++)
    {
      printf(" %d ", mat[i][j]);
      if(j==nbCols)
      {
        printf("\n");
      }
    }
  }
  /* Libération de la matrice */
  FreeMat(mat, nbRows);
 
  return 0;
}
 
 
 
/* Pour les tableaux */
int* AllocTab( int n )
{
  int *tab=(int*)calloc(n, sizeof(int));
  return tab;
}
 
void RandTab( int *tab, int n, int a, int b )
{
  srand(time(NULL));
  int i;
  for(i=0; i<n; i++)
  {
    tab[i]=(a + rand()%(b-a));
  }
}
 
void FreeTab( int *tab )
{
  free(tab);
}
 
 
 
 
/* Pour les matrices */
int** AllocMat( int nbRows, int nbCols )
{
  int i;
  int **mat=(int**)calloc(nbRows, sizeof(int));
  for(i=0; i<nbRows; i++)
  {
    mat[i]=(int*)calloc(nbCols, sizeof(int));
  }
  return mat;
}
 
void RandMat( int **mat, int nbRows, int nbCols, int a, int b )
{
  int i, j;
  for(i=0; i<nbRows; i++)
  {
    for(j=0; j<nbCols; j++)
    {
      mat[i][j]=(a + rand()%(b-a));
    }
  }
}
 
void FreeMat( int **mat, int nbRows )
{
  printf(" ");
  int i;
  for(i=0; i<nbRows; i++)
  {
    free(mat[i]);
  }
  free(mat);
}

Je compile avec gcc de cette manière sous linux: gcc -Wall EXO4.c -o EXO4.out
Si quelqu'un comprend le problème, je serait ravie de le savoir.

Je vous remercie d'avance pour votre aide

[emixam16]

Bonjour,

Ah, les problèmes de mémoire en C.


Dans ton code, tu libère ton tableau et tu le lis après

Code :

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

1
2
3
4
5
6
7
8
  /* Libération du tableau */
  FreeTab(tab);
  /* Affichage du tableau */
  for(i=0; i<n; i++)
  {
    printf("\n%d", tab[i]);
  }

Tu as un accès invalide et le comportement de ton programme devient indéterminé. Il peut alors planter ou ne pas planter (ce qui est encore pire).
D'autre part vérifie ta ligne d'allocation de ta matrice...

Pour corriger des erreurs mémoires par la suite tu as deux possibilité (non exclusives),
- Soit tu trouves les erreurs manuellement
- Soit tu utilises Valgrind qui checke les accès mémoire pour toi.

[olafsergent]

Merci pour ta réponse. J'ai modifié mon code et résolu le problème. En fait je me suis trompé lorsque j'ai donné la taille des cases de la matrices, j'ai déclaré la matrice mat de cette manière:

Code :

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

  int **mat=(int**)calloc(nbRows, sizeof(int));

Au lieu de:

Code :

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

  int **mat=(int**)calloc(nbRows, sizeof(int*));

C'est tout bête mais merci quand même. Je l'ai trouvé grâce à Valgrind !

[Sve@r]

Bonjour

En fait je me suis trompé lorsque j'ai donné la taille des cases de la matrices, j'ai déclaré la matrice mat de cette manière:

Code :

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

  int **mat=(int**)calloc(nbRows, sizeof(int));

Au lieu de:

Code :

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

  int **mat=(int**)calloc(nbRows, sizeof(int*));

Tu pourrais te simplifier la vie si tu considérais ta matrice comme un tableau 1D (ce qu'elle est d'ailleurs en réalité). Tu te ferais plus suer dans tes allocations et libérations en cascade. Un simple malloc à l'entrée, un free à la fin et c'est tout.

Le seul détail serait de devoir convertir un indice "i" en "lig/col" lors de l'accès aux éléments mais ça se règle assez facilement car lig=i/nbRows et col=i%nbRows. Et dans l'autre sens, i=lig * nbRows + col...