Bueno, vengo a dejar por acá este source para modificar el lugar donde apareces luego de morir en los mapas ya que hace mucho no publicaba algo y quiero hacer uno mejor y dinámico.

Recuerden que en caso de querer agregar coordenadas para mapas custom deben agregarlas a la función y modificar la cantidad de mapas.

Source


MapRespawnManager.ini



Creditos
Natzugen

Very Nice )) This is cool config)

@Natzugen  source for Added Newsboard UI system

Tambiens se lo puede configurar desde el Gate cambiando o creando uno nuevo.

+ Un excelente aporte a la comunidad.

hola, disculpen mi ignoracia.. lo agregue pero no se si esta funcionando .. como chequeo eso? gracias

hola, disculpen mi ignoracia.. lo agregue pero no se si esta funcionando .. como chequeo eso? gracias

 jajaja jajaja jajaja jajaja jajaja jajaja jajaja jajaja jajaja

Anda a algun mapa de la lista o algun mapa custom que tengas agregado, deja que te mate algun mob o player y verifica si revives en las coordenadas que pusiste en dicho mapa, ez

hola, disculpen mi ignoracia.. lo agregue pero no se si esta funcionando .. como chequeo eso? gracias

 jajaja jajaja jajaja jajaja jajaja jajaja jajaja jajaja jajaja

Anda a algun mapa de la lista o algun mapa custom que tengas agregado, deja que te mate algun mob o player y verifica si revives en las coordenadas que pusiste en dicho mapa, ez

gracias

Muy bueno, pero pudiste a ver optimizado más el código obteniendo el identificador desde el mismo archivo, ya que así el usuario puede agregar su mapa custom y normalmente su respawn desde un archivo .txt sin tener que ir a codear el núevo identificador de mapa.

Muy bueno, pero pudiste a ver optimizado más el código obteniendo el identificador desde el mismo archivo, ya que así el usuario puede agregar su mapa custom y normalmente su respawn desde un archivo .txt sin tener que ir a codear el núevo identificador de mapa.

sigue dando igual por qué la de mapas va especificado en el source, de nada sirve dar la opción de agregar más mapas si el gas soporta menos

no era mejor con vectores y usar iterator para evitar la especificacion cada array en el codigo ? pregunto nomas.

no era mejor con vectores y usar iterator para evitar la especificacion cada array en el codigo ? pregunto nomas.

exacto a esto me refería algo como esto:

Code: [Select]

#include "stdafx.h"
#include "RespawnLocation.h"
#include "Gate.h"
#include "Map.h"
#include "KanturuBattleUserMng.h"
#include "RaklionBattleUserMng.h"
#include "MemScript.h"
#include "Util.h"

cRespawnLocation gRespawnLocation;

cRespawnLocation::cRespawnLocation()
{
	this->m_RespLoc.clear();
}

cRespawnLocation::~cRespawnLocation()
{
	
}

void cRespawnLocation::Load(char* path)
{
	CMemScript* lpMemScript = new CMemScript;

	if(lpMemScript == 0)
	{
		ErrorMessageBox(MEM_SCRIPT_ALLOC_ERROR,path);
		return;
	}

	if(lpMemScript->SetBuffer(path) == 0)
	{
		ErrorMessageBox(lpMemScript->GetLastError());
		delete lpMemScript;
		return;
	}

	this->m_RespLoc.clear();

	try
	{
		while(true)
		{
			if(lpMemScript->GetToken() == TOKEN_END)
			{
				break;
			}

			if(strcmp("end",lpMemScript->GetString()) == 0)
			{
				break;
			}

			RESPLOC_DATA info;

			info.Map = lpMemScript->GetNumber();

			info.MoveGate = lpMemScript->GetAsNumber();

			info.Gate = lpMemScript->GetAsNumber();

			this->m_RespLoc.push_back(info);
		}
	}
	catch(...)
	{
		ErrorMessageBox(lpMemScript->GetLastError());
	}

	delete lpMemScript;
}

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (it->Map == lpObj->Map)
		{
			return true;
		}
	}
	return false;
}

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	bool result = 0;

	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (lpObj->Map == it->Map)
		{
			if (it->MoveGate == 0)
			{
				return 0;
			}
			result = gGate.GetGate(it->Gate,gate,map,x,y,dir,level);

			if (lpObj->Map == MAP_KANTURU3)
			{
				gKanturuBattleUserMng.DelUserData(lpObj->Index);
			}

			if (lpObj->Map == MAP_RAKLION2)
			{
				gRaklionBattleUserMng.DelUserData(lpObj->Index);
			}
		}
	}
	return result;
}

y el archivo txt quedaría como esto:

Code: [Select]

