conversion to non-scalar type requested !

Hello,

Je suis nouveau dans C++ je tente de récupérer un objet (il s'agit d'un itérateur) de l'objet courant (this) dans une méthode membre et j'obtiens cette erreur : conversion to non-scalar type requested.

La méthode membre en question :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

void AC_Nef_Polyhedron::draw() const { // cela ne marche pas ("conversion to non-scalar type requested") Facet_iterator monIterator=facets_begin();   // cela fonctionne AC_Nef_Polyhedron np (NULL,NULL); Facet_iterator t=np.facets_begin();   // cela fonctionne AC_Nef_Polyhedron* hj=new AC_Nef_Polyhedron(NULL,NULL); Facet_iterator o=hj->facets_begin(); }

---

Qu'est-ce que je fais faux?

Bonjour,
Il faudrait que tu nous montres la définition de Facet_iterator et facets_begin()

Ouiais, il s'agit d'une déclaration d'une bibliothèque tierce énorme ! (CGAL)...

-> Le diagnostic (avec notamment mes exemples "qui marche") n'est-il pas faisable ?

?

Il faudrait que tu donnes un peu plus de détails. Quelle ligne produit l'erreur? Quel est le code et le message d'erreur? Plateforme? Compilateur?

Hello,

La plate-forme est ubuntu, le compilateur est gcc.

L'erreur se produit là :

Code:

---

1 2 3

... Facet_iterator monIterator=facets_begin(); ...

---

Le message "complet" de l'erreur est ici :

Code:

---

erreur: conversion from «CGAL::CGALi::In_place_list_const_iterator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > >, std::allocator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > > > >» to non-scalar type «CGAL::CGALi::In_place_list_iterator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > >, std::allocator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > > > >» requested

---

:oops:

Je pense avoir trouvé une piste :D

Code:

---

1 2 3 4 5 6 7 8 9 10

erreur: conversion from «   CGAL::CGALi::In_place_list_const_iterator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > >, std::allocator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > > > >»     to non-scalar type     CGAL::CGALi::In_place_list_iterator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > >, std::allocator<CGAL::HalfedgeDS_in_place_list_face<CGAL::I_Polyhedron_facet<CGAL::HalfedgeDS_face_base<CGAL::HalfedgeDS_list_types<CGAL::Simple_cartesian<double>, CGAL::I_Polyhedron_derived_items_3<CGAL::Polyhedron_items_3>, std::allocator<int> >, CGAL::Boolean_tag<true>, CGAL::Plane_3<CGAL::Simple_cartesian<double> > > > > > >» requested

---

La seule différence c'est :

Code:

---

1 2 3

CGAL::CGALi::In_place_list_const_iterator CGAL::CGALi::In_place_list_iterator

---

Et dans ton premier post, ta méthode AC_Nef_Polyhedron::draw() était const. Donc il faut peut être que tu fasses quelque chose comme :

Code:

---

1 2	Facet_const_iterator monIterator = facets_begin();

---

Je vais m'empresser d'essayer !!!

Merci !

Te donnes des nouvelles !

Gilles

...ca marche !!!

Je n'ai même pas vérifier dans la doc, mais un iterateur const existe !!

-> Donc une méthode membre déclarée const ne peut pas 'employé' une autre entité non-const ou quelque chose comme ça ??

-> J'ai lu quelques part qu'il était bon de déclarer const les méthodes membres qui ne sont pas sensées modifier l'objet ! Dans ce cas-là, j'ai perdu quelques heures !!!


-> Merci mille fois !

Gilles

En fait, ça explique très bien le comportement que tu as observé dans le post #1.

Quand une fonction membre est const, cela veut dire qu'à l'intérieur de cette fonction tu ne peux pas modifier l'instance courante, ce qui implique de ne pouvoir appeler que les fonctions membres const de l'instance courante ou les fonctions membres const des objets membres de l'instance courante .

Or la fonction begin() est probablement surchargée comme ceci :

Code:

---

1 2 3 4 5 6 7 8 9

iterator begin() { return ...; }   const_iterator begin() const { return ...; }

---

Et renvoi donc un iterator ou un const_iterator en fonction de la constness de la fonction.

Donc en gros :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18

void AC_Nef_Polyhedron::draw() const {   // erreur, conversion de Facet_const_iterator vers Facet_iterator impossible Facet_iterator monIterator=facets_begin();   // Sauf qu'ici on déclare une NOUVELLE instance de AC_Nef_Polyhedron // elle n'est pas affecté epar le const de la fonction // qui ne s'applique qu'à l'instance en cours et à ses membres AC_Nef_Polyhedron np (NULL,NULL); Facet_iterator t = np.facets_begin(); // Ok   // par contre on retomberait sur la même erreur en faisant... const AC_Nef_Polyhedron np (NULL,NULL); Facet_iterator t = np.facets_begin(); }

---

Citation:

---

J'ai lu quelques part qu'il était bon de déclarer const les méthodes membres qui ne sont pas sensées modifier l'objet ! Dans ce cas-là, j'ai perdu quelques heures !!!

---

Ça reste un très bon conseil. Les avantages du respect de la constness dépasse largement les inconvénients. Même si c'est vrai que les messages d'erreur du au const sont souvent un peu étranges. On s'y fait.