Discussion :

[yayel59]

Bonjour à tous,

Il doit y avoir une subtilité du C++ qui m'échappe. Si quelqu'un peut m'expliquer pourquoi j'ai un problème avec un constructeur qui prend une chaine (string) comme paramètre...

Voici un morceau de code qui illustre le problème. Je l'ai compilé avec g++ (version 4.4 et 4.5 sous linux).

Voici le Main :

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
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
// fichier Main.cpp
#include "A.hpp"
#include <iostream>
 
using namespace std;
 
int main(int argc, char* argv[])
{
 
  cout <<"hello" << endl;
 
  string s("coucou");  
  A a(s);
 
  cout << " ---- " << endl;
 
  if (argc==2)
    {
 
      /* Tout ce qui suit fonctionne ! */
      string s2 = argv[1];
      A b(s2);
 
      A c=A(string(argv[1]));
 
      A d=string(argv[1]);
 
 
      char * cs = argv[1];
      string ps = string(cs);
 
      A e(argv[1]);
      e.affiche(string(argv[1]));
 
 
      /* Le problème commence ici */
      A f(string(argv[1]));
 
      /* Le compilateur signale :
	 Main.cpp:46: warning: the address of ' f(std::string*)' will always evaluate as 'true'
      */
 
      /* f n'est même plus un objet... */
      //f.affiche(string(argv[1]));
      cout << f << endl;           
    }  
 
}

Le code de la classe A est assez standard...

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
27
28
 
#include <iostream>
 
 
class A
{
private :
  std::string message;
 
public :
  // Constructeur par défaut
  A()  
  {
    std::cout << "message par défaut" << std::endl;
  } 
 
  // constructeur avec une chaine
  A(std::string message)
  {
    std::cout << message << std::endl;
  }
 
  // fonction avec une chaine
  void affiche(std::string message)
  {
    std::cout << message << std::endl;
  }
};

Et je mets tout de même le Makefile (bien que simplicime...) car on ne sait jamais...

Code :

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

1
2
3
4
5
6
7
8
 
all: Main
 
Main : Main.cpp
	g++ -W -Wall Main.cpp -o Main
 
clean :
	rm Main

Si quelqu'un a une idée de ce comportement ça m'intéresse !

[gl]

Il faudrait peut être aussi indiquer quel est précisément le problème rencontré.

Au minimum il manque

Code :

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

#include <string>

[yayel59]

Il faudrait peut être aussi indiquer quel est précisément le problème rencontré.

Oui, j'aurais du préciser qu'en plus du message bizarre à la compilation (que j'ai indiqué dans le source).

L'exécution produit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
./Main developpez
hello
coucou
 ---- 
developpez
developpez
developpez
developpez
developpez
1

Donc le dernier constructeur n'est pas appelé (Il y 6 appels de constructeurs qui affichent la chaine en paramètre). De plus, f vaut 1 (ce qui semble compatible avec l'indication du compilateur comme quoi c'est toujours vrai...) or, c'est un objet (dont le flux de sortie n'est même pas défini...).

Au minimum il manque

Code :

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

#include <string>

J'avais testé avec et sans, ça ne change pas...

[Klaim]

Le problème (ce qui ne va pas) n'est pas clair du tout.

J'essaie d'interpréter comme je peu:

Donc le dernier constructeur n'est pas appelé (Il y 6 appels de constructeurs qui affichent la chaine en paramètre).

Si je comprends bien, après avoir analysé le code, cette ligne (que tu aurais pu isolé ) :

Code :

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

1
2
 /* Le problème commence ici */
      A f(string(argv[1]));

N'affiche pas le string comme pour les autres initializations.

Si c'est ça, je ne suis pas sur du problème, mais il faudrait vérifier si cette ligne n'est pas interprettée comme une signature de fonction. D'après le warning, j'ai l'impression que c'est le cas.

Code :

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

cout << f << endl;

Je ne comprends pas pourquoi cette ligne compile. A part si ma précédente supposition est vrai, il est impossible que cette ligne compile puisque dans le code fournis tu n'as pas défini d'opérateur de flux (operator<<).

[Joel F]

ca manque de const& tout ces passages d'arguments

[3DArchi]

Salut,

Code :

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

 A f(std::string(argv[1]));

est compris comme une déclaration de fonction qui prend en argument un std::string* et retourne un A. Pas comme un objet de type A. Ainsi :

Code :

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

1
2
std::string s;
A ret = f(&s);

Ce code compile.


Dans ce cas, vaut mieux faire :

Code :

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

A f = A(std::string(argv[1]));

Cf F.A.Q. Y a-t-il une différence quelconque entre MaClasse x; et MaClasse x(); ?

[gl]

en plus du message bizarre à la compilation (que j'ai indiqué dans le source).

Au temps pour moi, j'ai zappé le commentaire.

[yayel59]

ca manque de const& tout ces passages d'arguments

Je suis d'accord, mais ça ne change rien au problème (j'avais aussi testé avec et sans avant de poster).

[yayel59]

Dans ce cas, vaut mieux faire :

Code :

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

A f = A(std::string(argv[1]));

Cf F.A.Q. Y a-t-il une différence quelconque entre MaClasse x; et MaClasse x(); ?

Merci ! Maintenant, j'ai bien compris. C'est vrai que j'avais oublié ce "piège" de la fausse déclaration de fonction. C'est un peu plus clair dans ma petite tête