definition de type non reconnu par g++ ou erreur?

Version imprimable

Bonjour,

Jusqu a present, mon code compilé sous visual sans souci... le passage a g++ est tout autre. J aurais donc besoin de vos lumieres pour savoir si l erreur vient de moins et du laxisme de Visual.

Voila donc une partie de mon .h:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
typedef techsoft::matrix<double> dMatrix;
typedef std::valarray<double> dVector;
 
class Tensor
{
public:
	Tensor ();
	Tensor (dMatrix &val, int x, int y, int z);
	(...)
private:
	int _x, _y, _z;
	dMatrix _val;
        (...)
};

La partie concernée de mon .cpp

Code:

1
2
3
4
5
6
7
 
Tensor
Tensor::operator+ (const Tensor & T)
{
	Tensor *res = new Tensor (_val + T._val, _x, _y, _z); 
	return *res;
}

et l'erreur rejetée:

Code:

1
2
3
4
5
6
 
Tensor.cpp: In member function `Tensor Tensor::operator+(const Tensor&)':
Tensor.cpp:28: error: no matching function for call to `Tensor::Tensor(techsoft::matrix<double>, int&, int&, int&)'
Tensor.h:16: note: candidates are: Tensor::Tensor(const Tensor&)
Tensor.cpp:14: note:                 Tensor::Tensor(dMatrix&, int, int, int)
Tensor.cpp:3: note:                 Tensor::Tensor()

08/06/2006, 11h41
Jean-Marc.Bourguet

Tu essaies de passer un temporaire a une reference non constante. g++ a raison de refuser.
08/06/2006, 11h50
harsh

_val + T._val n'a rien de temporaire :? (l'operateur + de la lib utilisée crée bien une nouvelle matrice)...
08/06/2006, 13h29
harsh
En revanche, ce code compile sans probleme:
Code:

1 2 3 4 5 6 7 8 9 10 Tensor Tensor::operator+ (const Tensor & T) { dMatrix tmp (3, 3); tmp = _val + T._val; Tensor *res = new Tensor (tmp, _x, _y, _z); return *res; }
8O 8O 8O

Je n'ai peut etre pas bien saisi la remarque de Jean Marc B... alors si j'ai repondu a cote, expliquez moi... Car effectivement si je met un const dans mon constructeur ca compile... :?

Merci
08/06/2006, 14h52
Jean-Marc.Bourguet

Citation:

Envoyé par harsh

_val + T._val n'a rien de temporaire :? (l'operateur + de la lib utilisée crée bien une nouvelle matrice)...

Et le retourne par valeur je suppose. Donc c'est un temporaire.
08/06/2006, 14h54
Jean-Marc.Bourguet

Citation:

Envoyé par harsh

Car effectivement si je met un const dans mon constructeur ca compile... :?

Quand une fonction demande une reference constante on peut lui passer un temporaire.

Quand une fonction demande une reference non constante, on ne peut pas.

Pour les types definis par l'utilisateur, les temporaires sont formes par des fonctions (et des operateurs) ayant une valeur comme resultat.
08/06/2006, 15h02
Matthieu Brucher
Citation:
Envoyé par harsh

En revanche, ce code compile sans probleme:

Code:

1 2 3 4 5 6 7 8 9 10 Tensor Tensor::operator+ (const Tensor & T) { dMatrix tmp (3, 3); tmp = _val + T._val; Tensor *res = new Tensor (tmp, _x, _y, _z); return *res; }

8O 8O 8O

Je n'ai peut etre pas bien saisi la remarque de Jean Marc B... alors si j'ai repondu a cote, expliquez moi... Car effectivement si je met un const dans mon constructeur ca compile... :?

Merci
Avec une belle fuite mémoire comme conséquence...

Bonsoir ,

Pourquoi ne pas essayer :

Code:

1
2
3
4
5
6
7
8
9
10
 
Tensor
Tensor::operator+ (const Tensor & T)
{
  dMatrix tmp (3, 3);
  tmp = _val + T._val; 
  Tensor res (tmp, _x, _y, _z); 
 
  return res;
}

Cordialement.