[Release] Monster Buffs

[Blouflash]

[Project.cpp]
search for:

#ifdef __IMPROVE_MAP_SYSTEM
#ifdef __CLIENT
	m_MapInformationManager.LoadMapInformationData();
#endif // __CLIENT
#endif // __IMPROVE_MAP_SYSTEM

and add this UNDER it:

#if defined(__MONSTER_BUFFS) && defined(__WORLDSERVER)
	LoadMonsterBuffs("MonsterBuffs.inc");
#endif // __MONSTER_BUFFS

search for:

void CProject::SetGlobal( UINT type, float fValue )

and add this ABOVE it:

#ifdef __MONSTER_BUFFS
BOOL CProject::LoadMonsterBuffs(LPCTSTR lpszFilename)
{
	CScript script;
	if (script.Load(lpszFilename) == FALSE)
	{
		Error("Failed to load: %s", lpszFilename);
		return FALSE;
	}

	int monsterID = script.GetNumber();
	while (script.tok != FINISHED)
	{
		int buffID = script.GetNumber();
		m_mapMonsterBuffs.insert(make_pair(monsterID, buffID));
		monsterID = script.GetNumber();
	}
	return TRUE;
}
#endif // __MONSTER_BUFFS

[Project.h]
search for:

CUser*			GetUser( OBJID objid );

and add this ABOVE it:

#ifdef __MONSTER_BUFFS
	map<int, int>   m_mapMonsterBuffs;
	BOOL			LoadMonsterBuffs(LPCTSTR lpszFilename);
#endif // __MONSTER_BUFFS

[AttackArbiter.cpp]
search for:

OnDiedSchoolEvent();

and add this UNDER it:

#ifdef __MONSTER_BUFFS
	map< int, int >::iterator it = prj.m_mapMonsterBuffs.find(m_pDefender->m_dwIndex);
	if (prj.m_mapMonsterBuffs.end() != it)
	{
		ItemProp* pProp = prj.GetItemProp(it->second);
		if (pProp)
			m_pAttacker->DoApplySkill(m_pAttacker, pProp, NULL);
	}
#endif // __MONSTER_BUFFS

[Resource]
Make a new file in your resource folder named MonsterBuffs.inc with this content:

MI_AIBATT1	II_SYS_SYS_LS_LOVE
MI_AIBATT2	II_SYS_SYS_LS_BLESSING

[WorldServer/VersionCommon.h]

#define __MONSTER_BUFFS

For those who can't read. Here are some screenshot on where to put the code.

 Spoiler

This thread took me longer than the actual coding lol.


UPDATE 1 (not tested): The buff will now be applied to all partymember no matter where they are.

 Spoiler

[AttackArbiter.cpp]
replace:

#ifdef __MONSTER_BUFFS
	map< int, int >::iterator it = prj.m_mapMonsterBuffs.find(m_pDefender->m_dwIndex);
	if (prj.m_mapMonsterBuffs.end() != it)
	{
		ItemProp* pProp = prj.GetItemProp(it->second);
		if (pProp)
			m_pAttacker->DoApplySkill(m_pAttacker, pProp, NULL);
	}
#endif // __MONSTER_BUFFS

with:

#ifdef __MONSTER_BUFFS
	map< int, int >::iterator it = prj.m_mapMonsterBuffs.find(m_pDefender->m_dwIndex);
	if (prj.m_mapMonsterBuffs.end() != it)
	{
		ItemProp* pProp = prj.GetItemProp(it->second);
		if (pProp)
		{
			CMover *pMember;
			CParty* pParty = g_PartyMng.GetParty(m_pAttacker->m_idparty);

			if (pParty)
			{
				for (int i = 0; i < pParty->m_nSizeofMember; i++)
				{
					pMember = (CMover *)g_UserMng.GetUserByPlayerID(pParty->m_aMember[i].m_uPlayerId);
					if (IsValidObj(pMember))
						((CUser*)pMember)->DoApplySkill(pMember, pProp, NULL);
				}
			}
			else 
				((CUser*)m_pAttacker)->DoApplySkill(m_pAttacker, pProp, NULL);
		}
	}
#endif // __MONSTER_BUFFS

[cholai2210]

Man this helps thanks <3

[Zeljian]

buff for how many online and their brackets., thanks

[cholai2210]

Originally Posted by naruto66620

Very nice idea, we can make a "debuff"
For exemple, a toxic monster, when you kill it boom, debuff "speed-50%"

If you thinking about debuffing when killing a toxic monster . Im thinking also a suicide bomber monster on darkon driller monster hahahaha.

[Drabur]

