STL List container with classes

I have made a class that looks like this:

Code:

class DLLIMPORT Object
{
    private:
        float x, y, z;
    
    public:
        LPDIRECT3DVERTEXBUFFER9 vertexbuffer;    //Main point here

        Object();
        void AddVertexBuffer(VERTEXBUFFER buf);
};

Than I made the STL list and iterator:

Code:

        list<Object> objects;
        list<Object>::iterator iter;

Than I tried to use the iter to get the vertexbuffer from the class and use it:

Code:

    for (iter = objects.begin(); iter != objects.end(); iter++)
    {
        device->SetStreamSource(0, iter->vertexbuffer, 0, sizeof(CUSTOMVERTEX));    //Main point here
    }
    device->SetFVF(D3DFVF_CUSTOMVERTEX);
    device->DrawPrimitive(D3DPT_TRIANGLELIST, 0, objects.size());

It compiles fine but when I run it, it gives a windows error saying the program had to close. Am I not alowed to use a varaible from the class or is there a better way? I can not use STL vectors or stacks/deques for certain reasons I already looked over.

Hi,

I don't think the operator<() is defined for iterators. I used != instead, and it worked for me:

Code:

#include <iostream>
#include <list>

using namespace std;

class Object
{
private:
	int num;

public:
	Object(int n)
	{
		num = n;
	}
	int getNum()
	{
		return num;
	}
};

int main()
{
	Object obj0(10);
	Object obj1(20);
	Object obj2(30);

	list<Object> objects;
	objects.push_back(obj0);
	objects.push_back(obj1);
	objects.push_back(obj2);

	list<Object>::iterator iter;
	for(iter = objects.begin(); iter != objects.end(); iter++)
	{
		cout<<iter->getNum()<<endl;
	}


    return 0;
}

class DLLIMPORT Object

That doesn't cause a compiler error? Two word variable name?

Warning, I could be wrong about this.

(*iter)->VertexBuffer

Dereferencing the iterator gives you the address of the object instance.

Code:

class DLLIMPORT Object

DLLIMPORT is a predefined varaible (#define) that tells about the dll exporting.

I don't think the operator<() is defined for iterators. I used != instead, and it worked for me:

I did use != ?

Originally Posted by Eber Kain

Warning, I could be wrong about this.

(*iter)->VertexBuffer

Dereferencing the iterator gives you the address of the object instance.

I'm afraid I don't get what you mean. It apears to work just like a vector container would, except vector containers worked in this situation.

Your Object class doesn't have a copy constructor, but vertexbuffer looks like a pointer. Be sure you aren't destroying that pointer or making it invalid in your destructor or elsewhere.

I did use != ?

Sorry, I must be seeing things.

Originally Posted by Rune Hunter

I have made a class that looks like this:

Code:

class DLLIMPORT Object
{
    private:
        float x, y, z;
    
    public:
        LPDIRECT3DVERTEXBUFFER9 vertexbuffer;    //Main point here

        Object();
        void AddVertexBuffer(VERTEXBUFFER buf);
};

snip...

Code:

    for (iter = objects.begin(); iter != objects.end(); iter++)
    {
        device->SetStreamSource(0, iter->vertexbuffer, 0, sizeof(CUSTOMVERTEX));    //Main point here
    }
    device->SetFVF(D3DFVF_CUSTOMVERTEX);
    device->DrawPrimitive(D3DPT_TRIANGLELIST, 0, objects.size());

have you initialised the vertexbuffer pointer?
BTW use ++iter, not iter++ (avoids creating a temporary iterator object)

I have declared and filled the vertexbuffer, and ++iter I will use now.


Well I now have reason to belive the class is causing all these problems. Because I did a test to see if could change a varaible inside the class and this is what the code looks like:

Code:

class object
{

public:

int x;
object();
};

object::object()
{
x = 5;
}


object obj();
obj.x = 10;

After I do that x will still equal 5, even though I changed it. I also typed that up inside the browser some there might be spelling errors, etc. Otherwise that is a short version of the code I have.

After I do that x will still equal 5, even though I changed it.

Huh? Prove it.

Note: you are not calling the default constructor properly.

Just remove the () from object obj(); and as long as obj.x is inside of a function like main you are set. Outside won't work.

in otherwords I am using the winapi main entry point so if I have obj.x = 5 inside that it should work? Well I'll go look over every peice of code and start debuging it than.