Discussion :

[sandball22]

Bonjour, j'ai une erreur de segmentation. Je pense avoir identifié où elle se trouve.
Elle se trouve dans la fonction suivante:

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
 
void FrequencyTransposition(double *signal_reel, float tk, int N)
{
  int i;
  float fp;
 
   double signal_re[N];
  double cosinus[N];
  double sinus[N];
  double signal_re_transpose[N];
  double signal_im_transpose[N];
 
 
 
 
  FILE * FicSortie;
  printf("\nN=%d",N);
  printf("\ntk=%f",tk);
 
  for(i=0;i<N;i++)
  { 
	signal_re[i]=signal_reel[i];      /*Récupération du signal partie réelle*/ 
  }	
 
 
  printf("\n\nFréquence de la porteuse en MHz :");   
  scanf("%f",&fp);
 
  printf("\n\nSignal partie réelle transposé par une porteuse de %lf MHz :",fp);       
 
/*Partie reelle*/
  for(i=0;i<N;i++)
  {
   cosinus[i] = cos(2*3.1416*fp*i*tk);                      
	signal_re_transpose[i]=signal_re[i]*cosinus[i];            /*Multiplication du signal reel par cos(2*pi*fp*tk)*/
 }
 
 
/*Partie imaginaire*/
 for(i=0;i<N;i++)
  {
    sinus[i] = sin(2*3.1416*fp*i*tk);                      
	signal_im_transpose[i]=signal_re[i]*sinus[i];            /*Multiplication du signal  par sin(2*pi*fp*tk)*/
 }
 
 
 
 
 
 
 
 
FicSortie= fopen("F:/Users/garnier/Matlab/imag.txt","w");
 
 for (i=0;i<N;++i)
 {fprintf(FicSortie,"%lf\n ",signal_im_transpose[i]); 
  }
 fclose(FicSortie);
 
 
FicSortie= fopen("F:/Users/garnier/Matlab/reel.txt","w");
 
 for (i=0;i<N;++i)
 {fprintf(FicSortie,"%lf\n ",signal_re_transpose[i]); 
  printf(" %lf ",signal_re_transpose[i]);
 }
 fclose(FicSortie);
 
  return;
}

Pouvez-vous m'aidez merci.

[juls64]

Es tu sur que signal_reel a été alloué et si oui que ton N n'est pas trop grand et donc va taper a une mauvaise adresse

[sandball22]

Je ne sais pas. Comment peut-on le savoir?

[juls64]

ca n'arrangera pas ton probleme mais pour faire une copie de chaine tu peux utiliser la fonction :
char *strncpy (char *dest, const char *src, size_t n);
char* est un pointeur vers ta chaine dest

[juls64]

au lieu d'utiliser un N fait un strlen() pour obtenir la taille et vérifie avant l('appel a la fonction que tu as alloué de la memoire chaine = (char*) malloc ( N*sizeof(char) )
ou bien initalisé comme tableau chaine[N]
tu peux utilisé un boolean lors de ton initialisation des qu'il est initilisé tu le passes a true et dans ta fonction tu test si c a true

[crocodilex]

Bonjour, j'ai une erreur de segmentation. Je pense avoir identifié où elle se trouve.
Elle se trouve dans la fonction suivante:

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
 
FicSortie= fopen("F:/Users/garnier/Matlab/imag.txt","w");
 
 for (i=0;i<N;++i)
 {fprintf(FicSortie,"%lf\n ",signal_im_transpose[i]); 
  }
 fclose(FicSortie);
 
 
FicSortie= fopen("F:/Users/garnier/Matlab/reel.txt","w");
 
 for (i=0;i<N;++i)
 {fprintf(FicSortie,"%lf\n ",signal_re_transpose[i]); 
  printf(" %lf ",signal_re_transpose[i]);
 }
 fclose(FicSortie);

Pouvez-vous m'aidez merci.

La fonction fopen peut planter. Il faut toujours vérifier son retour.

Code :

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

1
2
3
4
5
6
 
    if ((ptr_file = fopen("nomFichier","w")) == NULL)
    {
        fprintf(stderr,"Impossible d'ouvrir le fichier !!!\n");
        exit(EXIT_FAILURE);
    }

[sandball22]

Cela ne vient pas de la fonction fopen car je l'ai mise en commentaire.

[sandball22]

J'ai fait des modifs dans mon code. Cela marche mieus mais quand j'augmente tk et N il y a toujours des erreurs de segmentation mais à des valeurs supérieurs ç celles ou cela ne fonctionnait pas.

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
 
