Discussion :

[adel25]

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

Code :

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

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 :

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

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

[3DArchi]

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 C++11 :

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

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

[Klaim]

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 C++11 :

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

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

[Emmanuel Deloget]

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

[3DArchi]

Ca serait pas plutot std::find_if() ?

si

Code :

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

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

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