Ich arbeite gerade an einem Kartenspiel, und bei der Funktion, wo die Spieler 5 Karten ziehen, wird von einem Vector zu einem anderen Vector Zufallselemente importiert. Danach wird das gleiche Element im ersten Vector gelöscht.

Das Problem:
Wenn die for Schleife durchläuft, stürzt das Program beim zweiten Durchlauf ab. Er weist auf diese Funktion in _algobase.h hin:

template <class _RandomAccessIter, class _OutputIter, class _Distance>
inline _OutputIter
__copy(_RandomAccessIter __first, _RandomAccessIter __last,
       _OutputIter __result, const random_access_iterator_tag &, _Distance*) {
  for (_Distance __n = __last - __first; __n > 0; --__n) {
    *__result = *__first; // FEHLER!
    ++__first;
    ++__result;
  }
  return __result;
}

Meine Funktion schaut so aus:

void give_cards(vector<card> cards) {
                vector<card>::iterator pos;
                int card_nr;
                for (short int loop = 0; loop < 5; ++loop) {
                        card_nr = rand() % cards.size();
                        advance(pos, card_nr);
                        player_cards.push_back(cards[card_nr]);
                        cards.erase(pos);
                }
        }

Gruß,
Savadiv

Welche Variablen denn? Wenn du "player_cards" meinst, die ist ein Teil der Klasse. Und cards ist ja sowieso ein Parameter, pos und card_nr werden auch initiliaziert bevor sie benützt werden? Oder täusch ich mich? (Ist nähmlich erst eines der ersten Übungsprogramme mit C++)

wo denn?

\edit: verdammt, wieder fehlen mir verdammte siebzehn sekunden

Keine Sorge, ich bin nur selten im Weg.

camper schrieb:

Keine Sorge, ich bin nur selten im Weg.

wenn du's nicht bist, ist das immer jemand anders... aber ich bin immer zu spät^^

...
pos = cards.begin();
...

Da "vector" allerdings random-access Iteratoren hat kannst du auf "pos" und "advance" ganz verzichten und es einfach so schreiben:

cards.erase(cards.begin() + card_nr);

Ich habe da noch eine Frage, ich lösche ja dann das Element pos aus dem vector. Jedoch treten beim nächsten Aufruf, gleiche Karten bei unterschiedlichen Spielern auf... Pos ist doch gleich wie card_nr, oder?

Die aktuelle Funktion:

vector<card> give_cards(vector<card> cards) {
                vector<card>::iterator pos = cards.begin();
                int card_nr = 0;
                for (short int loop = 0; loop < 5; ++loop) {
                        card_nr = rand() % cards.size();
                        advance(pos, card_nr);
                        player_cards.push_back(cards[card_nr]);
                        cards.erase(pos);
                }
                return cards;
        }

Ich benutze die Funktion so, damit "cards" aktualisiert wird:

cards = pl1.give_cards(cards);
cards = pl2.give_cards(cards);

Dennoch krieg ich eine Karte zwei Mal.

ansonsten wie gesagt: schmeiss die "advance" funktion und "card_nr" komplett raus. Hast doch random access aufs ganze array... wählst ne beliebige karte und schmeisst die raus, fertig...

vector<card> give_cards(vector<card> cards)

Die Funktion bekommt eine Kopie deines Kartenstapels übergeben und nimmt dann Karten von dieser Kopie. Der Originalstapel wird davon überhaupt nicht berührt. Als Lösung mußt du den Kartenstapel als Referenz übergeben:

void give_cards(vector<card>&cards);//beachte das &

Nein, muss er nicht, denn er gibt den modifizierten Kartenstapel als Returnwert zurück.

OK, stimmt - dann Kommando zurück. Aber die Version mit der Referenz ist imho günstiger (und drückt klarer aus, was die Funktion macht - von den Performance-Vorteilen will ich gar nicht erst anfangen).