void FrequencyTransposition(double *signal, float tk, int N)
{
  int i;
  float fp;
 
//  double signal_re[N];
  double signal_re_transpose[N];
  double signal_im_transpose[N];
 
 
 
 
//  FILE * FicSortie;
  printf("\nN=%d",N);
  printf("\ntk=%f",tk);
 
//  for(i=0;i<N;i++)
//  { 
//	signal_re[i]=signal_reel[i];      /*Récupération du signal partie réelle*/ 
//  }	
 
 
  printf("\n\nFréquence de la porteuse en MHz :");   
  scanf("%f",&fp);
 
  printf("\n\nSignal partie réelle transposé par une porteuse de %lf MHz :",fp);       
 
/*Partie reelle*/
  for(i=0;i<N;i++)
  {
  // cosinus[i] = cos(2*3.1416*fp*i*tk);                      
	signal_re_transpose[i]=signal[i]*cos(2*3.1416*fp*i*tk);            /*Multiplication du signal reel par cos(2*pi*fp*tk)*/
 }
 
 
/*Partie imaginaire*/
 for(i=0;i<N;i++)
  {
   // sinus[i] = sin(2*3.1416*fp*i*tk);                      
	signal_im_transpose[i]=signal[i]*sin(2*3.1416*fp*i*tk);            /*Multiplication du signal  par sin(2*pi*fp*tk)*/
 }
 
 
 
 
 
 
//  printf("\n\nSignal partie imaginaire transposé par une porteuse de %f MHz :",fp);        
//  for(i=0;i<N;i++)
//  {
//    sinus[i] = sin(2*3.1416*fp*i*tk);    
//	signal_im_transpose[i]=signal_im[i]*sinus[i];              /*Multiplication du signal imaginaire par sin(2*pi*fp*tk)*/
//	printf("\n%d  %f ",i+1,signal_im_transpose[i]);	
//  }
 
 
//FicSortie= fopen("F:/Users/garnier/Matlab/imag.txt","w");
//  
// for (i=0;i<N;++i)
// {fprintf(FicSortie,"%lf\n ",signal_im_transpose[i]); 
//  }
// fclose(FicSortie);
 
 
//FicSortie= fopen("F:/Users/garnier/Matlab/reel.txt","w");
 
// for (i=0;i<N;++i)
// {fprintf(FicSortie,"%lf\n ",signal_re_transpose[i]); 
//  printf(" %lf ",signal_re_transpose[i]);
// }
// fclose(FicSortie);
 
  return;
}

[reptils]

question, ça marche ça en c99?

Code :

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

double signal_re[N];

[gl]

question, ça marche ça en c99?

Code :

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

double signal_re[N];

Oui, c'est ce qui est appele Variable Length Array (VLA). Par contre ce n'est pas forcement correctement supporte par tout les compilateurs (aux dernieres nouvelles l'implementation des VLA de gcc n'est pas encore conforme)

[TheGzD]

Tu pourrais peut-être envisager de remplacer la portion de code suivante :

Code :

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

1
2
  double signal_re_transpose[N];
  double signal_im_transpose[N];

par celle-ci :

Code :

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

1
2
3
4
  double * signal_re_transpose;
  double * signal_im_transpose;
  signal_re_transpose = malloc(N*sizeof(double));
  signal_im_transpose = malloc(N*sizeof(double));

(pense à faire des free en fin d'exécution aussi)

Je trouve ça déjà plus propre.


Autre chose : es-tu certain que le la taille du contenu pointé par signal est toujours égale à N ?

[diogene]

J'ai fait des modifs dans mon code. Cela marche mieus mais quand j'augmente tk et N il y a toujours des erreurs de segmentation mais à des valeurs supérieurs ç celles ou cela ne fonctionnait pas

Le problème peut venir de ces déclarations:

Code :

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

1
2
  double signal_re_transpose[N];
  double signal_im_transpose[N];

qui réserve sur la pile 2N*sizeof(double) donc probablement qq chose comme 16*N octets. Or la place disponible sur la pile est limitée et pour des valeurs élevées de N le programme plantera.
La solution est alors de créer ces variables sur le tas, c'est à dire par malloc comme suggéré par TheGzD

[Skyounet]

Et si tu utilisais un debuggeur non ?

Lui il te dirait où exactement ça plante.

C'est bien plus pratique que des suppositions.

[berg]

Tu pourrais peut-être envisager de remplacer la portion de code suivante :

Code :

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

1
2
  double signal_re_transpose[N];
  double signal_im_transpose[N];

par celle-ci :

Code :

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

1
2
3
4
  double * signal_re_transpose;
  double * signal_im_transpose;
  signal_re_transpose = malloc(N*sizeof(double));
  signal_im_transpose = malloc(N*sizeof(double));

(pense à faire des free en fin d'exécution aussi)

Je trouve ça déjà plus propre.


Autre chose : es-tu certain que le la taille du contenu pointé par signal est toujours égale à N ?

je ne vois pas en quoi un malloc est plus propore vu qu il connait deja la taille des tableaux

[Emmanuel Delahaye]

je ne vois pas en quoi un malloc est plus propore vu qu il connait deja la taille des tableaux

Simple question de bon sens.

Parce que la ressource 'mémoire automatique' est bien plus limitée que la ressource 'mémoire allouée'.

De plus, la mémoire allouée est contrôlée. Si il a échec, on est prévenu (NULL). Avec la mémoire automatique, c'est l'inconnu...

[gl]

De plus, la mémoire allouée est contrôlée. Si il a échec, on est prévenu (NULL). Avec la mémoire automatique, c'est l'inconnu...

C'est d'ailleur, a mon avis, le principal defaut des VLA ce manque de controle sur le resultat de la reservation.

Quoiqu'avec les gestionnaires de memoire "optimiste" qui, pour gagner du temps lors de l'allocation et considerant que la memoire n'est plus une ressource limitee, fournisse toujours une adresse valide et alloue reelement la memoire au moment de l'utilisation (avec les risques que ca comporte si il n'y a plus de memoire disponible a ce moment la) on peut avoir le meme manque de controle avec la memoire allouee.

[Emmanuel Delahaye]

Quoiqu'avec les gestionnaires de memoire "optimiste" qui, pour gagner du temps lors de l'allocation et considerant que la emoire n'est plus une ressource limitee, fournisse toujours une adresse valide et alloue reelement la memoire au moment de l'utilisation (avec les risques que ca comporte si il n'y a plus de memoire disponible a ce moment la) on peut avoir le meme manque de controle avec la memoire allouee.

Oui, mais c'est alors un problème de système, pas de C. Ca doit pouvoir se configurer...

[gl]

Oui, mais c'est alors un problème de système, pas de C.

Tout a fait.