Ich würde das System dahin ändern, dass nicht ein extra Datei nötig ist. Einfacher wäre es das ganze über die propMover.txt zu machen einfach ein neues Parameter dran und fertig.

[Professor Linebeck]

Originally Posted by Drabur

Ich würde das System dahin ändern, dass nicht ein extra Datei nötig ist. Einfacher wäre es das ganze über die propMover.txt zu machen einfach ein neues Parameter dran und fertig.

Ich finde es genau deshalb übersichtlicher. Man braucht nicht noch mehr unnötige Zeichen in der propMover, denn genau das macht es eben so unübersichtlich. Eine entsprechend benannte Datei ist mir um einiges anschaulicher und man kann ebenso schnell darauf zugreifen.
Außerdem nervt es mich ständig nach rechts scrollen zu müssen um zu den hinteren Parametern zu kommen :D

[Drabur]

Originally Posted by Professor Linebeck

Ich finde es genau deshalb übersichtlicher. Man braucht nicht noch mehr unnötige Zeichen in der propMover, denn genau das macht es eben so unübersichtlich. Eine entsprechend benannte Datei ist mir um einiges anschaulicher und man kann ebenso schnell darauf zugreifen.
Außerdem nervt es mich ständig nach rechts scrollen zu müssen um zu den hinteren Parametern zu kommen

Nicht wirklich. Der Resource wird immer voller mit Datein. Außerdem wenn man ein Tool für die Resource hat geht das wesentlich schneller als irgendwie anders

[Professor Linebeck]

Originally Posted by Drabur

Nicht wirklich. Der Resource wird immer voller mit Datein.

Und was ist daran jetzt schlimm?

Originally Posted by Drabur

Außerdem wenn man ein Tool für die Resource hat geht das wesentlich schneller als irgendwie anders

Und das Tool musst du dann jedes mal umschreiben wenn du neue Parameter reinhaust, meh. Ich bleib bei meinem guten alten Notepad :D

[Drabur]

Originally Posted by Professor Linebeck

Und das Tool musst du dann jedes mal umschreiben wenn du neue Parameter reinhaust, meh. Ich bleib bei meinem guten alten Notepad

Wieso sollte man immer an dem Tool was umschreiben müssen?

[Professor Linebeck]

Originally Posted by Drabur

Wieso sollte man immer an dem Tool was umschreiben müssen?

Wenn wir hier von einem vernünftigen Tool sprechen, willst du die Dateien mit Parametern und co laden und verwalten können, right?

[Drabur]

Originally Posted by Professor Linebeck

Wenn wir hier von einem vernünftigen Tool sprechen, willst du die Dateien mit Parametern und co laden und verwalten können, right?

Ja mein Tool lädt die Parameter gibt denen die gleichen Namen wie in der Source und ermöglicht es diese zu ändern und zeigt halt an was alles geht bei denen wo die defines genutzt werden

[Professor Linebeck]

Originally Posted by Drabur

Ja mein Tool lädt die Parameter gibt denen die gleichen Namen wie in der Source und ermöglicht es diese zu ändern und zeigt halt an was alles geht bei denen wo die defines genutzt werden

Mich würde es schon nerven bei jedem neuem Parameter der dazu kommt das Tool darauf anzupassen, kommt halt auch drauf an wie oft man neue Parameter hinzufügt und für was sie genau sind.

[Sedrika]

Originally Posted by Drabur

Ich würde das System dahin ändern, dass nicht ein extra Datei nötig ist. Einfacher wäre es das ganze über die propMover.txt zu machen einfach ein neues Parameter dran und fertig.

Hmm, lese ich lieber eine neue Datei ein und speichere die Daten in eine Map oder speichere ich die Daten lieber in jeden Mover der gespawnt wird... Kann mich nicht wirklich entscheiden was effizienter und resourcen schonender ist.

[Lumi]

Es soll jeder benutzen wie er es mag. Ich persönlich finde eine extra Datei auch angenehmer, da es einfach mehr übersichtlicher ist. Jeder verwendet seine Resource anders. Damit müsst ihr leben.

[Drabur]

Originally Posted by Sedrika

Hmm, lese ich lieber eine neue Datei ein und speichere die Daten in eine Map oder speichere ich die Daten lieber in jeden Mover der gespawnt wird... Kann mich nicht wirklich entscheiden was effizienter und resourcen schonender ist.

Wenn man auf Perfektion setzt kannst du ja auch alle Parameter einzeln machen aber das tut auch keiner.

Hab keine Ahnung was schneller geladen wird aber ich kann es mir vorstellen das ein Datei Zugriff länger dauert als die eine Variable neu zu machen.

Mir ist es egal und es sollte auch kein Flame sein. Nur ein Vorschlag wie ich es ändern würde.