Wypisywanie za pomocą iteratora kontenera vector

Question

Taki kod:

    template <class T> void wypisz2( vector< vector<T> >& v){
        for( auto it=v.cbegin(); it!=v.cend(); ++it){
            for(auto jt=v.cbegin(); jt!=it; ++jt){
                   cout << *jt << ' ';
 		}
        }
    }

Daje taką odpowiedź na kompilatora g++ 4.8.2 jest następująca:
 

vectorek.cpp: In instantiation of ‘void wypisz2(std::vector<std::vector<T> >&) [with T = int]’:
vectorek.cpp:37:19:   required from here
vectorek.cpp:18:25: error: cannot bind ‘std::ostream {aka std::basic_ostream<char>}’ lvalue to ‘std::basic_ostream<char>&&’
                    cout << *jt << ' ';

I z tego co rozumiem to operator << ma jakiś problem z przesłaniem iteratora. Tylko nie bardzo rozumiem o co chodzi i jak to naprawić. 
    

Answer 1

Tak o wypisywaniu danych z kontenera, to zawsze można:

for(auto dana : kontener) cout<<dana<<endl;

Jakby ktoś tu przyszedł i nie wiedział

Comments

Albo lepiej:

for(const auto &dana : kontener) cout<<dana<<endl;

Ode mnie .

---

dokładnie! dałbym Ci plusa gdybym mógł. :D

---

Czekałem aż ktoś to tu napisze. Ode mnie +1.

---

efik, czemu więc sam nie napisałeś? :P

---

Dałem szansę komu innemu!  Moim zdaniem to lepsze rozwiązanie niż tyle zmiennych. Iterator ja raczej wykorzystywałem do hmm szukania elementu w kontenerze. ( Kontener klasy Book )

---

aha, no i trzeba też pamiętać, że to rozwiązanie wymaga standardu 11, więc jak coś, to dodaj do linkera po prostu -std=gnu++11

---

albo choćby jeszcze można:

for(int a=0; a<kontener.size(); a++) cout<<kontener[a]<<endl;

Nie rozumiem po co oni cisną te iteratory w tym przypadku :/

A jak chcesz się odwołać do adresu zmiennej, to piszesz &kontener[a] i to wszystko:)

---

Bo to miało być na iteratorach. Niestety :(

Answer 2

Nie do końca rozumiem, co znaczy ten error, ale myśle, że kompilator troche zgłupiał bo napisałaś coś kompletnie bez sensu. Dlaczego druga pętla też operuje na iteratorze "głównego" vectora? Powinno to wyglądać tak:

    for( auto it=v.cbegin(); it!=v.cend(); ++it){
        for(auto jt = it->cbegin(); jt != it->cend(); ++jt){
               cout << *jt << ' ';

Druga pętla, skoro wypisuje zawartości vectorów zawartych w głównym vectorze powinna operować na iteratorach vectorów ktorych elementy wypisuje, prawda? :D 

Answer 3

Może to cie zadowoli:

template <class T> void wypisz2(vector< vector<T> > & v){
	for (int x = 0; x < v.size();x++){
		vector<T> fork = v[x];
		for (int y = 0; y < fork.size(); y++)
		{
			cout << fork[y] << " ";
		}
		cout << endl;
	}
}

Więcej kodu, ale działa...Jak i przejrzystszy jest dla mnie.

Answer 4

Z tego kodu wynika, że wypisujesz obiekt kontenera, a przecież nie ma to sensu - spróbuj wyciągnąć z iteratora obiekt std::vector, a dopiero później z niego odpowiednie dane.