Creating A Inventory System - GameCreators Forum

Author

Message

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 27th Jul 2010 00:05 Edited at: 27th Jul 2010 00:09

Link

Okay so I have this basic layout of an inventory but I need soem help on making it more dynamic.(See the picture attached)

Now in this each slot is the Grey boxes and there are mouse over states for them and click states but here are the classes that I used:

Inventory Class:

void Inventory::Initialize(Slot *slot[10][10])
{

TreasureIterator = RenderList.begin();

for(int i=0; i < 4; i++)
		{
			TreasureObjects = *TreasureIterator;
			TreasureObjects->SetPosition(i*60,0);
			TreasureIterator++;
		}

m_PosX = 0; 
	m_PosY = 0;

id   = 1;
	cols = 10;
	rows = 10;

for(int i = 0; i < cols; i++)
	{
		for(int j = 0; j < rows; j++)
		{
			slot[i][j] = new Slot(id, i * 60, j * 60);
			id += 4;
		}
	}

}

void Inventory::Update(Slot *slot[10][10])
{
	
	for(int i = 0; i < cols; i++)
	{
		for(int j = 0; j < rows; j++)
		{
			slot[i][j]->Update();

if(slot[i][j]->LeftUp())
				slot[i][j]->Select();
		}
	}

TreasureIterator = RenderList.begin();
	while(TreasureIterator != RenderList.end())
	{
		TreasureObjects = *TreasureIterator;
		TreasureObjects->Update();
		TreasureIterator++;
	}

char czPOSX[10];
	wsprintf(czPOSX, "Mouse X: %d", mouse.GetMousePosX());
	dbText(10, dbScreenHeight() - 40, czPOSX);

char czPOSY[10];
	wsprintf(czPOSY, "Mouse Y: %d", mouse.GetMousePosY());
	dbText(10, dbScreenHeight() - 20, czPOSY);
}

void Inventory::Remove()
{

}

+ Code Snippet

void Inventory::Initialize(Slot *slot[10][10])
{

	TreasureIterator = RenderList.begin();



		for(int i=0; i < 4; i++)
		{
			TreasureObjects = *TreasureIterator;
			TreasureObjects->SetPosition(i*60,0);
			TreasureIterator++;
		}


	m_PosX = 0; 
	m_PosY = 0;

	id   = 1;
	cols = 10;
	rows = 10;

	for(int i = 0; i < cols; i++)
	{
		for(int j = 0; j < rows; j++)
		{
			slot[i][j] = new Slot(id, i * 60, j * 60);
			id += 4;
		}
	}


	
}

void Inventory::Update(Slot *slot[10][10])
{
	
	for(int i = 0; i < cols; i++)
	{
		for(int j = 0; j < rows; j++)
		{
			slot[i][j]->Update();

			if(slot[i][j]->LeftUp())
				slot[i][j]->Select();
		}
	}

	TreasureIterator = RenderList.begin();
	while(TreasureIterator != RenderList.end())
	{
		TreasureObjects = *TreasureIterator;
		TreasureObjects->Update();
		TreasureIterator++;
	}


	char czPOSX[10];
	wsprintf(czPOSX, "Mouse X: %d", mouse.GetMousePosX());
	dbText(10, dbScreenHeight() - 40, czPOSX);

	char czPOSY[10];
	wsprintf(czPOSY, "Mouse Y: %d", mouse.GetMousePosY());
	dbText(10, dbScreenHeight() - 20, czPOSY);
}

void Inventory::Remove()
{

}

Slot Class:

Slot::Slot(int id, int x, int y)
{
	/*	
	Assigning the Slot ID numbers for when we call dbSprite or dbPasteSprite
	Each Slot needs 4 sprite IDs || slotID is also the idleID
	So if we start with passing in an ID of 1, the next Slot will need an ID of 5
	Slot(1, 0, 0);
	Slot(5, 0, 0);
	Slot(9, 0, 0);
	*/
	slotID     = id;
	overID     = id + 1;
	pressedID  = id + 2;
	selectedID = id + 3;
 
	m_PosX = x;
	m_PosY = y;

bPressed = false;
	bSelected = false;

// Each Slot will start in the Idle State
	slotState = eIdle;

dbLoadImage("Media\\idle.png",     1);
	dbLoadImage("Media\\over.png",     2);
	dbLoadImage("Media\\pressed.png",  3);
	dbLoadImage("Media\\selected.png", 4);

dbSprite(slotID,     m_PosX, m_PosY, 1);
	dbSprite(overID,     m_PosX, m_PosY, 2);
	dbSprite(pressedID,  m_PosX, m_PosY, 3);
	dbSprite(selectedID, m_PosX, m_PosY, 4);

width  = dbSpriteWidth(slotID);
	height = dbSpriteWidth(slotID);

dbHideAllSprites();
}

