Discussion :

[koala01]

Salut,

J'ai (peut etre mal) cherché sur le net, et sur le site, mais je n'ai pas trouvé de solution satisfaisante...

J'ai une classe qui utilise un vecteur sur des chaines, et une méthode qui devrait arriver à supprimer l'une des chaine en fonction de son index...

S'il est clair que je dois utiliser la méthode erase des vecteurs, je me trouve confronté au fait qu'il faut lui passer... un iterateur en argument...

Bien sur, je pourrais travailler sous la forme de

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
void Suppress(const unsigned int index)
{
std::vector<std::string>::iterator it=Description.begin();
for(unsigned int i=0;i<index;i++)
{
    ++it;
    if(it==Description.end())
       throw std::out_of_range("indice hors limite");
}
Description.erase(it);
}

Mais je me demandais s'il n'y aurait pas moyen d'obtenir directement l'itérateur sur l'élément dont l'index est donné en argument...

Un peu comme si on réalisait un code qui prendrait la forme

Code :

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

1
2
3
4
5
6
7
8
 
void Suppress(const unsigned int index)
{
    std::string str=Description.at(index);//ou peut etre autre chose,  en
                                         // somme :p
    //convertion de str en l'itérateur 
    Description.erase(l_iterateur);
}

J'ai bien conscience que, sémantiquement parlant, un itérateur est sensé parcourrir un conteneur de manière... itérative, et non de manière aléatoire, à l'instar de ce que je voudrais faire...

Mais je voudrais confirmation/infirmation de la chose...

Merci d'avance

[Jean-Marc.Bourguet]

Il y a des itérateurs à accès aléatoire et celui de vector en est un. Tu peux utiliser + dessus avec le sens que tu devines. Grosse différence avec le code que tu montres, il n'y a pas de validation du résultat (donc un problème si tu sorts du vector).

[koala01]

Il y a des itérateurs à accès aléatoire et celui de vector en est un. Tu peux utiliser + dessus avec le sens que tu devines. Grosse différence avec le code que tu montres, il n'y a pas de validation du résultat (donc un problème si tu sorts du vector).

Ah, bon...

Mais alors, comment convertir le résultat fournis par

Code :

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

1
2
3
4
 
(...)Description.at(index);
ou 
(...)Description[index];

en itérateur (le tout en sachant que j'aurais une préférence sur le fait d'utiliser at , et en n'oubliant pas de remplir la partie manquante )

Je n'ai aucune raison de mettre ta parole en doute, mais il semble que je sois passé à coté de quelque chose d'important car tout ce que j'ai pu essayer m'a provoqué une erreur de compilation...

Ou alors, autre alternative possible, comment utiliser un iterateur sur un vecteur de manière aléatoire?

[Matthieu Brucher]

vecteur.begin() + un std::advance devrait résoudre ton pb, non ?

[Jean-Marc.Bourguet]

vecteur.begin() + un std::advance devrait résoudre ton pb, non ?

vecteur.begin() + i suffit.

[Laurent Gomila]

Description.begin() + index suffira.
std::advance c'est justement pour les itérateurs à accès non aléatoire.

Pour la validité de l'itérateur suffira de tester l'index

Code :

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

1
2
3
4
5
if (index < Description.size())
{
    vector<std::string>::iterator It = Description.begin() + index;
    Description.erase(It);
}

[koala01]

houlala Je devais etre vachement fatigué au moment où j'ai lu la première réponse...

Je n'arrivais pas à interpréter le bout de phrase

Tu peux utiliser + dessus

Mais grace à vos réponses, ca devient évidemment limpide comme de l'eau de roche

A se demander comment j'ai fait pour ne meme pas y penser avant de poser ma question

Enfin, je marque le sujet comme résolu, et vous remercie bien for
(je vais effectivement tester la valeur de index, et lancer une exception si on est hors limites )

[Matthieu Brucher]

C'est à JM, le texte que tu as cité, pas à moi

[koala01]

C'est à JM, le texte que tu as cité, pas à moi

non pas la tete...

Excuses moi, JM... bon... (ou plutot au lit, parce que ca devient grave là...) Merci à tous

[loufoque]

S'il est clair que je dois utiliser la méthode erase des vecteurs, je me trouve confronté au fait qu'il faut lui passer... un iterateur en argument...

Il y a plusieurs définitions de la fonction erase, dont une qui prend des indexes.

[koala01]

Il y a plusieurs définitions de la fonction erase, dont une qui prend des indexes.

Hé bien, il semblerait que non...

Je l'avais déjà essayé, mais avec une methode

Code :

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

1
2
3
4
5
 
void Suppress(const unsigned int index)
{
    Description.erase(index);
}

j'obtiens le gentil message

Code :

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

1
2
3
4
 
src\strprojet\cprojet.cpp:168: error: no matching function for call to `std::vector<std::string, std::allocator<std::string> >::erase(const unsigned int&)'
<snip> note: candidates are: typename std::vector<_Tp, _Alloc>::iterator std::vector<_Tp, _Alloc>::erase(__gnu_cxx::__normal_iterator<typename _Alloc::pointer, std::vector<_Tp, _Alloc> >) [with _Tp = std::string, _Alloc = std::allocator<std::string>]
<snip> note:                 typename std::vector<_Tp, _Alloc>::iterator std::vector<_Tp, _Alloc>::erase(__gnu_cxx::__normal_iterator<typename _Alloc::pointer, std::vector<_Tp, _Alloc> >, __gnu_cxx::__normal_iterator<typename _Alloc::pointer, std::vector<_Tp, _Alloc> >) [with _Tp = std::string, _Alloc = std::allocator<std::string>]

Autrement dit, erase ne fonctionne qu'avec des itérateurs...
(du moins, sous mingw...)

[Jean-Marc.Bourguet]

Il y a plusieurs définitions de la fonction erase, dont une qui prend des indexes.

Pour les std::string oui, pas pour les std::vector.