//cos

BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
nu e necesar sa faci asa
nu ai nevoie de pointer pentru character in clasa CHARACTER
e suficient sa faci asa
if (CountSpecifyItem(30093)<0)

poti face chiar si asa if (bTime == 1 && CountSpecifyItem(30093) < 0 )
sa nu fie atatea linii

si nu e o functie ce ai facut aici

ElJose scrie:BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
nu e necesar sa faci asa
nu ai nevoie de pointer pentru character in clasa CHARACTER
e suficient sa faci asa
if (CountSpecifyItem(30093)<0)

poti face chiar si asa if (bTime == 1 && CountSpecifyItem(30093) < 0 )
sa nu fie atatea linii

si nu e o functie ce ai facut aici

Este acelasi lucru care l-am facut.
Functioneaza.
Si, de ce nu ai facut-o tu pana acum?

Dragos A.k.A Chitra scrie:

ElJose scrie:BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
nu e necesar sa faci asa
nu ai nevoie de pointer pentru character in clasa CHARACTER
e suficient sa faci asa
if (CountSpecifyItem(30093)<0)

poti face chiar si asa if (bTime == 1 && CountSpecifyItem(30093) < 0 )
sa nu fie atatea linii

si nu e o functie ce ai facut aici

Este acelasi lucru care l-am facut.
Functioneaza.
Si, de ce nu ai facut-o tu pana acum?

BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
de ce sa faci asta cand nu e necesara totusi?
si nu am facut asa ceva pt ca nu am nevoie
si sa stii pe viitor cand chiar o sa ai nevoie de asta sa verifici mai intai pointerul daca e null, sa eviti un posibil crash core

ElJose scrie:

Dragos A.k.A Chitra scrie:

ElJose scrie:BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
nu e necesar sa faci asa
nu ai nevoie de pointer pentru character in clasa CHARACTER
e suficient sa faci asa
if (CountSpecifyItem(30093)<0)

poti face chiar si asa if (bTime == 1 && CountSpecifyItem(30093) < 0 )
sa nu fie atatea linii

si nu e o functie ce ai facut aici

Este acelasi lucru care l-am facut.
Functioneaza.
Si, de ce nu ai facut-o tu pana acum?

BYTE bCount = quest::CQuestManager::instance().GetCurrentCharacterPtr()->CountSpecifyItem(30093);
de ce sa faci asta cand nu e necesara totusi?
si nu am facut asa ceva pt ca nu am nevoie
si sa stii pe viitor cand chiar o sa ai nevoie de asta sa verifici mai intai pointerul daca e null, sa eviti un posibil crash core

Vezi ca dai coment-uri pentru +1. Te asigur ca nu are crashcore si daca te uiti mai bine vezi acolo.
Testeaza si dupa vii si comentezi.

daca nu pricepi tu nu inseamna ca e pt +1, e problema ta, nu m-am referit strict aici unde chiar nici nu e necesar
poate altcineva o sa inteleaga si o sa aibe nevoie si asa o sa stie sa evite prostii postate aiurea

ElJose scrie:daca nu pricepi tu nu inseamna ca e pt +1, e problema ta
poate altcineva o sa inteleaga si o sa aibe nevoie si asa o sa stie sa evite prostii postate aiurea

Testeaza si dupa vii sa comentezi.

am dat un edit: "nu m-am referit strict aici unde chiar nici nu e necesar"
nu am venit ca sa fac hate ci doar sa explic ceva

ElJose scrie:daca nu pricepi tu nu inseamna ca e pt +1, e problema ta, nu m-am referit strict aici unde chiar nici nu e necesar
poate altcineva o sa inteleaga si o sa aibe nevoie si asa o sa stie sa evite prostii postate aiurea

Lasa-l frate..e pe lumea lui..a facut un cod de 50 de linii care se putea face in 15 ,dar ce mai conteaza? daca merge )
Omul le stie bine...glumesc

ON: FOlositor pt incepatori..Tine-o tot asa

DWORD ConvertTypeTimeSelectedToVnum(BYTE bTime)
{
	switch(bTime)
	{
		case 1: return ITEM_OFFLINESHOP_TIME_A;
		case 2: return ITEM_OFFLINESHOP_TIME_B;
		case 3: return ITEM_OFFLINESHOP_TIME_C;
	}
}
	
void OpenShop(const char * c_pszSign, TShopItemTable* pTable, BYTE bItemCount, BYTE bTime)
{
	DWORD dwItemVnum = ConvertTypeTimeSelectedToVnum(bTime);
	if (CountSpecifyItem(dwItemVnum))
		// something
		RemoveSpecifyItem(dwItemVnum, 1);
	else
		// something
}