//Map   MoveGate   Gate
0       1          17      // ->LORENCIA
1       1          17      // ->DUNGEON
2       1          22      // ->DEVIAS
3       1          27      // ->NORIA
4       1          42      // ->LOST TOWER
6       0          *       // ->ARENA
7       1          49      // ->ATLANS
8       1          57      // ->TARKAN
10      1          22      // ->ICARUS
31      1          106     // ->LAND OF TRIALS
33      1          119     // ->AIDA
37      1          138     // ->KANTURU1
38      1          139     // ->KANTURU2
39      1          137     // ->KANTURU3
40      1          402     // ->SILENT
41      1          256     // ->BARRAKS
42      1          256     // ->REFUGE
51      1          267     // ->ELBELAND
56      1          273     // ->SWAMP OF CALMNESS
57      1          286     // ->RAKLION
58      1          286     // ->RAKLION2
62      1          22      // ->SANTA TOWN
63      1          137     // ->VULCANUS
79      1          333     // ->LOREN MARKET
80      1          335     // ->KARUTAN1
81      1          335     // ->KARUTAN2
83      1          426     // ->ACHERON
84      1          443     // ->DEVENTER
85      1          452     // ->URUK
86      1          453     // ->FEREA
87      1          *       // ->TEAMVSTEAM
88		1		   *	   // ->NARS
89		1		   455	   // ->NIXIE LAKE
90		1		   17	   // ->DEEP DUNGEON 1
91		1		   17	   // ->DEEP DUNGEON 2
92		1		   17	   // ->DEEP DUNGEON 3
93		1		   17	   // ->DEEP DUNGEON 4
94		1		   17	   // ->DEEP DUNGEON 5
95		1		   *	   // ->SWAMP OF DARKNESS
96		1		   *	   // ->S14 MAP
end

debe tener su uso especifico, aunque si no mal cuando agregas un mapa tienes que declarar el gate que se va a usar como respawn y en gate.txt pones las cordenadas y cuando la borrar esa linea de ese gate la persona renace en la misma cordenada en al que murio.

no era mejor con vectores y usar iterator para evitar la especificacion cada array en el codigo ? pregunto nomas.

exacto a esto me refería algo como esto:

Code: [Select]

#include "stdafx.h"
#include "RespawnLocation.h"
#include "Gate.h"
#include "Map.h"
#include "KanturuBattleUserMng.h"
#include "RaklionBattleUserMng.h"
#include "MemScript.h"
#include "Util.h"

cRespawnLocation gRespawnLocation;

cRespawnLocation::cRespawnLocation()
{
	this->m_RespLoc.clear();
}

cRespawnLocation::~cRespawnLocation()
{
	
}

void cRespawnLocation::Load(char* path)
{
	CMemScript* lpMemScript = new CMemScript;

	if(lpMemScript == 0)
	{
		ErrorMessageBox(MEM_SCRIPT_ALLOC_ERROR,path);
		return;
	}

	if(lpMemScript->SetBuffer(path) == 0)
	{
		ErrorMessageBox(lpMemScript->GetLastError());
		delete lpMemScript;
		return;
	}

	this->m_RespLoc.clear();

	try
	{
		while(true)
		{
			if(lpMemScript->GetToken() == TOKEN_END)
			{
				break;
			}

			if(strcmp("end",lpMemScript->GetString()) == 0)
			{
				break;
			}

			RESPLOC_DATA info;

			info.Map = lpMemScript->GetNumber();

			info.MoveGate = lpMemScript->GetAsNumber();

			info.Gate = lpMemScript->GetAsNumber();

			this->m_RespLoc.push_back(info);
		}
	}
	catch(...)
	{
		ErrorMessageBox(lpMemScript->GetLastError());
	}

	delete lpMemScript;
}

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (it->Map == lpObj->Map)
		{
			return true;
		}
	}
	return false;
}

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	bool result = 0;

	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (lpObj->Map == it->Map)
		{
			if (it->MoveGate == 0)
			{
				return 0;
			}
			result = gGate.GetGate(it->Gate,gate,map,x,y,dir,level);

			if (lpObj->Map == MAP_KANTURU3)
			{
				gKanturuBattleUserMng.DelUserData(lpObj->Index);
			}

			if (lpObj->Map == MAP_RAKLION2)
			{
				gRaklionBattleUserMng.DelUserData(lpObj->Index);
			}
		}
	}
	return result;
}

y el archivo txt quedaría como esto:

Code: [Select]