void Slot::Update()
{
	//Object->Update();
	if(MouseOver() && bSelected == false)
	{
		slotState = eOver;
	}
	else if(MouseOver() == false && bSelected == false)
	{
		slotState = eIdle;
	}
		
	if(bSelected)
	{
		slotState = eSelected;
	}
	
	if(LeftPressed())
	{
		slotState = ePressed;
	}

switch(slotState)
	{
	case eIdle:
		{
			dbPasteSprite(slotID, m_PosX, m_PosY);
		}	break;
	case eOver:
		{
			dbPasteSprite(overID, m_PosX, m_PosY);
		}	break;
	case ePressed:
		{
			dbPasteSprite(pressedID, m_PosX, m_PosY);
		}	break;
	case eSelected:
		{
			dbPasteSprite(selectedID, m_PosX, m_PosY);
		}	break;
	case eCombine:
		{
			dbPasteSprite(selectedID, m_PosX, m_PosY);
		}	break;
	}
}

void Slot::Remove()
{
	for(int i = 0; i < 4; i++)
		dbDeleteImage(i);
	
	dbDeleteSprite(slotID);
	dbDeleteSprite(overID);
	dbDeleteSprite(pressedID);
	dbDeleteSprite(selectedID);
}

bool Slot::MouseOver()
{
	if(mouse.GetMousePosX() > m_PosX && mouse.GetMousePosX() < m_PosX + width &&
	   mouse.GetMousePosY() > m_PosY && mouse.GetMousePosY() < m_PosY + height)
	{
		return true;
	}
	else
	{
		bPressed = false;
		return false;
	}
}

bool Slot::LeftPressed()
{
	if(MouseOver() && mouse.ButtonDown(1))
	{
		bPressed = true;
		return true;
	}
	else
		return false;
}

bool Slot::LeftUp()
{
	if(MouseOver() && mouse.GetButtonState(1) == btn_Released && bPressed)
	{
		bPressed = false;
		return true;
	}
	else
		return false;
}

bool Slot::RightPressed()
{
	if(MouseOver() && mouse.ButtonDown(2))
	{
		bPressed = true;
		return true;
	}
	else
		return false;
}

bool Slot::RightUp()
{
	if(MouseOver() && mouse.GetButtonState(2) == btn_Released && bPressed)
	{
		bPressed = false;
		return true;
	}
	else
		return false;
}

void Slot::Select()
{
	if(bSelected == false)
		bSelected = true;
	else if(bSelected)
		bSelected = false;
}

+ Code Snippet

Slot::Slot(int id, int x, int y)
{
	/*	
	Assigning the Slot ID numbers for when we call dbSprite or dbPasteSprite
	Each Slot needs 4 sprite IDs || slotID is also the idleID
	So if we start with passing in an ID of 1, the next Slot will need an ID of 5
	Slot(1, 0, 0);
	Slot(5, 0, 0);
	Slot(9, 0, 0);
	*/
	slotID     = id;
	overID     = id + 1;
	pressedID  = id + 2;
	selectedID = id + 3;
 
	m_PosX = x;
	m_PosY = y;

	bPressed = false;
	bSelected = false;

	// Each Slot will start in the Idle State
	slotState = eIdle;

	dbLoadImage("Media\\idle.png",     1);
	dbLoadImage("Media\\over.png",     2);
	dbLoadImage("Media\\pressed.png",  3);
	dbLoadImage("Media\\selected.png", 4);

	dbSprite(slotID,     m_PosX, m_PosY, 1);
	dbSprite(overID,     m_PosX, m_PosY, 2);
	dbSprite(pressedID,  m_PosX, m_PosY, 3);
	dbSprite(selectedID, m_PosX, m_PosY, 4);

	width  = dbSpriteWidth(slotID);
	height = dbSpriteWidth(slotID);

	dbHideAllSprites();
}

