Discussion :

[sheep_one]

Bonjour,
Je tombe sur un problème récurrent et plutot simple mais je ne vois pas pourquoi mon code ne fonctionne pas.
Donc pour vous expliquer j'ai une classe Point

Code :

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

1
2
3
4
5
6
7
8
 
class Point 
{
    int x;
    int y;
    public :
      Point (int _x, int _y);
};

et une deuxieme classe :

Code :

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

1
2
3
4
5
6
7
8
 
classe Droite
{
    Point A;
    Point B;
    public :
      Droite (Point A, Point B);
};

Dans le main j'arrive à creer des objet de ma classe Point mais pas de la classe Droite j'ai cette erreur :

terminate called after throwing an instance of 'std::logic_error'
what(): basic_string::_S_construct NULL not valid

Voici mon main au cas ou :

Code :

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

1
2
3
4
5
6
7
 
int main() 
{
Point unPointA (5,4);
Point unPointB(3,5);
Droite uneDroite(unPointA, unPointB);
}

Voila, j'espère que l'exemple que j'ai choisi est clair. A ce que je comprend c'est qu'il cherche un constructeur par defaut, mais je ne vois pas pourquoi?
Merci

[Ekleog]

Le problème n'est pas dans le code que tu nous as montré. Il concerne un string quelque part.
Pourrait-on avoir tout le code ?

[kev753]

Dans ton constructeur Droite:

Code :

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

Droite (Point A, Point B);

tu a 2 point en paramètre qui porte le meme nom et qui sont de meme classe que les 2 point qui constitue ta classe :

Code :

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

1
2
3
4
5
 
classe Droite
{
    Point A;
    Point B;

je ne sais pas si c'est ça le problème mais dans tous les cas j'éviterais personnellement...

[Mat.M]

salut il me semble que le problème vient du fait que les constructeurs de Point et Droite n'aient pas de corps.
Et puis oui il faut éviter de donner les mêmes noms.
Testé sous VS2010 , si le constructeur n'a pas de corps de fonction j'ai une erreur de l'éditeur de lien

Erreur 1 error LNK2019: symbole externe non résolu "public: __thiscall Point :: Point(int,int)" (??0Point@@QAE@HH@Z) référencé dans la fonction "protected: virtual int __thiscall CtestMFC1Dlg::OnInitDialog(void)" (?OnInitDialog@CtestMFC1Dlg@@MAEHXZ) C:\Users\Documents\Visual Studio 2010\Projects\testMFC1\testMFC1\testMFC1Dlg.obj testMFC1

Avec ceci ça fonctionne

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
class Point 
{
    int x;
    int y;
    public :
Point (int _x, int _y){ x=_x;y=_y;};
 
};
 
BOOL CtestMFC1Dlg::OnInitDialog()
{
CDialogEx::OnInitDialog();
 
Point pt(10,20);
 
 
}

[Morbius]

Mat le constructeur d'un objet ne s'écrit pas comme ça en C++. Dans ce que tu as fait tu donnes les valeurs à x et y après la création de l'objet ce qui n'est pas bon.
Il faut écrire :

Code :

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

1
2
3
4
5
6
7
8
9
class Point 
{
    int x;
    int y;
    public :
 
        Point (int _x, int _y) : x(_x), y(_y){}
 
};

Petit point concernant le nom de tes données membres, prends l'habitude de faire commencer leur nom par une majuscule voir mieux comme ceci (plus propre et plus simple à comprendre)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class Point 
{
    private :
        int m_X;
        int m_Y;
 
    public :
        Point (int X, int Y) : m_X(X), m_Y(Y){}
        ~Point(){}  // ne pas oublier le destructeur même si il ne fait rien
 
}; // class Point

[therwald]

Testé sous VS2010 , si le constructeur n'a pas de corps de fonction j'ai une erreur de l'éditeur de lien

Euh, le PO montre une erreur runtime...visiblement son code compile donc.
Par contre, comme souligné par Ekleog, le problème semble venir d'une string, donc il faut chercher ailleurs.

Nous ne voyons ici que des déclarations de classes, il nous faudrait les implémentations...sans doute y-a-t'il là une construction de string avec un argument null...soit directement soit indirectement

[sheep_one]

salut,
Alors voila le code exactement. Je voulais juste montrer un exemple clair pour illustrer mon problème.

la classe Piece

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
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
class Piece
	{
   private:
		/*
		 * TODO: Rajouter nom de la piece
		 */
	   std::string id;
	   std::vector<Point> listePoints;
   public:
	   Piece(const std::string &_id = 0);
 
	   int addPoint(Point unPoint);
 
	   std::vector<Point> getPoints () const;
	   string  getIdPiece() const;
	   int getIdPoint (Type type);
	   void affichage ()const;
 
	   Piece & operator = (const Piece & _piece);
	};
 
}
	Piece::Piece(const std::string &_id):listePoints(0)
	{
		id = _id;
	}
 
	int Piece::addPeripherique(Point unPoint)
	{
		listePoints.push_back(unPoint);
		return peripheriques.size();
	}//addunPoint()
 
	std::vector<unPoint> Piece::getPoints () const
	{
		return listePoints;
	}//getPoints()
 
 
	int Piece::getIdPoints (Type type)
	{
		for(size_t i = 0, size = listePoints.size(); i < size; ++i )
		{
			Point& p1 = Point[i];
			if (p1.getType() == type)
				return p1.getId();
		}
		return -1;
	}//--getIdPoints()
	string Piece::getIdPiece() const
	{
		return id;
	}
	void Piece::affichage() const
	{
 
	}
 
	Piece & Piece::operator = (const Piece &_piece)
	{
		id= _piece.getIdPiece();
		for (size_t i(0); i  <_piece.getPoints().size(); ++i)
		{
			this->addPoints(_piece.getPoints()[i]);
		}
		return *this;
	}

Et mon autre classe:

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
 
class Controller {	
	bool 	mode;
	double 	echelle;;
	double 	ecart;
	Piece	unePiece;
 
  public:
	Controller(Piece _piece);
 
	// ---------les getters et les setters----------------------------------
 
};
 
Controller::Controller(Piece _piece) {
	echelle 	= 0;
	unePiece   = _piece;
	mode	= true;
        ecart         = 0;   
}

le main :

Code :

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

1
2
3
 
        Piece unePiece ("test");
	Controller unControle (unePiece);

J'ai enlevé pas mal de chose pour que ce soit plus claire, je me demande si le problème ne vient pas de l'initialisation de mes vecteurs..
Merci pour vos réponses

[therwald]

Sinon, sous quel environnement travailles-tu? Sous linux tu peux avoir une idée de la source du problème en compilant en mode debug et un coup de valgrind pour avoir la stacktrace...

[Ekleog]

[...]

Code :

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

1
2
3
4
5
6
7
 
class Piece
	{
// ...
	   Piece(const std::string &_id = 0);
// ...
	};

[...]

Il est probable que le problème vienne de cette ligne.
En effet, le 0 sera converti en char const * (car 0 <=> NULL).
Tu essaies donc de construire l'argument _id avec un pointeur nul.
Ce qui est probablement la source de l'erreur.
Remplacer par cette ligne aidera sûrement.

Code :

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

	   Piece(const std::string & _id = "");

[sheep_one]

C'était bien cette histoire de valeur par defaut, merci.