C++ Speedwow [Mount Command]

[frytiks 29]

Chcel by som sa poradiť čo som urobil zle a či by tento script vôbec mohol fungovať ak niečo nájdete tak prosím povedzte.

 

src/server/game/chat/Chat.cpp

 

{ "mount",		   SEC_PLAYER,		 false, OldHandler<&ChatHandler::HandleMountCommand>,				"", NULL },

 

src/server/game/chat/chat.h

 

bool HandleMountCommand(const char* args);

 

src/server/game/chat/level0.cpp

 

 

bool ChatHandler::HandleMountCommand(const char* /*args*/)
{

#define CENA                   100000   // Default: 100000(cooper) => 10G

Player *chr = m_session->GetPlayer();
Player *CENA = m_session->GetMoney();
Player *-CENA = m_session->ModifyMoney();

   if(chr->isInFlight()) //nemožno použiť v lietaní
   {
       SendSysMessage(LANG_YOU_IN_FLIGHT);
       SetSentErrorMessage(true);
       return false;
   }

   if(chr->isInArena()) //nemožno použiť v aréne
   {
       SendSysMessage(LANG_YOU_IN_ARENA);
       SetSentErrorMessage(true);
       return false;
   }

   if(chr->isInCombat()) //proti combatu
   {
       SendSysMessage(LANG_YOU_IN_COMBAT);
       SetSentErrorMessage(true);
       return false;
   }

   if (pPlayer->GetMoney() >= CENA)
   {
       chr->SendSysMessage("Dostal si mounta !");
           chr->CastSpell(player,54811,true);
           chr->CastSpell(player,54811,false);

  pPlayer->ModifyMoney(-CENA);
 break;
              default:
                      return false;
   }

   return true;
}

Edited March 9, 2012 by frytiks

[Nex 159]

Co to je za kravinu používat v podmínce default... když tam žádnej switch nemáš.

 

Zkus tohle: http://pastebin.com/3e5GSNm8

[Hkarta 584]

V kódu máš odemě komentáře. Nevím jestli to bude fungovat, ale mělo by to bejt správně (z toho co jsi mi dal za kód, funkce jsem v jádru nekontroloval).

bool ChatHandler::HandleMountCommand(const char* /*args*/)
{

#define CENA				   100000   // Default: 100000(cooper) => 10G

Player *chr = m_session->GetPlayer();
/*
Player *CENA = m_session->GetMoney(); // totální píčovina, smysl by dávalo unit32 cena = chr->GetMoney();
Player *-CENA = m_session->ModifyMoney(); // další píčovina, pochybuju že v c++ můžeš začínat název proměné mínusem, navíc pokud vím tak ModifyMoney vrací void*/

/*
pokud nepotřebuješ posílat chybovou hlášku, úplně v klidu se to dá zredukovat

if(chr->isInFlight()) //nemožno použiť v lietaní
{
	SendSysMessage(LANG_YOU_IN_FLIGHT);
	SetSentErrorMessage(true);
	return false;
}

if(chr->isInArena()) //nemožno použiť v aréne
{
	SendSysMessage(LANG_YOU_IN_ARENA);
	SetSentErrorMessage(true);
	return false;
}

if(chr->isInCombat()) //proti combatu
{
	SendSysMessage(LANG_YOU_IN_COMBAT);
	SetSentErrorMessage(true);
	return false;
}
*/

  if(chr->isInFlight() || chr->isInArena() || chr->isInCombat() || chr->IsMounted())
  {
  chr->SendSysMessage("Prikaz nelze pouzit v pripade ze uz mas mounta, za letu, v boji a v areně.");
  return false;
  }

//if (pPlayer->GetMoney() >= CENA) // wtf? kde jsi vzal pointer pPlayer?
if(chr->GetMoney() >= CENA)
{
	chr->SendSysMessage("Dostal si mounta !"); // jsi!
	chr->CastSpell(player,54811,true);
	//chr->CastSpell(player,54811,false); // nevím co přesně dělá bool triggered, ale dvakrát to určitě castit nepotřebuješ

   chr->ModifyMoney(-CENA);
   return true;
}

return false; // jinak by to vrátilo true i kdyby hráč neměl dost peněz
}

edit: omlouvám se za rozhozenej kód, na vině je místní editor

Edited March 10, 2012 by Hkarta

[frytiks 29]

ďakujem, viem pokúšal som sa o svoj druhý script a hkarta > aj mne to jeb...

[frytiks 29]

Vyriešené

Edited March 11, 2012 by frytiks

[Murko 69]

tak tam daj podmienku

[HyN3Q 18]

Noo.. oba tam mate chyby, ale pouziju Hkarty verzi..

 

bool ChatHandler::HandleMountCommand(const char* /*args*/)
{


Player *chr = m_session->GetPlayer();
uint64 cena = chr->GetMoney();


if(chr->isInFlight() || chr->isInArena() || chr->isInCombat() || chr->IsMounted())
{
	chr->SendSysMessage("Prikaz nelze pouzit v pripade ze uz mas mounta, za letu, v boji a v areně.");
	SetSentErrorMessage(true);
	return false;
}

if(chr->GetMoney() >= 10*GOLD)
{
	chr->SendSysMessage("Dostal jsi mounta !");
       chr->CastSpell(player, 54811, false);

	chr->ModifyMoney(-(CENA*GOLD));
	return true;
}
else
	chr->GetSession()->SendNotification(" Nemas dostatek penez, musis mit minimalne 10 goldu.");

return false; // jinak by to vrátilo true i kdyby hráč neměl dost peněz
}

Takhle ti to uz pujde,

Edited March 11, 2012 by HyN3Q

[Gelidus 42]

HyN3Q ešte to trocha uprav ...

uint64 cena = chr->GetMoney();

toto nikde nevyužívaš

 

chr->ModifyMoney(-(CENA*GOLD));

to by asi malo byť 10*GOLD ... cena je hore definovaná ako 10g takže to bude checkovať práve pre 10g ale zoberie 100

 

Edit : celé

 

#define CENA 100000
bool ChatHandler::HandleMountCommand(const char* /*args*/)
{
Player *chr = m_session->GetPlayer();
if(chr->isInFlight() || chr->isInArena() || chr->isInCombat() || chr->IsMounted())
{
 chr->SendSysMessage("Prikaz nelze pouzit v pripade ze uz mas mounta, za letu, v boji a v areně.");
 SetSentErrorMessage(true);
 return false;
}
if(chr->GetMoney() >= CENA)
{
 chr->SendSysMessage("Dostal jsi mounta !");
 chr->CastSpell(player, 54811, false);
 chr->ModifyMoney(-(CENA));
 return true;
}
else
 chr->GetSession()->SendNotification(" Nemas dostatek penez, musis mit minimalne 10 goldu.");
return false;
}

Edited March 11, 2012 by Gelidus

[HyN3Q 18]

Ajo, diky za opravu

 

edit:

 

ale ta definice tam je zbytecna, ja bych tam radeji dal 10*GOLD protoze nerad vidim nekde

 

#define xxx yyyy

 

a nebo to definovat do handleru a ne mimo.

Edited March 11, 2012 by HyN3Q

[frytiks 29]

Ja som si už tamto hkartove opravil sam ale aj tak dík

Edited March 11, 2012 by frytiks

[Murko 69]

zaujímavé že si opravíš všetko po tom ako ti to niekto napíše do topicu teraz som si to všimol

[frytiks 29]

To mi robíš naschval ? vidím čo mi v outlogu vypíše ... nesedím nonstop za wowresource tak som asi troška neskôr odpísal