void Slot::Update()
{
	//Object->Update();
	if(MouseOver() && bSelected == false)
	{
		slotState = eOver;
	}
	else if(MouseOver() == false && bSelected == false)
	{
		slotState = eIdle;
	}
		
	if(bSelected)
	{
		slotState = eSelected;
	}
	
	if(LeftPressed())
	{
		slotState = ePressed;
	}

	switch(slotState)
	{
	case eIdle:
		{
			dbPasteSprite(slotID, m_PosX, m_PosY);
		}	break;
	case eOver:
		{
			dbPasteSprite(overID, m_PosX, m_PosY);
		}	break;
	case ePressed:
		{
			dbPasteSprite(pressedID, m_PosX, m_PosY);
		}	break;
	case eSelected:
		{
			dbPasteSprite(selectedID, m_PosX, m_PosY);
		}	break;
	case eCombine:
		{
			dbPasteSprite(selectedID, m_PosX, m_PosY);
		}	break;
	}
}

void Slot::Remove()
{
	for(int i = 0; i < 4; i++)
		dbDeleteImage(i);
	
	dbDeleteSprite(slotID);
	dbDeleteSprite(overID);
	dbDeleteSprite(pressedID);
	dbDeleteSprite(selectedID);
}

bool Slot::MouseOver()
{
	if(mouse.GetMousePosX() > m_PosX && mouse.GetMousePosX() < m_PosX + width &&
	   mouse.GetMousePosY() > m_PosY && mouse.GetMousePosY() < m_PosY + height)
	{
		return true;
	}
	else
	{
		bPressed = false;
		return false;
	}
}

bool Slot::LeftPressed()
{
	if(MouseOver() && mouse.ButtonDown(1))
	{
		bPressed = true;
		return true;
	}
	else
		return false;
}

bool Slot::LeftUp()
{
	if(MouseOver() && mouse.GetButtonState(1) == btn_Released && bPressed)
	{
		bPressed = false;
		return true;
	}
	else
		return false;
}

bool Slot::RightPressed()
{
	if(MouseOver() && mouse.ButtonDown(2))
	{
		bPressed = true;
		return true;
	}
	else
		return false;
}

bool Slot::RightUp()
{
	if(MouseOver() && mouse.GetButtonState(2) == btn_Released && bPressed)
	{
		bPressed = false;
		return true;
	}
	else
		return false;
}

void Slot::Select()
{
	if(bSelected == false)
		bSelected = true;
	else if(bSelected)
		bSelected = false;
}

Treasure Class:

+ Code Snippet

void Treasure::Initialize(int id, char *name, int value, Inventory *inventory,Slot* slot[10][10])
{
	treasureID = id;

	dbLoadImage("Media\\Treasure\\treasure.png", 5);

	dbSprite(treasureID, 0, 0, 5);
	dbSizeSprite(treasureID, 60, 60);

	dbHideAllSprites();


	inventory->RenderList.push_back(this);
}

void Treasure::Update()
{
	dbPasteSprite(treasureID, m_PosX, m_PosY);
}

void Treasure::SetPosition(int x, int y)
{
	m_PosX = x;
	m_PosY = y;
}

int Treasure::GetPositonX()
{
	return m_PosX;
}

int Treasure::GetPositionY()
{
	return m_PosY;
}

Main:

+ Code Snippet

#include "Globals.h"
#include "Inventory.h"
#include "Treasure.h"

Inventory	inventory;
Treasure	emerald0;
Treasure	emerald1;
Treasure	emerald2;
Treasure	emerald3;
Slot		*slot[10][10];

void DarkGDK(void)
{
	dbSyncOn();
	dbSyncRate(60);

	dbSetDisplayMode(600, 600, 32);
	dbSetWindowTitle("Inventory System - GRID");

	emerald0.Initialize(500, "bob", 1000, &inventory,slot);
	emerald1.Initialize(502, "bob", 1000, &inventory,slot);
	emerald2.Initialize(504, "bob", 1000, &inventory,slot);
	emerald3.Initialize(506, "bob", 1000, &inventory,slot);
	
	inventory.Initialize(slot);

	while(LoopGDK())
	{
		keyboard.Poll();
		mouse.Poll();

		inventory.Update( slot);

		dbSync();
	}

	return;
}

Now the main problem I am running into is how do I know what game object is in each slot? I want to be able to select a slot and then see a description of the object in that slot. Does any one have a good way of doing this? I am a little stumped on how to create a dynamic inventory system.

