Discussion :

[sandball22]

Bonjour, j'ai un problème au niveau de la multiplication des valeurs de 2 tableaux.

Voici le code:

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
 
void FrequencyTransposition(float *signal_reel, float tk, int taille_signal)
{
  int i;
  int N=taille_signal;               /*récupération de la taille des tableaux*/
  float fp,cosinus[N];
  float *signal_re=NULL;
  float *signal_re_transpose=NULL;
 
  FILE * FicSortie;
 
 
  signal_re=malloc(sizeof(float) * (N));
  signal_re_transpose=malloc(sizeof(float) * (N));
 
  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 %f MHz :",fp);       
  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)*/
  }
 
 
FicSortie= fopen("F:/Users/garnier/Matlab/reel.txt","w");
 
 for (i=0;i<N;++i)
 {fprintf(FicSortie,"%lf\n ",signal_re_transpose[i]); 
  printf(" %f ",signal_re_transpose[i]);
 }
 fclose(FicSortie);
 
  return;
}

J'ai des erreurs du style j'obtiens:-588189646392141076285668743708672.000000.Or je multiplie un cosinus donc compris entre +1 et -1 avec des coefficients de dépassant pas 1000. DOnc cela n'est pas cohérent.

Quelqu'un peut-il m'aider? Merci.

[fearyourself]

Voici quelques remarques :

- Pourquoi utilises tu N si tu passes la taille du signal en paramètre ?
- Pourquoi utiliser malloc si la taille est fixe ?
- Pas de tests après les mallocs, fopen
- Pas de libération de mémoire.

des coefficients de dépassant pas 1000. DOnc cela n'est pas cohérent.

Certes pas cohérent mais on a aucune preuve que tes coefficients ne dépassent pas 1000...

Jc

[sandball22]

- Pourquoi utilises tu N si tu passes la taille du signal en paramètre ?
Oui c'est sur, surtout que je ne midifie pas cette valeur dans cette fonction.

- Pourquoi utiliser malloc si la taille est fixe ?
La taille dépend du paramètre qui n'est pas connue.

- Pas de tests après les mallocs, fopen
Je ne vois pas ce que tu veux dire.


- Pas de libération de mémoire.
A quel endroit faut-il faire la libération

[DaZumba]

J'ai des erreurs du style j'obtiens:-588189646392141076285668743708672.000000.Or je multiplie un cosinus donc compris entre +1 et -1 avec des coefficients de dépassant pas 1000.

Ce genre d'erreur est souvent du a l'oubli de #include <math.h>. Du coup, cos() se retrouve declaree implicitement comme retournant un int, ce qui fait un peu mal.

[fearyourself]

- Pourquoi utiliser malloc si la taille est fixe ?
La taille dépend du paramètre qui n'est pas connue.

Pourtant tu passes une taille fixe à malloc...

- Pas de tests après les mallocs, fopen
Je ne vois pas ce que tu veux dire.

malloc et fopen retournent des pointeurs, il faut vérifier si ces derniers ne sont pas nuls avant de continuer.

- Pas de libération de mémoire.
A quel endroit faut-il faire la libération

[/quote]
A la fin de la fonction il faut le faire et ton return à la fin est inutile...

Jc

[diogene]

Toujours dans le même genre de remarques que celles de fearyourself

- Pourquoi un tableau supplémentaire signal_re ? On peut aussi bien travailler sur signal_reel.

- Pourquoi un tableau cosinus ? On peut travailler avec une simple variable float cosinus.

- D'ailleurs, pouquoi un tableau signal_re_transpose ? on peut faire le calcul pour chaque point et l'écrire aussitot. On n'a besoin de créer aucun tableau au final.