Discussion :

[fab87]

Bonjour

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
class Matrice
{
    protected:
 
        typedef float *ligne;
        ligne *matrice; // données de la matrice
        int nbLig; // Nombre de lignes de la matrice
        int nbCol; // Nombre de colonnes de la matrice
...
}
 
class Vecteur : public Matrice
{
...
}
 
Vecteur Vecteur::operator*(const Matrice &mat)
{
    Vecteur ret;
 
    ret.x = mat.matrice[0][0]*x + mat.matrice[0][1]*y + mat.matrice[0][2]*z + mat.matrice[0][3];
    ret.y = mat.matrice[1][0]*x + mat.matrice[1][1]*y + mat.matrice[1][2]*z + mat.matrice[1][3];
    ret.z = mat.matrice[2][0]*x + mat.matrice[2][1]*y + mat.matrice[2][2]*z + mat.matrice[2][3];
 
    return ret;
}

Le compilateur me dit que je ne peux pas accéder à l'attribut matrice (qui correspond aux éléments de ma matrice) alors que je l'ai mis en protected et que j'ai fait un héritage..
Je ne comprends pas le problème, merci de bien vouloir m'aider.

[Flo.]

Salut,

Dans ta fonction

Code :

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

Vecteur Vecteur::operator*(const Matrice &mat)

mat est marquée comme const. Tu ne peux donc pas accéder à ses attributs comme tu le fais excepté si tu les rends "mutable" ou si tu supprimes le const de l'argument de ta fonction

Code :

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

mutable ligne *matrice;

ou

Code :

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

Vecteur Vecteur::operator*(Matrice &mat)

Flo.

[fab87]

Merci çà fonctionne!!
Mais je viens de remarquer que ma fonction va retourner une matrice! =)

[Flo.]

Attention, parce que mes 2 solutions ne sont peut-être pas bonnes d'un point de vue philosophie. Effectivement dans ta fonction, il est logique de mettre mat en const puisqu'elle n'est pas modifiée. Et mettre ses attributs en mutable n'est pas également une solution.

La bonne (et troisième) solution à mon avis serait de proposer une fonction

Code :

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

1
2
3
4
const double Matrice::get(const unsigned int & i, const unsigned int & j) const 
{
    return matrice[i][j];
}

que tu pourrais utiliser comme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
Vecteur Vecteur::operator*(const Matrice &mat)
{
    Vecteur ret;
 
    ret.x = mat.get(0,0)*x + mat.get(0,1)*y + mat.get(0,2)*z + mat.get(0,3);
    ret.y = mat.get(1,0)*x + mat.get(1,1)*y + mat.get(1,2)*z + mat.get(1,3);
    ret.z = mat.get(2,0)*x + mat.get(2,1)*y + mat.get(2,2)*z + mat.get(2,3);
 
    return ret;
}

Flo.

[fab87]

Merci!
En effet je pense que c'est plus correcte.
Néanmoins j'ai toujours deux erreurs :

- pour cette ligne : Vecteur operator*(const Matrice &);

ISO C++ forbids declaration of `Matrice' with no type

expected ',' or '...' before '&' token

[fab87]

En fait, je suis dans la classe vecteur et le produit d'un vecteur par une matrice me retourne une matrice normalement donc çà va pas.

Comment puis-je faire pour implémenter une méthode permettant de multiplier une matrice par un vecteur dans ma classe Vecteur?
Je suis obligé de mettre deux paramètres nan? Mais pour une surcharge il ne faut pas même je crois qu'on ne peut pas..

Merci de bien m'aider

Vecteur Vecteur::operator*(const Vecteur &vect, const Matrice &mat) serait une idiotie je crois mais je ne vois pas comment faire..