probleme avec les constantes

Bonjour
j ai un probleme avec une constante,
cette fonction est censé renvoyer un pointeur vers un vecteur.

Code:

1
2
3
4
5
6
7
  Interval* Schedule::return_interval(Interval* i)const{
	for (Schedule::const_iterator it = begin(); it != end(); it++){
		if ((*it)->getInterval()->getId()==i->getId() && (*it)->getInterval()->getAvailabilityStart()==i->getAvailabilityStart()){
			return *it->getInterval() ;
		}
	}
}

lors de la compilation, j obtient cette erreur, pouvez vous m'aider SVP

Code:

1
2
src/Schedule.cpp: In member function ‘Interval* Schedule::return_interval(Interval*) const’:
src/Schedule.cpp:104:16: error: request for member ‘getInterval’ in ‘* it.std::_Rb_tree_const_iterator<_Tp>::operator-> [with _Tp = const Allocation*, std::_Rb_tree_const_iterator<_Tp>::pointer = const Allocation* const*]()’, which is of non-class type ‘const Allocation* const’

tout apport va etre utilise
Merci

Bonjour,
C'est difficile de répondre comme ça. Il nous en faudrait un peu plus sur Schedule et sur Interval.
A tout hasard, j'avance ça :
Interval const* Schedule::return_interval(Interval* i)const.

Pour faire des recherches, il existe l'algorithme std::find (exemple en C++1, mais se fait à l'identique en C++98 avec un foncteur) :
Code:

1 2 3 4 5 6 7 8 9 Interval const* Schedule::return_interval(Interval* i)const { return std::find(begin(),end(),[i](decltype(*begin()) elt_) { return elt_->getInterval()->getId()==i->getId() && elt_->getInterval()->getAvailabilityStart()==i->getAvailabilityStart } )->getInterval(); }
Il est difficile de manipuler correctement des pointeurs. C'est pour cela qu'on préfère utiliser les pointeurs intelligents (cf Présentation des pointeurs intelligents en C++ par Loïc Joly).

Citation:
Envoyé par 3DArchi
Pour faire des recherches, il existe l'algorithme std::find (exemple en C++1, mais se fait à l'identique en C++98 avec un foncteur) :
Code:

1 2 3 4 5 6 7 8 9 Interval const* Schedule::return_interval(Interval* i)const { return std::find(begin(),end(),[i](decltype(*begin()) elt_) { return elt_->getInterval()->getId()==i->getId() && elt_->getInterval()->getAvailabilityStart()==i->getAvailabilityStart } )->getInterval(); }
Ca serait pas plutot std::find_if() ?

Je pencherais dans un premier temps pour : getInterval() n'est pas const et ne renvoie pas un objet const.

Citation:

Envoyé par Klaim

Ca serait pas plutot std::find_if() ?

si
Code:

1 2 3 4 5 6 7 8 9 Interval const* Schedule::return_interval(Interval const* i)const { return std::find_if(begin(),end(),[i](decltype(*begin()) elt_) { return elt_->getInterval()->getId()==i->getId() && elt_->getInterval()->getAvailabilityStart()==i->getAvailabilityStart } )->getInterval(); }
Citation:

Envoyé par Emmanuel Deloget

Je pencherais dans un premier temps pour : getInterval() n'est pas const et ne renvoie pas un objet const.

Yes. Ce qui explique pourquoi i n'est pas passée par pointeur constant (d'ailleurs, référence constante serait mieux puisque le pointeur ne peut être nul d'après ce qui est proposé au début).