Objekte auf dem Heap -> Methodenzugriff?



  • Neras schrieb:

    Ich hab vor die Schleifen eine size Abfrage gestellt und heraus kam, dass der vector "spielKarten" 10 Elemente hat und der vector "karten" 0 Elemente. In den Schleifen sollte der vector "karten" ja dann gefüllt werden. Erscheint mir also richtig, oder? oO Auffällig waren nur die unterschiedlichen schreibweisen der beiden size-Abfragen:

    cout << spielKarten.size() << endl;
    cout << karten->size() << endl;
    

    Grüße,
    Neras

    Das ist klar, dass die leer sind. Du hast ja noch nichts befüllt. Das geschieht in der Schleife. Also mach das lieber einmal nach dem befüllen.

    spielKarten gehe ich einmal davon aus, dass das mittlerweile richtig klappt, also musst du nur noch das befüllen richtig machen. Dann klappt das.

    karten ist ja ein Array von vectoren, also müsste der Zugriff so gehen:

    karten[0].size();
    

    Allerdings funktioniert die Version :

    karten->size();
    

    auch, da karten auf das erste Element des Array verweist.

    EDIT:
    Schau mal, wie das ganze aussieht kurz bevor das cout kommt. Also noch bevor der Fehler kommt.



  • Zudem würde ich von sowas abraten:

    std::vector<const Karte*> karten[10];
    

    Willst du wirklich ein Array mit 10 Vectoren vom Typ const Karte* ? Also nicht nur einen Container? Falls ja, würde ich eher std::tr1::array verwenden:

    std::tr1::array<std::vector<const Karte*>, 10> karten;
    

    Geht aber nur mit TR1-Unterstützung (ansonsten boost::array ). Es braucht folgenden Header:

    #include <array>
    


  • Ich habe die Schreibweise von drakon übernommen. Im Endeffekt wollte ich ja nur 10 Zeiger auf die erste Stelle im vector "spielKarten" erstellen.

    @drakon:
    Übringens kommt der Fehler nicht von dem cout, sondern von deiner Schleife. Wenn ich kein cout mache, kommt der Error trotzdem.

    Grüße,
    Neras



  • Neras schrieb:

    Ich habe die Schreibweise von drakon übernommen. Im Endeffekt wollte ich ja nur 10 Zeiger auf die erste Stelle im vector "spielKarten" erstellen.

    Eben, warum brauchst du dann ein Array von Containern? Da reicht ein std::vector<const Karte*> ...

    Neras schrieb:

    Übringens kommt der Fehler nicht von dem cout, sondern von deiner Schleife. Wenn ich kein cout mache, kommt der Error trotzdem.

    Das sagen wir ja die ganze Zeit! Du greifst auf ungültige Indizes zu.



  • Nexus, dass habe ich wohl verstanden 😉
    Den Array von vectors habe ich bereits verschwinden lassen.
    Die Schleife, die den Error ausspuckt ist ja nur übernommen aus dem Post von drakon.
    Nexus, wie würdest du 10 Zeiger auf das erste Element in dem vector "spielKarte" erzeugen?

    Grüße,
    Neras



  • Neras schrieb:

    Nexus, wie würdest du 10 Zeiger auf das erste Element in dem vector "spielKarte" erzeugen?

    Zehn Zeiger auf das gleiche Element?

    Ich bin mir ehrlich gesagt nicht ganz sicher, was du machen willst. Sag am besten nochmals ausführlich, wie die Container deklariert sind und was sie speichern sollen.



  • Also das mit dem Array von std::vector habe ich mich auch gesträube zu machen, aber da ich immernoch nicht richtig weiss, was er eigentlich machen will habe ich einfach mal das nächstligenste gemacht, weil er halt gesagt hat 10 container von Zeigern auf Karten. Natürlich hätte es ein array (oder halt ein container) von 100 Elementen auch getan, aber das ganze ist ein wenig konfus.. 🙄

    Nexus, wie würdest du 10 Zeiger auf das erste Element in dem vector "spielKarte" erzeugen?

    Das haben wir doch bereits (wie ich mich entsinne) erfolgreich gemacht.. ?!



  • Ok, vergessen wir alles, was wir vorher gesagt haben.

    Meine Idee:
    Ich möchte ein Kartenspiel machen. Dieses Kartenspiel hat 10 Stapel a 10 Karten, also 100 Karten insgesamt.
    So, jetzt habe ich die Werte der Karten aus einer .txt Datei eingelesen und jeweils eine Karte jedes Stapels auf dem Heap erzeugt. Wenn ich jetzt für jede Karte ein Objekt machen würde, hätte ich 100 Objekte, die ich wahrscheinlich nochnichtmal alle brauche, weil sie im Spiel nachher nicht gezogen werden. Soweit verstanden? Wenn ja, dann weiter; wenn nein, lieber erst nochmal nachfragen, bevor es wieder zu so einem durcheinander kommt, wie vorher 😉

    Und jetzt kommt der Part, den ich die ganze Zeit eigentlich nur wissen wollte:
    Anstatt jetzt 100 Objekte zu erzeugen, wollte ich nur von jedem Kartentyp ein Objekt erstellen und dann 10 Zeiger auf jedes dieser Kartenobjekte zeigen lassen.
    Die Kartenobjekte liegen auf dem Heap und die Zeiger, die darauf zeigen, liegen in dem vector "spielKarten". Soweit klar? xD

    Grüße,
    Neras



  • Warum benutzt du nicht einfach immer den gleichen Container, wo die Zeiger drin sind?

    std::vector<const karte*> karten;
    

    Und wenn du dir Karteninformationen irgendwo in einer Funktion brauchst, dann kannst du das Objekt kopieren, oder lediglich eine Referenz auf den vector übergeben. Ich verstehe nicht, warum du so viele Zeiger haben willst, wenn es alle die gleichen sind. Da liegt dein logisches Proble..


Anmelden zum Antworten