Discussion :

[Cpowa]

Bonjour,

Voila j'ai besoin d'une classe ChampVectoriel mais sachant que j'ai déjà une classe ChampScalaire j'ai deux choix possible pour faire celle-ci.

Ma classe ChampScalaire peut se résumer à ceci

Code :

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

1
2
3
4
5
6
7
8
9
 
class ChampScalaire
{
private:
    float m_pas; //Pas de discrétisation suivant X et Y.
    unsigned int m_tailleX; //nb de cases suivant X.
    unsigned int m_tailleY; //nb de cases suivant Y.
    std::vector<float>  m_champ; //tableau contenant les valeurs du champ en chaque point
};

Et donc comme premier choix pour ma classe ChampVectoriel je peux faire ça:

Code :

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

1
2
3
4
5
6
7
 
class ChampVectoriel
{
private:
ChampScalaire m_x;
ChampScalaire m_y;
};

Avantage:
-Réutilisation de la classe précédente
Inconvénient:
-Obligé de faire les calculs composante pas composante (répétition de code)


La deuxième possibilité est la suivante:

Code :

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

1
2
3
4
5
6
7
8
9
 
class ChampVectoriel
{
private:
    float m_pas; //Pas de discrétisation suivant X et Y.
    unsigned int m_tailleX; //nb de cases suivant X.
    unsigned int m_tailleY; //nb de cases suivant Y.
    std::vector<Vecteur>  m_champ; //tableau contenant les vecteurs du champ en chaque point
};

Inconvénient:
-Avoir une classe vecteur (en fait je l'ai déjà mais bon le code devient plus gros)
-Niveau performance c'est mieux d'avoir un gros tableau (mon champ) avec plein de petits tableau de deux cases (mes vecteurs) ou alors deux gros tableaux (champ suivant x et y)?

Avantage:
-Possibilité d’utiliser les surcharges d'opérateur de la classe Vecteur et donc le code devient plus propre quand je fais des calculs, par ex:

Code :

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

1
2
3
4
 
ChampVectoriel champ(10,20) //champ 10*20 cases
 
champ(i,j)=champ(i,j)*2+vecteur;

Qu’en pensez vous ?

Merci.

[JolyLoic]

J'en pense que je séparerais la notion de champ de la notion de ce qui est dans le champ, de manière à avoir des champs scalaires, vectoriels, matriciels...

Code :

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

1
2
3
4
5
6
7
8
9
template<class T>
class Champ
{
private:
    float m_pas; //Pas de discrétisation suivant X et Y.
    unsigned int m_tailleX; //nb de cases suivant X.
    unsigned int m_tailleY; //nb de cases suivant Y.
    std::vector<T>  m_champ; //tableau contenant les valeurs du champ en chaque point
};

[Cpowa]

Oui j'y ai également pensé mais je trouve que c'est compliquer le code inutilement, m'a classe n'a pas pour but d'être généraliste.
En plus les templates obligent à écrire les fonctions dans le .h, c'est pas très jolie.

Et je me soucis également des performances, mieux vaut avoir deux gros tableaux ou un gros et plein de petits ?
(je vais parcourir les tableaux un grand nombre de fois et effectuer des calculs sur chaque case)

[Ylvin]

Oui j'y ai également pensé mais je trouve que c'est compliquer le code inutilement, m'a classe n'a pas pour but d'être généraliste.
En plus les templates obligent à écrire les fonctions dans le .h, c'est pas très jolie.

Pas nécessairement, tu peux aussi écrire quelque chose comme :

Code :

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

1
2
template <class T>
T myclass<T>::func () {..}

Et à ce moment là l'écrire dans le fichier .cpp.
Cf : http://www.cplusplus.com/doc/tutorial/templates/

[3DArchi]

Salut,

Pas nécessairement, tu peux aussi écrire quelque chose comme :

Code :

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

1
2
template <class T>
T myclass<T>::func () {..}

Et à ce moment là l'écrire dans le fichier .cpp.
Cf : http://www.cplusplus.com/doc/tutorial/templates/

Cela provoquera une erreur à l'édition de lien. Que la définition ne soit pas dans la classe n'empêche pas qu'elle est nécessaire en général dans le .h.

Oui j'y ai également pensé mais je trouve que c'est compliquer le code inutilement, m'a classe n'a pas pour but d'être généraliste.
En plus les templates obligent à écrire les fonctions dans le .h, c'est pas très jolie.

Une solution est de définir les fonctions templates dans une fichier séparé et d'inclure celui-ci à la fin du fichier de définition de la classe :

Code class_tpl.h :

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

1
2
3
4
5
6
7
 
template<typename T> struct dummy
{
    void do_something();
};
//...
#include "class_tpl.impl"

Code class_tpl_impl.h :

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

1
2
3
4
5
6
 
template<typename T> 
void dummy<T>::do_something()
{
//...
};

Et je me soucis également des performances, mieux vaut avoir deux gros tableaux ou un gros et plein de petits ?
(je vais parcourir les tableaux un grand nombre de fois et effectuer des calculs sur chaque case)

Je pense que ça va dépendre de comment ils sont parcourus.

[Joel F]

Si on veut pouvoir paralleliser proprement, on prefere des Structure of Arrays que des Array of Structures.

[Ylvin]

Salut,

Cela provoquera une erreur à l'édition de lien. Que la définition ne soit pas dans la classe n'empêche pas qu'elle est nécessaire en général dans le .h.

Oui, évidemment, il faut inclure les .cpp correspondants. Sinon le compilateur peut pas faire les .o.
Reste que les déclarations sont séparées. Et personnellement, je préfère le faire avec des .cpp plutôt que des .h.