Ich habe einen Vector vom Typ einer eigenen Klasse (cSprite), welchen ich erst mit Werten füllen möchte, und dann mit random_shuffle() neu mischen möchte, um eine Art Zufallseffekt für ein Spiel zu bekommen.

Hier mal die ungefähre Vorgehensweise:

//Größe des Vectors
const int vector_max = 3;

//Der Vector vom Type einer Klasse
vector<CSprite> Objekt(vector_max);

/*
Vector mit Werten füllen...
*/

//Vector neu mischen  -> führt zum Absturz des Programms
random_shuffle(Objekt.begin(), Objekt.end());

Alles funktioniert gut, nur wenn ich random_shuffle() mit reinnehme, stürzt das Programm immer ab.
Der Debugger sagt was von einem Zugriffsfehler beim Lesen...
Habe es genau so mit einem Vector vom Typ int probiert, was ohne Probleme läuft.

Kann der Vector nicht vom Typ einer Klasse sein, also Instanzen der Klasse beinhalten?
Habe sonst mit Arrays gearbeitet (CSprite Objekt[5]), nur da haben ich ja leider keine Funktion wie random_shuffle()...

Kann mir einer das Problem erklären?
Greife ich auf nicht vorhandene Elemente zu? Eigentlich dürft das nicht der Fall sein, da ich ja mit Objekt.begin() und Objekt.end() arbeite...

MfG
Hundefutter

/*
Vector mit Werten füllen...
*/

?

//Objekt 1
//Textur laden
for(int i=0;i<50;i++)
{
	sprintf(Filename,"./Sprites/Objekt/00%d.png",i+1);
	Objekt[0].AddTexture(Filename);
}

//Bewegungsrichtung festlegen
Objekt[0].Set_direction(RIGHT);

//Position u Schrittgröße setzen
Objekt[0].SetPosition(200,600);
Objekt[0].SetStep(1.5,2);

// Konstruktor - Initialisierungen
CSprite::CSprite(void)
{
    m_Rotation = 0;

    m_StepX    = 0;
    m_StepY    = 0;

    m_Width    = 0;
    m_Height   = 0;

    m_AnimationSpeed = 0;
    m_AnimationDelay = 0;
    m_CurrentImage   = 0;
    m_animation = false;
    m_animzaehler = 1;

    m_lpDevice = NULL;                  
    m_lpSprite = NULL;
}

Kann es deshalb also zu Fehlern kommen, weil die Variablen neu gesetzt werden?

Einen Kopier-Konstruktor habe ich eigentlich nicht....

Wenn das Zeiger sind, solltest du vermutlich einen brauchen

Hundefutter schrieb:

m_lpDevice = NULL;                  
    m_lpSprite = NULL;

Sind also Zeiger...
Wie muss ich das dann mit dem Kopier-Konstruktor lösen?
Habe bis jetzt noch nie damit gearbeitet....

Kann ich den dann auch einfach definieren und der wird dann automatisch bei einem Kopiervorgang aufgerufen, genau wie der Konstruktor beim Erstellen?
Habe dazu folgenden Syntax gefunden:

Klassenname(const Klassenname&)

Ist das so richtig?

Ich würde eher die Klasse non-copyable machen, und boost::shared_ptr<CSprite> in den vector reinstecken anstelle von CSprite.

Wenn deine CSprite Klasse wirklich kopierbar sein soll, dann solltest du AddRef() auf das Device aufrufen.

noncopyable ist keine Klasse sondern eine Eigenschaft einer Klasse. Um das zu erreichen, genügt es eigentlich den CopyCtor und den Assignment-operator private zu machen.

In der Tat bietet boost eine Klasse mit Namen "noncopyable" an. Also ist es, unter anderem, eine Klasse!

In der Tat bietet boost eine Klasse mit Namen "noncopyable" an. Also ist es, unter anderem, eine Klasse!

Genau und der Konstruktor ist auch eine Klasse: class Konstruktor {};

David_pb schrieb:

In der Tat bietet boost eine Klasse mit Namen "noncopyable" an. Also ist es, unter anderem, eine Klasse!

Genau und der Konstruktor ist auch eine Klasse: class Konstruktor {};

Stimmt!