Discussion :

[nicoblade77240]

Bonjour à tous,
Je souhaite créer un tableau 2D de complexes avec la bibliothèque standard <complex>

Je n'arrive pas à trouver la syntaxe pour créer ce tableau dynamique, et je ne comprends pas d'où vient l'erreur.

Merci à tous pour votre aide !

Code :

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

1
2
3
4
5
6
7
8
 
  complex **Pixels;
  Pixels = new complex *[Hauteur+2];
  for(int i=0 ; i<Hauteur+2 ; i++)
 
    {
      Pixels[i]=new complex[Largeur+2];
    }

[JolyLoic]

Déjà, complex est un template, car on peut vouloir avoir des nombres complexes basés sur une représentation flottante, ou une représentation double (voire éventuellement d'autres représentations, mais ce n'est je crois pas supporté par tous les compilateurs).

Donc, pour utiliser ce type, il faut préciser : complex<double> ou complex<float> (et c'est probablement complex<double> qui a le plus de sens si tu hésites entre les eux.

Ensuite, pour gérer un tableau dynamique, il y a bien plus simple que ce que tu essayes de faire. La solution passe par vector, un autre template de la bibliothèque standard. Une retranscription directe de ton code donnerait :

Code :

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

1
2
3
4
5
vector<vector<complex<double> > > Pixels; // Attention aus espaces entre '>'
for(int i=0 ; i<Hauteur+2 ; i++)
{
    Pixels.push_back(vector<complex<double> (Largeur+2));
}

On n'a pas l'air de beaucoup y gagner, mais la libération de la mémoire est alors effectuée automatiquement. Mais souvent, pour un tableau où chaque ligne a la même largeur, il est encore plus simple d'écrire :

Code :

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

1
2
3
vector<complex<double> > Pixels(Hauteur+2 * Largeur+2);
// Pour accéder au Pixel i,j
Pixels [i * ( Largeur+2) + j] = ...;

Je te conseille grandement de te documenter sur vector, c'est un miracle en terme d'économie de bugs !

[nicoblade77240]

Déjà, complex est un template, car on peut vouloir avoir des nombres complexes basés sur une représentation flottante, ou une représentation double (voire éventuellement d'autres représentations, mais ce n'est je crois pas supporté par tous les compilateurs).

Donc, pour utiliser ce type, il faut préciser : complex<double> ou complex<float> (et c'est probablement complex<double> qui a le plus de sens si tu hésites entre les eux.

Ensuite, pour gérer un tableau dynamique, il y a bien plus simple que ce que tu essayes de faire. La solution passe par vector, un autre template de la bibliothèque standard. Une retranscription directe de ton code donnerait :

Code :

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

1
2
3
4
5
vector<vector<complex<double> > > Pixels; // Attention aus espaces entre '>'
for(int i=0 ; i<Hauteur+2 ; i++)
{
    Pixels.push_back(vector<complex<double> (Largeur+2));
}

On n'a pas l'air de beaucoup y gagner, mais la libération de la mémoire est alors effectuée automatiquement. Mais souvent, pour un tableau où chaque ligne a la même largeur, il est encore plus simple d'écrire :

Code :

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

1
2
3
vector<complex<double> > Pixels(Hauteur+2 * Largeur+2);
// Pour accéder au Pixel i,j
Pixels [i * ( Largeur+2) + j] = ...;

Je te conseille grandement de te documenter sur vector, c'est un miracle en terme d'économie de bugs !

Merci beaucoup pour ton aide mais dans mon cas je travaille sur des tableaux de pixels, et j'ai déja programmé la grosse classe en entier en tableaux, je ne voudrais pas tout recommencer, est il possible de ne pas utiliser les vector et garder la structure en tableaux ?
Merci beaucoup pour ton aide (ici c'est un projet scolaire, donc le but et de ne pas utiliser vector et de tout faire à la main, et je dois avouer, je galère pas mal)
Merci encore pour ton aide précieuse

[JolyLoic]

est il possible de ne pas utiliser les vector et garder la structure en tableaux ?

Oui, c'est possible, c'est juste plus compliqué. A la base, ton code d'initialisation à l'air correct, à partir du moment où tu remplaces complex par complex<double>, mais je l'écrirais pour simplifier :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
  typedef std::complex<double> Complexe;
  typedef Complexe * Ligne;
  typedef Ligne * Tableau;
 
  Tableau Pixels;
  Pixels = new Ligne[Hauteur+2];
  for(int i=0 ; i<Hauteur+2 ; i++)
 
    {
      Pixels[i]=new Complexe[Largeur+2];
    }

Et le conseil de travailler à une seule dimension est tout aussi valable si tu travailles avec de la gestion manuelle de la mémoire :

Code :

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

1
2
3
4
// Tiens, j'ai oublié les () dans le message précédent...
complex<double> *Pixels = new complex<double>[(Hauteur+2) * (Largeur+2)];
// Pour accéder au Pixel i,j
Pixels [i * ( Largeur+2) + j] = ...;

[nicoblade77240]

Merci beaucoup pour toutes ces indications.

[koala01]

Salut

<snip>
Merci beaucoup pour ton aide (ici c'est un projet scolaire, donc le but et de ne pas utiliser vector et de tout faire à la main, et je dois avouer, je galère pas mal)

Ne serait-ce pas l'occasion de, justement, faire découvrir à ton prof les joies de la programmation en C++ "moderne"

S'il est ne serait-ce qu'un tantinet ouvert à la discussion, cela pourrait non seulement être l'occasion d'une heure de cours pour le moins intéressante, mais aussi l'inciter à revoir entièrement la manière dont il présente le C++ à ses élèves pour la suite

Et tu peux me croire (je l'ai moi-même fait), si le prof n'est pas plus buté qu'une mule corse et que tu arrive à lui présenter des avantages directs (fini les new et delete dans tous les sens), il se ralliera surement à ton optique