Discussion :

[harsh]

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 :

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

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 :

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

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 :

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

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()

[Jean-Marc.Bourguet]

Tu essaies de passer un temporaire a une reference non constante. g++ a raison de refuser.

[harsh]

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

[harsh]

En revanche, ce code compile sans probleme:

Code :

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

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;
}

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

[Jean-Marc.Bourguet]

_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.

[Jean-Marc.Bourguet]

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.

[Matthieu Brucher]

En revanche, ce code compile sans probleme:

Code :

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

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;
}

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...

[dj.motte]

Bonsoir ,

Pourquoi ne pas essayer :

Code :

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

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.