Comment obtenir l'avant-dernier élément d'une liste ?

Question

J'ai un std::list<double> foo;

J'utilise

if (foo.size() >= 2){
    double penultimate = *(--foo.rbegin());
}

mais cela me donne toujours une valeur arbitraire de penultimate .

Qu'est-ce que je fais de mal ?

Answer 1

Plutôt que de décrémentation rbegin vous devez l'incrémenter, comme indiqué ici : 1

double penultimate = *++foo.rbegin();

comme rbegin() renvoie un itérateur inverse donc ++ est l'opérateur qui se déplace en arrière dans le conteneur. Notez que j'ai également supprimé les parenthèses superflues : ce n'est pas du goût de tout le monde.

Actuellement, le comportement de votre programme est indéfini puisque vous déménagez en fait à end() et vous n'êtes pas autorisé à le déréférencer. La nature arbitraire de la sortie est une manifestation de ce comportement indéfini.

---

1 Conservez le contrôle de la taille minimale que vous avez actuellement.

Answer 2

La façon la plus claire, à mon avis, est d'utiliser la construction conçue à cette fin (C++11) :

double penultimate = *std::prev(foo.end(), 2)

Answer 3

Je ferais juste *--(--foo.end()) ; pas besoin d'itérateurs inversés. C'est aussi moins déroutant.