ich will einen vector in dem ich strings speichere, der vector wird über eine funktion gefüllt und über eine andere wieder ausgelesen, beim auslesen wird aber immer die erste stelle im vector falsch angezeigt ->"(null)", der rest ist richtig.

folgender code:
main.cpp:

#include <iostream>
#include <cstdio>
#include "klasse.h"

int main()
{
    Concert *con = new Concert();
    con->addSupportBand("Velvet Underground");
    con->addSupportBand("Black Angels");
    con->addSupportBand("The Duke Spirit");

    for (std::vector<std::string>::iterator itt = con->getSupportBands().begin(); itt != con->getSupportBands().end(); ++itt) {
        printf("Support: %s\n", itt->c_str());
    }

    return 0;
}

klasse.h

#include <vector>
#include <string>

class Concert {
    public:
        std::vector<std::string> getSupportBands() const { return supportBands; }
        void addSupportBand(std::string name) { supportBands.push_back(name); }

    private:
        std::vector<std::string> supportBands;
};

Ausgabe:

Support: (null)
Support: Black Angels
Support: The Duke Spirit

also wieso stimmt hier die erste Stelle nicht?
wenn ich allerdings den vector "supportBands" public mach und ich dann bei der Ausgabe direkt (ohne getsupportbands) darauf zugreife funktionierts. wieso geht das nicht mit der get-funktion?

so, zurück zum Thema:
ich gebe jetzt wie vorgeschlagen beim getSupportBands den Vektor als Referenz zurück, die Ausgabe ist jetzt auch richtig, danke. Nur damit ich das auch verstehe, wenn ich den vektor in getSupportBand als Wert zurück liefere wird dieser kopiert, müsste es dann nicht trotzdem funktionieren? oder warum ist dann nur das erste Element falsch?

#include <iostream>

using namespace std;

int main()
{
    Concert *con = new Concert;
    con->addSupportBand("Velvet Underground");
    con->addSupportBand("Black Angels");
    con->addSupportBand("The Duke Spirit");

    std::vector<std::string> buf = con->getSupportBands();

    for (std::vector<std::string>::iterator itt = buf.begin(); itt != buf.end(); ++itt) {
        cout << (*itt) << endl;

    }
    delete con;

}

Durch das Zwischenspeichern in buf bleiben die Zeigeradressen (iterator ist nix anderes) valid da sie vom selben container stammen

drakon schrieb:

btw:
Das du hier std::vector ad absurdum führst ist dir schon klar, oder?
Und immer schön wieder Speicher freigeben am Ende..

nein, ist mir nicht klar (wirklich nicht), warum?
und keine sorge, im eigentlichen Programm geb ich den Speicher auch wieder frei, das war halt nur der entscheidende Teil den ich hier reinkopiert habe.

@NAJA: ok, danke, das ist einläuchtend.

drakon schrieb:

btw:
Das du hier std::vector ad absurdum führst ist dir schon klar, oder?
Und immer schön wieder Speicher freigeben am Ende..

nein, ist mir nicht klar (wirklich nicht), warum?
und keine sorge, im eigentlichen Programm geb ich den Speicher auch wieder frei, das war halt nur der entscheidende Teil den ich hier reinkopiert habe.

Vielleicht meint er weil deine Klasse kaum einen Sinn ergibt weil du vector auf ein push_back reduzierst. Somit könntest du auch einfach

typedef std::vector<std::string> Concert_t

schreiben und bräuchstest keine Klasse.

Jedoch ist das sicherlich nur ein klitzekleiner Ausschnitt deines Programms, richtig?

...

Zum einen das und zum anderen wüsste ich nicht, warum man einen Standardcontainer per new ( ob in einer Klasse gewrappt, oder nicht ) anfordern sollte.

#include <vector>
#include <string>

class Concert {
    public:
        typedef std::vector <std::string> bandContainer;

        const bandContainer& getSupportBands() const { return supportBands; }
        void addSupportBand(const std::string &name) { supportBands.push_back(name); }

    private:
        bandContainer supportBands;
};

#include <iostream>

#include "concert.h"

int main ()
{
Convert bla;
//füllen
for (Concert::bandContainer::const_iterator iter(bla.getSupportBands().begin()), end(bla.getSupportBands().end()); iter != end; ++iter)
{
std::cout << "Support-Band: " *iter << "\r\n";
}
}

bb