//Map   MoveGate   Gate
0       1          17      // ->LORENCIA
1       1          17      // ->DUNGEON
2       1          22      // ->DEVIAS
3       1          27      // ->NORIA
4       1          42      // ->LOST TOWER
6       0          *       // ->ARENA
7       1          49      // ->ATLANS
8       1          57      // ->TARKAN
10      1          22      // ->ICARUS
31      1          106     // ->LAND OF TRIALS
33      1          119     // ->AIDA
37      1          138     // ->KANTURU1
38      1          139     // ->KANTURU2
39      1          137     // ->KANTURU3
40      1          402     // ->SILENT
41      1          256     // ->BARRAKS
42      1          256     // ->REFUGE
51      1          267     // ->ELBELAND
56      1          273     // ->SWAMP OF CALMNESS
57      1          286     // ->RAKLION
58      1          286     // ->RAKLION2
62      1          22      // ->SANTA TOWN
63      1          137     // ->VULCANUS
79      1          333     // ->LOREN MARKET
80      1          335     // ->KARUTAN1
81      1          335     // ->KARUTAN2
83      1          426     // ->ACHERON
84      1          443     // ->DEVENTER
85      1          452     // ->URUK
86      1          453     // ->FEREA
87      1          *       // ->TEAMVSTEAM
88		1		   *	   // ->NARS
89		1		   455	   // ->NIXIE LAKE
90		1		   17	   // ->DEEP DUNGEON 1
91		1		   17	   // ->DEEP DUNGEON 2
92		1		   17	   // ->DEEP DUNGEON 3
93		1		   17	   // ->DEEP DUNGEON 4
94		1		   17	   // ->DEEP DUNGEON 5
95		1		   *	   // ->SWAMP OF DARKNESS
96		1		   *	   // ->S14 MAP
end

Si Asi mismo yo lo tengo se evita la declaracion de variables y trabaja dependiendo a la necesidad del lector

Así es como lo tengo actualmente en mis files, el que publique es el que usaba hace como dos años

Yo remplazaria esto en el .h

Code: [Select]

std::vector<RESPLOC_DATA> m_RespLoc;

por esto

Code: [Select]

std::map<int,RESPLOC_DATA> m_RespLoc;

y en el cpp

en void cRespawnLocation::Load(char* path)

cambiaria esto:

Code: [Select]

this->m_RespLoc.push_back(info);

por esto:

Code: [Select]

this->m_RespLoc.insert(std::pair<int,RESPLOC_DATA>(info.Map, info));

cambiaria esto

Code: [Select]

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (it->Map == lpObj->Map)
		{
			return true;
		}
	}
	return false;
}

por esto

Code: [Select]

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	std::map<int,RESPLOC_DATA>::iterator it = this->m_RespLoc.find(lpObj->Map);

	if (it == this->m_CustomMixInfo.end())
	{
		return 0;
	}
	else
	{
		return 1;
	}
}

y por ultimo cambiaría esto

Code: [Select]

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	bool result = 0;

	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (lpObj->Map == it->Map)
		{
			if (it->MoveGate == 0)
			{
				return 0;
			}
			result = gGate.GetGate(it->Gate,gate,map,x,y,dir,level);

			if (lpObj->Map == MAP_KANTURU3)
			{
				gKanturuBattleUserMng.DelUserData(lpObj->Index);
			}

			if (lpObj->Map == MAP_RAKLION2)
			{
				gRaklionBattleUserMng.DelUserData(lpObj->Index);
			}
		}
	}
	return result;
}

por esto

Code: [Select]

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	std::map<int,RESPLOC_DATA>::iterator it = this->m_RespLoc.find(lpObj->Map);
	
	if (it == this->m_CustomMixInfo.end() || it->second.MoveGate == 0)
	{
		return 0;
	}

	if (lpObj->Map == MAP_KANTURU3)
	{
		gKanturuBattleUserMng.DelUserData(lpObj->Index);
	}

	if (lpObj->Map == MAP_RAKLION2)
	{
		gRaklionBattleUserMng.DelUserData(lpObj->Index);
	}
	
	return gGate.GetGate(it->second.Gate,gate,map,x,y,dir,level);
}

eso seria todo entonces no tenemos que recorrer todo un vector si no que vamos directamente al que necesitamos, aunque no parezca aceleramos mucho la lectura, perdón si me equivoque en algo lo hice a ojo. Saludos a todos.

Yo remplazaria esto en el .h

Code: [Select]

std::vector<RESPLOC_DATA> m_RespLoc;

por esto

Code: [Select]

std::map<int,RESPLOC_DATA> m_RespLoc;

y en el cpp

en void cRespawnLocation::Load(char* path)

cambiaria esto:

Code: [Select]

this->m_RespLoc.push_back(info);

por esto:

Code: [Select]

this->m_RespLoc.insert(std::pair<int,RESPLOC_DATA>(info.Map, info));

cambiaria esto

Code: [Select]

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (it->Map == lpObj->Map)
		{
			return true;
		}
	}
	return false;
}

por esto