Back to top

Profile PM

Hassan

15

Years of Service

User Offline

Joined: 4th May 2009

Location: <script> alert(1); </script>

Posted: 27th Jul 2010 11:21

Link

no need to read your classes, you can very simply adjust your structure of the system, the easiest and best way to do this is the following:

bag (inventory) class contains
-number of slots
-array of slots (( INVENTORYITEM items[10][10]; ))

INVENTORYITEM is a structure that can look like the following
struct INVENTORYITEM
{
bool bFilled; //the slot is filled?
int imageID; //image id of the item
int spriteID; //sprite id of the item
bool (*USE) ( void ); //the item's trigger function
};

So i guess everything's clear except the USE function, you simply create a function for each item in your game, let's say it's a health potion, it should look like this:

+ Code Snippet

bool HealthPotionFunction ( void )
{
if ( player.health == player.maxhealth )
return false; //dont use the item


player.health += 10; //increase health by 10
return true; //use the item (delete from inventory)
}

i designed it so that it returns true if the item should be used, otherwise return false (the item's use condition is false), you can modify this to fit your needs

hope that helps a bit

Back to top

Profile PM Website

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 28th Jul 2010 00:57 Edited at: 28th Jul 2010 04:52

Link

Ahh okay that makes since I am going to have to try it out and for the Use function I am not to worried about it because I just have to do the inventory system and thats it. No game play is really involved but thanks for the post and I will post back with what I got in the next couple of days.

Back to top

Profile PM

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 29th Jul 2010 18:42

Link

Hey I got a question on adding items to the inventory. So I went a head and did what you suggested but I am having trouble adding items to the inventory I guess dynamically. What I mean is I derived from the Slot class to make new items with new images and descriptions and what not. Now I want to be able to add items by calling the AddItem function which when I first created it looks like his:

+ Code Snippet

void Inventory::AddItem(Slot*item, Slot *slot[6][6])
{
	
	for(int j = 0; j < rows; j++)
	{

		for(int i = 0; i < cols; i++)
		{
			if (!slot[i][j]->bFilled)
			{
				OldID = slot[i][j]->GetID();
				OldPosX = slot[i][j]->GetPosX();
				OldPosY = slot[i][j]->GetPosY();
				////Want to add 'Item' as a Chain not a slot but want to be able to add other items in the future//// item = new   Slot(OldID,OldPosX,OldPosY);
				//slot[i][j]->Remove();
				slot[i][j] = item;
				j = rows;
				break;
				

			}
			
		}
	}

so basically it loops through my items in the inventory and checks if they are filled, if they aren't then it adds an item to the inventory. Few problems with this first off I want the slots to be what ever Item I create. Meaning when I derive a class from Slot called Chain I want to be able to put that new chain item in the parameters of the AddItem and then insert it into the slot that isn't filled. How would I do this? Or does anyone have a better way of adding items to the inventory?

Back to top

Profile PM

Matty H

15

Years of Service

User Offline

Joined: 7th Oct 2008

Location: England

Posted: 29th Jul 2010 22:11 Edited at: 29th Jul 2010 22:13

Link

I don't completely understand your code but here is how I would approach the problem.

I would have a Slot class, maybe each slot object would have a SpriteID, posx, posY etc. It would also hold a pointer to an Item object(Item should not be derived from Slot imo), when you draw your slot object it uses the image from the Item object it has a pointer to, if it does not point to an Item object yet you could have a default image it could use.

Here are some class functions Slot may contain:

+ Code Snippet

void setItem(Item *anItem); // Set pointer and update image
Item* getItem();
int getImageID(); // Returns default image if pointer == 0;

I would then perhaps have a Slot_Group class, an object of this class may hold a number of slots(array or vector etc) and position them on the screen and perhaps update them all with one call.

I don't think this is the kind of answer you were looking for but it may be some of your class design choices are now making things a bit harder. I have not put a great deal of though into this so feel free to tell me if there is a glaring issue I may have missed.

fulcrum physics - free collision/physics plug-in for DarkGDK

Back to top

Profile PM Email

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 29th Jul 2010 22:48

Link

Yeah that's what I started to do up in my first post. I was getting stuck on a few things which the GetItem() function I think will help me in. I am going to try it today and see if I can go back to what I original wanted to do because I like the idea of actually using game objects rather than trying to just fake it with slots. I will post back if I run into any big issues, and maybe you can help me along the way.

Back to top

Profile PM

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 29th Jul 2010 23:57

Link

Hey for the SetItem() function are you saying to do something like this:

+ Code Snippet

class Slot
{
protected:
////This is the variable that I will set the the pointer to
GameObject* SlotObject;
};
////This is where I set the SlotObject to the Object that I want the slot to hold.
GameObject* Slot::SetItem(GameObject* Object)
{
SlotObject =  Object;
}

then in GetItem() I just return SlotObject Right?

Back to top

Profile PM

Hassan

15

Years of Service

User Offline

Joined: 4th May 2009

Location: <script> alert(1); </script>

Posted: 30th Jul 2010 00:20 Edited at: 30th Jul 2010 00:25

Link

in the additem method, why are you passing the slot place for the item? i believe the inventory system should decide where the item's position should be

anyway, in conclusion, you should have the following:

Item structure : contains :
-image ID of the item
-a pointer to a function that executes the item
-possibly an ID (if you want it to be polished)
****no sprite ID should be used, since you may have more than one slot containing that item, and they cant have the same sprite ID, alternatively let the inventory system pick a suitable sprite ID

Slot structure : contains :
-a pointer to the item
-quantity of the item ****a slot may have a stack of items (multiple items of the same type in one slot)
-position in the inventory (x/y)
****to find out whether its filled or not use this ( if ( !items.size ( ) ) { not filled ( 0 items ) } )

InventorySystem class : contains :
-a vector (or an array) of Slots
-number of rows/cols

i believe those should cover all the base system, you can add additional stuff to fill your needs

sorry if that just repeats what's posted above

and hope it helped

Back to top

Profile PM Website

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 30th Jul 2010 00:44

Link

What are the differences in making them a Struct or class? Or is it really just a preference? I mean I know structs are basically a class but everything is public. Is this true or is there more to it?

Thanks for the help so far I am trying to get this to work right now so hopefully I will have something working in the next couple of hours.

Back to top

Profile PM

Matty H

15

Years of Service

User Offline

Joined: 7th Oct 2008

Location: England

Posted: 30th Jul 2010 00:44

Link

Quote: "Hey for the SetItem() function are you saying to do something like this:
then in GetItem() I just return SlotObject Right? "

Yeh, pretty much, your slot(and inventory) then has access to any item it holds and you can change it dynamically which is where you seemed to be having a problem. If you are quite new to creating this kind of access just be careful that when you delete the item, any slot object which is using it is told. This may mean that the item holds a pointer to the slot it belongs to and informs the slot of its destruction, or you may do it different its up to you.

@Hassan - I was of the understanding that you can't use function pointers inside a class, have you done something like this before(in a class)?

fulcrum physics - free collision/physics plug-in for DarkGDK

Back to top

Profile PM Email

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 30th Jul 2010 01:41 Edited at: 30th Jul 2010 01:42

Link

Hey Matty or Hassan,
So I have a GameObject Pointer in my slot class and I want to call the Update of my GameObject in my slot class but since the GameObject doesn't exist yet I am getting errors. I tried setting an if check to make sure it wouldn't call the update if it is NULL but I keep getting the same error:
C2027: use of undefined type 'GameObject'
So what do I need to do to let my slot class know to wait or to allow me to call these functions until there is something in the slot?

Back to top

Profile PM

Matty H

15

Years of Service

User Offline

Joined: 7th Oct 2008

Location: England

Posted: 30th Jul 2010 02:03

Link

Have you included the GameObject.h in Slot.

Each slot could hold a bool hasObject, set to true when setObject is called, only update object if hasObject == true.

fulcrum physics - free collision/physics plug-in for DarkGDK

Back to top

Profile PM Email

Greg_C

14

Years of Service

User Offline

Joined: 22nd May 2010

Location:

Posted: 30th Jul 2010 02:06

Link

haha man I can't believe I forgot that. Thanks I think I got it to work now.

Back to top

Profile PM

Hassan

15

Years of Service

User Offline

Joined: 4th May 2009

Location: <script> alert(1); </script>

Posted: 30th Jul 2010 02:27 Edited at: 30th Jul 2010 02:30

Link

Quote: "What are the differences in making them a Struct or class? Or is it really just a preference? I mean I know structs are basically a class but everything is public. Is this true or is there more to it?"

there is no difference, the only difference is that struct is public by default and class is private by default, but it's more of a habit to me, because i never used a function in a struct ( in C not C++ ), and when C++ came out (classes also), i then figured out that you can have functions in classes (and structs) so i started doing it

Quote: "@Hassan - I was of the understanding that you can't use function pointers inside a class, have you done something like this before(in a class)?"

i'm using this

struct _hxRegisterDesc
{
	_hxRegisterDesc ( )
	{
		classStyle = CS_HREDRAW | CS_VREDRAW;
		classIcon = NULL;
		classCursor = LoadCursorA ( NULL, MAKEINTRESOURCEA(32512) );
		menuName = (LPSTR)GlobalAlloc ( GPTR, 1024 );
		menuName = NULL;
		classIcon_sm = NULL;
		windowMenu = NULL;
		windowParent = NULL;
		windowStyle = WS_CAPTION | WS_VISIBLE | WS_OVERLAPPEDWINDOW;
		windowStyleEx = WS_EX_OVERLAPPEDWINDOW;
	};
	~_hxRegisterDesc ( )
	{
		GlobalFree ( menuName );
	}

DWORD classStyle;
	HICON classIcon;
	HCURSOR classCursor;
	HICON classIcon_sm;
	LPSTR menuName;

DWORD windowStyle;
	DWORD windowStyleEx;
	HMENU windowMenu;
	HWND windowParent;

void (*menuProc) ( HMENU, UINT );
};

_hxRegisterDesc* desc = mCore->GetRegisterInfo ( );
	desc->menuName = MAKEINTRESOURCEA ( ID_MENU );
	desc->windowMenu = menu;
	desc->menuProc = MENU_CALLBACK; /************function pointer***************/
	desc->classIcon = icon;
	desc->classIcon_sm = icon;

void MENU_CALLBACK ( HMENU menuWnd, UINT buttonID )
{
	switch ( buttonID )
	{
		case ID_MENU_EXIT:
		{
			int result = MessageBoxA ( mCore->GetWindow ( ), "Are you sure you want to exit?",
							APPNAME, MB_YESNO | MB_ICONWARNING );
			if ( result == IDYES )
				PostQuitMessage ( 0 );
			break;
		}
		case ID_MENU_IMPORT:
			ImportObject ( );
			break;
		case ID_MENU_EXPORT:
			ExportObject ( );
			break;
		case ID_MENU_CTERRAIN:
		{
			if ( OBJECT->Vaild ( ) )
			{
				int result = MessageBoxA ( mCore->GetWindow ( ),
"An object is already in the scene, adding a terrain will delete it, Are you sure you want to create the terrain?",
													APPNAME, MB_YESNO | MB_ICONWARNING );
				if ( result == IDNO )
					return;
			}
			DialogBoxA ( mCore->GetInstance ( ), MAKEINTRESOURCEA ( ID_CTERRAIN ),
							mCore->GetWindow ( ), (DLGPROC)CTERRAINDLGPROC );
			SetActiveWindow ( mCore->GetWindow ( ) );
		}
		case ID_MENU_SMOOTH:
			curWireMode = 0;
			break;
		case ID_MENU_WIREFRAME:
			curWireMode = 1;
			break;
		case ID_MENU_SWIREFRAME:
			curWireMode = 2;
			break;
		case ID_MENU_BRUSH:
			brush = !brush;
			break;
		case ID_MENU_SCALER:
			scaler = !scaler;
			break;
		case ID_MENU_FULLSCREEN:
			PlayInFullScreen ( );
			break;
		case ID_MENU_EFFECTEDITOR:
			FXEON = !FXEON;
			if ( !FXEON )
			{
				GetDlgItemTextA ( FXBox, ID_FXE_EDIT, FXTEXT, 10000 );
				mCore->UnbindWindow ( FXBox );
				DestroyWindow ( FXBox );
			}
			else
			{
				FXBox = CreateDialogA ( mCore->GetInstance ( ), MAKEINTRESOURCEA ( ID_FXE ),
											mCore->GetWindow ( ), (DLGPROC)DialogProc );
				SetDlgItemTextA ( FXBox, ID_FXE_EDIT, FXTEXT );
				mCore->BindWindow ( FXBox, HX_WINDOWBIND_LEFT, FXBOXOFFSET );
			}
			break;
		case ID_MENU_LTEX:
			LoadTexture ( );
			break;
	}
}

+ Code Snippet

struct _hxRegisterDesc
{
	_hxRegisterDesc ( )
	{
		classStyle = CS_HREDRAW | CS_VREDRAW;
		classIcon = NULL;
		classCursor = LoadCursorA ( NULL, MAKEINTRESOURCEA(32512) );
		menuName = (LPSTR)GlobalAlloc ( GPTR, 1024 );
		menuName = NULL;
		classIcon_sm = NULL;
		windowMenu = NULL;
		windowParent = NULL;
		windowStyle = WS_CAPTION | WS_VISIBLE | WS_OVERLAPPEDWINDOW;
		windowStyleEx = WS_EX_OVERLAPPEDWINDOW;
	};
	~_hxRegisterDesc ( )
	{
		GlobalFree ( menuName );
	}

	DWORD classStyle;
	HICON classIcon;
	HCURSOR classCursor;
	HICON classIcon_sm;
	LPSTR menuName;

	DWORD windowStyle;
	DWORD windowStyleEx;
	HMENU windowMenu;
	HWND windowParent;

	void (*menuProc) ( HMENU, UINT );
};







	_hxRegisterDesc* desc = mCore->GetRegisterInfo ( );
	desc->menuName = MAKEINTRESOURCEA ( ID_MENU );
	desc->windowMenu = menu;
	desc->menuProc = MENU_CALLBACK; /************function pointer***************/
	desc->classIcon = icon;
	desc->classIcon_sm = icon;










void MENU_CALLBACK ( HMENU menuWnd, UINT buttonID )
{
	switch ( buttonID )
	{
		case ID_MENU_EXIT:
		{
			int result = MessageBoxA ( mCore->GetWindow ( ), "Are you sure you want to exit?",
							APPNAME, MB_YESNO | MB_ICONWARNING );
			if ( result == IDYES )
				PostQuitMessage ( 0 );
			break;
		}
		case ID_MENU_IMPORT:
			ImportObject ( );
			break;
		case ID_MENU_EXPORT:
			ExportObject ( );
			break;
		case ID_MENU_CTERRAIN:
		{
			if ( OBJECT->Vaild ( ) )
			{
				int result = MessageBoxA ( mCore->GetWindow ( ),
"An object is already in the scene, adding a terrain will delete it, Are you sure you want to create the terrain?",
													APPNAME, MB_YESNO | MB_ICONWARNING );
				if ( result == IDNO )
					return;
			}
			DialogBoxA ( mCore->GetInstance ( ), MAKEINTRESOURCEA ( ID_CTERRAIN ),
							mCore->GetWindow ( ), (DLGPROC)CTERRAINDLGPROC );
			SetActiveWindow ( mCore->GetWindow ( ) );
		}
		case ID_MENU_SMOOTH:
			curWireMode = 0;
			break;
		case ID_MENU_WIREFRAME:
			curWireMode = 1;
			break;
		case ID_MENU_SWIREFRAME:
			curWireMode = 2;
			break;
		case ID_MENU_BRUSH:
			brush = !brush;
			break;
		case ID_MENU_SCALER:
			scaler = !scaler;
			break;
		case ID_MENU_FULLSCREEN:
			PlayInFullScreen ( );
			break;
		case ID_MENU_EFFECTEDITOR:
			FXEON = !FXEON;
			if ( !FXEON )
			{
				GetDlgItemTextA ( FXBox, ID_FXE_EDIT, FXTEXT, 10000 );
				mCore->UnbindWindow ( FXBox );
				DestroyWindow ( FXBox );
			}
			else
			{
				FXBox = CreateDialogA ( mCore->GetInstance ( ), MAKEINTRESOURCEA ( ID_FXE ),
											mCore->GetWindow ( ), (DLGPROC)DialogProc );
				SetDlgItemTextA ( FXBox, ID_FXE_EDIT, FXTEXT );
				mCore->BindWindow ( FXBox, HX_WINDOWBIND_LEFT, FXBOXOFFSET );
			}
			break;
		case ID_MENU_LTEX:
			LoadTexture ( );
			break;
	}
}

and its working just fine, so you are either using it in a different way or structs and classes are different at this

Back to top

Profile PM Website

Sorry your browser is not supported!

Dark GDK / Creating A Inventory System