Code: [Select]

bool cRespawnLocation::IsMap(LPOBJ lpObj)
{
	std::map<int,RESPLOC_DATA>::iterator it = this->m_RespLoc.find(lpObj->Map);

	if (it == this->m_CustomMixInfo.end())
	{
		return 0;
	}
	else
	{
		return 1;
	}
}

y por ultimo cambiaría esto

Code: [Select]

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	bool result = 0;

	for(std::vector<RESPLOC_DATA>::iterator it=this->m_RespLoc.begin();it != this->m_RespLoc.end();it++)
	{
		if (lpObj->Map == it->Map)
		{
			if (it->MoveGate == 0)
			{
				return 0;
			}
			result = gGate.GetGate(it->Gate,gate,map,x,y,dir,level);

			if (lpObj->Map == MAP_KANTURU3)
			{
				gKanturuBattleUserMng.DelUserData(lpObj->Index);
			}

			if (lpObj->Map == MAP_RAKLION2)
			{
				gRaklionBattleUserMng.DelUserData(lpObj->Index);
			}
		}
	}
	return result;
}

por esto

Code: [Select]

bool cRespawnLocation::GetUserRespawnLocation(LPOBJ lpObj,int* gate,int* map,int* x,int* y,int* dir,int* level)
{
	std::map<int,RESPLOC_DATA>::iterator it = this->m_RespLoc.find(lpObj->Map);
	
	if (it == this->m_CustomMixInfo.end() || it->second.MoveGate == 0)
	{
		return 0;
	}

	if (lpObj->Map == MAP_KANTURU3)
	{
		gKanturuBattleUserMng.DelUserData(lpObj->Index);
	}

	if (lpObj->Map == MAP_RAKLION2)
	{
		gRaklionBattleUserMng.DelUserData(lpObj->Index);
	}
	
	return gGate.GetGate(it->second.Gate,gate,map,x,y,dir,level);
}

eso seria todo entonces no tenemos que recorrer todo un vector si no que vamos directamente al que necesitamos, aunque no parezca aceleramos mucho la lectura, perdón si me equivoque en algo lo hice a ojo. Saludos a todos.

como dato , te comento que la busqueda entre un vector y un map es practicamente la misma , el map no hace magia no va a ir directamente a una locacion solo con pasarle un valor, este tambien tiene que recorer por igual que lo harias manual con el vector

Hola @YolaxD tengo entendido que al indexar un objeto como lo estamos haciendo en su insert. es mas rápida la lectura obviamente que va a recorrer el array pero de otra forma, en cosas tan pequeñas como estas no va a hacer diferencia ni en mili segundos, pero en arrays grandes puede hacer una pequeña diferencia o por lo menos es lo que tengo entendido, si no es así te agradezco por el comentario.

Hola @YolaxD tengo entendido que al indexar un objeto como lo estamos haciendo en su insert. es mas rápida la lectura obviamente que va a recorrer el array pero de otra forma, en cosas tan pequeñas como estas no va a hacer diferencia ni en mili segundos, pero en arrays grandes puede hacer una pequeña diferencia o por lo menos es lo que tengo entendido, si no es así te agradezco por el comentario.

de echo hay varios metodos de leer un array pero la más comun es lineal, ya que es la que en su mayoría hasta un programador principiante conoce, y es por eso que se maneja eso al compartir, luego la mayoría de los user de aquí copian y pegan que luego cuando necesitan agregar una condición adicional comienzan a tener problemas y generar comentarios como: "no sirve tu código".

es por eso que cuando se comparte se trata de dar un código sencillo y factible a la lectura del algoritmo de ejecución.

pero si hablamos de optimizar el código uff el mu le hace falta mucho. nada mejor que asm pero desgraciadamente no todo es tan fácil y hay cosas que es más fácil arreglar por c que por asm. por la complejidad de código.

Hola @YolaxD tengo entendido que al indexar un objeto como lo estamos haciendo en su insert. es mas rápida la lectura obviamente que va a recorrer el array pero de otra forma, en cosas tan pequeñas como estas no va a hacer diferencia ni en mili segundos, pero en arrays grandes puede hacer una pequeña diferencia o por lo menos es lo que tengo entendido, si no es así te agradezco por el comentario.

Efectivamente hay métodos más rápidos de lectura como dices y es importantísimo saber que método usar en cada ocasion, pero mi comentario fue porque mencionas te no tener que recorrer el array lo cual el find() internamente lo hace y me pareció importante destacar que no era así para no mal informar :)

Its not @LTP Team code? I saw it on hes old 1.6.0 EMU update....
Anyway good job!

Its not @LTP Team code? I saw it on hes old 1.6.0 EMU update....
Anyway good job!

Nope, i had it on my files since long time ago